20 Jul
2021
20 Jul
'21
7:56 p.m.
Hi Rob,
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
I had preferred that the names was fixed in a dedicated patch - to avoid mixing things.
But thats just nitpicking, so: Reviewed-by: Sam Ravnborg sam@ravnborg.org