Re: [PATCH 1/2] dt-bindings: rtc: ds1307: Convert to json-schema

Rob Herring
2021-12-18T18:08:39Z

On Fri, 17 Dec 2021 18:03:10 +0100, Thierry Reding wrote:
> From: Thierry Reding <treding@nvidia.com>
>
> Convert the DS1307 (and compatible) RTC bindings from the free-form text
> format to json-schema.
>
> Signed-off-by: Thierry Reding <treding@nvidia.com>
> ---
> .../devicetree/bindings/rtc/rtc-ds1307.txt | 52 ---------
> .../devicetree/bindings/rtc/rtc-ds1307.yaml | 104 ++++++++++++++++++
> 2 files changed, 104 insertions(+), 52 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/rtc/rtc-ds1307.txt
> create mode 100644 Documentation/devicetree/bindings/rtc/rtc-ds1307.yaml
>

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/1570210

ds1307@68: $nodename:0: 'ds1307@68' does not match '^rtc(@.*|-[0-9a-f])*$'
arch/arm/boot/dts/imx6q-tbs2910.dt.yaml

ds1339@68: $nodename:0: 'ds1339@68' does not match '^rtc(@.*|-[0-9a-f])*$'
arch/arm/boot/dts/imx53-tx53-x03x.dt.yaml
arch/arm/boot/dts/imx53-tx53-x13x.dt.yaml

m41t00@68: $nodename:0: 'm41t00@68' does not match '^rtc(@.*|-[0-9a-f])*$'
arch/arm/boot/dts/imx51-ts4800.dt.yaml

m41t0m6@68: $nodename:0: 'm41t0m6@68' does not match '^rtc(@.*|-[0-9a-f])*$'
arch/arm/boot/dts/imx7d-colibri-aster.dt.yaml
arch/arm/boot/dts/imx7d-colibri-emmc-aster.dt.yaml
arch/arm/boot/dts/imx7d-colibri-emmc-eval-v3.dt.yaml
arch/arm/boot/dts/imx7d-colibri-eval-v3.dt.yaml
arch/arm/boot/dts/imx7s-colibri-aster.dt.yaml
arch/arm/boot/dts/imx7s-colibri-eval-v3.dt.yaml
arch/arm/boot/dts/vf500-colibri-eval-v3.dt.yaml
arch/arm/boot/dts/vf610-colibri-eval-v3.dt.yaml

mcp79400@6f: $nodename:0: 'mcp79400@6f' does not match '^rtc(@.*|-[0-9a-f])*$'
arch/arm/boot/dts/am335x-pdu001.dt.yaml

mcp7940x@6f: $nodename:0: 'mcp7940x@6f' does not match '^rtc(@.*|-[0-9a-f])*$'
arch/arm/boot/dts/imx6dl-tx6s-8034-mb7.dt.yaml
arch/arm/boot/dts/imx6dl-tx6s-8035-mb7.dt.yaml
arch/arm/boot/dts/imx6dl-tx6u-8033-mb7.dt.yaml
arch/arm/boot/dts/imx6dl-tx6u-80xx-mb7.dt.yaml
arch/arm/boot/dts/imx6dl-tx6u-81xx-mb7.dt.yaml
arch/arm/boot/dts/imx6qp-tx6qp-8037-mb7.dt.yaml
arch/arm/boot/dts/imx6qp-tx6qp-8137-mb7.dt.yaml
arch/arm/boot/dts/imx6q-tx6q-1036-mb7.dt.yaml
arch/arm/boot/dts/imx6q-tx6q-10x0-mb7.dt.yaml
arch/arm/boot/dts/imx6q-tx6q-11x0-mb7.dt.yaml
arch/arm/boot/dts/imx6ul-tx6ul-mainboard.dt.yaml

rtc@6f: interrupts-extended: [[1, 0, 2, 1], [154, 1060]] is too long
arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml