patch 2/2:
Remove clearing bmdma status from cdrom_decode_status() since ATA devices
might need it as well.
(http://lkml.org/lkml/2006/12/4/201 and http://lkml.org/lkml/2006/11/15/94)
Signed-off-by: Albert Lee <[EMAIL PROTECTED]>
---
diff -Nrup 01_add_to_ide_intr/drivers/ide/ide-cd.c
02_remove_from_ide_cd/drivers/ide/ide-cd.c
--- 01_add_to_ide_intr/drivers/ide/ide-cd.c 2007-01-31 11:01:03.000000000
+0800
+++ 02_remove_from_ide_cd/drivers/ide/ide-cd.c 2007-01-31 13:28:11.000000000
+0800
@@ -687,15 +687,8 @@ static void ide_dump_status_no_sense(ide
static int cdrom_decode_status(ide_drive_t *drive, int good_stat, int
*stat_ret)
{
struct request *rq = HWGROUP(drive)->rq;
- ide_hwif_t *hwif = HWIF(drive);
int stat, err, sense_key;
- /* We may have bogus DMA interrupts in PIO state here */
- if (HWIF(drive)->dma_status && hwif->atapi_irq_bogon) {
- stat = hwif->INB(hwif->dma_status);
- /* Should we force the bit as well ? */
- hwif->OUTB(stat, hwif->dma_status);
- }
/* Check for errors. */
stat = HWIF(drive)->INB(IDE_STATUS_REG);
if (stat_ret)
diff -Nrup 01_add_to_ide_intr/drivers/ide/pci/piix.c
02_remove_from_ide_cd/drivers/ide/pci/piix.c
--- 01_add_to_ide_intr/drivers/ide/pci/piix.c 2007-01-31 13:38:42.000000000
+0800
+++ 02_remove_from_ide_cd/drivers/ide/pci/piix.c 2007-01-31
13:42:40.000000000 +0800
@@ -502,10 +502,6 @@ static void __devinit init_hwif_piix(ide
/* This is a painful system best to let it self tune for now */
return;
}
- /* ESB2 appears to generate spurious DMA interrupts in PIO mode
- when in native mode */
- if (hwif->pci_dev->device == PCI_DEVICE_ID_INTEL_ESB2_18)
- hwif->atapi_irq_bogon = 1;
hwif->autodma = 0;
hwif->tuneproc = &piix_tune_drive;
diff -Nrup 01_add_to_ide_intr/include/linux/ide.h
02_remove_from_ide_cd/include/linux/ide.h
--- 01_add_to_ide_intr/include/linux/ide.h 2007-01-31 11:10:27.000000000
+0800
+++ 02_remove_from_ide_cd/include/linux/ide.h 2007-01-31 13:28:11.000000000
+0800
@@ -797,7 +797,6 @@ typedef struct hwif_s {
unsigned sg_mapped : 1; /* sg_table and sg_nents are ready */
unsigned no_io_32bit : 1; /* 1 = can not do 32-bit IO ops */
unsigned err_stops_fifo : 1; /* 1=data FIFO is cleared by an
error */
- unsigned atapi_irq_bogon : 1; /* Generates spurious DMA
interrupts in PIO mode */
struct device gendev;
struct completion gendev_rel_comp; /* To deal with device release() */
-
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
