[PATCH v5] introduce macro spin_event_timeout()
Josh Boyer
jwboyer at linux.vnet.ibm.com
Wed Mar 11 11:44:58 EST 2009
On Wed, Mar 11, 2009 at 10:59:11AM +1100, Benjamin Herrenschmidt wrote:
>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...
Did I say Alan was right? I'm just explaining why Timur probably posted it
as arch-specific.
josh
More information about the Linuxppc-dev
mailing list