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

On 18/10/2022 09:21, Aakarsh Jain wrote:

quoted

-----Original Message-----
From: Krzysztof Kozlowski [mailto:krzk@kernel.org]
Sent: 18 October 2022 18:40
To: Aakarsh Jain <redacted>; 'Rob Herring'
[off-list ref]
Cc: linux-fsd@tesla.com; linux-media@vger.kernel.org;
pankaj.dubey@samsung.com; linux-arm-kernel@lists.infradead.org;
dillon.minfei@gmail.com; devicetree@vger.kernel.org; krzk+dt@kernel.org;
smitha.t@samsung.com; benjamin.gaignard@collabora.com;
stanimir.varbanov@linaro.org; jernej.skrabec@gmail.com;
robh+dt@kernel.org; aswani.reddy@samsung.com; mchehab@kernel.org;
hverkuil-cisco@xs4all.nl; mark.rutland@arm.com;
m.szyprowski@samsung.com; linux-kernel@vger.kernel.org;
alim.akhtar@samsung.com; andi@etezian.org; andrzej.hajda@intel.com;
ezequiel@vanguardiasur.com.ar; david.plowman@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:

quoted

quoted

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


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel