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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Rob,

Thank you for the patch.

On Tue, Jul 20, 2021 at 11:20:25AM -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 <mripard@xxxxxxxxxx>
> Cc: Chen-Yu Tsai <wens@xxxxxxxx>
> Cc: Thierry Reding <thierry.reding@xxxxxxxxx>
> Cc: Sam Ravnborg <sam@xxxxxxxxxxxx>
> Cc: Rui Miguel Silva <rmfrfs@xxxxxxxxx>
> Cc: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
> Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>
> Cc: "David S. Miller" <davem@xxxxxxxxxxxxx>
> Cc: Jakub Kicinski <kuba@xxxxxxxxxx>
> Cc: Mark Brown <broonie@xxxxxxxxxx>
> Cc: Robert Marko <robert.marko@xxxxxxxxxx>
> Cc: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx>
> Cc: Alessandro Zummo <a.zummo@xxxxxxxxxxxx>
> Cc: Alexandre Belloni <alexandre.belloni@xxxxxxxxxxx>
> Cc: Ramesh Shanmugasundaram <rashanmu@xxxxxxxxx>
> Cc: "G. Jaya Kumaran" <vineetha.g.jaya.kumaran@xxxxxxxxx>
> Cc: Linus Walleij <linus.walleij@xxxxxxxxxx>
> Cc: Oleksij Rempel <o.rempel@xxxxxxxxxxxxxx>
> Cc: ChiYuan Huang <cy_huang@xxxxxxxxxxx>
> Cc: Wei Xu <xuwei5@xxxxxxxxxxxxx>
> Cc: Dilip Kota <eswara.kota@xxxxxxxxxxxxxxx>
> Cc: Karol Gugala <kgugala@xxxxxxxxxxxx>
> Cc: Mateusz Holenko <mholenko@xxxxxxxxxxxx>
> Cc: Olivier Moysan <olivier.moysan@xxxxxx>
> Cc: Peter Ujfalusi <peter.ujfalusi@xxxxxx>
> Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx
> Cc: linux-media@xxxxxxxxxxxxxxx
> Cc: netdev@xxxxxxxxxxxxxxx
> Cc: linux-rtc@xxxxxxxxxxxxxxx
> Cc: alsa-devel@xxxxxxxxxxxxxxxx
> Signed-off-by: Rob Herring <robh@xxxxxxxxxx>
> ---
>  .../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";
> -

I would have kept the blank line here.

Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>

>          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>;
>  

-- 
Regards,

Laurent Pinchart



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux