RE: [PATCH v4 1/3] dt-bindings: mfd: atmel,flexcom: Convert to json-schema

From: Kavyasree.Kotagiri
Date: Mon Jun 20 2022 - 07:46:07 EST


> > Convert the Atmel flexcom device tree bindings to json schema.
> >
> > Signed-off-by: Kavyasree Kotagiri <kavyasree.kotagiri@xxxxxxxxxxxxx>
> > ---
> > v3 -> v4:
> > - Corrected format of enum used for compatible string.
> >
> > v2 -> v3:
> > - used enum for compatible string.
> > - changed irq flag to IRQ_TYPE_LEVEL_HIGH in example.
> > - fixed dtschema errors.
> >
> > v1 -> v2:
> > - Fix title.
> >
> > .../bindings/mfd/atmel,flexcom.yaml | 104 ++++++++++++++++++
> > .../devicetree/bindings/mfd/atmel-flexcom.txt | 63 -----------
> > 2 files changed, 104 insertions(+), 63 deletions(-)
> > create mode 100644
> Documentation/devicetree/bindings/mfd/atmel,flexcom.yaml
> > delete mode 100644 Documentation/devicetree/bindings/mfd/atmel-
> flexcom.txt
> >
>
> Running 'make dtbs_check' with the schema in this patch gives the
> following warnings. Consider if they are expected or the schema is
> incorrect. These may not be new warnings.
>
> Note that it is not yet a requirement to have 0 warnings for dtbs_check.
> This will change in the future.
>
> Full log is available here: https://patchwork.ozlabs.org/patch/
>
>
> flexcom@f0000000: spi@400:compatible:0: 'atmel,at91rm9200-spi' was
> expected
> arch/arm/boot/dts/at91-sam9x60ek.dtb
>
> flexcom@f0000000: spi@400:compatible: ['microchip,sam9x60-spi',
> 'atmel,at91rm9200-spi'] is too long
> arch/arm/boot/dts/at91-sam9x60ek.dtb

These errors are expected because "microchip,sam9x60-spi" and "atmel,at91rm9200-spi"
are documented in Documentation/devicetree/bindings/spi/spi_atmel.txt.
which I added in description as
" description: See ../spi/spi_atmel.txt for details of SPI bindings."