All of lore.kernel.org
 help / color / mirror / Atom feed
From: Will Deacon <will.deacon-5wv7dgnIgG8@public.gmane.org>
To: Robin Murphy <robin.murphy-5wv7dgnIgG8@public.gmane.org>
Cc: iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org,
	laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH v2] iommu/io-pgtable: Rationalise quirk handling
Date: Fri, 12 Feb 2016 17:27:17 +0000	[thread overview]
Message-ID: <20160212172716.GH17901@arm.com> (raw)
In-Reply-To: <62860b753f304beef7b4b6f1e13d7094252f2f37.1455294748.git.robin.murphy-5wv7dgnIgG8@public.gmane.org>

On Fri, Feb 12, 2016 at 05:09:46PM +0000, Robin Murphy wrote:
> As the number of io-pgtable implementations grows beyond 1, it's time
> to rationalise the quirks mechanism before things have a chance to
> start getting really ugly and out-of-hand.
> 
> To that end:
> - Indicate exactly which quirks each format can/does support.
> - Fail creating a table if a caller wants unsupported quirks.
> - Properly document where each quirk applies and why.
> 
> Signed-off-by: Robin Murphy <robin.murphy-5wv7dgnIgG8@public.gmane.org>
> ---
> 
> Just to allay any confusion, I intend this as a followup to the
> short-descriptor[1] and Mediatek IOMMU[2] patches, rather than a
> crucial addition, so it can either get picked up along with those
> or wait 'til later.

I'd like to see it included in whichever tree takes the short-descriptor
code so, in case that's not my tree:

Acked-by: Will Deacon <will.deacon-5wv7dgnIgG8@public.gmane.org>

Will

WARNING: multiple messages have this Message-ID (diff)
From: will.deacon@arm.com (Will Deacon)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2] iommu/io-pgtable: Rationalise quirk handling
Date: Fri, 12 Feb 2016 17:27:17 +0000	[thread overview]
Message-ID: <20160212172716.GH17901@arm.com> (raw)
In-Reply-To: <62860b753f304beef7b4b6f1e13d7094252f2f37.1455294748.git.robin.murphy@arm.com>

On Fri, Feb 12, 2016 at 05:09:46PM +0000, Robin Murphy wrote:
> As the number of io-pgtable implementations grows beyond 1, it's time
> to rationalise the quirks mechanism before things have a chance to
> start getting really ugly and out-of-hand.
> 
> To that end:
> - Indicate exactly which quirks each format can/does support.
> - Fail creating a table if a caller wants unsupported quirks.
> - Properly document where each quirk applies and why.
> 
> Signed-off-by: Robin Murphy <robin.murphy@arm.com>
> ---
> 
> Just to allay any confusion, I intend this as a followup to the
> short-descriptor[1] and Mediatek IOMMU[2] patches, rather than a
> crucial addition, so it can either get picked up along with those
> or wait 'til later.

I'd like to see it included in whichever tree takes the short-descriptor
code so, in case that's not my tree:

Acked-by: Will Deacon <will.deacon@arm.com>

Will

  parent reply	other threads:[~2016-02-12 17:27 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-26 17:13 [PATCH v3 0/3] io-pgtable ARM short descriptor format Robin Murphy
2016-01-26 17:13 ` Robin Murphy
     [not found] ` <cover.1453723096.git.robin.murphy-5wv7dgnIgG8@public.gmane.org>
2016-01-26 17:13   ` [PATCH v3 1/3] iommu/io-pgtable: Add ARMv7 short descriptor support Robin Murphy
2016-01-26 17:13     ` Robin Murphy
     [not found]     ` <066c4af0e29c9cc658bd5e48edc5e2d7ba7140a1.1453723096.git.robin.murphy-5wv7dgnIgG8@public.gmane.org>
2016-01-27  1:16       ` Yong Wu
2016-01-27  1:16         ` Yong Wu
2016-01-28 13:41         ` Robin Murphy
2016-01-28 13:41           ` Robin Murphy
     [not found]           ` <56AA1A89.6080706-5wv7dgnIgG8@public.gmane.org>
2016-02-17 14:31             ` Will Deacon
2016-02-17 14:31               ` Will Deacon
     [not found]               ` <20160217143106.GC1459-5wv7dgnIgG8@public.gmane.org>
2016-02-17 14:34                 ` Robin Murphy
2016-02-17 14:34                   ` Robin Murphy
2016-02-10 15:48       ` Will Deacon
2016-02-10 15:48         ` Will Deacon
     [not found]         ` <20160210154815.GO1052-5wv7dgnIgG8@public.gmane.org>
2016-02-11 16:13           ` [PATCH 4/3] iommu/io-pgtable: Rationalise quirk handling Robin Murphy
2016-02-11 16:13             ` Robin Murphy
     [not found]             ` <ea1f3d3feec0dc9c18d43393678370b50b1275f5.1455206840.git.robin.murphy-5wv7dgnIgG8@public.gmane.org>
2016-02-11 20:39               ` Laurent Pinchart
2016-02-11 20:39                 ` Laurent Pinchart
2016-02-12 12:08               ` Will Deacon
2016-02-12 12:08                 ` Will Deacon
     [not found]                 ` <20160212120857.GN25087-5wv7dgnIgG8@public.gmane.org>
2016-02-12 12:32                   ` Laurent Pinchart
2016-02-12 12:32                     ` Laurent Pinchart
2016-02-12 15:23                     ` Robin Murphy
2016-02-12 15:23                       ` Robin Murphy
     [not found]                       ` <56BDF8D8.4010200-5wv7dgnIgG8@public.gmane.org>
2016-02-12 15:55                         ` Will Deacon
2016-02-12 15:55                           ` Will Deacon
2016-03-01 12:01     ` [PATCH v3 1/3] iommu/io-pgtable: Add ARMv7 short descriptor support Geert Uytterhoeven
2016-03-01 12:01       ` Geert Uytterhoeven
2016-03-01 13:13       ` Laurent Pinchart
2016-03-01 13:13         ` Laurent Pinchart
2016-03-01 13:51       ` Robin Murphy
2016-03-01 13:51         ` Robin Murphy
2016-01-26 17:13   ` [PATCH v3 2/3] iommu/io-pgtable: Add helper functions for TLB ops Robin Murphy
2016-01-26 17:13     ` Robin Murphy
2016-01-26 17:13   ` [PATCH v3 3/3] iommu/io-pgtable: Avoid redundant TLB syncs Robin Murphy
2016-01-26 17:13     ` Robin Murphy
     [not found]     ` <81a5e784494a04713e152659c95b34e92d944f32.1453723096.git.robin.murphy-5wv7dgnIgG8@public.gmane.org>
2016-02-10 14:21       ` Will Deacon
2016-02-10 14:21         ` Will Deacon
2016-02-12 17:09   ` [PATCH v2] iommu/io-pgtable: Rationalise quirk handling Robin Murphy
2016-02-12 17:09     ` Robin Murphy
     [not found]     ` <62860b753f304beef7b4b6f1e13d7094252f2f37.1455294748.git.robin.murphy-5wv7dgnIgG8@public.gmane.org>
2016-02-12 17:24       ` Laurent Pinchart
2016-02-12 17:24         ` Laurent Pinchart
2016-02-12 17:25         ` Robin Murphy
2016-02-12 17:25           ` Robin Murphy
2016-02-12 17:27       ` Will Deacon [this message]
2016-02-12 17:27         ` Will Deacon

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=20160212172716.GH17901@arm.com \
    --to=will.deacon-5wv7dgnigg8@public.gmane.org \
    --cc=iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org \
    --cc=laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=robin.murphy-5wv7dgnIgG8@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 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.