2021-10-29 14:14:00

by David Heidelberg

[permalink] [raw]
Subject: [PATCH] powerpc/fsl: fix the schema check errors for fsl,tmu-calibration

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@30260000: 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 <[email protected]>
---
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@f0000 {
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@f0000 {
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


2022-01-20 12:57:02

by David Heidelberg

[permalink] [raw]
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 <[email protected]>
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@30260000: 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 <[email protected]>
> ---
> 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@f0000 {
> 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@f0000 {
> 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
>