Re: [PATCH] pinctrl: imx: PINCTRL_IMX_SCMI should depend on ARCH_MXC

[Geert Uytterhoeven]

Hi Peng,

On Thu, 26 Feb 2026 at 11:46, Peng Fan <peng.fan@xxxxxxx> wrote:
> > Subject: [PATCH] pinctrl: imx: PINCTRL_IMX_SCMI should depend on
> >
> > i.MX95 SCMI firmware is only present on NXP i.MX94 and i.MX95 SoCs.
> > Hence add a dependency on ARCH_MXC, to prevent asking the user
> > about this driver when configuring a kernel without NXP i.MX SoC
> > family support.
> >
> > Fixes: b755521fd6eb22ac ("pinctrl: imx: support SCMI pinctrl protocol
> > for i.MX95")
>
> Not sure this deserves a fix tag.

I can drop it if you want.

> > Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>

> > --- a/drivers/pinctrl/freescale/Kconfig
> > +++ b/drivers/pinctrl/freescale/Kconfig
> > @@ -10,6 +10,7 @@ config PINCTRL_IMX
> >  config PINCTRL_IMX_SCMI
> >         tristate "i.MX95 pinctrl driver using SCMI protocol interface"
> >         depends on ARM_SCMI_PROTOCOL && OF
> > +       depends on ARCH_MXC || COMPILE_TEST
>
> How about this?
> depends on (ARM_SCMI_PROTOCOL && OF && ARCH_MXC) || COMPILE_TEST

Sure, these dependencies can indeed be relaxed, as the driver
compiles fine without ARM_SCMI_PROTOCOL or OF.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds