From: Robin Murphy <robin.murphy@arm.com> To: Nipun Gupta <nipun.gupta@nxp.com>, will.deacon@arm.com, mark.rutland@arm.com, catalin.marinas@arm.com Cc: iommu@lists.linux-foundation.org, robh+dt@kernel.org, hch@lst.de, m.szyprowski@samsung.com, gregkh@linuxfoundation.org, joro@8bytes.org, leoyang.li@nxp.com, shawnguo@kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linuxppc-dev@lists.ozlabs.org, bharat.bhushan@nxp.com, stuyoder@gmail.com, laurentiu.tudor@nxp.com Subject: Re: [PATCH 1/6] Docs: dt: add fsl-mc iommu-parent device-tree binding Date: Mon, 5 Mar 2018 14:53:21 +0000 [thread overview] Message-ID: <5cdeded1-ca3c-339a-bf73-73401e7dd4ed@arm.com> (raw) In-Reply-To: <1520260166-29387-2-git-send-email-nipun.gupta@nxp.com> On 05/03/18 14:29, Nipun Gupta wrote: > The existing IOMMU bindings cannot be used to specify the relationship > between fsl-mc devices and IOMMUs. This patch adds a binding for > mapping fsl-mc devices to IOMMUs, using a new iommu-parent property. Given that allowing "msi-parent" for #msi-cells > 1 is merely a backward-compatibility bodge full of hard-coded assumptions, why would we want to knowingly introduce a similarly unpleasant equivalent for IOMMUs? What's wrong with "iommu-map"? > Signed-off-by: Nipun Gupta <nipun.gupta@nxp.com> > --- > .../devicetree/bindings/misc/fsl,qoriq-mc.txt | 31 ++++++++++++++++++++++ > 1 file changed, 31 insertions(+) > > diff --git a/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt b/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt > index 6611a7c..011c7d6 100644 > --- a/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt > +++ b/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt > @@ -9,6 +9,24 @@ blocks that can be used to create functional hardware objects/devices > such as network interfaces, crypto accelerator instances, L2 switches, > etc. > > +For an overview of the DPAA2 architecture and fsl-mc bus see: > +drivers/staging/fsl-mc/README.txt > + > +As described in the above overview, all DPAA2 objects in a DPRC share the > +same hardware "isolation context" and a 10-bit value called an ICID > +(isolation context id) is expressed by the hardware to identify > +the requester. IOW, precisely the case for which "{msi,iommu}-map" exist. Yes, I know they're currently documented under bindings/pci, but they're not really intended to be absolutely PCI-specific. Robin. > +The generic 'iommus' property is cannot be used to describe the relationship > +between fsl-mc and IOMMUs, so an iommu-parent property is used to define > +the same. > + > +For generic IOMMU bindings, see > +Documentation/devicetree/bindings/iommu/iommu.txt. > + > +For arm-smmu binding, see: > +Documentation/devicetree/bindings/iommu/arm,smmu.txt. > + > Required properties: > > - compatible > @@ -88,14 +106,27 @@ Sub-nodes: > Value type: <phandle> > Definition: Specifies the phandle to the PHY device node associated > with the this dpmac. > +Optional properties: > + > +- iommu-parent: Maps the devices on fsl-mc bus to an IOMMU. > + The property specifies the IOMMU behind which the devices on > + fsl-mc bus are residing. > > Example: > > + smmu: iommu@5000000 { > + compatible = "arm,mmu-500"; > + #iommu-cells = <1>; > + stream-match-mask = <0x7C00>; > + ... > + }; > + > fsl_mc: fsl-mc@80c000000 { > compatible = "fsl,qoriq-mc"; > reg = <0x00000008 0x0c000000 0 0x40>, /* MC portal base */ > <0x00000000 0x08340000 0 0x40000>; /* MC control reg */ > msi-parent = <&its>; > + iommu-parent = <&smmu>; > #address-cells = <3>; > #size-cells = <1>; > >
WARNING: multiple messages have this Message-ID (diff)
From: robin.murphy@arm.com (Robin Murphy) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 1/6] Docs: dt: add fsl-mc iommu-parent device-tree binding Date: Mon, 5 Mar 2018 14:53:21 +0000 [thread overview] Message-ID: <5cdeded1-ca3c-339a-bf73-73401e7dd4ed@arm.com> (raw) In-Reply-To: <1520260166-29387-2-git-send-email-nipun.gupta@nxp.com> On 05/03/18 14:29, Nipun Gupta wrote: > The existing IOMMU bindings cannot be used to specify the relationship > between fsl-mc devices and IOMMUs. This patch adds a binding for > mapping fsl-mc devices to IOMMUs, using a new iommu-parent property. Given that allowing "msi-parent" for #msi-cells > 1 is merely a backward-compatibility bodge full of hard-coded assumptions, why would we want to knowingly introduce a similarly unpleasant equivalent for IOMMUs? What's wrong with "iommu-map"? > Signed-off-by: Nipun Gupta <nipun.gupta@nxp.com> > --- > .../devicetree/bindings/misc/fsl,qoriq-mc.txt | 31 ++++++++++++++++++++++ > 1 file changed, 31 insertions(+) > > diff --git a/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt b/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt > index 6611a7c..011c7d6 100644 > --- a/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt > +++ b/Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt > @@ -9,6 +9,24 @@ blocks that can be used to create functional hardware objects/devices > such as network interfaces, crypto accelerator instances, L2 switches, > etc. > > +For an overview of the DPAA2 architecture and fsl-mc bus see: > +drivers/staging/fsl-mc/README.txt > + > +As described in the above overview, all DPAA2 objects in a DPRC share the > +same hardware "isolation context" and a 10-bit value called an ICID > +(isolation context id) is expressed by the hardware to identify > +the requester. IOW, precisely the case for which "{msi,iommu}-map" exist. Yes, I know they're currently documented under bindings/pci, but they're not really intended to be absolutely PCI-specific. Robin. > +The generic 'iommus' property is cannot be used to describe the relationship > +between fsl-mc and IOMMUs, so an iommu-parent property is used to define > +the same. > + > +For generic IOMMU bindings, see > +Documentation/devicetree/bindings/iommu/iommu.txt. > + > +For arm-smmu binding, see: > +Documentation/devicetree/bindings/iommu/arm,smmu.txt. > + > Required properties: > > - compatible > @@ -88,14 +106,27 @@ Sub-nodes: > Value type: <phandle> > Definition: Specifies the phandle to the PHY device node associated > with the this dpmac. > +Optional properties: > + > +- iommu-parent: Maps the devices on fsl-mc bus to an IOMMU. > + The property specifies the IOMMU behind which the devices on > + fsl-mc bus are residing. > > Example: > > + smmu: iommu at 5000000 { > + compatible = "arm,mmu-500"; > + #iommu-cells = <1>; > + stream-match-mask = <0x7C00>; > + ... > + }; > + > fsl_mc: fsl-mc at 80c000000 { > compatible = "fsl,qoriq-mc"; > reg = <0x00000008 0x0c000000 0 0x40>, /* MC portal base */ > <0x00000000 0x08340000 0 0x40000>; /* MC control reg */ > msi-parent = <&its>; > + iommu-parent = <&smmu>; > #address-cells = <3>; > #size-cells = <1>; > >
next prev parent reply other threads:[~2018-03-05 14:53 UTC|newest] Thread overview: 106+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-03-05 14:29 [PATCH 0/6] Support for fsl-mc bus and its devices in SMMU Nipun Gupta 2018-03-05 14:29 ` Nipun Gupta 2018-03-05 14:29 ` Nipun Gupta 2018-03-05 14:29 ` [PATCH 1/6] Docs: dt: add fsl-mc iommu-parent device-tree binding Nipun Gupta 2018-03-05 14:29 ` Nipun Gupta 2018-03-05 14:29 ` Nipun Gupta 2018-03-05 14:53 ` Robin Murphy [this message] 2018-03-05 14:53 ` Robin Murphy 2018-03-05 15:00 ` Nipun Gupta 2018-03-05 15:00 ` Nipun Gupta 2018-03-05 15:00 ` Nipun Gupta 2018-03-05 15:37 ` Robin Murphy 2018-03-05 15:37 ` Robin Murphy 2018-03-05 15:37 ` Robin Murphy 2018-03-05 15:54 ` Nipun Gupta 2018-03-05 15:54 ` Nipun Gupta 2018-03-05 15:54 ` Nipun Gupta 2018-03-07 22:40 ` Rob Herring 2018-03-07 22:40 ` Rob Herring 2018-03-07 22:40 ` Rob Herring 2018-03-08 12:32 ` Nipun Gupta 2018-03-08 12:32 ` Nipun Gupta 2018-03-08 12:32 ` Nipun Gupta 2018-03-08 12:32 ` Nipun Gupta 2018-03-05 14:29 ` [PATCH 2/6] iommu: support iommu configuration for fsl-mc devices Nipun Gupta 2018-03-05 14:29 ` Nipun Gupta 2018-03-05 14:29 ` Nipun Gupta 2018-03-05 14:29 ` [PATCH 3/6] iommu: arm-smmu: Add support for the fsl-mc bus Nipun Gupta 2018-03-05 14:29 ` Nipun Gupta 2018-03-05 14:29 ` Nipun Gupta 2018-03-05 14:29 ` [PATCH 4/6] bus: fsl-mc: remove dma ops setup from driver Nipun Gupta 2018-03-05 14:29 ` Nipun Gupta 2018-03-05 14:29 ` Nipun Gupta 2018-03-05 14:29 ` [PATCH 5/6] dma-mapping: support fsl-mc bus Nipun Gupta 2018-03-05 14:29 ` Nipun Gupta 2018-03-05 14:29 ` Nipun Gupta 2018-03-05 15:08 ` Christoph Hellwig 2018-03-05 15:08 ` Christoph Hellwig 2018-03-05 15:48 ` Robin Murphy 2018-03-05 15:48 ` Robin Murphy 2018-03-05 15:48 ` Robin Murphy 2018-03-05 18:39 ` Christoph Hellwig 2018-03-05 18:39 ` Christoph Hellwig 2018-03-05 18:39 ` Christoph Hellwig 2018-03-05 18:51 ` Robin Murphy 2018-03-05 18:51 ` Robin Murphy 2018-03-05 18:51 ` Robin Murphy 2018-03-06 4:41 ` Nipun Gupta 2018-03-06 4:41 ` Nipun Gupta 2018-03-06 4:41 ` Nipun Gupta 2018-03-06 4:41 ` Nipun Gupta 2018-03-08 7:41 ` Christoph Hellwig 2018-03-08 7:41 ` Christoph Hellwig 2018-03-08 7:41 ` Christoph Hellwig 2018-03-09 18:29 ` Nipun Gupta 2018-03-09 18:29 ` Nipun Gupta 2018-03-09 18:29 ` Nipun Gupta 2018-03-09 18:29 ` Nipun Gupta 2018-03-09 18:50 ` Robin Murphy 2018-03-09 18:50 ` Robin Murphy 2018-03-09 18:50 ` Robin Murphy 2018-03-05 14:29 ` [PATCH 6/6] dts: fsl-ls208x: updated DT with SMMU support for fsl-mc Nipun Gupta 2018-03-05 14:29 ` Nipun Gupta 2018-03-05 14:29 ` Nipun Gupta 2018-04-17 10:21 ` [PATCH 0/6 v2] Support for fsl-mc bus and its devices in SMMU Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-17 10:21 ` [PATCH 1/6 v2] Docs: dt: add fsl-mc iommu-map device-tree binding Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-17 10:21 ` [PATCH 2/6 v2] iommu: of: make of_pci_map_rid() available for other devices too Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-17 16:52 ` Robin Murphy 2018-04-17 16:52 ` Robin Murphy 2018-04-17 16:52 ` Robin Murphy 2018-04-18 5:09 ` Bharat Bhushan 2018-04-18 5:09 ` Bharat Bhushan 2018-04-18 5:09 ` Bharat Bhushan 2018-04-18 5:09 ` Bharat Bhushan 2018-04-18 5:09 ` Bharat Bhushan 2018-04-18 6:21 ` Nipun Gupta 2018-04-18 6:21 ` Nipun Gupta 2018-04-18 6:21 ` Nipun Gupta 2018-04-18 6:21 ` Nipun Gupta 2018-04-18 6:21 ` Nipun Gupta 2018-04-17 10:21 ` [PATCH 3/6 v2] iommu: support iommu configuration for fsl-mc devices Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-17 10:21 ` [PATCH 4/6 v2] iommu: arm-smmu: Add support for the fsl-mc bus Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-17 10:21 ` [PATCH 5/6 v2] bus: fsl-mc: supoprt dma configure for devices on " Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-26 0:00 ` kbuild test robot 2018-04-26 0:00 ` kbuild test robot 2018-04-26 0:00 ` kbuild test robot 2018-04-26 0:00 ` kbuild test robot 2018-04-17 10:21 ` [PATCH 6/6 v2] arm64: dts: ls208xa: comply with the iommu map binding for fsl_mc Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta 2018-04-17 10:21 ` Nipun Gupta
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=5cdeded1-ca3c-339a-bf73-73401e7dd4ed@arm.com \ --to=robin.murphy@arm.com \ --cc=bharat.bhushan@nxp.com \ --cc=catalin.marinas@arm.com \ --cc=devicetree@vger.kernel.org \ --cc=gregkh@linuxfoundation.org \ --cc=hch@lst.de \ --cc=iommu@lists.linux-foundation.org \ --cc=joro@8bytes.org \ --cc=laurentiu.tudor@nxp.com \ --cc=leoyang.li@nxp.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linuxppc-dev@lists.ozlabs.org \ --cc=m.szyprowski@samsung.com \ --cc=mark.rutland@arm.com \ --cc=nipun.gupta@nxp.com \ --cc=robh+dt@kernel.org \ --cc=shawnguo@kernel.org \ --cc=stuyoder@gmail.com \ --cc=will.deacon@arm.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.