Subject: CVS commit: src/sys/dev/ic
To: None <source-changes@NetBSD.org>
From: Charles M. Hannum <mycroft@netbsd.org>
List: source-changes
Date: 08/12/2004 03:39:11
Module Name:	src
Committed By:	mycroft
Date:		Thu Aug 12 03:39:11 UTC 2004

Modified Files:
	src/sys/dev/ic: mb89352.c

Log Message:
Close a race condition in the datain loop.  If we receive another byte between
reading the FIFO status and reading the interrupt status, we could end up
leaving it in the FIFO.  Force another round through the loop after reading the
interrupt status until the FIFO reads empty again.

Also, there is no point in having the extra loop to wait for the transfer
command to the controller to be acknowledged, because the transfer loop handles
that just fine -- and getting rid of it fixes another race condition.


To generate a diff of this commit:
cvs rdiff -r1.31 -r1.32 src/sys/dev/ic/mb89352.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.