[U-Boot] Regression after "distro: not taint environment variables if possible"
a.heider at gmail.com
Mon Sep 9 10:17:47 UTC 2019
wrt my other mail "TI mmc env bug?", Nuno sent a proper patch, yet the
commit has the syntax error (missing "="):
On 27/08/2019 02:17, Tom Rini wrote:
> On Wed, Jul 10, 2019 at 01:46:57PM +0200, Nuno Gonçalves wrote:
>> I found out that my Beaglebone didn't boot after:
>> I digged out the reason that this patch makes devnum a local variable,
>> and it ends up shadowed by other code that sets devnum as a env
>> For example to boot on the beaglebone I had to remove the setenv as in
>> the patch below.
>> This only fixes for my board. Many other will likely have regressions.
>> Fixing it for all boards in a reliable way I think is very complex,
>> unless the strategy is to wait for board maintainers to fix it as they
>> need it, but I wonder how many latent bugs this will create for corner
>> boot cases.
>> Maybe this change is not worth it?
> I've checked other usages here and only the TI case looks to have been a
> problem. I reworded the commit message as well.
> Applied to u-boot/master, thanks!
> U-Boot mailing list
> U-Boot at lists.denx.de
More information about the U-Boot