From: will.deacon@arm.com (Will Deacon)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v10] iopoll: Introduce memory-mapped IO polling macros
Date: Tue, 16 Dec 2014 09:45:27 +0000 [thread overview]
Message-ID: <20141216094527.GC6580@arm.com> (raw)
In-Reply-To: <1418687243-16395-1-git-send-email-mitchelh@codeaurora.org>
On Mon, Dec 15, 2014 at 11:47:23PM +0000, Mitchel Humpherys wrote:
> From: Matt Wagantall <mattw@codeaurora.org>
>
> It is sometimes necessary to poll a memory-mapped register until its value
> satisfies some condition. Introduce a family of convenience macros that do
> this. Tight-looping, sleeping, and timing out can all be accomplished using
> these macros.
>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: Will Deacon <will.deacon@arm.com>
> Cc: Arnd Bergmann <arnd@arndb.de>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Cc: Robert Elliott <elliott@hp.com>
> Signed-off-by: Matt Wagantall <mattw@codeaurora.org>
> Signed-off-by: Mitchel Humpherys <mitchelh@codeaurora.org>
> ---
> v9..10:
> - Actually added the comments mentioned in v8..v9 (doh!)
>
> v8..v9:
> - Added note in comments about max sleep time (Rob Elliott)
>
> v7..v8:
> - sorted helper macros by size (b, w, l, q)
> - removed some of the more esoteric (or flat-out bogus) helper macros
>
> This patch was originally part of a series [1] for adding support for IOMMU
> address translations through an ARM SMMU hardware register. The other
> patch in the series (the one that actually uses these macros and implements
> said hardware address translations) was Ack'd by the driver maintainer
> there (Will Deacon) so I've pulled this patch out to avoid resending an
> already Ack'd patch over and over again.
>
> In short, please see [1] for previous discussion and the first user of
> these macros.
>
> Will also acked this patch in [2]. I didn't retain his Ack here because I
> added to the macro comments.
You can keep the ack, it still looks good to me and I'm not really fussed
about the comments.
Will
next prev parent reply other threads:[~2014-12-16 9:45 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-15 23:47 [PATCH v10] iopoll: Introduce memory-mapped IO polling macros Mitchel Humpherys
2014-12-16 9:45 ` Will Deacon [this message]
2015-01-14 19:42 ` Mitchel Humpherys
2015-01-15 10:25 ` Will Deacon
2015-01-19 12:43 ` Joerg Roedel
2015-01-19 14:40 ` Will Deacon
2015-01-19 15:19 ` Arnd Bergmann
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=20141216094527.GC6580@arm.com \
--to=will.deacon@arm.com \
--cc=linux-arm-kernel@lists.infradead.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).