Re: [PATCH RFC v2 0/3] Add Amlogic stateless H.264 video decoder for S4

From: Krzysztof Kozlowski <krzk@kernel.org>
To: Zhentao Guo <zhentao.guo@amlogic.com>, Rob Herring <robh@kernel.org>
Cc: Jerome Brunet <jbrunet@baylibre.com>,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	Neil Armstrong <neil.armstrong@linaro.org>,
	Kevin Hilman <khilman@baylibre.com>,
	Mauro Carvalho Chehab <mchehab@kernel.org>,
	Martin Blumenstingl <martin.blumenstingl@googlemail.com>,
	devicetree@vger.kernel.org, Conor Dooley <conor+dt@kernel.org>,
	linux-amlogic@lists.infradead.org, linux-media@vger.kernel.org,
	Krzysztof Kozlowski <krzk+dt@kernel.org>
Subject: Re: [PATCH RFC v2 0/3] Add Amlogic stateless H.264 video decoder for S4
Date: Tue, 25 Nov 2025 12:29:48 +0100	[thread overview]
Message-ID: <ae1fe925-3e3b-428d-9bae-a587ca5e37f6@kernel.org> (raw)
In-Reply-To: <5faeb411-91d0-42e2-9bf8-3051045ff42c@amlogic.com>

On 25/11/2025 04:24, Zhentao Guo wrote:
>> (or use b4 which does this automatically)
>>
>> New warnings running 'make CHECK_DTBS=y for arch/arm64/boot/dts/amlogic/' for 20251124-b4-s4-vdec-upstream-v2-0-bdbbce3f11a6@amlogic.com:
>>
>> arch/arm64/boot/dts/amlogic/meson-s4-s805x2-aq222.dtb: video-codec@fe320000 (amlogic,s4-vcodec-dec): 'amlogic,canvas' does not match any of the regexes: '^pinctrl-[0-9]+$'
>>          from schema $id: http://devicetree.org/schemas/media/amlogic,vcodec-dec.yaml
> Yes, this is a problem. We renamed the dt-binding file and added new 
> node to the dts in this patch series, but the dt-binding was not fully 
> updated. We'll fix this warning in the next version.

The problem is that you did not test it before sending.

Best regards,
Krzysztof

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