On Sun, 21 Sep 2008 01:59:33 +0400, Sergei Shtylyov <sshtylyov@xxxxxxxxxxxxx> wrote:
> > +static int tx4939ide_dma_test_irq(ide_drive_t *drive)
> > +{
> > + ide_hwif_t *hwif = HWIF(drive);
> > + void __iomem *base = TX4939IDE_BASE(hwif);
> > + u16 ctl = tx4939ide_readw(base, TX4939IDE_int_ctl);
> > + u8 dma_stat, stat;
> > + u16 ide_int;
> > + int found = 0;
> > +
> > + tx4939ide_check_error_ints(hwif, ctl);
> > + ide_int = ctl & (TX4939IDE_INT_XFEREND | TX4939IDE_INT_HOST);
>
> Well, since you're effectively ignoring the BUSERR interrupt, there's
> no point in enabling it...
The BUSERR is not ignored. tx4939ide_check_error_ints() will print a
message. It would be better than just ignoring.
---
Atsushi Nemoto
