On Wed, Nov 13, 2019 at 11:40:37AM +0100, Uwe Kleine-König wrote: > On Wed, Nov 13, 2019 at 11:27:29AM +0100, Geert Uytterhoeven wrote: > > The RZA2MEVB sub board has 64 MiB of SDRAM at 0x0C000000 (CS3 space). > > This wording is misleading. You don't adapt the limit because 128 MiB > > the amount of RAM on that platform but because the alignment of the CS3 > space isn't a multiple of 128 MiB. So I suggest: > > On the RZA2MEVB the SDRAM's base is at 0x0C000000 which isn't > aligned to 128 MiB. So to ensure the assumptions of the > decompressor are valid the used alignment must be decreased. > > > Hence the mask for CONFIG_AUTO_ZRELADDR needs to be changed, otherwise > > the system will crash because it will try to decompress a zImage or > > uImage to a non-RAM garbage address. > > > > Based on a patch in the BSP by Chris Brandt <chris.brandt@xxxxxxxxxxx>. > > > > Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > > --- > > No idea what to do with the rest of the comment, or if this breaks > > existing platforms. > > I would assume that it indeed breaks existing platforms. So maybe better > make this configurable, default to 128 MiB and select it to 64 MiB on > the affected platform? If the resulting kernel supports also other > machines a warning (at compile time) might be a good idea. Are we going back to non-multi-platform kernels? ;) -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up According to speedtest.net: 11.9Mbps down 500kbps up
