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 21FB3C71159 for ; Mon, 16 Jun 2025 15:28:27 +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=iLnV8BEM+D2PNKdH0g/PxvrKQF2q/qZCKGSlCSPvydc=; b=m4rzEkVJ+BF6kLrUg3bFdRWTJf jjHNQdO7jydwzPu2iyS5eTMMBpGxamAnJX4P/rzvnzc/QAD8Lzr3+mhewtiI31EoX4+7p0Y/EMeDz eeKAWq0f18+aCNsAgZNO68zt2lF4gaTMOdZ/GLgKQzMTo+dIuh9iQvFECaOgRQh7QdAWkVIlwmRmL 1UM+8yZiWZEkXdzNj9pZyZJLaeGEkxTMg1h7Is0QUtfnGuAUOQT/FUTs4+Hbs6Cn4zLMUa3yE2RHt BIgtZVqzW6+vA4rVsJPyBNAZJsT141n27tNb9tlHwwCzNh9iDoUhWeQjasX/ccuFbT/Y8DtWcNFes z9XdMK+w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1uRBkz-00000004sKc-0YLO; Mon, 16 Jun 2025 15:28:21 +0000 Received: from tor.source.kernel.org ([2600:3c04:e001:324:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1uRBXK-00000004o20-08zN; Mon, 16 Jun 2025 15:14:14 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by tor.source.kernel.org (Postfix) with ESMTP id 7565A6115E; Mon, 16 Jun 2025 15:14:13 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id D2D51C4CEED; Mon, 16 Jun 2025 15:14:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1750086853; bh=oVNq1HKgH5AsdLLp2/oc3d/YzPzDQJprNg0ypJQvF5g=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=PX/3X52xu3kaMjy4O0uJXkY0H6d569uWQLxiivcPb8otZdkmozLr37/eGCZohQ17T oFoOccSPG+bsR4FgnAe+PubjvwayN/Ks4Bxez77C+J1E9sArCYo02bUlZY1f3Ak2lI MZZQctzQzyq9iMWROBGPEXG6ffe2ekqloegWqLh6gd2ThRxY40gocWRFxoBwo0eIPA CAubRuQxNHcH2O1rCBpaj2oZ62ToeHGbFpmKzN66ez/+r1KVb1HvKKdA+GZbtjte3b JiDdx7JsZI6Ly49Sr/bsk1R9Dujl1wNEAHvF+wMs9s4uEV5apub+euT2LSRC8SW//l O9mX5WdLzDAsg== Date: Mon, 16 Jun 2025 16:14:07 +0100 From: Conor Dooley To: Benjamin Gaignard Cc: joro@8bytes.org, will@kernel.org, robin.murphy@arm.com, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, heiko@sntech.de, nicolas.dufresne@collabora.com, p.zabel@pengutronix.de, mchehab@kernel.org, iommu@lists.linux.dev, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-media@vger.kernel.org, kernel@collabora.com Subject: Re: [PATCH 2/5] dt-bindings: iommu: verisilicon: Add binding for VSI IOMMU Message-ID: <20250616-winter-strict-db98f85db22d@spud> References: <20250616145607.116639-1-benjamin.gaignard@collabora.com> <20250616145607.116639-3-benjamin.gaignard@collabora.com> MIME-Version: 1.0 In-Reply-To: <20250616145607.116639-3-benjamin.gaignard@collabora.com> X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============2424567390571441090==" Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org --===============2424567390571441090== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="9fJcQZWqMIbTgGsZ" Content-Disposition: inline --9fJcQZWqMIbTgGsZ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Jun 16, 2025 at 04:55:50PM +0200, Benjamin Gaignard wrote: > Add a device tree binding for the Verisilicon (VSI) IOMMU. This IOMMU sits > in front of hardware encoder and decoder blocks on SoCs using Verisilicon= IP, > such as the Rockchip RK3588. >=20 > Signed-off-by: Benjamin Gaignard > --- > .../bindings/iommu/verisilicon,iommu.yaml | 71 +++++++++++++++++++ > 1 file changed, 71 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iommu/verisilicon,i= ommu.yaml >=20 > diff --git a/Documentation/devicetree/bindings/iommu/verisilicon,iommu.ya= ml b/Documentation/devicetree/bindings/iommu/verisilicon,iommu.yaml > new file mode 100644 > index 000000000000..acef855fc61d > --- /dev/null > +++ b/Documentation/devicetree/bindings/iommu/verisilicon,iommu.yaml > @@ -0,0 +1,71 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/iommu/verisilicon,iommu.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Verisilicon IOMMU > + > +maintainers: > + - Benjamin Gaignard > + > +description: |+ > + A Versilicon iommu translates io virtual addresses to physical address= es for > + its associated video decoder. > + > +properties: > + compatible: > + oneOf: > + - items: > + - const: verisilicon,iommu You're missing a soc-specific compatible at the very least here, but is there really no versioning on the IP at all? I'd be surprised if verisilicon only produced exactly one version of an iommu IP. > + > + reg: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + clocks: > + items: > + - description: Core clock > + - description: Interface clock > + > + clock-names: > + items: > + - const: aclk > + - const: iface Why "aclk" rather than core, to match the description? > + > + "#iommu-cells": > + const: 0 > + > + power-domains: > + maxItems: 1 > + > +required: > + - compatible > + - reg > + - interrupts > + - clocks > + - clock-names > + - "#iommu-cells" > + > +additionalProperties: false > + > +examples: > + - | > + #include > + #include > + > + bus { > + #address-cells =3D <2>; > + #size-cells =3D <2>; > + > + vsi_mmu: iommu@fdca0000 { The "vsi_mmu" label can be dropped here, it has no users. Cheers, Conor. > + compatible =3D "verisilicon,iommu"; > + reg =3D <0x0 0xfdca0000 0x0 0x600>; > + interrupts =3D ; > + clocks =3D <&cru ACLK_AV1>, <&cru PCLK_AV1>; > + clock-names =3D "aclk", "iface"; > + #iommu-cells =3D <0>; > + }; > + }; > --=20 > 2.43.0 >=20 --9fJcQZWqMIbTgGsZ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCaFA0vwAKCRB4tDGHoIJi 0vT6AQC6scfk381tN82ARmoQ+PHZMrSLuYi9ck4P/3vTospnlgEA8JzI7GkSRolM exwAAerU1ffK1uHS3p3k6fwqqJumHgY= =9xSJ -----END PGP SIGNATURE----- --9fJcQZWqMIbTgGsZ-- --===============2424567390571441090== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip --===============2424567390571441090==--