Re: [PATCH v3] arm: imx8m: add boot_device and boot_instance env variables

[Peng Fan <peng.fan@oss.nxp.com>]

On Wed, Feb 11, 2026 at 02:18:28PM +0100, William MARTIN wrote:
>
>Le 11/02/2026 à 12:49, Peng Fan a écrit :
>> Hi William,
>> 
>> Thanks for your patch!
>> 
>> > Subject: [PATCH v3] arm: imx8m: add boot_device and boot_instance
>> > env variables
>> > 
>> > Some SoCs have supports for 'boot_device' and 'boot_instance'
>> > environment variable. Those variables help to choose the right boot
>> > media during the execution of the boot command.
>> > 
>> > The i.MX8M SoC doesn't have support for those variables but provides
>> > internally all the needed information to create those variables.
>> > 
>> > Fill this lack adding support for those variables in the i.MX8M SoC.
>> > 
>> > Signed-off-by: William MARTIN <william.martin@muxen.fr>
>> > ---
>> No change log here.
>I will add in next patch
>> 
>> >   arch/arm/mach-imx/imx8m/soc.c | 72
>> > +++++++++++++++++++++++++++++++++++
>> >   1 file changed, 72 insertions(+)
>> > 
>> > diff --git a/arch/arm/mach-imx/imx8m/soc.c b/arch/arm/mach-
>> > imx/imx8m/soc.c index 1fe083ae94f..498b565183b 100644
>> > --- a/arch/arm/mach-imx/imx8m/soc.c
>> > +++ b/arch/arm/mach-imx/imx8m/soc.c
>> > @@ -1472,6 +1472,76 @@ void reset_cpu(void)  #endif
>> > 
>> >   #if IS_ENABLED(CONFIG_ARCH_MISC_INIT)
>> > +static void setup_boot_mode(void)
>> > +{
>> > +       enum boot_device dev = get_boot_device();
>> > +
>> > +       log_debug("setup_boot_mode = %u\n", dev);
>> > +
>> > +       switch (dev) {
>> > +       case USB_BOOT:
>> > +               env_set("boot_device", "usb");
>> > +               env_set("boot_instance", "0");
>> > +               break;
>> > +
>> > +       case USB2_BOOT:
>> > +               env_set("boot_device", "usb");
>> > +               env_set("boot_instance", "1");
>> > +               break;
>> > +
>> > +       case QSPI_BOOT:
>> > +               env_set("boot_device", "flexspi");
>> i.MX8MQ does not have flexspi. It is qspi.
>> i.MX8MM/N/P has flexspi.
>> 
>> Regards
>> Peng.
>
>Thanks for your feedback.
>
>Does the use of IS_ENABLED(CONFIG_IMX8MQ) to change boot_device from flexspi
>to qspi is enought ?
>

if (IS_ENABLED(CONFIG_IMX8MQ))
	env_set("boot_device", "qspi");
else
	env_set("boot_device", "flexspi");

Regards
Peng