Received: by 2002:a05:6a10:1287:0:0:0:0 with SMTP id d7csp4799309pxv; Tue, 20 Jul 2021 11:43:12 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwrPKwNB5l80zK84/1zIQDnKGcoK/YC2ir8L2RXElmzk0iqI1WHIMnZQmtoEw6aUdESDyeG X-Received: by 2002:a05:6402:c9:: with SMTP id i9mr43062104edu.48.1626806592230; Tue, 20 Jul 2021 11:43:12 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1626806592; cv=none; d=google.com; s=arc-20160816; b=hpeg/lFwuGJ85vRzcowwPPEl5dhyOiiNF82kSH0PxCba8+Oh1dV3cnXbAaLPG3UdvC 0FemK3tmBYzQcMUvaCrcEBt2l0iFbLjAi3V+e/IBuoMfAE+7vssxYhJa4grSTJuz2CsL RfoJxRyhQsqohpkEnzSZRDcBDs/2sRX33ld3uR8Yp5oSy5l94MF0Cl3rHrBzyYWqx/sY CX9Ig5PjeA/9CvgqN6WKjKCiHNBhgzpim9OQ65ysFeCp5PtoOT6o/YY7KFarF32b3nQD /z6I58rdZUcRaP9n8wuIVMJfchx03NSftF05XYY91cSBllxxV+ebtUWHGcZ3lORHU1rr ffhQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date; bh=WAWZOQKlToXjJ4zyi91afnQgk5vgzMYMYz26aZy8oT0=; b=RZCOunaLkWInPNg8Z6rIBCq448ryQoldztfwJKJlREYJx27LBC59kGyZOFVH62KwWU 0uWPeBEJB3MKyl3pVGBg/tixesQ67P4AJPtGJc1j7snevCVpfSMFZYI1zRz03UDQUPAV 5MEvlTKA/HfoYeOiU2z/sRcw/TFoLAlTx1iVs8J1lljN5sq6bewIK70bx3FtkP2aUVBG oWaTpCcj6++6fslWPF5JCZZexeDVYB4nHVXK3j9Xv3/tpSZuSwI9bL04glz9wSSnpff5 T3LlN6DlfbJc0owgjG8PTmQwC6K6k2l79FvGukdsbjnP5d/B0NEhNDIWQtna5xHL7B0l tx+g== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id rh15si13840172ejb.613.2021.07.20.11.42.47; Tue, 20 Jul 2021 11:43:12 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229818AbhGTSAn (ORCPT + 99 others); Tue, 20 Jul 2021 14:00:43 -0400 Received: from relay7-d.mail.gandi.net ([217.70.183.200]:51467 "EHLO relay7-d.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231452AbhGTSAR (ORCPT ); Tue, 20 Jul 2021 14:00:17 -0400 Received: (Authenticated sender: alexandre.belloni@bootlin.com) by relay7-d.mail.gandi.net (Postfix) with ESMTPSA id BA49E20004; Tue, 20 Jul 2021 18:40:42 +0000 (UTC) Date: Tue, 20 Jul 2021 20:40:42 +0200 From: Alexandre Belloni To: Rob Herring Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Maxime Ripard , Chen-Yu Tsai , Thierry Reding , Sam Ravnborg , Rui Miguel Silva , Laurent Pinchart , Mauro Carvalho Chehab , "David S. Miller" , Jakub Kicinski , Mark Brown , Robert Marko , Philipp Zabel , Alessandro Zummo , Ramesh Shanmugasundaram , "G. Jaya Kumaran" , Linus Walleij , Oleksij Rempel , ChiYuan Huang , Wei Xu , Dilip Kota , Karol Gugala , Mateusz Holenko , Olivier Moysan , Peter Ujfalusi , dri-devel@lists.freedesktop.org, linux-media@vger.kernel.org, netdev@vger.kernel.org, linux-rtc@vger.kernel.org, alsa-devel@alsa-project.org Subject: Re: [PATCH] dt-bindings: Remove "status" from schema examples Message-ID: References: <20210720172025.363238-1-robh@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210720172025.363238-1-robh@kernel.org> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 20/07/2021 11:20:25-0600, 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). > > Enabling qca,ar71xx causes a warning, so let's fix the node names: > > Documentation/devicetree/bindings/net/qca,ar71xx.example.dt.yaml: phy@3: '#phy-cells' is a required property > From schema: schemas/phy/phy-provider.yaml > > Cc: Maxime Ripard > Cc: Chen-Yu Tsai > Cc: Thierry Reding > Cc: Sam Ravnborg > Cc: Rui Miguel Silva > Cc: Laurent Pinchart > Cc: Mauro Carvalho Chehab > Cc: "David S. Miller" > Cc: Jakub Kicinski > Cc: Mark Brown > Cc: Robert Marko > Cc: Philipp Zabel > Cc: Alessandro Zummo > Cc: Alexandre Belloni > Cc: Ramesh Shanmugasundaram > Cc: "G. Jaya Kumaran" > Cc: Linus Walleij > Cc: Oleksij Rempel > Cc: ChiYuan Huang > Cc: Wei Xu > Cc: Dilip Kota > Cc: Karol Gugala > Cc: Mateusz Holenko > Cc: Olivier Moysan > Cc: Peter Ujfalusi > Cc: dri-devel@lists.freedesktop.org > Cc: linux-media@vger.kernel.org > Cc: netdev@vger.kernel.org > Cc: linux-rtc@vger.kernel.org > Cc: alsa-devel@alsa-project.org > Signed-off-by: Rob Herring Acked-by: Alexandre Belloni > --- > .../display/allwinner,sun8i-a83t-dw-hdmi.yaml | 2 -- > .../display/panel/boe,tv101wum-nl6.yaml | 1 - > .../bindings/media/nxp,imx7-mipi-csi2.yaml | 2 -- > .../bindings/media/renesas,drif.yaml | 1 - > .../bindings/net/intel,dwmac-plat.yaml | 2 -- > .../bindings/net/intel,ixp4xx-ethernet.yaml | 2 -- > .../bindings/net/nfc/samsung,s3fwrn5.yaml | 3 --- > .../devicetree/bindings/net/qca,ar71xx.yaml | 25 ++++--------------- > .../regulator/richtek,rt6245-regulator.yaml | 1 - > .../regulator/vqmmc-ipq4019-regulator.yaml | 1 - > .../reset/hisilicon,hi3660-reset.yaml | 1 - > .../bindings/reset/intel,rcu-gw.yaml | 1 - > .../bindings/rtc/microcrystal,rv3032.yaml | 1 - > .../soc/litex/litex,soc-controller.yaml | 1 - > .../bindings/sound/st,stm32-sai.yaml | 2 -- > .../bindings/sound/ti,j721e-cpb-audio.yaml | 2 -- > .../sound/ti,j721e-cpb-ivi-audio.yaml | 2 -- > 17 files changed, 5 insertions(+), 45 deletions(-) > > diff --git a/Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-dw-hdmi.yaml b/Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-dw-hdmi.yaml > index 5d42d36608d9..4951b5ef5c6a 100644 > --- a/Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-dw-hdmi.yaml > +++ b/Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-dw-hdmi.yaml > @@ -174,7 +174,6 @@ examples: > phy-names = "phy"; > pinctrl-names = "default"; > pinctrl-0 = <&hdmi_pins>; > - status = "disabled"; > > ports { > #address-cells = <1>; > @@ -233,7 +232,6 @@ examples: > phy-names = "phy"; > pinctrl-names = "default"; > pinctrl-0 = <&hdmi_pins>; > - status = "disabled"; > > ports { > #address-cells = <1>; > diff --git a/Documentation/devicetree/bindings/display/panel/boe,tv101wum-nl6.yaml b/Documentation/devicetree/bindings/display/panel/boe,tv101wum-nl6.yaml > index 38bc1d1b511e..b87a2e28c866 100644 > --- a/Documentation/devicetree/bindings/display/panel/boe,tv101wum-nl6.yaml > +++ b/Documentation/devicetree/bindings/display/panel/boe,tv101wum-nl6.yaml > @@ -70,7 +70,6 @@ examples: > avee-supply = <&ppvarp_lcd>; > pp1800-supply = <&pp1800_lcd>; > backlight = <&backlight_lcd0>; > - status = "okay"; > port { > panel_in: endpoint { > remote-endpoint = <&dsi_out>; > diff --git a/Documentation/devicetree/bindings/media/nxp,imx7-mipi-csi2.yaml b/Documentation/devicetree/bindings/media/nxp,imx7-mipi-csi2.yaml > index 7c09eec78ce5..877183cf4278 100644 > --- a/Documentation/devicetree/bindings/media/nxp,imx7-mipi-csi2.yaml > +++ b/Documentation/devicetree/bindings/media/nxp,imx7-mipi-csi2.yaml > @@ -200,8 +200,6 @@ examples: > clock-names = "pclk", "wrap", "phy", "axi"; > power-domains = <&mipi_pd>; > > - status = "disabled"; > - > ports { > #address-cells = <1>; > #size-cells = <0>; > diff --git a/Documentation/devicetree/bindings/media/renesas,drif.yaml b/Documentation/devicetree/bindings/media/renesas,drif.yaml > index 2867d11fe156..9403b235e976 100644 > --- a/Documentation/devicetree/bindings/media/renesas,drif.yaml > +++ b/Documentation/devicetree/bindings/media/renesas,drif.yaml > @@ -242,7 +242,6 @@ examples: > power-domains = <&sysc R8A7795_PD_ALWAYS_ON>; > resets = <&cpg 513>; > renesas,bonding = <&drif11>; > - status = "disabled"; > }; > > drif11: rif@e6f70000 { > diff --git a/Documentation/devicetree/bindings/net/intel,dwmac-plat.yaml b/Documentation/devicetree/bindings/net/intel,dwmac-plat.yaml > index c1948ce00081..79fa04f5e40d 100644 > --- a/Documentation/devicetree/bindings/net/intel,dwmac-plat.yaml > +++ b/Documentation/devicetree/bindings/net/intel,dwmac-plat.yaml > @@ -116,8 +116,6 @@ examples: > snps,mtl-rx-config = <&mtl_rx_setup>; > snps,mtl-tx-config = <&mtl_tx_setup>; > snps,tso; > - status = "okay"; > - > mdio0 { > #address-cells = <1>; > #size-cells = <0>; > diff --git a/Documentation/devicetree/bindings/net/intel,ixp4xx-ethernet.yaml b/Documentation/devicetree/bindings/net/intel,ixp4xx-ethernet.yaml > index f2e91d1bf7d7..378ed2d3b003 100644 > --- a/Documentation/devicetree/bindings/net/intel,ixp4xx-ethernet.yaml > +++ b/Documentation/devicetree/bindings/net/intel,ixp4xx-ethernet.yaml > @@ -71,7 +71,6 @@ examples: > ethernet@c8009000 { > compatible = "intel,ixp4xx-ethernet"; > reg = <0xc8009000 0x1000>; > - status = "disabled"; > queue-rx = <&qmgr 4>; > queue-txready = <&qmgr 21>; > intel,npe-handle = <&npe 1>; > @@ -82,7 +81,6 @@ examples: > ethernet@c800c000 { > compatible = "intel,ixp4xx-ethernet"; > reg = <0xc800c000 0x1000>; > - status = "disabled"; > queue-rx = <&qmgr 3>; > queue-txready = <&qmgr 20>; > intel,npe-handle = <&npe 2>; > diff --git a/Documentation/devicetree/bindings/net/nfc/samsung,s3fwrn5.yaml b/Documentation/devicetree/bindings/net/nfc/samsung,s3fwrn5.yaml > index 081742c2b726..64995cbb0f97 100644 > --- a/Documentation/devicetree/bindings/net/nfc/samsung,s3fwrn5.yaml > +++ b/Documentation/devicetree/bindings/net/nfc/samsung,s3fwrn5.yaml > @@ -90,14 +90,11 @@ examples: > # UART example on Raspberry Pi > - | > uart0 { > - status = "okay"; > - > nfc { > compatible = "samsung,s3fwrn82"; > > en-gpios = <&gpio 20 GPIO_ACTIVE_HIGH>; > wake-gpios = <&gpio 16 GPIO_ACTIVE_HIGH>; > > - status = "okay"; > }; > }; > diff --git a/Documentation/devicetree/bindings/net/qca,ar71xx.yaml b/Documentation/devicetree/bindings/net/qca,ar71xx.yaml > index f0db22645d73..cf4d35edaa1b 100644 > --- a/Documentation/devicetree/bindings/net/qca,ar71xx.yaml > +++ b/Documentation/devicetree/bindings/net/qca,ar71xx.yaml > @@ -101,8 +101,6 @@ examples: > > phy-mode = "gmii"; > > - status = "disabled"; > - > fixed-link { > speed = <1000>; > full-duplex; > @@ -148,32 +146,24 @@ examples: > reg = <0x1>; > phy-handle = <&phy_port0>; > phy-mode = "internal"; > - > - status = "disabled"; > }; > > switch_port2: port@2 { > reg = <0x2>; > phy-handle = <&phy_port1>; > phy-mode = "internal"; > - > - status = "disabled"; > }; > > switch_port3: port@3 { > reg = <0x3>; > phy-handle = <&phy_port2>; > phy-mode = "internal"; > - > - status = "disabled"; > }; > > switch_port4: port@4 { > reg = <0x4>; > phy-handle = <&phy_port3>; > phy-mode = "internal"; > - > - status = "disabled"; > }; > }; > > @@ -183,34 +173,29 @@ examples: > > interrupt-parent = <&switch10>; > > - phy_port0: phy@0 { > + phy_port0: ethernet-phy@0 { > reg = <0x0>; > interrupts = <0>; > - status = "disabled"; > }; > > - phy_port1: phy@1 { > + phy_port1: ethernet-phy@1 { > reg = <0x1>; > interrupts = <0>; > - status = "disabled"; > }; > > - phy_port2: phy@2 { > + phy_port2: ethernet-phy@2 { > reg = <0x2>; > interrupts = <0>; > - status = "disabled"; > }; > > - phy_port3: phy@3 { > + phy_port3: ethernet-phy@3 { > reg = <0x3>; > interrupts = <0>; > - status = "disabled"; > }; > > - phy_port4: phy@4 { > + phy_port4: ethernet-phy@4 { > reg = <0x4>; > interrupts = <0>; > - status = "disabled"; > }; > }; > }; > diff --git a/Documentation/devicetree/bindings/regulator/richtek,rt6245-regulator.yaml b/Documentation/devicetree/bindings/regulator/richtek,rt6245-regulator.yaml > index 796ceac87445..e983d0e70c9b 100644 > --- a/Documentation/devicetree/bindings/regulator/richtek,rt6245-regulator.yaml > +++ b/Documentation/devicetree/bindings/regulator/richtek,rt6245-regulator.yaml > @@ -77,7 +77,6 @@ examples: > > rt6245@34 { > compatible = "richtek,rt6245"; > - status = "okay"; > reg = <0x34>; > enable-gpios = <&gpio26 2 0>; > > diff --git a/Documentation/devicetree/bindings/regulator/vqmmc-ipq4019-regulator.yaml b/Documentation/devicetree/bindings/regulator/vqmmc-ipq4019-regulator.yaml > index 6f45582c914e..dd7a2f92634c 100644 > --- a/Documentation/devicetree/bindings/regulator/vqmmc-ipq4019-regulator.yaml > +++ b/Documentation/devicetree/bindings/regulator/vqmmc-ipq4019-regulator.yaml > @@ -39,6 +39,5 @@ examples: > regulator-min-microvolt = <1500000>; > regulator-max-microvolt = <3000000>; > regulator-always-on; > - status = "disabled"; > }; > ... > diff --git a/Documentation/devicetree/bindings/reset/hisilicon,hi3660-reset.yaml b/Documentation/devicetree/bindings/reset/hisilicon,hi3660-reset.yaml > index 9bf40952e5b7..b0c41ab1a746 100644 > --- a/Documentation/devicetree/bindings/reset/hisilicon,hi3660-reset.yaml > +++ b/Documentation/devicetree/bindings/reset/hisilicon,hi3660-reset.yaml > @@ -72,6 +72,5 @@ examples: > resets = <&iomcu_rst 0x20 3>; > pinctrl-names = "default"; > pinctrl-0 = <&i2c0_pmx_func &i2c0_cfg_func>; > - status = "disabled"; > }; > ... > diff --git a/Documentation/devicetree/bindings/reset/intel,rcu-gw.yaml b/Documentation/devicetree/bindings/reset/intel,rcu-gw.yaml > index 6b2d56cc3f38..13bf6bb3f097 100644 > --- a/Documentation/devicetree/bindings/reset/intel,rcu-gw.yaml > +++ b/Documentation/devicetree/bindings/reset/intel,rcu-gw.yaml > @@ -57,7 +57,6 @@ examples: > }; > > pwm: pwm@e0d00000 { > - status = "disabled"; > compatible = "intel,lgm-pwm"; > reg = <0xe0d00000 0x30>; > clocks = <&cgu0 1>; > diff --git a/Documentation/devicetree/bindings/rtc/microcrystal,rv3032.yaml b/Documentation/devicetree/bindings/rtc/microcrystal,rv3032.yaml > index a2c55303810d..9593840a4a2b 100644 > --- a/Documentation/devicetree/bindings/rtc/microcrystal,rv3032.yaml > +++ b/Documentation/devicetree/bindings/rtc/microcrystal,rv3032.yaml > @@ -53,7 +53,6 @@ examples: > rtc@51 { > compatible = "microcrystal,rv3032"; > reg = <0x51>; > - status = "okay"; > pinctrl-0 = <&rtc_nint_pins>; > interrupts-extended = <&gpio1 16 IRQ_TYPE_LEVEL_HIGH>; > trickle-resistor-ohms = <7000>; > diff --git a/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml b/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml > index c8b57c7fd08c..ecae9fa8561b 100644 > --- a/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml > +++ b/Documentation/devicetree/bindings/soc/litex/litex,soc-controller.yaml > @@ -35,7 +35,6 @@ examples: > soc_ctrl0: soc-controller@f0000000 { > compatible = "litex,soc-controller"; > reg = <0xf0000000 0xc>; > - status = "okay"; > }; > > ... > diff --git a/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml b/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml > index 06e83461705c..f97132400bb6 100644 > --- a/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml > +++ b/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml > @@ -180,7 +180,6 @@ examples: > pinctrl-names = "default", "sleep"; > pinctrl-0 = <&sai2a_pins_a>, <&sai2b_pins_b>; > pinctrl-1 = <&sai2a_sleep_pins_a>, <&sai2b_sleep_pins_b>; > - status = "okay"; > > sai2a: audio-controller@4400b004 { > #sound-dai-cells = <0>; > @@ -190,7 +189,6 @@ examples: > dma-names = "tx"; > clocks = <&rcc SAI2_K>; > clock-names = "sai_ck"; > - status = "okay"; > }; > }; > > diff --git a/Documentation/devicetree/bindings/sound/ti,j721e-cpb-audio.yaml b/Documentation/devicetree/bindings/sound/ti,j721e-cpb-audio.yaml > index ec06789b21df..6806f53a4aed 100644 > --- a/Documentation/devicetree/bindings/sound/ti,j721e-cpb-audio.yaml > +++ b/Documentation/devicetree/bindings/sound/ti,j721e-cpb-audio.yaml > @@ -127,8 +127,6 @@ examples: > compatible = "ti,j721e-cpb-audio"; > model = "j721e-cpb"; > > - status = "okay"; > - > ti,cpb-mcasp = <&mcasp10>; > ti,cpb-codec = <&pcm3168a_1>; > > diff --git a/Documentation/devicetree/bindings/sound/ti,j721e-cpb-ivi-audio.yaml b/Documentation/devicetree/bindings/sound/ti,j721e-cpb-ivi-audio.yaml > index ee9f960de36b..859d369c71e2 100644 > --- a/Documentation/devicetree/bindings/sound/ti,j721e-cpb-ivi-audio.yaml > +++ b/Documentation/devicetree/bindings/sound/ti,j721e-cpb-ivi-audio.yaml > @@ -119,8 +119,6 @@ examples: > compatible = "ti,j721e-cpb-ivi-audio"; > model = "j721e-cpb-ivi"; > > - status = "okay"; > - > ti,cpb-mcasp = <&mcasp10>; > ti,cpb-codec = <&pcm3168a_1>; > > -- > 2.27.0 > -- Alexandre Belloni, co-owner and COO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com