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

[Peng Fan]

> Subject: Re: [PATCH] pinctrl: imx: PINCTRL_IMX_SCMI should depend
> on ARCH_MXC
> 
> 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.

Yes, please.

> 
> > > 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.

I just give a build with x86_64_defconfig + COMPILE_TEST, no
ARCH_MXC, no SCMI, no OF, it build pass.

Regards
Peng

> 
> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-
> m68k.org
> 
> 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