On Mon, Dec 9, 2019 at 3:34 AM Maxime Ripard <maxime@xxxxxxxxxx> wrote: > > The older Allwinner SoCs have a system controller that is supported in > Linux, with a matching Device Tree binding. > > Now that we have the DT validation in place, let's convert the device tree > bindings for that controller over to a YAML schemas. > > Signed-off-by: Maxime Ripard <maxime@xxxxxxxxxx> > --- > .../allwinner,sun4i-a10-system-control.yaml | 140 ++++++++++++++++++ > .../devicetree/bindings/sram/sunxi-sram.txt | 113 -------------- > 2 files changed, 140 insertions(+), 113 deletions(-) > create mode 100644 Documentation/devicetree/bindings/sram/allwinner,sun4i-a10-system-control.yaml > delete mode 100644 Documentation/devicetree/bindings/sram/sunxi-sram.txt > > diff --git a/Documentation/devicetree/bindings/sram/allwinner,sun4i-a10-system-control.yaml b/Documentation/devicetree/bindings/sram/allwinner,sun4i-a10-system-control.yaml > new file mode 100644 > index 000000000000..3a368fbdff58 > --- /dev/null > +++ b/Documentation/devicetree/bindings/sram/allwinner,sun4i-a10-system-control.yaml > @@ -0,0 +1,140 @@ > +# SPDX-License-Identifier: GPL-2.0+ > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/sram/allwinner,sun4i-a10-system-control.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Allwinner A10 System Control Device Tree Bindings > + > +maintainers: > + - Chen-Yu Tsai <wens@xxxxxxxx> > + - Maxime Ripard <mripard@xxxxxxxxxx> > + > +description: > + The SRAM controller found on most Allwinner devices is represented > + by a regular node for the SRAM controller itself, with sub-nodes > + representing the SRAM handled by the SRAM controller. > + > +properties: > + "#address-cells": > + const: 1 > + > + "#size-cells": > + const: 1 > + > + compatible: > + oneOf: > + - const: allwinner,sun4i-a10-sram-controller > + deprecated: true > + - const: allwinner,sun4i-a10-system-control > + - const: allwinner,sun5i-a13-system-control > + - items: > + - const: allwinner,sun7i-a20-system-control > + - const: allwinner,sun4i-a10-system-control > + - const: allwinner,sun8i-a23-system-control > + - const: allwinner,sun8i-h3-system-control > + - const: allwinner,sun50i-a64-sram-controller > + deprecated: true > + - const: allwinner,sun50i-a64-system-control > + - const: allwinner,sun50i-h5-system-control > + - items: > + - const: allwinner,sun50i-h6-system-control > + - const: allwinner,sun50i-a64-system-control > + - items: > + - const: allwinner,suniv-f1c100s-system-control > + - const: allwinner,sun4i-a10-system-control > + > + reg: > + maxItems: 1 > + > + ranges: true > + > +patternProperties: > + "^sram@[a-z0-9]+": > + type: object > + > + properties: > + compatible: > + const: mmio-sram > + > + patternProperties: > + "^sram-section?@[a-f0-9]+$": > + type: object > + > + compatible: This should be under 'properties'. I fixed it up. I think this should have triggered an error before, but I reworked the meta-schema a bit today as there were several cases where we weren't descending levels correctly. Rob
