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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Sat, 26 Sep 2020 15:22:17 +0200
From:   Uwe Kleine-König <u.kleine-koenig@...gutronix.de>
To:     Krzysztof Kozlowski <krzk@...nel.org>
Cc:     Thierry Reding <thierry.reding@...il.com>,
        Lee Jones <lee.jones@...aro.org>,
        Rob Herring <robh+dt@...nel.org>,
        Shawn Guo <shawnguo@...nel.org>,
        Sascha Hauer <s.hauer@...gutronix.de>,
        Pengutronix Kernel Team <kernel@...gutronix.de>,
        Fabio Estevam <festevam@...il.com>,
        NXP Linux Team <linux-imx@....com>,
        Philipp Zabel <p.zabel@...gutronix.de>,
        linux-pwm@...r.kernel.org, devicetree@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] dt-bindings: pwm: imx: document i.MX compatibles

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@...0000: 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@...nel.org>
> ---
>  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/ |

Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)