* Re: [PATCH] iommu/ipmmu-vmsa: Add r8a7795 DT binding
2016-02-29 14:33 [PATCH] iommu/ipmmu-vmsa: Add r8a7795 DT binding Magnus Damm
@ 2016-03-02 0:30 ` Simon Horman
2016-03-02 8:17 ` Geert Uytterhoeven
` (2 subsequent siblings)
3 siblings, 0 replies; 9+ messages in thread
From: Simon Horman @ 2016-03-02 0:30 UTC (permalink / raw)
To: Magnus Damm
Cc: devicetree, mark.rutland, laurent.pinchart+renesas, geert+renesas,
pawel.moll, ijc+devicetree, joro, linux-renesas-soc, iommu,
robh+dt, galak
On Mon, Feb 29, 2016 at 11:33:09PM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm+renesas@opensource.se>
>
> Update the IPMMU DT binding documentation to include the r8a7795 compat
> string as well as the "renesas,ipmmu-main" property that on r8a7795 will
> be used to describe the topology and the relationship between the various
> cache IPMMU instances and the main IPMMU.
>
> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
Reviewed-by: Simon Horman <horms+renesas@verge.net.au>
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH] iommu/ipmmu-vmsa: Add r8a7795 DT binding
2016-02-29 14:33 [PATCH] iommu/ipmmu-vmsa: Add r8a7795 DT binding Magnus Damm
2016-03-02 0:30 ` Simon Horman
@ 2016-03-02 8:17 ` Geert Uytterhoeven
2016-03-02 14:55 ` Joerg Roedel
2017-02-03 14:10 ` Laurent Pinchart
3 siblings, 0 replies; 9+ messages in thread
From: Geert Uytterhoeven @ 2016-03-02 8:17 UTC (permalink / raw)
To: Magnus Damm
Cc: devicetree@vger.kernel.org, Mark Rutland, Laurent Pinchart,
Geert Uytterhoeven, Pawel Moll, Ian Campbell, Joerg Roedel,
linux-renesas-soc, iommu, Rob Herring, Kumar Gala, Simon Horman
Hi Magnus,
On Mon, Feb 29, 2016 at 3:33 PM, Magnus Damm <magnus.damm@gmail.com> wrote:
> From: Magnus Damm <damm+renesas@opensource.se>
>
> Update the IPMMU DT binding documentation to include the r8a7795 compat
> string as well as the "renesas,ipmmu-main" property that on r8a7795 will
> be used to describe the topology and the relationship between the various
> cache IPMMU instances and the main IPMMU.
>
> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
Thanks for your patch!
> ---
>
> Written against linux-next tag next-20160229
>
> Documentation/devicetree/bindings/iommu/renesas,ipmmu-vmsa.txt | 15 ++++++++--
> 1 file changed, 13 insertions(+), 2 deletions(-)
>
> --- 0001/Documentation/devicetree/bindings/iommu/renesas,ipmmu-vmsa.txt
> +++ work/Documentation/devicetree/bindings/iommu/renesas,ipmmu-vmsa.txt 2016-02-29 23:25:15.540513000 +0900
> @@ -7,23 +7,34 @@ connected to the IPMMU through a port ca
>
> Required Properties:
>
> - - compatible: Must contain SoC-specific and generic entries from below.
> + - compatible: Must contain SoC-specific and generic entry below in case
> + the device is compatible with the R-Car Gen2 VMSA-compatible IPMMU.
>
> - "renesas,ipmmu-r8a73a4" for the R8A73A4 (R-Mobile APE6) IPMMU.
> - "renesas,ipmmu-r8a7790" for the R8A7790 (R-Car H2) IPMMU.
> - "renesas,ipmmu-r8a7791" for the R8A7791 (R-Car M2-W) IPMMU.
> - "renesas,ipmmu-r8a7793" for the R8A7793 (R-Car M2-N) IPMMU.
> - "renesas,ipmmu-r8a7794" for the R8A7794 (R-Car E2) IPMMU.
> + - "renesas,ipmmu-r8a7795" for the R8A7795 (R-Car H3) IPMMU.
> - "renesas,ipmmu-vmsa" for generic R-Car Gen2 VMSA-compatible IPMMU.
>
> - reg: Base address and size of the IPMMU registers.
> - interrupts: Specifiers for the MMU fault interrupts. For instances that
> support secure mode two interrupts must be specified, for non-secure and
> secure mode, in that order. For instances that don't support secure mode a
> - single interrupt must be specified.
> + single interrupt must be specified. Not required for cache IPMMUs.
>
> - #iommu-cells: Must be 1.
>
> +Optional properties:
> +
> + - renesas,ipmmu-main: reference to the main IPMMU instance in two cells.
> + The first cell is a phandle to the main IPMMU and the second cell is
> + the interrupt bit number associated with the particular cache IPMMU device.
> + The interrupt bit number needs to match the main IPMMU IMSSTR register.
> + Only used by cache IPMMU instances.
> +
> +
I think it would be good to include an example of how the optional properties
should be used.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH] iommu/ipmmu-vmsa: Add r8a7795 DT binding
2016-02-29 14:33 [PATCH] iommu/ipmmu-vmsa: Add r8a7795 DT binding Magnus Damm
2016-03-02 0:30 ` Simon Horman
2016-03-02 8:17 ` Geert Uytterhoeven
@ 2016-03-02 14:55 ` Joerg Roedel
2016-03-02 15:46 ` Magnus Damm
2017-02-03 14:10 ` Laurent Pinchart
3 siblings, 1 reply; 9+ messages in thread
From: Joerg Roedel @ 2016-03-02 14:55 UTC (permalink / raw)
To: Magnus Damm
Cc: mark.rutland-5wv7dgnIgG8, devicetree-u79uwXL29TY76Z2rM5mHXA,
laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw,
geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ, pawel.moll-5wv7dgnIgG8,
ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA,
iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, galak-sgV2jX0FEOL9JmXXK+q4OQ,
horms+renesas-/R6kz+dDXgpPR4JQBCEnsQ
On Mon, Feb 29, 2016 at 11:33:09PM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm+renesas-yzvPICuk2ACczHhG9Qg4qA@public.gmane.org>
>
> Update the IPMMU DT binding documentation to include the r8a7795 compat
> string as well as the "renesas,ipmmu-main" property that on r8a7795 will
> be used to describe the topology and the relationship between the various
> cache IPMMU instances and the main IPMMU.
>
> Signed-off-by: Magnus Damm <damm+renesas-yzvPICuk2ACczHhG9Qg4qA@public.gmane.org>
> ---
>
> Written against linux-next tag next-20160229
>
> Documentation/devicetree/bindings/iommu/renesas,ipmmu-vmsa.txt | 15 ++++++++--
> 1 file changed, 13 insertions(+), 2 deletions(-)
Through which tree should this go?
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] iommu/ipmmu-vmsa: Add r8a7795 DT binding
2016-03-02 14:55 ` Joerg Roedel
@ 2016-03-02 15:46 ` Magnus Damm
2016-03-03 0:05 ` Simon Horman
0 siblings, 1 reply; 9+ messages in thread
From: Magnus Damm @ 2016-03-02 15:46 UTC (permalink / raw)
To: Joerg Roedel
Cc: devicetree@vger.kernel.org, Mark Rutland, Laurent Pinchart,
Geert Uytterhoeven, Pawel Moll, ijc+devicetree@hellion.org.uk,
linux-renesas-soc, iommu, Rob Herring, Kumar Gala, Simon Horman
On Wed, Mar 2, 2016 at 11:55 PM, Joerg Roedel <joro@8bytes.org> wrote:
> On Mon, Feb 29, 2016 at 11:33:09PM +0900, Magnus Damm wrote:
>> From: Magnus Damm <damm+renesas@opensource.se>
>>
>> Update the IPMMU DT binding documentation to include the r8a7795 compat
>> string as well as the "renesas,ipmmu-main" property that on r8a7795 will
>> be used to describe the topology and the relationship between the various
>> cache IPMMU instances and the main IPMMU.
>>
>> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
>> ---
>>
>> Written against linux-next tag next-20160229
>>
>> Documentation/devicetree/bindings/iommu/renesas,ipmmu-vmsa.txt | 15 ++++++++--
>> 1 file changed, 13 insertions(+), 2 deletions(-)
>
> Through which tree should this go?
Unless anyone objects may I suggest going through your tree please!
/ magnus
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] iommu/ipmmu-vmsa: Add r8a7795 DT binding
2016-03-02 15:46 ` Magnus Damm
@ 2016-03-03 0:05 ` Simon Horman
2016-03-03 9:59 ` Joerg Roedel
0 siblings, 1 reply; 9+ messages in thread
From: Simon Horman @ 2016-03-03 0:05 UTC (permalink / raw)
To: Magnus Damm
Cc: Joerg Roedel, devicetree@vger.kernel.org, Mark Rutland,
Laurent Pinchart, Geert Uytterhoeven, Pawel Moll,
ijc+devicetree@hellion.org.uk, linux-renesas-soc, iommu,
Rob Herring, Kumar Gala
On Thu, Mar 03, 2016 at 12:46:30AM +0900, Magnus Damm wrote:
> On Wed, Mar 2, 2016 at 11:55 PM, Joerg Roedel <joro@8bytes.org> wrote:
> > On Mon, Feb 29, 2016 at 11:33:09PM +0900, Magnus Damm wrote:
> >> From: Magnus Damm <damm+renesas@opensource.se>
> >>
> >> Update the IPMMU DT binding documentation to include the r8a7795 compat
> >> string as well as the "renesas,ipmmu-main" property that on r8a7795 will
> >> be used to describe the topology and the relationship between the various
> >> cache IPMMU instances and the main IPMMU.
> >>
> >> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> >> ---
> >>
> >> Written against linux-next tag next-20160229
> >>
> >> Documentation/devicetree/bindings/iommu/renesas,ipmmu-vmsa.txt | 15 ++++++++--
> >> 1 file changed, 13 insertions(+), 2 deletions(-)
> >
> > Through which tree should this go?
>
> Unless anyone objects may I suggest going through your tree please!
FWIW that would be my preferred option.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] iommu/ipmmu-vmsa: Add r8a7795 DT binding
2016-03-03 0:05 ` Simon Horman
@ 2016-03-03 9:59 ` Joerg Roedel
0 siblings, 0 replies; 9+ messages in thread
From: Joerg Roedel @ 2016-03-03 9:59 UTC (permalink / raw)
To: Simon Horman
Cc: Magnus Damm, devicetree@vger.kernel.org, Mark Rutland,
Laurent Pinchart, Geert Uytterhoeven, Pawel Moll,
ijc+devicetree@hellion.org.uk, linux-renesas-soc, iommu,
Rob Herring, Kumar Gala
On Thu, Mar 03, 2016 at 09:05:39AM +0900, Simon Horman wrote:
> On Thu, Mar 03, 2016 at 12:46:30AM +0900, Magnus Damm wrote:
> > On Wed, Mar 2, 2016 at 11:55 PM, Joerg Roedel <joro@8bytes.org> wrote:
> > > On Mon, Feb 29, 2016 at 11:33:09PM +0900, Magnus Damm wrote:
> > >> From: Magnus Damm <damm+renesas@opensource.se>
> > >>
> > >> Update the IPMMU DT binding documentation to include the r8a7795 compat
> > >> string as well as the "renesas,ipmmu-main" property that on r8a7795 will
> > >> be used to describe the topology and the relationship between the various
> > >> cache IPMMU instances and the main IPMMU.
> > >>
> > >> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> > >> ---
> > >>
> > >> Written against linux-next tag next-20160229
> > >>
> > >> Documentation/devicetree/bindings/iommu/renesas,ipmmu-vmsa.txt | 15 ++++++++--
> > >> 1 file changed, 13 insertions(+), 2 deletions(-)
> > >
> > > Through which tree should this go?
> >
> > Unless anyone objects may I suggest going through your tree please!
>
> FWIW that would be my preferred option.
Okay, applied.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] iommu/ipmmu-vmsa: Add r8a7795 DT binding
2016-02-29 14:33 [PATCH] iommu/ipmmu-vmsa: Add r8a7795 DT binding Magnus Damm
` (2 preceding siblings ...)
2016-03-02 14:55 ` Joerg Roedel
@ 2017-02-03 14:10 ` Laurent Pinchart
2017-02-08 21:28 ` Rob Herring
3 siblings, 1 reply; 9+ messages in thread
From: Laurent Pinchart @ 2017-02-03 14:10 UTC (permalink / raw)
To: robh+dt
Cc: Magnus Damm, devicetree, mark.rutland, laurent.pinchart+renesas,
geert+renesas, pawel.moll, ijc+devicetree, joro,
linux-renesas-soc, iommu, galak, horms+renesas
Hi Rob,
On Monday 29 Feb 2016 23:33:09 Magnus Damm wrote:
> From: Magnus Damm <damm+renesas@opensource.se>
>
> Update the IPMMU DT binding documentation to include the r8a7795 compat
> string as well as the "renesas,ipmmu-main" property that on r8a7795 will
> be used to describe the topology and the relationship between the various
> cache IPMMU instances and the main IPMMU.
>
> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
This DT binding has been merged already, but isn't used upstream (nor in the
dts nor in the drivers). We plan to start using it, and I'd like to seize this
last opportunity to change it if needed and ask you for a review.
> ---
>
> Written against linux-next tag next-20160229
>
> Documentation/devicetree/bindings/iommu/renesas,ipmmu-vmsa.txt | 15 ++++-
> 1 file changed, 13 insertions(+), 2 deletions(-)
>
> --- 0001/Documentation/devicetree/bindings/iommu/renesas,ipmmu-vmsa.txt
> +++
> work/Documentation/devicetree/bindings/iommu/renesas,ipmmu-vmsa.txt
2016-02
> -29 23:25:15.540513000 +0900 @@ -7,23 +7,34 @@ connected to the IPMMU
> through a port ca
>
> Required Properties:
>
> - - compatible: Must contain SoC-specific and generic entries from below.
> + - compatible: Must contain SoC-specific and generic entry below in case
> + the device is compatible with the R-Car Gen2 VMSA-compatible IPMMU.
>
> - "renesas,ipmmu-r8a73a4" for the R8A73A4 (R-Mobile APE6) IPMMU.
> - "renesas,ipmmu-r8a7790" for the R8A7790 (R-Car H2) IPMMU.
> - "renesas,ipmmu-r8a7791" for the R8A7791 (R-Car M2-W) IPMMU.
> - "renesas,ipmmu-r8a7793" for the R8A7793 (R-Car M2-N) IPMMU.
> - "renesas,ipmmu-r8a7794" for the R8A7794 (R-Car E2) IPMMU.
> + - "renesas,ipmmu-r8a7795" for the R8A7795 (R-Car H3) IPMMU.
> - "renesas,ipmmu-vmsa" for generic R-Car Gen2 VMSA-compatible IPMMU.
>
> - reg: Base address and size of the IPMMU registers.
> - interrupts: Specifiers for the MMU fault interrupts. For instances that
> support secure mode two interrupts must be specified, for non-secure and
> secure mode, in that order. For instances that don't support secure mode a
> - single interrupt must be specified.
> + single interrupt must be specified. Not required for cache IPMMUs.
>
> - #iommu-cells: Must be 1.
>
> +Optional properties:
> +
> + - renesas,ipmmu-main: reference to the main IPMMU instance in two cells.
> + The first cell is a phandle to the main IPMMU and the second cell is
> + the interrupt bit number associated with the particular cache IPMMU
> device.
> + The interrupt bit number needs to match the main IPMMU IMSSTR register.
> + Only used by cache IPMMU instances.
> +
> +
> Each bus master connected to an IPMMU must reference the IPMMU in its
> device node with the following property:
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH] iommu/ipmmu-vmsa: Add r8a7795 DT binding
2017-02-03 14:10 ` Laurent Pinchart
@ 2017-02-08 21:28 ` Rob Herring
0 siblings, 0 replies; 9+ messages in thread
From: Rob Herring @ 2017-02-08 21:28 UTC (permalink / raw)
To: Laurent Pinchart
Cc: mark.rutland-5wv7dgnIgG8, devicetree-u79uwXL29TY76Z2rM5mHXA,
laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw,
geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ, pawel.moll-5wv7dgnIgG8,
ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg, Magnus Damm,
linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA,
iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA,
galak-sgV2jX0FEOL9JmXXK+q4OQ,
horms+renesas-/R6kz+dDXgpPR4JQBCEnsQ
On Fri, Feb 03, 2017 at 04:10:30PM +0200, Laurent Pinchart wrote:
> Hi Rob,
>
> On Monday 29 Feb 2016 23:33:09 Magnus Damm wrote:
> > From: Magnus Damm <damm+renesas-yzvPICuk2ACczHhG9Qg4qA@public.gmane.org>
> >
> > Update the IPMMU DT binding documentation to include the r8a7795 compat
> > string as well as the "renesas,ipmmu-main" property that on r8a7795 will
> > be used to describe the topology and the relationship between the various
> > cache IPMMU instances and the main IPMMU.
> >
> > Signed-off-by: Magnus Damm <damm+renesas-yzvPICuk2ACczHhG9Qg4qA@public.gmane.org>
>
> This DT binding has been merged already, but isn't used upstream (nor in the
> dts nor in the drivers). We plan to start using it, and I'd like to seize this
> last opportunity to change it if needed and ask you for a review.
Looks pretty standard.
Rob
^ permalink raw reply [flat|nested] 9+ messages in thread