On Thu, Jul 30, 2020 at 12:51:10PM +0200, Stefan Roese 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).
> 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.
>
> Also complete remove bi_memstart & bi_memsize from the board-info
> struct. As now bi_dram[] is always enabled and should be used instead.
> This removes the redundant varriables.
>
> Signed-off-by: Stefan Roese <[email protected]>
> Cc: Daniel Schwierzeck <[email protected]>
> Cc: Tom Rini <[email protected]>
> Cc: Ramon Fried <[email protected]>
> Cc: Simon Glass <[email protected]>
> Cc: Michal Simek <[email protected]>
> Reviewed-by: Daniel Schwierzeck <[email protected]>I don't see quite how, but this is breaking running test/nokia_rx51_test.sh (or, my fixup of this to apply to current master was wrong, and is what's breaking the test). Please rebase and confirm that test passes as well, thanks! -- Tom
signature.asc
Description: PGP signature
