Hi Julius, On 7 October 2015 at 04:01, Julius Werner <jwer...@chromium.org> wrote: > Well then... a few hours and a significant reduction in sanity later, > I found that I'm shadowing the new 'ret' variable from changing the > API around because I forgot to remove the declaration part from the > 'ret' that already existed in the else block. It would be nice if > U-Boot compiled with -Wshadow...
I saw an 'unused variable' warning which I mentioned in my email. It may have been another symptom. You could send a patch to change the flags if you like. > > This made the (outer) return variable uninitialized in the compression > error case, which on my native x86_64 happened to produce a non-zero > value... but with an i686 cross-compiler I could reproduce the error > you saw. Fixed in the next version and moved the config to > sandbox_defconfig as you suggested. Seems to work now, so we are good. Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
