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

[Aakarsh Jain]

> -----Original Message-----
> From: Krzysztof Kozlowski [mailto:krzk at kernel.org]
> Sent: 18 October 2022 18:40
> To: Aakarsh Jain <aakarsh.jain at samsung.com>; 'Rob Herring'
> <robh at kernel.org>
> Cc: linux-fsd at tesla.com; linux-media at vger.kernel.org;
> pankaj.dubey at samsung.com; linux-arm-kernel at lists.infradead.org;
> dillon.minfei at gmail.com; devicetree at vger.kernel.org; krzk+dt at kernel.org;
> smitha.t at samsung.com; benjamin.gaignard at collabora.com;
> stanimir.varbanov at linaro.org; jernej.skrabec at gmail.com;
> robh+dt at kernel.org; aswani.reddy at samsung.com; mchehab at kernel.org;
> hverkuil-cisco at xs4all.nl; mark.rutland at arm.com;
> m.szyprowski at samsung.com; linux-kernel at vger.kernel.org;
> alim.akhtar at samsung.com; andi at etezian.org; andrzej.hajda at intel.com;
> ezequiel at vanguardiasur.com.ar; david.plowman at raspberrypi.com
> 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 at 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 at f1700000: memory-region: [[51], [52]] is too long
> >> 	arch/arm/boot/dts/s5pv210-fascinate4g.dtb
> >>
> >> codec at 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 .

> Best regards,
> Krzysztof