Skip to contentSkip to conversation

Kconfig: Migrate CONFIG_NR_DRAM_BANKS

PATCH In review Ramon Fried 3 participants 5 messages Aug 13, 2018 – Aug 17, 2018

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Kconfig: Migrate CONFIG_NR_DRAM_BANKS

Move CONFIG_NR_DRAM_BANKS from headers to Kconfig.

---
 Kconfig | 6 ++++++
 1 file changed, 6 insertions(+)
Kconfig +6 -0
index db0f545e45..d96e3373c1 100644
@@ -104,6 +104,12 @@ config ENV_VARS_UBOOT_CONFIG
 	  - CONFIG_SYS_VENDOR
 	  - CONFIG_SYS_SOC
 
+config NR_DRAM_BANKS
+	int "Number of DRAM banks"
+	default 4
+	help
+	  This defines the number of DRAM banks.
+
 config SYS_BOOT_GET_CMDLINE
 	bool "Enable kernel command line setup"
 	help
Signature · 1 line
2.18.0
#1 lore raw

PATCH 1/1 Kconfig: Migrate CONFIG_NR_DRAM_BANKS

Quoted 3 lines · Ramon Fried

Move CONFIG_NR_DRAM_BANKS from headers to Kconfig.

Somehow this seems to be missing a lot of cleanups in include/configs/ ... look at moveconfig.py

Quoted 22 lines
---
 Kconfig | 6 ++++++
 1 file changed, 6 insertions(+)
Kconfig +6 -0
index db0f545e45..d96e3373c1 100644
@@ -104,6 +104,12 @@ config ENV_VARS_UBOOT_CONFIG
 	  - CONFIG_SYS_VENDOR
 	  - CONFIG_SYS_SOC
 
+config NR_DRAM_BANKS
+	int "Number of DRAM banks"
+	default 4
+	help
+	  This defines the number of DRAM banks.
+
 config SYS_BOOT_GET_CMDLINE
 	bool "Enable kernel command line setup"
 	help
Signature · 2 lines
Best regards,
Marek Vasut
#2 lore raw

PATCH 1/1 Kconfig: Migrate CONFIG_NR_DRAM_BANKS

Quoted 8 lines · Marek Vasut
Quoted 3 lines · Ramon Fried

Move CONFIG_NR_DRAM_BANKS from headers to Kconfig.

Somehow this seems to be missing a lot of cleanups in include/configs/ ... look at moveconfig.py

Indeed, Tom said he'll handle that :)

Quoted 29 lines
Quoted 13 lines
---
 Kconfig | 6 ++++++
 1 file changed, 6 insertions(+)
Kconfig +0 -0
index db0f545e45..d96e3373c1 100644
@@ -104,6 +104,12 @@ config ENV_VARS_UBOOT_CONFIG
        - CONFIG_SYS_VENDOR
        - CONFIG_SYS_SOC

Kconfig config ENV_VARS_UBOOT_CONFIG · line 115

Quoted 10 lines
Kconfig +6 -0
 
+config NR_DRAM_BANKS
+     int "Number of DRAM banks"
+     default 4
+     help
+       This defines the number of DRAM banks.
+
 config SYS_BOOT_GET_CMDLINE
      bool "Enable kernel command line setup"
      help

--
Best regards,
Marek Vasut

#3 lore raw

PATCH 1/1 Kconfig: Migrate CONFIG_NR_DRAM_BANKS

Quoted 8 lines · Ramon Fried

On Tue, Aug 14, 2018 at 1:11 AM Marek Vasut <[email protected] <mailto:[email protected]>> wrote:

On 08/14/2018 12:00 AM, Ramon Fried wrote:
> Move CONFIG_NR_DRAM_BANKS from headers to Kconfig. >

Quoted 7 lines

> Signed-off-by: Ramon Fried <[email protected] <mailto:[email protected]>>

Somehow this seems to be missing a lot of cleanups in include/configs/ ... look at moveconfig.py

Indeed, Tom said he'll handle that :)

This looks like it's gonna be quite involved. If Tom wants to do it, that's fine by me, but this patch shouldn't be applied without the cleanup.

Quoted 28 lines
Quoted 18 lines

---
  Kconfig | 6 ++++++

  1 file changed, 6 insertions(+)
Kconfig +0 -0
index db0f545e45..d96e3373c1 100644
@@ -104,6 +104,12 @@ config ENV_VARS_UBOOT_CONFIG

         - CONFIG_SYS_VENDOR
         - CONFIG_SYS_SOC

Kconfig +6 -0
+config NR_DRAM_BANKS
+     int "Number of DRAM banks"
+     default 4
+     help
+       This defines the number of DRAM banks.
+
Quoted 4 lines

  config SYS_BOOT_GET_CMDLINE
       bool "Enable kernel command line setup"
       help

    -- 
    Best regards,
    Marek Vasut
Signature · 2 lines
Best regards,
Marek Vasut
#4 lore raw
In reply to Ramon Fried
Quoted 3 lines · Ramon Fried

Move CONFIG_NR_DRAM_BANKS from headers to Kconfig.

Applied to u-boot/master, thanks!

Signature · 8 lines
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180816/a4d792ca/attachment.sig>