linux-kernel - Re: [PATCH 2/2] arm64: dts: imx: Add i.MX8M Plus Gateworks gw7400 dts support

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] [day] [month] [year] [list]

Message-ID: <7dc8b66d-0569-183a-554c-4a0d273bbf0b@linaro.org>
Date:   Sat, 9 Apr 2022 12:42:53 +0200
From:   Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To:     Tim Harvey <tharvey@...eworks.com>
Cc:     Device Tree Mailing List <devicetree@...r.kernel.org>,
        open list <linux-kernel@...r.kernel.org>,
        Linux ARM Mailing List <linux-arm-kernel@...ts.infradead.org>,
        Shawn Guo <shawnguo@...nel.org>,
        Rob Herring <robh+dt@...nel.org>,
        Fabio Estevam <festevam@...il.com>,
        NXP Linux Team <linux-imx@....com>,
        Krzysztof Kozlowski <krzk+dt@...nel.org>,
        Sascha Hauer <s.hauer@...gutronix.de>,
        Pengutronix Kernel Team <kernel@...gutronix.de>
Subject: Re: [PATCH 2/2] arm64: dts: imx: Add i.MX8M Plus Gateworks gw7400 dts
 support

On 08/04/2022 17:52, Tim Harvey wrote:
>>> +     pps {
>>> +             compatible = "pps-gpio";
>>> +             pinctrl-names = "default";
>>> +             pinctrl-0 = <&pinctrl_pps>;
>>> +             gpios = <&gpio1 12 GPIO_ACTIVE_HIGH>;
>>> +             status = "okay";
>>
>> No need for status.
> 
> ok. What is the best way to know if a binding uses status or not?
> 
> Documentation/devicetree/bindings/net/fsl,fec.yaml for example does
> not mention it uses status, yet it does.

status is not part of binding at all. All nodes have by default
status=okay, just the DTSI disables many of them. Therefore you need to
add status=okay only to these which are explicitly disabled.

Best regards,
Krzysztof