Re: [RESEND PATCH] powerpc/fsl: fix the schema check errors for fsl,tmu-calibration

Michael Ellerman
2023-12-19T06:36:13Z

David Heidelberg <david@ixit.cz> writes:
> On 13/12/2023 06:17, Michael Ellerman wrote:
>> David Heidelberg <david@ixit.cz> writes:
>>> 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
>> This file name doesn't match the one you're modifying?
> Hello Michael. I mentioned this as a example of same problem (I
> originally sent more patches to fix it across DTS files)
>>
>>> , 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
>>> ...
>> I don't see any errors like this. I guess I'm running the tests wrong?
>
> Currently I see also no error :(

At least it's not just me :)

> If you look into
> Documentation/devicetree/bindings/thermal/qoriq-thermal.yaml you can see
> that
>
> `fsl,tmu-calibration` matches the change I'm introducing here, most
> likely there is a bug in validator or syntax is not 100% correct.

OK. I'll merge the patch as "correct per documentation" and just edit
the commit message to drop the error message so as not to confuse anyone.

cheers