Re: [Patch v3 01/15] dt-bindings: media: s5p-mfc: Add new DT schema for MFC

[Krzysztof Kozlowski]

On 18/10/2022 09:21, Aakarsh Jain wrote:
> 
> 
>> -----Original Message-----
>> From: Krzysztof Kozlowski [mailto:krzk@xxxxxxxxxx]
>> Sent: 18 October 2022 18:40
>> To: Aakarsh Jain <aakarsh.jain@xxxxxxxxxxx>; 'Rob Herring'
>> <robh@xxxxxxxxxx>
>> Cc: linux-fsd@xxxxxxxxx; linux-media@xxxxxxxxxxxxxxx;
>> pankaj.dubey@xxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx;
>> dillon.minfei@xxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; krzk+dt@xxxxxxxxxx;
>> smitha.t@xxxxxxxxxxx; benjamin.gaignard@xxxxxxxxxxxxx;
>> stanimir.varbanov@xxxxxxxxxx; jernej.skrabec@xxxxxxxxx;
>> robh+dt@xxxxxxxxxx; aswani.reddy@xxxxxxxxxxx; mchehab@xxxxxxxxxx;
>> hverkuil-cisco@xxxxxxxxx; mark.rutland@xxxxxxx;
>> m.szyprowski@xxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
>> alim.akhtar@xxxxxxxxxxx; andi@xxxxxxxxxxx; andrzej.hajda@xxxxxxxxx;
>> ezequiel@xxxxxxxxxxxxxxxxxxxx; david.plowman@xxxxxxxxxxxxxxx
>> Subject: Re: [Patch v3 01/15] dt-bindings: media: s5p-mfc: Add new DT
>> schema for MFC
>>
>> On 18/10/2022 06:48, Aakarsh Jain wrote:
>>>>
>>>> codec@f1700000: clock-names:1: 'sclk_mfc' was expected
>>>> 	arch/arm/boot/dts/s5pv210-aquila.dtb
>>>> 	arch/arm/boot/dts/s5pv210-fascinate4g.dtb
>>>> 	arch/arm/boot/dts/s5pv210-galaxys.dtb
>>>> 	arch/arm/boot/dts/s5pv210-goni.dtb
>>>> 	arch/arm/boot/dts/s5pv210-smdkc110.dtb
>>>> 	arch/arm/boot/dts/s5pv210-smdkv210.dtb
>>>> 	arch/arm/boot/dts/s5pv210-torbreck.dtb
>>>>
>>>> codec@f1700000: memory-region: [[51], [52]] is too long
>>>> 	arch/arm/boot/dts/s5pv210-fascinate4g.dtb
>>>>
>>>> codec@f1700000: memory-region: [[55], [56]] is too long
>>>> 	arch/arm/boot/dts/s5pv210-galaxys.dtb
>>>
>>>
>>>
>>> Hi Rob,
>>>
>>> We tried reproducing warnings as reported above, but I am not able to
>>> see these warnings after  running make dtbs_check & make
>>> DT_CHECKER_FLAGS=-m dt_binding_check.
>>> Packages used-
>>> yamllint 1.10.0
>>> $ dt-mk-schema --version
>>> 2022.9
>>
>>
>> Are you sure you are running these commands on proper config and arch?
>>
> yes. arm64/defconfig .

Which does not look like proper arch. Look at the warnings Rob robot
provided.


Best regards,
Krzysztof