[KJ] [Fwd: PATCH: 2.6.10 - Incorrect return from PCI ide controller]

Hi Guys,

Maybe someone could look through the IDE drivers and see what can be done?

Since the IDE party seems to be hosted by the -ac tree, submitting them
for inclusion on that tree might be a good idea.

Cheers,

Felipe Damasio

-------- Original Message --------
Subject: PATCH: 2.6.10 - Incorrect return from PCI ide controller
Date: Mon, 27 Dec 2004 14:37:38 +0000
From: Alan Cox <[email protected]>
To: [email protected], Linux Kernel Mailing List <[email protected]>, Bartlomiej Zolnierkiewicz <[email protected]>

Several IDE drivers return positive values as errors in the PCI setup code. Unfortunately the PCI layer considers positive values as success so the driver skips the device but still claims it and things then go downhill.

This fixes the IT8172 driver. There are other drivers with this bug (eg generic) but the -ac IDE is sufficiently diverged from base that someone else needs to generate/test the more divergent cases.

Alan

diff -u --new-file --recursive --exclude-from /usr/src/exclude linux.vanilla-2.6.10/drivers/ide/pci/it8172.c linux-2.6.10/drivers/ide/pci/it8172.c
@@ -270,7 +270,7 @@
 {
         if ((!(PCI_FUNC(dev->devfn) & 1) ||
             (!((dev->class >> 8) == PCI_CLASS_STORAGE_IDE))))
-                return 1; /* IT8172 is more than only a IDE controller */
+                return -EAGAIN; /* IT8172 is more than an IDE controller */
 	ide_setup_pci_device(dev, &it8172_chipsets[id->driver_data]);
 	return 0;
 }

_______________________________________________
Kernel-janitors mailing list
[email protected]
http://lists.osdl.org/mailman/listinfo/kernel-janitors