On Thu, 31 Jan 2008 12:19:11 -0800
David Brownell <[EMAIL PROTECTED]> wrote:

> +             while (!(read_STAT(drv_data) & BIT_STAT_SPIF))
> +                     cpu_relax();

I'd suggest that this commonly-occurring code sequence be implemented in a
standalone function.  That'll probably produce less code and you can also
add a timeout+printk+BUG (or whatever) to that function, rather than just
mysteriously locking up if something goes wrong.


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
spi-devel-general mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/spi-devel-general

Reply via email to