| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-Id: <SXTW5R.DQG57Y1RMSI9@ixit.cz>
Date: Tue, 18 Jan 2022 15:37:04 +0100
From: David Heidelberg <david@...t.cz>
To: Rob Herring <robh+dt@...nel.org>,
Michael Ellerman <mpe@...erman.id.au>,
Benjamin Herrenschmidt <benh@...nel.crashing.org>,
Paul Mackerras <paulus@...ba.org>
Cc: ~okias/devicetree@...ts.sr.ht, devicetree@...r.kernel.org,
linuxppc-dev@...ts.ozlabs.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] powerpc/fsl: fix the schema check errors for
fsl,tmu-calibration
Hello, can I ask for review please? Thanks.
David Heidelberg
On Fri, Oct 29 2021 at 14:17:33 +0200, David Heidelberg <david@...t.cz>
wrote:
> fsl,tmu-calibration is in u32-matrix format. Use matching property
> syntax.
> No functional changes. Fixes warnings as:
> $ make dtbs_check
> ...
> arch/arm64/boot/dts/freescale/imx8mq-librem5-r3.dt.yaml:
> tmu@...60000: fsl,tmu-calibration:0: Additional items are not allowed
> (1, 41, 2, 47, 3, 53, 4, 61, 5, 67, 6, 75, 7, 81, 8, 87, 9, 95, 10,
> 103, 11, 111
> , 65536, 27, 65537, 35, 65538, 43, 65539, 51, 65540, 59, 65541, 67,
> 65542, 75, 65543, 85, 65544, 93, 65545, 103, 65546, 112, 131072, 23,
> 131073, 35, 131074, 45, 131075, 55, 131076, 65, 131077, 75, 131078,
> 87, 13
> 1079, 99, 131080, 111, 196608, 21, 196609, 33, 196610, 45, 196611,
> 57, 196612, 69, 196613, 83, 196614, 95, 196615, 113 were unexpected)
> From schema:
> Documentation/devicetree/bindings/thermal/qoriq-thermal.yaml
> ...
>
> Signed-off-by: David Heidelberg <david@...t.cz>
> ---
> arch/powerpc/boot/dts/fsl/t1023si-post.dtsi | 79
> +++++++++++----------
> arch/powerpc/boot/dts/fsl/t1040si-post.dtsi | 71 +++++++++---------
> 2 files changed, 76 insertions(+), 74 deletions(-)
>
> diff --git a/arch/powerpc/boot/dts/fsl/t1023si-post.dtsi
> b/arch/powerpc/boot/dts/fsl/t1023si-post.dtsi
> index d552044c5afc..aa5152ca8120 100644
> --- a/arch/powerpc/boot/dts/fsl/t1023si-post.dtsi
> +++ b/arch/powerpc/boot/dts/fsl/t1023si-post.dtsi
> @@ -367,45 +367,46 @@ tmu: tmu@...00 {
> reg = <0xf0000 0x1000>;
> interrupts = <18 2 0 0>;
> fsl,tmu-range = <0xb0000 0xa0026 0x80048 0x30061>;
> - fsl,tmu-calibration = <0x00000000 0x0000000f
> - 0x00000001 0x00000017
> - 0x00000002 0x0000001e
> - 0x00000003 0x00000026
> - 0x00000004 0x0000002e
> - 0x00000005 0x00000035
> - 0x00000006 0x0000003d
> - 0x00000007 0x00000044
> - 0x00000008 0x0000004c
> - 0x00000009 0x00000053
> - 0x0000000a 0x0000005b
> - 0x0000000b 0x00000064
> -
> - 0x00010000 0x00000011
> - 0x00010001 0x0000001c
> - 0x00010002 0x00000024
> - 0x00010003 0x0000002b
> - 0x00010004 0x00000034
> - 0x00010005 0x00000039
> - 0x00010006 0x00000042
> - 0x00010007 0x0000004c
> - 0x00010008 0x00000051
> - 0x00010009 0x0000005a
> - 0x0001000a 0x00000063
> -
> - 0x00020000 0x00000013
> - 0x00020001 0x00000019
> - 0x00020002 0x00000024
> - 0x00020003 0x0000002c
> - 0x00020004 0x00000035
> - 0x00020005 0x0000003d
> - 0x00020006 0x00000046
> - 0x00020007 0x00000050
> - 0x00020008 0x00000059
> -
> - 0x00030000 0x00000002
> - 0x00030001 0x0000000d
> - 0x00030002 0x00000019
> - 0x00030003 0x00000024>;
> + fsl,tmu-calibration =
> + <0x00000000 0x0000000f>,
> + <0x00000001 0x00000017>,
> + <0x00000002 0x0000001e>,
> + <0x00000003 0x00000026>,
> + <0x00000004 0x0000002e>,
> + <0x00000005 0x00000035>,
> + <0x00000006 0x0000003d>,
> + <0x00000007 0x00000044>,
> + <0x00000008 0x0000004c>,
> + <0x00000009 0x00000053>,
> + <0x0000000a 0x0000005b>,
> + <0x0000000b 0x00000064>,
> +
> + <0x00010000 0x00000011>,
> + <0x00010001 0x0000001c>,
> + <0x00010002 0x00000024>,
> + <0x00010003 0x0000002b>,
> + <0x00010004 0x00000034>,
> + <0x00010005 0x00000039>,
> + <0x00010006 0x00000042>,
> + <0x00010007 0x0000004c>,
> + <0x00010008 0x00000051>,
> + <0x00010009 0x0000005a>,
> + <0x0001000a 0x00000063>,
> +
> + <0x00020000 0x00000013>,
> + <0x00020001 0x00000019>,
> + <0x00020002 0x00000024>,
> + <0x00020003 0x0000002c>,
> + <0x00020004 0x00000035>,
> + <0x00020005 0x0000003d>,
> + <0x00020006 0x00000046>,
> + <0x00020007 0x00000050>,
> + <0x00020008 0x00000059>,
> +
> + <0x00030000 0x00000002>,
> + <0x00030001 0x0000000d>,
> + <0x00030002 0x00000019>,
> + <0x00030003 0x00000024>;
> #thermal-sensor-cells = <1>;
> };
>
> diff --git a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi
> b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi
> index f58eb820eb5e..27e6985d8bde 100644
> --- a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi
> +++ b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi
> @@ -447,41 +447,42 @@ tmu: tmu@...00 {
> reg = <0xf0000 0x1000>;
> interrupts = <18 2 0 0>;
> fsl,tmu-range = <0xa0000 0x90026 0x8004a 0x1006a>;
> - fsl,tmu-calibration = <0x00000000 0x00000025
> - 0x00000001 0x00000028
> - 0x00000002 0x0000002d
> - 0x00000003 0x00000031
> - 0x00000004 0x00000036
> - 0x00000005 0x0000003a
> - 0x00000006 0x00000040
> - 0x00000007 0x00000044
> - 0x00000008 0x0000004a
> - 0x00000009 0x0000004f
> - 0x0000000a 0x00000054
> -
> - 0x00010000 0x0000000d
> - 0x00010001 0x00000013
> - 0x00010002 0x00000019
> - 0x00010003 0x0000001f
> - 0x00010004 0x00000025
> - 0x00010005 0x0000002d
> - 0x00010006 0x00000033
> - 0x00010007 0x00000043
> - 0x00010008 0x0000004b
> - 0x00010009 0x00000053
> -
> - 0x00020000 0x00000010
> - 0x00020001 0x00000017
> - 0x00020002 0x0000001f
> - 0x00020003 0x00000029
> - 0x00020004 0x00000031
> - 0x00020005 0x0000003c
> - 0x00020006 0x00000042
> - 0x00020007 0x0000004d
> - 0x00020008 0x00000056
> -
> - 0x00030000 0x00000012
> - 0x00030001 0x0000001d>;
> + fsl,tmu-calibration =
> + <0x00000000 0x00000025>,
> + <0x00000001 0x00000028>,
> + <0x00000002 0x0000002d>,
> + <0x00000003 0x00000031>,
> + <0x00000004 0x00000036>,
> + <0x00000005 0x0000003a>,
> + <0x00000006 0x00000040>,
> + <0x00000007 0x00000044>,
> + <0x00000008 0x0000004a>,
> + <0x00000009 0x0000004f>,
> + <0x0000000a 0x00000054>,
> +
> + <0x00010000 0x0000000d>,
> + <0x00010001 0x00000013>,
> + <0x00010002 0x00000019>,
> + <0x00010003 0x0000001f>,
> + <0x00010004 0x00000025>,
> + <0x00010005 0x0000002d>,
> + <0x00010006 0x00000033>,
> + <0x00010007 0x00000043>,
> + <0x00010008 0x0000004b>,
> + <0x00010009 0x00000053>,
> +
> + <0x00020000 0x00000010>,
> + <0x00020001 0x00000017>,
> + <0x00020002 0x0000001f>,
> + <0x00020003 0x00000029>,
> + <0x00020004 0x00000031>,
> + <0x00020005 0x0000003c>,
> + <0x00020006 0x00000042>,
> + <0x00020007 0x0000004d>,
> + <0x00020008 0x00000056>,
> +
> + <0x00030000 0x00000012>,
> + <0x00030001 0x0000001d>;
> #thermal-sensor-cells = <1>;
> };
>
> --
> 2.33.0
>
Powered by blists - more mailing lists