Re: Small error?? in IDE dma processing?

Mark Lord (mlord@pobox.com)
Wed, 16 Dec 1998 09:57:21 -0500

jason.harp@mail.ray.ca wrote:
..
> Should this not be:
> outb(inb(dma_base)&~1, dma_base); /* stop DMA operation*/
> dma_stat = inb(dma_base+2); /* get DMA status */
> stat = GET_STAT(); /* get drive status */
...

You're right, the current IDE BM documentation now has
that suggested order.

It shouldn't make any difference, but perhaps the code
should be updated regardless, just in case some new chipset
relies on the sequence.

Feel free to change it.
I don't send many patches these days.

Cheers.

-- 
mlord@pobox.com


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/