Hi Rob,
On Tue, 2021-07-20 at 11:20 -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).
Is this documented somewhere? If not, should it be? (Maybe in writing-
schema.rst -> Schema Contents -> examples?)
> 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 <mrip...@kernel.org>
> Cc: Chen-Yu Tsai <w...@csie.org>
> Cc: Thierry Reding <thierry.red...@gmail.com>
> Cc: Sam Ravnborg <s...@ravnborg.org>
> Cc: Rui Miguel Silva <rmf...@gmail.com>
> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Cc: Mauro Carvalho Chehab <mche...@kernel.org>
> Cc: "David S. Miller" <da...@davemloft.net>
> Cc: Jakub Kicinski <k...@kernel.org>
> Cc: Mark Brown <broo...@kernel.org>
> Cc: Robert Marko <robert.ma...@sartura.hr>
> Cc: Philipp Zabel <p.za...@pengutronix.de>
> Cc: Alessandro Zummo <a.zu...@towertech.it>
> Cc: Alexandre Belloni <alexandre.bell...@bootlin.com>
> Cc: Ramesh Shanmugasundaram <rasha...@gmail.com>
> Cc: "G. Jaya Kumaran" <vineetha.g.jaya.kuma...@intel.com>
> Cc: Linus Walleij <linus.wall...@linaro.org>
> Cc: Oleksij Rempel <o.rem...@pengutronix.de>
> Cc: ChiYuan Huang <cy_hu...@richtek.com>
> Cc: Wei Xu <xuw...@hisilicon.com>
> Cc: Dilip Kota <eswara.k...@linux.intel.com>
> Cc: Karol Gugala <kgug...@antmicro.com>
> Cc: Mateusz Holenko <mhole...@antmicro.com>
> Cc: Olivier Moysan <olivier.moy...@st.com>
> Cc: Peter Ujfalusi <peter.ujfal...@ti.com>
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-me...@vger.kernel.org
> Cc: net...@vger.kernel.org
> Cc: linux-...@vger.kernel.org
> Cc: alsa-de...@alsa-project.org
> Signed-off-by: Rob Herring <r...@kernel.org>
> ---
> .../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 -
[...]
Reviewed-by: Philipp Zabel <p.za...@pengutronix.de>
regards
Philipp
