ARC: CONFIG_NODES_SHIFT fix default values
diff mbox

Message ID 1474455108-13550-1-git-send-email-noamca@mellanox.com
State New
Headers show

Commit Message

Noam Camus Sept. 21, 2016, 10:51 a.m. UTC
From: Noam Camus <noamca@mellanox.com>

Seem like values assigned as absolute number and not and
shift value, i.e. should be 0 for one node (2^0) and 1 for
couple of nodes (2^1)

Signed-off-by: Noam Camus <noamca@mellanox.com>
---
 arch/arc/Kconfig |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

Comments

Vineet Gupta Sept. 26, 2016, 9:23 p.m. UTC | #1
On 09/21/2016 03:51 AM, Noam Camus wrote:
> From: Noam Camus <noamca@mellanox.com>
> 
> Seem like values assigned as absolute number and not and
> shift value, i.e. should be 0 for one node (2^0) and 1 for
> couple of nodes (2^1)
> 
> Signed-off-by: Noam Camus <noamca@mellanox.com>

Good catch ! Added to for-next.

Thx,
-Vineet

Patch
diff mbox

diff --git a/arch/arc/Kconfig b/arch/arc/Kconfig
index 6af0b43..f347715 100644
--- a/arch/arc/Kconfig
+++ b/arch/arc/Kconfig
@@ -383,8 +383,8 @@  endchoice
 
 config NODES_SHIFT
 	int "Maximum NUMA Nodes (as a power of 2)"
-	default "1" if !DISCONTIGMEM
-	default "2" if DISCONTIGMEM
+	default "0" if !DISCONTIGMEM
+	default "1" if DISCONTIGMEM
 	depends on NEED_MULTIPLE_NODES
 	---help---
 	  Accessing memory beyond 1GB (with or w/o PAE) requires 2 memory