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

From: Nicolas Ferre <nicolas.ferre@microchip.com>
To: Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Thierry Reding <thierry.reding@gmail.com>,
	Jonathan Hunter <jonathanh@nvidia.com>,
	"David Airlie" <airlied@gmail.com>,
	Daniel Vetter <daniel@ffwll.ch>,
	"Lars-Peter Clausen" <lars@metafoo.de>,
	Michael Hennerich <Michael.Hennerich@analog.com>,
	Cosmin Tanislav <cosmin.tanislav@analog.com>,
	Jonathan Cameron <jic23@kernel.org>,
	"David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
	Shawn Guo <shawnguo@kernel.org>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	Fabio Estevam <festevam@gmail.com>,
	NXP Linux Team <linux-imx@nxp.com>,
	Nandhini Srikandan <nandhini.srikandan@intel.com>,
	Rashmi A <rashmi.a@intel.com>,
	Kishon Vijay Abraham I <kishon@ti.com>,
	Vinod Koul <vkoul@kernel.org>, Sumit Gupta <sumitg@nvidia.com>,
	Mikko Perttunen <mperttunen@nvidia.com>,
	Claudiu Beznea <claudiu.beznea@microchip.com>,
	Joakim Zhang <qiangqing.zhang@nxp.com>
Cc: <devicetree@vger.kernel.org>, <linux-tegra@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>, <dri-devel@lists.freedesktop.org>,
	<linux-iio@vger.kernel.org>, <netdev@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-phy@lists.infradead.org>
Subject: Re: [PATCH] dt-bindings: Remove "status" from schema examples, again
Date: Mon, 17 Oct 2022 09:19:22 +0200	[thread overview]
Message-ID: <91250148-67b9-d514-6af0-bfcd719fde53@microchip.com> (raw)
In-Reply-To: <20221014205104.2822159-1-robh@kernel.org>

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

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel