[PATCH v3 01/12] dt-bindings: pinctrl: fsl,imx6ul-pinctrl: Convert i.MX35/5x/6 to YAML

[Marek Vasut]

On 10/22/24 5:25 AM, Shawn Guo wrote:
> On Thu, Oct 17, 2024 at 11:11:18PM +0200, Marek Vasut wrote:
>> The IOMUXC controller description is almost identical on i.MX35/5x/6 SoCs,
>> except for the configuration bits which differ across SoCs. Rename the
>> fsl,imx6ul-pinctrl.yaml to fsl,imx35-pinctrl.yaml, fill in compatible
>> strings for the other SoCs and fill in the various bits into desciption.
>> This way, i.MX35/5x/6 series SoCs can all be converted to DT schema.
>> Remove the old text DT bindings description.
>>
>> Reviewed-by: Rob Herring (Arm) <robh at kernel.org>
>> Signed-off-by: Marek Vasut <marex at denx.de>
>> ---
>> Cc: Alexander Stein <alexander.stein at ew.tq-group.com>
>> Cc: Conor Dooley <conor+dt at kernel.org>
>> Cc: Dong Aisheng <aisheng.dong at nxp.com>
>> Cc: Fabio Estevam <festevam at gmail.com>
>> Cc: Jacky Bai <ping.bai at nxp.com>
>> Cc: Krzysztof Kozlowski <krzk+dt at kernel.org>
>> Cc: Linus Walleij <linus.walleij at linaro.org>
>> Cc: Pengutronix Kernel Team <kernel at pengutronix.de>
>> Cc: Rob Herring <robh at kernel.org>
>> Cc: Sascha Hauer <s.hauer at pengutronix.de>
>> Cc: Shawn Guo <shawnguo at kernel.org>
>> Cc: Stefan Wahren <wahrenst at gmx.net>
>> Cc: devicetree at vger.kernel.org
>> Cc: imx at lists.linux.dev
>> Cc: kernel at dh-electronics.com
>> Cc: linux-arm-kernel at lists.infradead.org
>> Cc: linux-gpio at vger.kernel.org
>> ---
>> V2: - Use DT schema to refer to the fsl,imx35-pinctrl.yaml file
>>      - Special-case the iMX50 compatible string
>> V3: Add RB from Rob
>> ---
>>   .../bindings/pinctrl/fsl,imx35-pinctrl.txt    | 33 -------
>>   ...ul-pinctrl.yaml => fsl,imx35-pinctrl.yaml} | 88 ++++++++++++++++---
>>   .../bindings/pinctrl/fsl,imx50-pinctrl.txt    | 32 -------
>>   .../bindings/pinctrl/fsl,imx51-pinctrl.txt    | 32 -------
>>   .../bindings/pinctrl/fsl,imx53-pinctrl.txt    | 32 -------
>>   .../bindings/pinctrl/fsl,imx6dl-pinctrl.txt   | 38 --------
>>   .../bindings/pinctrl/fsl,imx6q-pinctrl.txt    | 38 --------
>>   .../bindings/pinctrl/fsl,imx6sl-pinctrl.txt   | 39 --------
>>   .../bindings/pinctrl/fsl,imx6sll-pinctrl.txt  | 40 ---------
>>   .../bindings/pinctrl/fsl,imx6sx-pinctrl.txt   | 36 --------
>>   10 files changed, 78 insertions(+), 330 deletions(-)
>>   delete mode 100644 Documentation/devicetree/bindings/pinctrl/fsl,imx35-pinctrl.txt
>>   rename Documentation/devicetree/bindings/pinctrl/{fsl,imx6ul-pinctrl.yaml => fsl,imx35-pinctrl.yaml} (50%)
>>   delete mode 100644 Documentation/devicetree/bindings/pinctrl/fsl,imx50-pinctrl.txt
>>   delete mode 100644 Documentation/devicetree/bindings/pinctrl/fsl,imx51-pinctrl.txt
>>   delete mode 100644 Documentation/devicetree/bindings/pinctrl/fsl,imx53-pinctrl.txt
>>   delete mode 100644 Documentation/devicetree/bindings/pinctrl/fsl,imx6dl-pinctrl.txt
>>   delete mode 100644 Documentation/devicetree/bindings/pinctrl/fsl,imx6q-pinctrl.txt
>>   delete mode 100644 Documentation/devicetree/bindings/pinctrl/fsl,imx6sl-pinctrl.txt
>>   delete mode 100644 Documentation/devicetree/bindings/pinctrl/fsl,imx6sll-pinctrl.txt
>>   delete mode 100644 Documentation/devicetree/bindings/pinctrl/fsl,imx6sx-pinctrl.txt
> 
> I'm not sure it makes a lot sense to have "fsl,imx6ul-pinctrl: " in the
> subject prefix.
I can change it to imx-pinctrl or something and send V4 , or can you 
tweak it while applying since the series is somewhat large ? Which do 
you prefer ?