iommu.lists.linux-foundation.org archive mirror
 help / color / mirror / Atom feed
From: Will Deacon <will.deacon-5wv7dgnIgG8@public.gmane.org>
To: Robin Murphy <Robin.Murphy-5wv7dgnIgG8@public.gmane.org>
Cc: "jroedel-l3A5Bk7waGM@public.gmane.org"
	<jroedel-l3A5Bk7waGM@public.gmane.org>,
	"arnd-r2nGTMty4D4@public.gmane.org"
	<arnd-r2nGTMty4D4@public.gmane.org>,
	"iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org"
	<iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org>,
	"thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org"
	<thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	"laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org"
	<laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>,
	"Varun.Sethi-KZfg59tc24xl57MIdRCFDg@public.gmane.org"
	<Varun.Sethi-KZfg59tc24xl57MIdRCFDg@public.gmane.org>,
	"dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org"
	<dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>,
	"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>
Subject: Re: [RFC PATCH v3 0/7] Introduce automatic DMA configuration for IOMMU masters
Date: Wed, 17 Sep 2014 02:19:11 +0100	[thread overview]
Message-ID: <20140917011910.GA3232@arm.com> (raw)
In-Reply-To: <541821AB.8040403-5wv7dgnIgG8@public.gmane.org>

On Tue, Sep 16, 2014 at 12:40:27PM +0100, Robin Murphy wrote:
> On 12/09/14 17:34, Will Deacon wrote:
> > Here is version three of the RFC I've previously posted here:
> >
> >    RFCv1: http://lists.infradead.org/pipermail/linux-arm-kernel/2014-August/283023.html
> >    RFCv2: http://lists.infradead.org/pipermail/linux-arm-kernel/2014-September/283752.html
> >
> > Changes since RFCv2 include:
> >
> >    - Put the iommu_ops in iommu_data so of_iommu_configure can avoid using
> >      the bus_type
> >    - Initialise the offset and DMA masks on the dev in of_dma_configure
> >      instead of in the arch callback (as this would cause a regression on
> >      some architectures)
> >    - Added deconfigure/teardown code based on ref counting the iommu_dma_mapping
> >    - A bunch of small fixes (_OF_DECLARE,  some code shuffling, fix multiple
> >      IOMMU parsing)
> >
> > All feedback welcome. Hopefully this is now at a point where people can
> > start looking to port dma-mapping and/or IOMMU drivers to it.
> >
> 
> What about AMBA devices? Playing with this on Juno and wondering why my 
> PL330 doesn't get any of_xlate callbacks, I see that 
> of_amba_device_create doesn't call of_dma_configure or anything from 
> that path. It's easy to work around by removing the arm,primecell 
> compatible, but that feels pretty dirty.

Yeah, that's just a bug in mainline. I'll look at fixing it if you don't
beat me to it.

Will

      parent reply	other threads:[~2014-09-17  1:19 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-12 16:34 [RFC PATCH v3 0/7] Introduce automatic DMA configuration for IOMMU masters Will Deacon
     [not found] ` <1410539695-29128-1-git-send-email-will.deacon-5wv7dgnIgG8@public.gmane.org>
2014-09-12 16:34   ` [RFC PATCH v3 1/7] iommu: provide early initialisation hook for IOMMU drivers Will Deacon
2014-09-18 14:31     ` Robin Murphy
     [not found]       ` <541AECDA.1000805-5wv7dgnIgG8@public.gmane.org>
2014-09-22 17:35         ` Will Deacon
2014-09-12 16:34   ` [RFC PATCH v3 2/7] dma-mapping: replace set_arch_dma_coherent_ops with arch_setup_dma_ops Will Deacon
2014-09-12 16:34   ` [RFC PATCH v3 3/7] iommu: add new iommu_ops callback for adding an OF device Will Deacon
     [not found]     ` <1410539695-29128-4-git-send-email-will.deacon-5wv7dgnIgG8@public.gmane.org>
2014-09-15 11:57       ` Marek Szyprowski
     [not found]         ` <5416D432.8020107-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2014-09-17  1:39           ` Will Deacon
2014-09-12 16:34   ` [RFC PATCH v3 4/7] iommu: provide helper function to configure an IOMMU for an of master Will Deacon
     [not found]     ` <1410539695-29128-5-git-send-email-will.deacon-5wv7dgnIgG8@public.gmane.org>
2014-09-18 11:13       ` Laurent Pinchart
2014-09-22 17:13         ` Will Deacon
     [not found]           ` <20140922171352.GF7936-5wv7dgnIgG8@public.gmane.org>
2014-10-14 13:12             ` Laurent Pinchart
2014-09-12 16:34   ` [RFC PATCH v3 5/7] dma-mapping: detect and configure IOMMU in of_dma_configure Will Deacon
     [not found]     ` <1410539695-29128-6-git-send-email-will.deacon-5wv7dgnIgG8@public.gmane.org>
2014-09-18 11:17       ` Laurent Pinchart
2014-09-22  9:29         ` Thierry Reding
2014-09-22 17:50           ` Will Deacon
     [not found]             ` <20140922175027.GK7936-5wv7dgnIgG8@public.gmane.org>
2014-10-14 12:53               ` Laurent Pinchart
2014-10-27 10:51                 ` Will Deacon
     [not found]                   ` <20141027105158.GE8768-5wv7dgnIgG8@public.gmane.org>
2014-10-27 11:12                     ` Marek Szyprowski
2014-10-27 11:30                     ` Laurent Pinchart
2014-10-27 16:02                       ` Will Deacon
     [not found]                         ` <20141027160216.GY8768-5wv7dgnIgG8@public.gmane.org>
2014-10-27 16:33                           ` jroedel-l3A5Bk7waGM
2014-09-22 17:46         ` Will Deacon
2014-09-12 16:34   ` [RFC PATCH v3 6/7] arm: call iommu_init before of_platform_populate Will Deacon
     [not found]     ` <1410539695-29128-7-git-send-email-will.deacon-5wv7dgnIgG8@public.gmane.org>
2014-09-22  9:36       ` Thierry Reding
2014-09-22 11:08         ` Arnd Bergmann
2014-09-22 11:40           ` Thierry Reding
2014-09-22 16:03             ` Arnd Bergmann
2014-09-23  7:02               ` Thierry Reding
2014-09-23  7:44                 ` Arnd Bergmann
2014-09-23  8:59                   ` Thierry Reding
2014-10-14 13:07                   ` Laurent Pinchart
2014-10-14 13:20                     ` Arnd Bergmann
2014-10-14 13:37                       ` Thierry Reding
2014-10-14 15:01                         ` Laurent Pinchart
2014-10-14 15:05                           ` Thierry Reding
2014-10-14 15:10                             ` Laurent Pinchart
2014-09-12 16:34   ` [RFC PATCH v3 7/7] arm: dma-mapping: plumb our iommu mapping ops into arch_setup_dma_ops Will Deacon
     [not found]     ` <1410539695-29128-8-git-send-email-will.deacon-5wv7dgnIgG8@public.gmane.org>
2014-09-22  9:19       ` Thierry Reding
2014-09-22  9:22         ` Laurent Pinchart
2014-09-22 17:43         ` Will Deacon
     [not found]           ` <20140922174337.GI7936-5wv7dgnIgG8@public.gmane.org>
2014-09-23  7:14             ` Thierry Reding
2014-09-24 16:33               ` Will Deacon
     [not found]                 ` <20140924163338.GF16244-5wv7dgnIgG8@public.gmane.org>
2014-09-25  6:40                   ` Thierry Reding
2014-09-30 16:00                     ` Will Deacon
     [not found]                       ` <20140930160035.GM2548-5wv7dgnIgG8@public.gmane.org>
2014-10-01  8:46                         ` Thierry Reding
2014-10-03 15:08                           ` Will Deacon
     [not found]                             ` <20141003150850.GA32451-5wv7dgnIgG8@public.gmane.org>
2014-10-06  9:52                               ` Thierry Reding
2014-10-06 10:50                                 ` Laurent Pinchart
2014-10-06 13:05                                   ` Thierry Reding
2014-09-16 11:40 ` [RFC PATCH v3 0/7] Introduce automatic DMA configuration for IOMMU masters Robin Murphy
     [not found]   ` <541821AB.8040403-5wv7dgnIgG8@public.gmane.org>
2014-09-17  1:19     ` Will Deacon [this message]

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=20140917011910.GA3232@arm.com \
    --to=will.deacon-5wv7dgnigg8@public.gmane.org \
    --cc=Robin.Murphy-5wv7dgnIgG8@public.gmane.org \
    --cc=Varun.Sethi-KZfg59tc24xl57MIdRCFDg@public.gmane.org \
    --cc=arnd-r2nGTMty4D4@public.gmane.org \
    --cc=dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org \
    --cc=iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org \
    --cc=jroedel-l3A5Bk7waGM@public.gmane.org \
    --cc=laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    /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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).