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? Rob
