From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Date: Mon, 28 Sep 2015 21:22:35 +0200 Subject: [U-Boot] [PATCH 03/11] Kconfig: add CONFIG_SYS_BOOTM_LEN In-Reply-To: <20150928151053.GH22966@bill-the-cat> References: <1443029143-22313-1-git-send-email-ryan.harkin@linaro.org> <1443029143-22313-4-git-send-email-ryan.harkin@linaro.org> <20150928151053.GH22966@bill-the-cat> Message-ID: <5609937B.8000309@redhat.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi, On 28-09-15 17:10, Tom Rini wrote: > On Wed, Sep 23, 2015 at 10:25:35AM -0700, Ryan Harkin wrote: > >> As config migrates from board config files to Kconfig, when adding >> CONFIG_SYS_BOOTM_LEN to a platform, I decided to add >> Kconfig support for CONFIG_SYS_BOOTM_LEN. >> >> Signed-off-by: Ryan Harkin >> Reviewed-by: Linus Walleij >> CC: Masahiro Yamada >> CC: Simon Glass >> CC: Linus Walleij > > Thanks for trying to do this. The problem however is that you need to > use tools/moveconfig.py so that all of the other boards (which is a lot) > get updated too, otherwise they fail to build. No, just no, not more polluting of defconfig files with things which really belong in a per SoC file not a per board file. I think we can fix the "make savedefconfig" warnings by doing something like this in a per SoC Kconfig file: if ARCH_SUNXI defconfig SYS_BOOTM_LEN default 0x0f000000 if ARCH_SUNXI endif The second "if ARCH_SUNXI" is necessary to stop the script calling "make savedefconfig" for all boards to clean the defconfig files from complaining. Please let set per SoC defaults this way rather then turning the defconfig files in a mixture of board / soc settings where as they should really only select the SoC and then contain board specific settings on top of that. Regards, Hans