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
