> Subject: Re: [PATCH 1/2] dt-bindings: arm: fsl: Update i.MX95 > compatible > > On 04/01/2025 13:13, Peng Fan (OSS) wrote: > > From: Peng Fan <peng.fan@xxxxxxx> > > > > i.MX95 features a System Controller and SCMI Spec 3.2 compatible > > firmware System Manager(SM) runs on the controller. > > Add "fsl,imx-sm" compatible string as fallback for "fsl,imx95" to > > indicate it is compatible with i.MX System Manager. > > I see little value in generic compatible like that. All these are > aarch64 so why not adding that compatible? > > How this generic compatible would be used? > > And by what exactly? There will be more i.MX9 chips with System Manager. I would not expand the list here each time to support a new SoC. https://elixir.bootlin.com/linux/v6.13-rc3/source/drivers/pinctrl/pinctrl-scmi.c#L508 https://elixir.bootlin.com/linux/v6.13-rc3/source/drivers/pinctrl/freescale/pinctrl-imx-scmi.c#L290 > > All this must be explained in the commit msg. If upper is ok, I will include in V2. Thanks, Peng. > > Best regards, > Krzysztof
