On Tue, Aug 18, 2020 at 12:31:32PM +0200, Wolfgang Denk wrote:
> Dear Stefan Roese,
>
> In message <20200813054800.469284-2...@denx.de> you wrote:
> > Since commit 86cf1c82850f ("configs: Migrate CONFIG_NR_DRAM_BANKS") &
> > commit 999a772d9f24 ("Kconfig: Migrate CONFIG_NR_DRAM_BANKS"),
> > CONFIG_NR_DRAM_BANKS is always defined with a value (4 is default).
>
> Has there been any evaluation about the impact this change had on
> both text and data sizes of the resulting U-Boot image?
> Especially the default value of 4 makes no sense to me - whiy is
> this not 1?The default value is 4 because that was the most commonly used value. Annoyingly enough, I don't have those logs around (either I did them on Google Compute, or it was on a build box we reinstalled and I didn't save the logs off of) but I'm fairly sure there wasn't a size change, that's one of my tests for these migrations (that does sometimes fail to catch things, I need to play with getting the data out of buildman in something I can filter more easily). -- Tom
signature.asc
Description: PGP signature
