[PATCH] armv8: fsl : fix bootcmd and mcinitcmd default value

Bedel, Alban alban.bedel at aerq.com
Wed Jun 23 15:08:01 CEST 2021


On Wed, 2021-06-16 at 14:19 +0200, Wasim Khan wrote:
> From: Wasim Khan <wasim.khan at nxp.com>
> 
> NXP platforms expect bootcmd and mcinitcmd to be updated
> as per boot source.
> 
> commit cbf77d201870f2d12227e2d95718a416b16ec98b breaks this
> behaviour.
> Revert commit cbf77d201870f2d12227e2d95718a416b16ec98b

As I already explained in the prior exchanges we had, I'm fully
convinced that reverting this patch is not the solution to your
problem. Please see the log of my patch for a full explanation, but
basically the old code not only rely on the a broken assumption, it
also fails to implement it correctly.

The current code set `bootcmd` and `mcinicmd` only if they are not set
which a simple and sane behaviour. When I submitted my patch these
variables were not set in the default so I suspect that another patch
now set these in the default env. Please look for the root cause of the
problem instead of re-enabling known broken code.

Alban



More information about the U-Boot mailing list