[PATCH v5] introduce macro spin_event_timeout()
Benjamin Herrenschmidt
benh at kernel.crashing.org
Wed Mar 11 10:59:11 EST 2009
On Tue, 2009-03-10 at 18:37 -0400, Josh Boyer wrote:
> On Tue, Mar 10, 2009 at 05:33:08PM -0500, Scott Wood wrote:
> > Timur Tabi wrote:
> >> The macro spin_event_timeout() takes a condition and timeout value
> >> (in microseconds) as parameters. It spins until either the condition is true
> >> or the timeout expires. It returns zero if the timeout expires first, non-zero
> >> otherwise.
> >>
> >> This primary purpose of this macro is to poll on a hardware register until a
> >> status bit changes. The timeout ensures that the loop still terminates if the
> >> bit doesn't change as expected. This macro makes it easier for driver
> >> developers to perform this kind of operation properly.
> >>
> >> Signed-off-by: Timur Tabi <timur at freescale.com>
> >> ---
> >>
> >> v5: ported to arch/powerpc, made it powerpc-specific, eliminated udelay
> >
> > Why make it powerpc-specific? This would be nice to have in
> > arch-independent code.
>
> That's just mean. He already posted it to lkml and was told to make it
> powerpc specific by Alan.
And ? We can disagree with Alan...
Ben.
More information about the Linuxppc-dev
mailing list