[PATCH 01/17] Convert CONFIG_FSL_FIXED_MMC_LOCATION et al to Kconfig

[Tom Rini]

On Fri, Jul 22, 2022 at 11:01:52AM -0700, Tim Harvey wrote:
> On Mon, Jun 20, 2022 at 5:08 AM Tom Rini <trini at konsulko.com> wrote:
> >
> > This converts the following to Kconfig:
> >    CONFIG_FSL_FIXED_MMC_LOCATION
> >    CONFIG_ESDHC_HC_BLK_ADDR
> >
> > Signed-off-by: Tom Rini <trini at 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20220722/32b5a0bd/attachment.sig>