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

Nicolas Ferre
2022-10-17T07:19:42Z

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@kernel.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@microchip.com>
Thanks Rob. Best regards,
Nicolas

--
Nicolas Ferre