Re: [PATCH] dt-bindings: pwm: imx: document i.MX compatibles

[Uwe Kleine-König]

On Fri, Sep 25, 2020 at 11:26:09PM +0200, Krzysztof Kozlowski wrote:
> Document all ARMv5, ARMv6 and ARMv7 i.MX compatibles to fix dtbs_check
> warnings like:
> 
>   arch/arm/boot/dts/imx6dl-colibri-eval-v3.dt.yaml: pwm@2080000: compatible:0:
>     'fsl,imx6q-pwm' is not one of ['fsl,imx8mm-pwm', 'fsl,imx8mn-pwm', 'fsl,imx8mp-pwm', 'fsl,imx8mq-pwm']
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> ---
>  Documentation/devicetree/bindings/pwm/imx-pwm.yaml | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/pwm/imx-pwm.yaml b/Documentation/devicetree/bindings/pwm/imx-pwm.yaml
> index 473863eb67e5..379d693889f6 100644
> --- a/Documentation/devicetree/bindings/pwm/imx-pwm.yaml
> +++ b/Documentation/devicetree/bindings/pwm/imx-pwm.yaml
> @@ -25,6 +25,17 @@ properties:
>            - fsl,imx27-pwm
>        - items:
>            - enum:
> +              - fsl,imx25-pwm

The driver actually used fsl,imx27-pwm to bind ...

Also since v5.1-rc1~38^2~17 the driver is split into pwm-imx27 and
pwm-imx1. So maybe this file should be renamed to fsl,imx27-pwm.yaml?
(And we need a volunteer to write fsl,imx1-pwm.yaml.)

> +              - fsl,imx31-pwm
> +              - fsl,imx50-pwm
> +              - fsl,imx51-pwm
> +              - fsl,imx53-pwm
> +              - fsl,imx6q-pwm
> +              - fsl,imx6sl-pwm
> +              - fsl,imx6sll-pwm
> +              - fsl,imx6sx-pwm
> +              - fsl,imx6ul-pwm
> +              - fsl,imx7d-pwm
>                - fsl,imx8mm-pwm
>                - fsl,imx8mn-pwm
>                - fsl,imx8mp-pwm

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |

Attachment: signature.asc
Description: PGP signature