On Wed, Jun 5, 2019 at 6:29 AM Maxime Ripard <maxime.ripard@xxxxxxxxxxx> wrote: > > Switch the DT binding to a YAML schema to enable the DT validation. > > Signed-off-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxx> > > --- > > Changes from v1: > - Fix the maintainers > --- > .../devicetree/bindings/i2c/i2c-mv64xxx.txt | 64 ----------- > .../bindings/i2c/marvell,mv64xxx-i2c.yaml | 105 ++++++++++++++++++ > 2 files changed, 105 insertions(+), 64 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt > create mode 100644 Documentation/devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml > diff --git a/Documentation/devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml b/Documentation/devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml > new file mode 100644 > index 000000000000..a1c631eaeafd > --- /dev/null > +++ b/Documentation/devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml > @@ -0,0 +1,105 @@ > +# SPDX-License-Identifier: GPL-2.0 > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/i2c/marvell,mv64xxx-i2c.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Marvell MV64XXX I2C Controller Device Tree Bindings > + > +maintainers: > + - Gregory CLEMENT <gregory.clement@xxxxxxxxxxx> > + > +properties: > + compatible: > + oneOf: > + - const: allwinner,sun4i-a10-i2c > + - items: > + - const: allwinner,sun7i-a20-i2c > + - const: allwinner,sun4i-a10-i2c > + - const: allwinner,sun6i-a31-i2c > + - items: > + - const: allwinner,sun8i-a23-i2c > + - const: allwinner,sun6i-a31-i2c > + - items: > + - const: allwinner,sun8i-a83t-i2c > + - const: allwinner,sun6i-a31-i2c > + - items: > + - const: allwinner,sun50i-a64-i2c > + - const: allwinner,sun6i-a31-i2c > + > + - const: marvell,mv64xxx-i2c > + - const: marvell,mv78230-i2c > + - const: marvell,mv78230-a0-i2c > + > + description: > + Only use "marvell,mv78230-a0-i2c" for a very rare, initial > + version of the SoC which had broken offload support. Linux > + auto-detects this and sets it appropriately. > + > + reg: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + clocks: > + minItems: 1 > + maxItems: 2 > + items: > + - description: Reference clock for the I2C bus > + - description: Bus clock (Only for Armada 7K/8K) > + > + clock-names: > + minItems: 1 > + maxItems: 2 > + items: > + - const: core > + - const: reg > + description: > + Mandatory if two clocks are used (only for Armada 7k and 8k). > + > + resets: > + maxItems: 1 > + > +required: > + - compatible > + - reg > + - interrupts > + > +allOf: > + - $ref: /schemas/i2c/i2c-controller.yaml# > + - if: > + properties: > + compatible: > + contains: > + enum: > + - allwinner,sun4i-a10-i2c > + - allwinner,sun6i-a31-i2c > + > + then: > + required: > + - clocks > + > + - if: > + properties: > + compatible: > + contains: > + const: allwinner,sun6i-a31-i2c > + > + then: > + required: > + - resets > + > +# FIXME: We should set it, but it would report all the generic > +# properties as additional properties. > +# additionalProperties: false > + > +examples: > + - | > + timer { timer? (and missing unit-address) > + compatible = "allwinner,sun4i-a10-timer"; > + reg = <0x01c20c00 0x400>; > + interrupts = <22>; > + clocks = <&osc>; > + }; > +... > -- > 2.21.0 >
