From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 73DE0CD6E57 for ; Wed, 11 Oct 2023 15:56:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Reply-To:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=zO9dctkNe60rp2UHYpo3UnruyHg3kzA7wsmjytbUwIQ=; b=A/k+qJ9JBGDn5QigYlxOyCHGDI QSQbRYs+CIsOeGhFS7lClIB3yBAZRohlR9O3XuKQLnsn4sBTMzNF/Gy2t+zCu4l/1ToKFcSlfGgZ8 syBgciBfrLTvEDL43RT0OPnlXiwWunorrK8Fd4ZU+mbcR6ySXfFGYkM5AL2168hZ7cxXzhBnft/6s t/NuGtZBH+B35ThYK71axkdOIwc3V4KFuiRMIRBMKD37mSMAhUChMymTcvXUHsSHjbnXe6nZF2+3W TzZ8b+YHImcFnwhFF8hUqxpsBXerhPobumXHGclmzCg+LeyDiUsf18jDBGo1pgssweQ6YC3L00D+N 7go6z8aQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qqbZ6-00GDr6-2t; Wed, 11 Oct 2023 15:56:04 +0000 Received: from sin.source.kernel.org ([2604:1380:40e1:4800::1]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qqbZ3-00GDpw-2h; Wed, 11 Oct 2023 15:56:03 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id CB5E9CE2461; Wed, 11 Oct 2023 15:55:59 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4D5EBC433C9; Wed, 11 Oct 2023 15:55:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1697039758; bh=l9SFEh6GumGFUG+DXwfjem9vY8NkT0tXT2M7UKxvr+4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=ixz6+3ToD5gCvDW97nhaKCvGkg+zuAa66qnUIYhJxKe8kNBhHm2Kcov3Ggelcst9i gCuJyp9AUNk/QELshcUjFj0lTDxx2hxty9VRcvCceAtmIYNmsQaSYciVbweYPfD+QT PaAS5v9GBATPOL+MQPemncG7EnSSAETrqt1nMcyVyZho0hjP69D9eJHZcSPQLh0gLq Zcwghmsthja+tv1QeN0SZmKhbZ87cTA90Ijeuct3i+wfsAUXKNED9EUCKIKU5HBlwY DqbyU8yLOZk9J+mr71itTN70mVjLDa/ElG2Fn+OZAaLbYJefXMY+gAPTBnvyrilY3U vhMRPbSWgnBQQ== Date: Wed, 11 Oct 2023 16:55:50 +0100 From: Conor Dooley To: Hugues Fruchet Cc: Ezequiel Garcia , Philipp Zabel , Andrzej Pietrasiewicz , Nicolas Dufresne , Sakari Ailus , Benjamin Gaignard , Laurent Pinchart , Daniel Almeida , Benjamin Mugnier , Heiko Stuebner , Mauro Carvalho Chehab , Hans Verkuil , linux-media@vger.kernel.org, Maxime Coquelin , Alexandre Torgue , linux-stm32@st-md-mailman.stormreply.com, Rob Herring , Krzysztof Kozlowski , Conor Dooley , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org, Marco Felsch , Adam Ford Subject: Re: [PATCH v2 1/5] dt-bindings: media: Document STM32MP25 VDEC & VENC video codecs Message-ID: <20231011-jogger-designing-0eb6e2daacba@spud> References: <20231010091643.3666290-1-hugues.fruchet@foss.st.com> <20231010091643.3666290-2-hugues.fruchet@foss.st.com> MIME-Version: 1.0 In-Reply-To: <20231010091643.3666290-2-hugues.fruchet@foss.st.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231011_085602_213357_0E61079C X-CRM114-Status: GOOD ( 17.59 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============3177090207656795227==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============3177090207656795227== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="HQXQfP5KdaxdfgWJ" Content-Disposition: inline --HQXQfP5KdaxdfgWJ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hey, On Tue, Oct 10, 2023 at 11:16:39AM +0200, Hugues Fruchet wrote: > Add STM32MP25 VDEC video decoder & VENC video encoder bindings. >=20 > Signed-off-by: Hugues Fruchet > --- > .../media/st,stm32mp25-video-codec.yaml | 58 +++++++++++++++++++ > 1 file changed, 58 insertions(+) > create mode 100644 Documentation/devicetree/bindings/media/st,stm32mp25-= video-codec.yaml >=20 > diff --git a/Documentation/devicetree/bindings/media/st,stm32mp25-video-c= odec.yaml b/Documentation/devicetree/bindings/media/st,stm32mp25-video-code= c.yaml > new file mode 100644 > index 000000000000..479566171568 > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/st,stm32mp25-video-codec.ya= ml > @@ -0,0 +1,58 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > + > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/media/st,stm32mp25-video-codec.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: STMicroelectronics STM32MP25 VDEC video decoder & VENC video enco= der > + > +maintainers: > + - Hugues Fruchet > + > +description: > + The STMicroelectronics STM32MP25 SOCs embeds a VDEC video hardware > + decoder peripheral based on Verisilicon VC8000NanoD IP (former Hantro = G1) > + and a VENC video hardware encoder peripheral based on Verisilicon > + VC8000NanoE IP (former Hantro H1). > + > +properties: > + compatible: > + items: This "items:" is not needed, the enum on its own is enough. > + - enum: > + - st,stm32mp25-vdec > + - st,stm32mp25-venc > + > + reg: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + clocks: > + maxItems: 1 > + > +required: > + - compatible > + - reg > + - interrupts > + - clocks > + > +additionalProperties: false > + > +examples: > + - | > + #include > + vdec: video-codec@580d0000 { The node labels for both nodes here are not used & should be dropped. Otherwise, this seems fine to me. Cheers, Conor. > + compatible =3D "st,stm32mp25-vdec"; > + reg =3D <0x580d0000 0x3c8>; > + interrupts =3D ; > + clocks =3D <&ck_icn_p_vdec>; > + }; > + - | > + venc: video-codec@580e0000 { > + compatible =3D "st,stm32mp25-venc"; > + reg =3D <0x580e0000 0x800>; > + interrupts =3D ; > + clocks =3D <&ck_icn_p_venc>; > + }; > --=20 > 2.25.1 >=20 --HQXQfP5KdaxdfgWJ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZSbFhgAKCRB4tDGHoIJi 0t0uAQCP6ge+y1j9+kMRyc0k7z2mmqF+5BxIUEKWEn3WUxp1vgD+NYj7GsuoXHij EdPldWL1H+xzD4wuI5OWRDd2WLQiMQY= =H1kq -----END PGP SIGNATURE----- --HQXQfP5KdaxdfgWJ-- --===============3177090207656795227== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============3177090207656795227==--