Re: [PATCH 3/4] dt-bindings: usb: tegra-xusb: Convert to json-schema

Rob Herring
2021-12-09T20:56:10Z

On Thu, 09 Dec 2021 17:53:38 +0100, Thierry Reding wrote:
> From: Thierry Reding <treding@nvidia.com>
>
> Convert the Tegra XUSB controller bindings from the free-form text
> format to json-schema.
>
> Signed-off-by: Thierry Reding <treding@nvidia.com>
> ---
> .../bindings/usb/nvidia,tegra124-xusb.txt | 132 -----------
> .../bindings/usb/nvidia,tegra124-xusb.yaml | 201 +++++++++++++++++
> .../bindings/usb/nvidia,tegra186-xusb.yaml | 182 +++++++++++++++
> .../bindings/usb/nvidia,tegra194-xusb.yaml | 188 ++++++++++++++++
> .../bindings/usb/nvidia,tegra210-xusb.yaml | 210 ++++++++++++++++++
> 5 files changed, 781 insertions(+), 132 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/usb/nvidia,tegra124-xusb.txt
> create mode 100644 Documentation/devicetree/bindings/usb/nvidia,tegra124-xusb.yaml
> create mode 100644 Documentation/devicetree/bindings/usb/nvidia,tegra186-xusb.yaml
> create mode 100644 Documentation/devicetree/bindings/usb/nvidia,tegra194-xusb.yaml
> create mode 100644 Documentation/devicetree/bindings/usb/nvidia,tegra210-xusb.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/1565847

usb@70090000: clock-names:4: 'xusb_ss_div2' was expected
arch/arm64/boot/dts/nvidia/tegra132-norrin.dt.yaml
arch/arm64/boot/dts/nvidia/tegra210-p2371-0000.dt.yaml
arch/arm64/boot/dts/nvidia/tegra210-p2371-2180.dt.yaml
arch/arm64/boot/dts/nvidia/tegra210-p2571.dt.yaml
arch/arm64/boot/dts/nvidia/tegra210-p2894-0050-a08.dt.yaml
arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dt.yaml
arch/arm64/boot/dts/nvidia/tegra210-smaug.dt.yaml
arch/arm/boot/dts/tegra124-apalis-eval.dt.yaml
arch/arm/boot/dts/tegra124-apalis-v1.2-eval.dt.yaml
arch/arm/boot/dts/tegra124-jetson-tk1.dt.yaml
arch/arm/boot/dts/tegra124-nyan-big.dt.yaml
arch/arm/boot/dts/tegra124-nyan-blaze.dt.yaml
arch/arm/boot/dts/tegra124-venice2.dt.yaml

usb@70090000: clock-names:5: 'xusb_ss_src' was expected
arch/arm64/boot/dts/nvidia/tegra132-norrin.dt.yaml
arch/arm64/boot/dts/nvidia/tegra210-p2371-0000.dt.yaml
arch/arm64/boot/dts/nvidia/tegra210-p2371-2180.dt.yaml
arch/arm64/boot/dts/nvidia/tegra210-p2571.dt.yaml
arch/arm64/boot/dts/nvidia/tegra210-p2894-0050-a08.dt.yaml
arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dt.yaml
arch/arm64/boot/dts/nvidia/tegra210-smaug.dt.yaml
arch/arm/boot/dts/tegra124-apalis-eval.dt.yaml
arch/arm/boot/dts/tegra124-apalis-v1.2-eval.dt.yaml
arch/arm/boot/dts/tegra124-jetson-tk1.dt.yaml
arch/arm/boot/dts/tegra124-nyan-big.dt.yaml
arch/arm/boot/dts/tegra124-nyan-blaze.dt.yaml
arch/arm/boot/dts/tegra124-venice2.dt.yaml

usb@70090000: dvdd-pex-pll-supply: [[35]] is not of type 'object'
arch/arm64/boot/dts/nvidia/tegra210-smaug.dt.yaml

usb@70090000: hvdd-pex-pll-e-supply: [[36]] is not of type 'object'
arch/arm64/boot/dts/nvidia/tegra210-smaug.dt.yaml