On Wed, Jun 19, 2019 at 8:46 AM Rob Herring <robh+dt@xxxxxxxxxx> wrote: > > On Wed, Jun 19, 2019 at 3:48 AM Maxime Ripard <maxime.ripard@xxxxxxxxxxx> wrote: > > > > Switch our Allwinner A10 EMAC controller binding to a YAML schema to enable > > the DT validation. > > > > Signed-off-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxx> > > > > --- > > > > Changes from v2: > > - Switch from the deprecated phy property to phy-handle > > --- > > Documentation/devicetree/bindings/net/allwinner,sun4i-a10-emac.yaml | 55 +++++++++++++++++++++++++++++++++++++++++++++++++++++++ > > Documentation/devicetree/bindings/net/allwinner,sun4i-emac.txt | 19 ------------------- > > 2 files changed, 55 insertions(+), 19 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/net/allwinner,sun4i-a10-emac.yaml > > delete mode 100644 Documentation/devicetree/bindings/net/allwinner,sun4i-emac.txt > > > > diff --git a/Documentation/devicetree/bindings/net/allwinner,sun4i-a10-emac.yaml b/Documentation/devicetree/bindings/net/allwinner,sun4i-a10-emac.yaml > > new file mode 100644 > > index 000000000000..2ff9e605cd26 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/net/allwinner,sun4i-a10-emac.yaml > > @@ -0,0 +1,55 @@ > > +# SPDX-License-Identifier: GPL-2.0 > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/net/allwinner,sun4i-a10-emac.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Allwinner A10 EMAC Ethernet Controller Device Tree Bindings > > + > > +allOf: > > + - $ref: "ethernet-controller.yaml#" > > + > > +maintainers: > > + - Chen-Yu Tsai <wens@xxxxxxxx> > > + - Maxime Ripard <maxime.ripard@xxxxxxxxxxx> > > + > > +properties: > > + compatible: > > + const: allwinner,sun4i-a10-emac > > + > > + reg: > > + maxItems: 1 > > + > > + interrupts: > > + maxItems: 1 > > + > > + clocks: > > + maxItems: 1 > > + > > + allwinner,sram: > > + description: Phandle to the device SRAM > > + $ref: /schemas/types.yaml#/definitions/phandle-array > > + > > +required: > > + - compatible > > + - reg > > + - interrupts > > + - clocks > > + - phy-handle > > Doesn't this throw an error if not listed in properties? NM, it doesn't. Reviewed-by: Rob Herring <robh@xxxxxxxxxx> Rob
