2021-11-27 13:03:30

by David Heidelberg

[permalink] [raw]
Subject: [PATCH v2] arm64: dts: imx8mq: fix the schema check errors for fsl,tmu-calibration

fsl,tmu-calibration is in u32-matrix. 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, 131079, 99, 131080, 111, 196608, 21, 196609, 33, 196610, 45, 196611, 57, 196612, 69, 196613, 83, 196614, 95, 196615, 113 were unexpected)
From schema: /home/ubuntu/projects_remote/linux/Documentation/devicetree/bindings/thermal/qoriq-thermal.yaml
...

Signed-off-by: David Heidelberg <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mq.dtsi | 86 +++++++++++------------
1 file changed, 43 insertions(+), 43 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
index 95d8b95d6120..c90a8befdd95 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
@@ -457,49 +457,49 @@ tmu: tmu@30260000 {
clocks = <&clk IMX8MQ_CLK_TMU_ROOT>;
little-endian;
fsl,tmu-range = <0xb0000 0xa0026 0x80048 0x70061>;
- fsl,tmu-calibration = <0x00000000 0x00000023
- 0x00000001 0x00000029
- 0x00000002 0x0000002f
- 0x00000003 0x00000035
- 0x00000004 0x0000003d
- 0x00000005 0x00000043
- 0x00000006 0x0000004b
- 0x00000007 0x00000051
- 0x00000008 0x00000057
- 0x00000009 0x0000005f
- 0x0000000a 0x00000067
- 0x0000000b 0x0000006f
-
- 0x00010000 0x0000001b
- 0x00010001 0x00000023
- 0x00010002 0x0000002b
- 0x00010003 0x00000033
- 0x00010004 0x0000003b
- 0x00010005 0x00000043
- 0x00010006 0x0000004b
- 0x00010007 0x00000055
- 0x00010008 0x0000005d
- 0x00010009 0x00000067
- 0x0001000a 0x00000070
-
- 0x00020000 0x00000017
- 0x00020001 0x00000023
- 0x00020002 0x0000002d
- 0x00020003 0x00000037
- 0x00020004 0x00000041
- 0x00020005 0x0000004b
- 0x00020006 0x00000057
- 0x00020007 0x00000063
- 0x00020008 0x0000006f
-
- 0x00030000 0x00000015
- 0x00030001 0x00000021
- 0x00030002 0x0000002d
- 0x00030003 0x00000039
- 0x00030004 0x00000045
- 0x00030005 0x00000053
- 0x00030006 0x0000005f
- 0x00030007 0x00000071>;
+ fsl,tmu-calibration = <0x00000000 0x00000023>,
+ <0x00000001 0x00000029>,
+ <0x00000002 0x0000002f>,
+ <0x00000003 0x00000035>,
+ <0x00000004 0x0000003d>,
+ <0x00000005 0x00000043>,
+ <0x00000006 0x0000004b>,
+ <0x00000007 0x00000051>,
+ <0x00000008 0x00000057>,
+ <0x00000009 0x0000005f>,
+ <0x0000000a 0x00000067>,
+ <0x0000000b 0x0000006f>,
+
+ <0x00010000 0x0000001b>,
+ <0x00010001 0x00000023>,
+ <0x00010002 0x0000002b>,
+ <0x00010003 0x00000033>,
+ <0x00010004 0x0000003b>,
+ <0x00010005 0x00000043>,
+ <0x00010006 0x0000004b>,
+ <0x00010007 0x00000055>,
+ <0x00010008 0x0000005d>,
+ <0x00010009 0x00000067>,
+ <0x0001000a 0x00000070>,
+
+ <0x00020000 0x00000017>,
+ <0x00020001 0x00000023>,
+ <0x00020002 0x0000002d>,
+ <0x00020003 0x00000037>,
+ <0x00020004 0x00000041>,
+ <0x00020005 0x0000004b>,
+ <0x00020006 0x00000057>,
+ <0x00020007 0x00000063>,
+ <0x00020008 0x0000006f>,
+
+ <0x00030000 0x00000015>,
+ <0x00030001 0x00000021>,
+ <0x00030002 0x0000002d>,
+ <0x00030003 0x00000039>,
+ <0x00030004 0x00000045>,
+ <0x00030005 0x00000053>,
+ <0x00030006 0x0000005f>,
+ <0x00030007 0x00000071>;
#thermal-sensor-cells = <1>;
};

--
2.33.0



2021-12-06 02:26:47

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH v2] arm64: dts: imx8mq: fix the schema check errors for fsl,tmu-calibration

On Sat, Nov 27, 2021 at 02:01:18PM +0100, David Heidelberg wrote:
> fsl,tmu-calibration is in u32-matrix. 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, 131079, 99, 131080, 111, 196608, 21, 196609, 33, 196610, 45, 196611, 57, 196612, 69, 196613, 83, 196614, 95, 196615, 113 were unexpected)
> From schema: /home/ubuntu/projects_remote/linux/Documentation/devicetree/bindings/thermal/qoriq-thermal.yaml

I made some edit to split the long line and use relative path for kernel
source.

> ...
>
> Signed-off-by: David Heidelberg <[email protected]>
> ---
> arch/arm64/boot/dts/freescale/imx8mq.dtsi | 86 +++++++++++------------
> 1 file changed, 43 insertions(+), 43 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> index 95d8b95d6120..c90a8befdd95 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> @@ -457,49 +457,49 @@ tmu: tmu@30260000 {
> clocks = <&clk IMX8MQ_CLK_TMU_ROOT>;
> little-endian;
> fsl,tmu-range = <0xb0000 0xa0026 0x80048 0x70061>;
> - fsl,tmu-calibration = <0x00000000 0x00000023
> - 0x00000001 0x00000029
> - 0x00000002 0x0000002f
> - 0x00000003 0x00000035
> - 0x00000004 0x0000003d
> - 0x00000005 0x00000043
> - 0x00000006 0x0000004b
> - 0x00000007 0x00000051
> - 0x00000008 0x00000057
> - 0x00000009 0x0000005f
> - 0x0000000a 0x00000067
> - 0x0000000b 0x0000006f
> -
> - 0x00010000 0x0000001b
> - 0x00010001 0x00000023
> - 0x00010002 0x0000002b
> - 0x00010003 0x00000033
> - 0x00010004 0x0000003b
> - 0x00010005 0x00000043
> - 0x00010006 0x0000004b
> - 0x00010007 0x00000055
> - 0x00010008 0x0000005d
> - 0x00010009 0x00000067
> - 0x0001000a 0x00000070
> -
> - 0x00020000 0x00000017
> - 0x00020001 0x00000023
> - 0x00020002 0x0000002d
> - 0x00020003 0x00000037
> - 0x00020004 0x00000041
> - 0x00020005 0x0000004b
> - 0x00020006 0x00000057
> - 0x00020007 0x00000063
> - 0x00020008 0x0000006f
> -
> - 0x00030000 0x00000015
> - 0x00030001 0x00000021
> - 0x00030002 0x0000002d
> - 0x00030003 0x00000039
> - 0x00030004 0x00000045
> - 0x00030005 0x00000053
> - 0x00030006 0x0000005f
> - 0x00030007 0x00000071>;
> + fsl,tmu-calibration = <0x00000000 0x00000023>,

I update the following lines to align with "<".

Applied, thanks!

Shawn

> + <0x00000001 0x00000029>,
> + <0x00000002 0x0000002f>,
> + <0x00000003 0x00000035>,
> + <0x00000004 0x0000003d>,
> + <0x00000005 0x00000043>,
> + <0x00000006 0x0000004b>,
> + <0x00000007 0x00000051>,
> + <0x00000008 0x00000057>,
> + <0x00000009 0x0000005f>,
> + <0x0000000a 0x00000067>,
> + <0x0000000b 0x0000006f>,
> +
> + <0x00010000 0x0000001b>,
> + <0x00010001 0x00000023>,
> + <0x00010002 0x0000002b>,
> + <0x00010003 0x00000033>,
> + <0x00010004 0x0000003b>,
> + <0x00010005 0x00000043>,
> + <0x00010006 0x0000004b>,
> + <0x00010007 0x00000055>,
> + <0x00010008 0x0000005d>,
> + <0x00010009 0x00000067>,
> + <0x0001000a 0x00000070>,
> +
> + <0x00020000 0x00000017>,
> + <0x00020001 0x00000023>,
> + <0x00020002 0x0000002d>,
> + <0x00020003 0x00000037>,
> + <0x00020004 0x00000041>,
> + <0x00020005 0x0000004b>,
> + <0x00020006 0x00000057>,
> + <0x00020007 0x00000063>,
> + <0x00020008 0x0000006f>,
> +
> + <0x00030000 0x00000015>,
> + <0x00030001 0x00000021>,
> + <0x00030002 0x0000002d>,
> + <0x00030003 0x00000039>,
> + <0x00030004 0x00000045>,
> + <0x00030005 0x00000053>,
> + <0x00030006 0x0000005f>,
> + <0x00030007 0x00000071>;
> #thermal-sensor-cells = <1>;
> };
>
> --
> 2.33.0
>