[PATCH v4 1/9] CONFIG_NR_DRAM_BANKS: Remove unreferenced code as its always defined
Stefan Roese
sr at denx.de
Tue Aug 18 13:33:07 CEST 2020
Hi Wolfgang,
(added Ramon to Cc)
On 18.08.20 12:31, Wolfgang Denk wrote:
> In message <20200813054800.469284-2-sr at 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?
Not sure. Ramon and / or Tom might comment on this. Even though the
git history shows that these Kconfig migration patches have been
committed ~2 years ago, so their memory might be foggy ;)
commit 86cf1c82850f7c226f23684e19616e526ffaf10f
Author: Tom Rini <trini at konsulko.com>
Date: Thu Aug 16 08:16:24 2018 -0400
configs: Migrate CONFIG_NR_DRAM_BANKS
We have the following cases:
- CONFIG_NR_DRAM_BANKS was defined, migrate normally
- CONFIG_NR_DRAM_BANKS_MAX was defined and then used for
CONFIG_NR_DRAM_BANKS after a check, just migrate it over now.
- CONFIG_NR_DRAM_BANKS was very oddly defined on p2771-0000-* (to
1024 +
2), set this to 8.
Signed-off-by: Tom Rini <trini at konsulko.com>
commit 999a772d9f24bf9b8d0726e1359c9d8c3bdad72e
Author: Ramon Fried <ramon.fried at gmail.com>
Date: Tue Aug 14 01:00:04 2018 +0300
Kconfig: Migrate CONFIG_NR_DRAM_BANKS
Move CONFIG_NR_DRAM_BANKS from headers to Kconfig.
Signed-off-by: Ramon Fried <ramon.fried at gmail.com>
> Especially the default value of 4 makes no sense to me - whiy is
> this not 1?
I can't really tell. I can only assume, that it originates from this
patch:
commit 6b6f216f9234c33881af05116057c902cb411a62
Author: Ramon Fried <ramon.fried at gmail.com>
Date: Tue Aug 14 00:35:42 2018 +0300
fdt_support: Use CONFIG_NR_DRAM_BANKS if necessary
If CONFIG_NR_DRAM_BANKS is bigger than the default
value (4) define MEMORY_BANKS_MAX as CONFIG_NR_DRAM_BANKS.
Fixes: 2a1f4f1758b5 ("Revert "fdt_support: Use CONFIG_NR_DRAM_BANKS
if defined"")
Signed-off-by: Ramon Fried <ramon.fried at gmail.com>
>> It makes no sense to still carry code that is guarded with
>> "#ifndef CONFIG_NR_DRAM_BANKS" (and similar). This patch removes
>> all these unreferenced code paths.
>
> ...thus futher hiding where we just lost another lof of memory, for
> no advantage.
>
> Sic...
I'm not sure, what you mean with "lost lot of memory"? We carried
referenced code for ~2 years, which this patch series now tries to
solve.
Thanks,
Stefan
More information about the U-Boot
mailing list