[U-Boot] BUG num banks 8 exceeds hardcoded limit 4

[Anand Moon]

Hi Lukasz and Ramon.

On Sun, 12 Aug 2018 at 13:21, Ramon Fried <ramon.fried at gmail.com> wrote:
>
> Adding Simon.
>
> On Sat, Aug 11, 2018 at 2:54 PM Lukasz Majewski <lukma at denx.de> wrote:
> >
> > Hi Anand,
> >
> > > Hi Lukasz / Minkyu,
> > >
> > > Today I wanted to test the latest u-boot on my Odroid-XU4
> > > and I landed onto this bug and it failed to boot up.
> > >
> > > include/configs/odroid.h:#define CONFIG_NR_DRAM_BANKS   8
> > > include/configs/odroid_xu3.h:#define CONFIG_NR_DRAM_BANKS       8
> > >
> > > seem to the the issue.
>
> The issue is my new patch:
> https://patchwork.ozlabs.org/patch/947431/
> Temporarily you adjust MEMORY_BANKS_MAX in common/fdt_support.c
> I have a suggestion on how to fix it for all boards, waiting for Simon
> to respond.
> Thanks,
> Ramon.
>

This issue is related to number of sub nodes in memory node.
Odroid XU3 / XU4

https://github.com/u-boot/u-boot/blob/master/arch/arm/dts/exynos5422-odroidxu3.dts#L21-L31

But this can be fix by using mainline memory node.
https://github.com/torvalds/linux/blob/master/arch/arm/boot/dts/exynos5422-odroid-core.dtsi#L17-L20

> >
> >
> >
> > Are you sure that you ran the code recently?
> >
> > According to git blame:
> > 973ae1e0858d3 (Przemyslaw Marczak 2015-02-17 14:50:26 +0100  22)
> > #define CONFIG_NR_DRAM_BANKS   8
> >
> > This change is quite old...
> >
> > Which u-boot version have you ran till now?
> >
I am using the mainline u-boot for now to test new feature.

Best Regards
-Anand