On Fri, Jul 22, 2022 at 11:01:52AM -0700, Tim Harvey wrote: > On Mon, Jun 20, 2022 at 5:08 AM Tom Rini <tr...@konsulko.com> wrote: > > > > This converts the following to Kconfig: > > CONFIG_FSL_FIXED_MMC_LOCATION > > CONFIG_ESDHC_HC_BLK_ADDR > > > > Signed-off-by: Tom Rini <tr...@konsulko.com> [snip] > Tom, > > I'm not sure this patch had the intended results. I just noticed that > default bootcmd was changed due to this for the the imx8m*venice > boards I maintain. > > How did you go about generating this? In my case > CONFIG_USE_BOOTCOMMAND is not set and prior to this patch was getting > defaulted in include/config_distro_bootcmd.h but now that is removed > and its getting set in tools/env/fw_env_private.h to something > different. > > To fix this, I'll have to submit a patch that does the following for > configs/imx8m*_venice_defconfig > +CONFIG_USE_BOOTCOMMAND=y > +CONFIG_BOOTCOMMAND="run distro_bootcmd" > > I figured it was worth mentioning and asking for others that run into this.
It's this commit, specifically? Looking at u-boot-initial-env (and disabling LTO for the moment), I don't see bootcmd set before nor after. I could see this being a merge related problem or similar from 970bf8603b877e2b66170290f751f9c23c120838 where I finished moving everyone that was in-tree at the time that should have been setting CONFIG_USE_BOOTCOMMAND and then CONFIG_BOOTCOMMAND (if not using the default) leading to this kind of problem. -- Tom
signature.asc
Description: PGP signature