2022-03-03 17:55:04

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH v2 17/18] dt-bindings: crypto: convert rockchip-crypto to yaml

On Wed, 02 Mar 2022 21:11:12 +0000, Corentin Labbe wrote:
> convert rockchip-crypto to yaml
>
> Signed-off-by: Corentin Labbe <[email protected]>
> ---
> .../crypto/rockchip,rk3288-crypto.yaml | 64 +++++++++++++++++++
> .../bindings/crypto/rockchip-crypto.txt | 28 --------
> 2 files changed, 64 insertions(+), 28 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/crypto/rockchip,rk3288-crypto.yaml
> delete mode 100644 Documentation/devicetree/bindings/crypto/rockchip-crypto.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/1600075


cypto-controller@ff8a0000: clock-names: ['aclk', 'hclk', 'sclk', 'apb_pclk'] is too long
arch/arm/boot/dts/rk3288-evb-act8846.dt.yaml
arch/arm/boot/dts/rk3288-evb-rk808.dt.yaml
arch/arm/boot/dts/rk3288-firefly-beta.dt.yaml
arch/arm/boot/dts/rk3288-firefly.dt.yaml
arch/arm/boot/dts/rk3288-firefly-reload.dt.yaml
arch/arm/boot/dts/rk3288-miqi.dt.yaml
arch/arm/boot/dts/rk3288-phycore-rdk.dt.yaml
arch/arm/boot/dts/rk3288-popmetal.dt.yaml
arch/arm/boot/dts/rk3288-r89.dt.yaml
arch/arm/boot/dts/rk3288-rock2-square.dt.yaml
arch/arm/boot/dts/rk3288-rock-pi-n8.dt.yaml
arch/arm/boot/dts/rk3288-tinker.dt.yaml
arch/arm/boot/dts/rk3288-tinker-s.dt.yaml
arch/arm/boot/dts/rk3288-veyron-brain.dt.yaml
arch/arm/boot/dts/rk3288-veyron-fievel.dt.yaml
arch/arm/boot/dts/rk3288-veyron-jaq.dt.yaml
arch/arm/boot/dts/rk3288-veyron-jerry.dt.yaml
arch/arm/boot/dts/rk3288-veyron-mickey.dt.yaml
arch/arm/boot/dts/rk3288-veyron-mighty.dt.yaml
arch/arm/boot/dts/rk3288-veyron-minnie.dt.yaml
arch/arm/boot/dts/rk3288-veyron-pinky.dt.yaml
arch/arm/boot/dts/rk3288-veyron-speedy.dt.yaml
arch/arm/boot/dts/rk3288-veyron-tiger.dt.yaml
arch/arm/boot/dts/rk3288-vyasa.dt.yaml

cypto-controller@ff8a0000: clocks: [[7, 199], [7, 461], [7, 125], [7, 193]] is too long
arch/arm/boot/dts/rk3288-evb-act8846.dt.yaml
arch/arm/boot/dts/rk3288-evb-rk808.dt.yaml
arch/arm/boot/dts/rk3288-firefly-beta.dt.yaml
arch/arm/boot/dts/rk3288-firefly.dt.yaml
arch/arm/boot/dts/rk3288-firefly-reload.dt.yaml
arch/arm/boot/dts/rk3288-miqi.dt.yaml
arch/arm/boot/dts/rk3288-phycore-rdk.dt.yaml
arch/arm/boot/dts/rk3288-popmetal.dt.yaml
arch/arm/boot/dts/rk3288-r89.dt.yaml
arch/arm/boot/dts/rk3288-rock2-square.dt.yaml
arch/arm/boot/dts/rk3288-rock-pi-n8.dt.yaml
arch/arm/boot/dts/rk3288-tinker.dt.yaml
arch/arm/boot/dts/rk3288-tinker-s.dt.yaml
arch/arm/boot/dts/rk3288-veyron-brain.dt.yaml
arch/arm/boot/dts/rk3288-veyron-fievel.dt.yaml
arch/arm/boot/dts/rk3288-veyron-jaq.dt.yaml
arch/arm/boot/dts/rk3288-veyron-jerry.dt.yaml
arch/arm/boot/dts/rk3288-veyron-mickey.dt.yaml
arch/arm/boot/dts/rk3288-veyron-mighty.dt.yaml
arch/arm/boot/dts/rk3288-veyron-minnie.dt.yaml
arch/arm/boot/dts/rk3288-veyron-pinky.dt.yaml
arch/arm/boot/dts/rk3288-veyron-speedy.dt.yaml
arch/arm/boot/dts/rk3288-veyron-tiger.dt.yaml
arch/arm/boot/dts/rk3288-vyasa.dt.yaml


2022-03-03 22:25:32

by Corentin LABBE

[permalink] [raw]
Subject: Re: [PATCH v2 17/18] dt-bindings: crypto: convert rockchip-crypto to yaml

Le Thu, Mar 03, 2022 at 07:44:38AM -0600, Rob Herring a ?crit :
> On Wed, 02 Mar 2022 21:11:12 +0000, Corentin Labbe wrote:
> > convert rockchip-crypto to yaml
> >
> > Signed-off-by: Corentin Labbe <[email protected]>
> > ---
> > .../crypto/rockchip,rk3288-crypto.yaml | 64 +++++++++++++++++++
> > .../bindings/crypto/rockchip-crypto.txt | 28 --------
> > 2 files changed, 64 insertions(+), 28 deletions(-)
> > create mode 100644 Documentation/devicetree/bindings/crypto/rockchip,rk3288-crypto.yaml
> > delete mode 100644 Documentation/devicetree/bindings/crypto/rockchip-crypto.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/1600075
>
>
> cypto-controller@ff8a0000: clock-names: ['aclk', 'hclk', 'sclk', 'apb_pclk'] is too long
> arch/arm/boot/dts/rk3288-evb-act8846.dt.yaml
> arch/arm/boot/dts/rk3288-evb-rk808.dt.yaml
> arch/arm/boot/dts/rk3288-firefly-beta.dt.yaml
> arch/arm/boot/dts/rk3288-firefly.dt.yaml
> arch/arm/boot/dts/rk3288-firefly-reload.dt.yaml
> arch/arm/boot/dts/rk3288-miqi.dt.yaml
> arch/arm/boot/dts/rk3288-phycore-rdk.dt.yaml
> arch/arm/boot/dts/rk3288-popmetal.dt.yaml
> arch/arm/boot/dts/rk3288-r89.dt.yaml
> arch/arm/boot/dts/rk3288-rock2-square.dt.yaml
> arch/arm/boot/dts/rk3288-rock-pi-n8.dt.yaml
> arch/arm/boot/dts/rk3288-tinker.dt.yaml
> arch/arm/boot/dts/rk3288-tinker-s.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-brain.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-fievel.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-jaq.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-jerry.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-mickey.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-mighty.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-minnie.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-pinky.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-speedy.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-tiger.dt.yaml
> arch/arm/boot/dts/rk3288-vyasa.dt.yaml
>
> cypto-controller@ff8a0000: clocks: [[7, 199], [7, 461], [7, 125], [7, 193]] is too long
> arch/arm/boot/dts/rk3288-evb-act8846.dt.yaml
> arch/arm/boot/dts/rk3288-evb-rk808.dt.yaml
> arch/arm/boot/dts/rk3288-firefly-beta.dt.yaml
> arch/arm/boot/dts/rk3288-firefly.dt.yaml
> arch/arm/boot/dts/rk3288-firefly-reload.dt.yaml
> arch/arm/boot/dts/rk3288-miqi.dt.yaml
> arch/arm/boot/dts/rk3288-phycore-rdk.dt.yaml
> arch/arm/boot/dts/rk3288-popmetal.dt.yaml
> arch/arm/boot/dts/rk3288-r89.dt.yaml
> arch/arm/boot/dts/rk3288-rock2-square.dt.yaml
> arch/arm/boot/dts/rk3288-rock-pi-n8.dt.yaml
> arch/arm/boot/dts/rk3288-tinker.dt.yaml
> arch/arm/boot/dts/rk3288-tinker-s.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-brain.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-fievel.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-jaq.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-jerry.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-mickey.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-mighty.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-minnie.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-pinky.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-speedy.dt.yaml
> arch/arm/boot/dts/rk3288-veyron-tiger.dt.yaml
> arch/arm/boot/dts/rk3288-vyasa.dt.yaml
>

The patch fixing thoses warning is in the serie.

Regards