Re: [PATCH 0/2] arm64: Add basic support for QNAP TS-433

From: "Heiko Stübner" <heiko@sntech.de>
To: "Rob Herring" <robh@kernel.org>,
	"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
Cc: Conor Dooley <conor+dt@kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	linux-rockchip@lists.infradead.org, devicetree@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 0/2] arm64: Add basic support for QNAP TS-433
Date: Tue, 27 Feb 2024 18:41:59 +0100	[thread overview]
Message-ID: <6020339.31tnzDBltd@diego> (raw)
In-Reply-To: <2ifrk72uv2j27eukuxeekeuluzf4bu5335qk73gaaykg7tcq4d@uhkutnknqqd2>

Am Dienstag, 27. Februar 2024, 14:55:57 CET schrieb Uwe Kleine-König:
> Hello Rob,
> 
> On Tue, Feb 27, 2024 at 07:45:06AM -0600, Rob Herring wrote:
> > New warnings running 'make CHECK_DTBS=y rockchip/rk3568-qnap-ts433.dtb' for cover.1709034476.git.ukleinek@debian.org:
> > 
> > arch/arm64/boot/dts/rockchip/rk3568-qnap-ts433.dtb: video-codec@fdea0400: 'interrupt-names' is a required property
> > 	from schema $id: http://devicetree.org/schemas/media/rockchip-vpu.yaml#
> > arch/arm64/boot/dts/rockchip/rk3568-qnap-ts433.dtb: i2s@fe420000: reset-names:0: 'm' is not one of ['tx-m', 'rx-m']
> > 	from schema $id: http://devicetree.org/schemas/sound/rockchip,i2s-tdm.yaml#
> 
> Yes, I saw these, too. But given that the added dts doesn't touch these
> two nodes, I thought this to be a problem of rk3568.dtsi that I can
> ignore for this series.

I think I found suitable fixes for those in [0].

The question would be, if the bot finding errors causes the patch to get
removed from the review queue, because the binding addition still would
profit from an Ack :-)

Heiko

[0] https://lore.kernel.org/linux-rockchip/20240227173526.710056-1-heiko@sntech.de/