mbox series

[0/5] dt-bindings: gpio: omap: Convert to json-schema

Message ID 20210521223236.18324-1-grygorii.strashko@ti.com
Headers show
Series dt-bindings: gpio: omap: Convert to json-schema | expand

Message

Grygorii Strashko May 21, 2021, 10:32 p.m. UTC 
Hi

Convert the OMAP GPIO Device Tree binding documentation to json-schema.
The GPIO hogs node names defined to end with a 'hog' suffix.

All existing GPIO Hogs fixed to follow above naming convention
before changing the binding to avoid dtbs_check warnings.

Grygorii Strashko (5):
  ARM: dts: am335x: align GPIO hog names with dt-schema
  ARM: dts: am437x: align gpio hog names with dt-schema
  ARM: dts: omap3: align gpio hog names with dt-schema
  ARM: dts: omap5-board-common: align gpio hog names with dt-schema
  dt-bindings: gpio: omap: Convert to json-schema

 .../devicetree/bindings/gpio/gpio-omap.txt    |  45 --------
 .../bindings/gpio/ti,omap-gpio.yaml           | 108 ++++++++++++++++++
 .../boot/dts/am335x-boneblack-wireless.dts    |   2 +-
 arch/arm/boot/dts/am335x-boneblue.dts         |   2 +-
 .../boot/dts/am335x-bonegreen-wireless.dts    |   4 +-
 arch/arm/boot/dts/am335x-icev2.dts            |   4 +-
 arch/arm/boot/dts/am335x-shc.dts              |   8 +-
 arch/arm/boot/dts/am437x-gp-evm.dts           |   4 +-
 arch/arm/boot/dts/am43x-epos-evm.dts          |   2 +-
 .../boot/dts/omap3-evm-processor-common.dtsi  |   2 +-
 arch/arm/boot/dts/omap3-gta04a5.dts           |   2 +-
 arch/arm/boot/dts/omap5-board-common.dtsi     |   2 +-
 12 files changed, 124 insertions(+), 61 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/gpio/gpio-omap.txt
 create mode 100644 Documentation/devicetree/bindings/gpio/ti,omap-gpio.yaml

Comments

Rob Herring May 24, 2021, 4:59 p.m. UTC | #1 
On Sat, 22 May 2021 01:32:36 +0300, Grygorii Strashko wrote:
> Convert the OMAP GPIO Device Tree binding documentation to json-schema.

> The GPIO hogs node names defined to end with a 'hog' suffix.

> 

> Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>

> ---

>  .../devicetree/bindings/gpio/gpio-omap.txt    |  45 --------

>  .../bindings/gpio/ti,omap-gpio.yaml           | 108 ++++++++++++++++++

>  2 files changed, 108 insertions(+), 45 deletions(-)

>  delete mode 100644 Documentation/devicetree/bindings/gpio/gpio-omap.txt

>  create mode 100644 Documentation/devicetree/bindings/gpio/ti,omap-gpio.yaml

> 


My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
on your patch (DT_CHECKER_FLAGS is new in v5.13):

yamllint warnings/errors:
./Documentation/devicetree/bindings/gpio/ti,omap-gpio.yaml:22:9: [warning] wrong indentation: expected 10 but found 8 (indentation)

dtschema/dtc warnings/errors:

See https://patchwork.ozlabs.org/patch/1482351

This check can fail if there are any dependencies. The base for a patch
series is generally the most recent rc1.

If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:

pip3 install dtschema --upgrade

Please check and re-submit.
Rob Herring May 24, 2021, 10:52 p.m. UTC | #2 
On Sat, May 22, 2021 at 01:32:36AM +0300, Grygorii Strashko wrote:
> Convert the OMAP GPIO Device Tree binding documentation to json-schema.
> The GPIO hogs node names defined to end with a 'hog' suffix.
> 
> Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
> ---
>  .../devicetree/bindings/gpio/gpio-omap.txt    |  45 --------
>  .../bindings/gpio/ti,omap-gpio.yaml           | 108 ++++++++++++++++++
>  2 files changed, 108 insertions(+), 45 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/gpio/gpio-omap.txt
>  create mode 100644 Documentation/devicetree/bindings/gpio/ti,omap-gpio.yaml

Other than the indentation,

Reviewed-by: Rob Herring <robh@kernel.org>