Le 28/01/2014 10:47, Andreas Färber a écrit : > Am 27.01.2014 13:53, schrieb Guillaume Gardet: >> Le 27/01/2014 13:16, Alexander Graf a écrit : >>> On 27.01.2014, at 13:06, Andreas Färber <afaer...@suse.de> wrote: >>> >>>> https://build.opensuse.org/request/show/215263 >>>> >>>> If you can clean up the patch and restore the build so that we can get >>>> it submitted into Factory, I'll be happy. >>> Sorry, I won't get around to anything except for KVM and QEMU patches for >>> the next few days. I'm moving houses tomorrow and will try to squeeze in as >>> many patch reviews as I can in between so that I don't miss the next merge >>> window. >>> >>> Guillaume, do you have some spare time atm? >> Not much time ATM. What is needed here? > https://build.opensuse.org/package/show/Base:System/u-boot-am335xevm > > This is failing with the mlo-ext2.patch (that Alex, Dirk and you all > have rebased in the past), now leading to an SPL too large for SRAM. > > As mentioned earlier, I was able to get that down to 104 bytes by > avoiding a duplicate variable assignment: > > [ 390s] ld.bfd: u-boot-spl section `.u_boot_list' will not fit in > region `.sram' > [ 390s] ld.bfd: region `.sram' overflowed by 104 bytes > > What's needed is to change the patch in whatever way necessary to get > the SPL code size small enough. Alex' suggestion was to replace FAT > support with ext support instead of just adding the latter.
Good idea. We could also disable CONFIG_SPL_OS_BOOT since we do not boot Linux directly. I will give a try. > > A lesser priority once our build is fixed would be to overhaul the patch > in such a way that some CONFIG_SPL_EXT_SUPPORT is used rather than > reusing CONFIG_SPL_FAT_SUPPORT and mentions of hacks for OMAP4 be > dropped, so that this can be submitted upstream and dropped as patch > with the next U-Boot update. Even the environment changes (fatload vs. > ext2load or load) could be upstreamed that way. I agree. We should upstream all our patches when possible. But this patch is not upstreamable without a big clean-up. ;) Guillaume > > Andreas > -- To unsubscribe, e-mail: opensuse-arm+unsubscr...@opensuse.org To contact the owner, e-mail: opensuse-arm+ow...@opensuse.org