Hi,
wrt my other mail "TI mmc env bug?", Nuno sent a proper patch, yet the
commit has the syntax error (missing "="):
https://gitlab.denx.de/u-boot/u-boot/commit/27e0f3bcf07530a9cd272953797efda54ebb8f5e
Regards,
Andre
On 27/08/2019 02:17, Tom Rini wrote:
On Wed, Jul 10, 2019 at 01:46:57PM +0200, Nuno Gonçalves wrote:
Hi,
I found out that my Beaglebone didn't boot after:
https://github.com/u-boot/u-boot/commit/13dd6665ed18f72380ca596931d609bc108d4b82
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
variable.
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?
Thanks,
Nuno
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@lists.denx.de
https://lists.denx.de/listinfo/u-boot
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
