netdev - Re: [PATCH] dt-bindings: Remove "status" from schema examples, again

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:   Mon, 17 Oct 2022 09:19:22 +0200
From:   Nicolas Ferre <nicolas.ferre@...rochip.com>
To:     Rob Herring <robh@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Thierry Reding <thierry.reding@...il.com>,
        Jonathan Hunter <jonathanh@...dia.com>,
        "David Airlie" <airlied@...il.com>,
        Daniel Vetter <daniel@...ll.ch>,
        "Lars-Peter Clausen" <lars@...afoo.de>,
        Michael Hennerich <Michael.Hennerich@...log.com>,
        Cosmin Tanislav <cosmin.tanislav@...log.com>,
        Jonathan Cameron <jic23@...nel.org>,
        "David S. Miller" <davem@...emloft.net>,
        Eric Dumazet <edumazet@...gle.com>,
        Jakub Kicinski <kuba@...nel.org>,
        Paolo Abeni <pabeni@...hat.com>,
        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>,
        Nandhini Srikandan <nandhini.srikandan@...el.com>,
        Rashmi A <rashmi.a@...el.com>,
        Kishon Vijay Abraham I <kishon@...com>,
        Vinod Koul <vkoul@...nel.org>, Sumit Gupta <sumitg@...dia.com>,
        Mikko Perttunen <mperttunen@...dia.com>,
        Claudiu Beznea <claudiu.beznea@...rochip.com>,
        Joakim Zhang <qiangqing.zhang@....com>
CC:     <devicetree@...r.kernel.org>, <linux-tegra@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>, <dri-devel@...ts.freedesktop.org>,
        <linux-iio@...r.kernel.org>, <netdev@...r.kernel.org>,
        <linux-arm-kernel@...ts.infradead.org>,
        <linux-phy@...ts.infradead.org>
Subject: Re: [PATCH] dt-bindings: Remove "status" from schema examples, again

On 14/10/2022 at 22:51, Rob Herring wrote:
> There's no reason to have "status" properties in examples. "okay" is the
> default, and "disabled" turns off some schema checks ('required'
> specifically).
> 
> A meta-schema check for this is pending, so hopefully the last time to
> fix these.
> 
> Fix the indentation in intel,phy-thunderbay-emmc while we're here.
> 
> Signed-off-by: Rob Herring <robh@...nel.org>
> ---
>   .../arm/tegra/nvidia,tegra-ccplex-cluster.yaml    |  1 -
>   .../display/tegra/nvidia,tegra124-dpaux.yaml      |  1 -
>   .../display/tegra/nvidia,tegra186-display.yaml    |  2 --
>   .../bindings/iio/addac/adi,ad74413r.yaml          |  1 -
>   .../devicetree/bindings/net/cdns,macb.yaml        |  1 -
>   .../devicetree/bindings/net/nxp,dwmac-imx.yaml    |  1 -
>   .../bindings/phy/intel,phy-thunderbay-emmc.yaml   | 15 +++++++--------
>   7 files changed, 7 insertions(+), 15 deletions(-)

[..]

> diff --git a/Documentation/devicetree/bindings/net/cdns,macb.yaml b/Documentation/devicetree/bindings/net/cdns,macb.yaml
> index 318f4efe7f6f..bef5e0f895be 100644
> --- a/Documentation/devicetree/bindings/net/cdns,macb.yaml
> +++ b/Documentation/devicetree/bindings/net/cdns,macb.yaml
> @@ -203,7 +203,6 @@ examples:
>                       power-domains = <&zynqmp_firmware PD_ETH_1>;
>                       resets = <&zynqmp_reset ZYNQMP_RESET_GEM1>;
>                       reset-names = "gem1_rst";
> -                    status = "okay";
>                       phy-mode = "sgmii";
>                       phys = <&psgtr 1 PHY_TYPE_SGMII 1 1>;
>                       fixed-link {


Acked-by: Nicolas Ferre <nicolas.ferre@...rochip.com>
Thanks Rob. Best regards,
   Nicolas

-- 
Nicolas Ferre