Greg:

This patch for the UHCI driver changes the test for over current inputs,
which can cause false Resume indications with some of Intel's USB
controllers.  Previously the code only checked for one specific controller
chip, but now another one has turned up that exhibits the same flaw.  It
seems best just to check for Intel being the manufacturer, especially
since it doesn't hurt to check for the condition when it isn't present or
is only temporary.

Please apply.

Alan Stern


===== uhci-hcd.c 1.53 vs edited =====
--- 1.53/drivers/usb/host/uhci-hcd.c    Wed Jul  2 12:28:41 2003
+++ edited/drivers/usb/host/uhci-hcd.c  Thu Jul 17 16:44:36 2003
@@ -2007,19 +2007,17 @@
        unsigned int io_addr = uhci->io_addr;
        int i;
 
-       if (!uhci->hcd.pdev ||
-            uhci->hcd.pdev->vendor != PCI_VENDOR_ID_INTEL ||
-            uhci->hcd.pdev->device != PCI_DEVICE_ID_INTEL_82371AB_2)
+       if (!uhci->hcd.pdev || uhci->hcd.pdev->vendor != PCI_VENDOR_ID_INTEL)
                return 1;
 
-       /* This is a 82371AB/EB/MB USB controller which has a bug that
-        * causes false resume indications if any port has an
-        * over current condition.  To prevent problems, we will not
-        * allow a global suspend if any ports are OC.
+       /* Some of Intel's USB controllers have a bug that causes false
+        * resume indications if any port has an over current condition.
+        * To prevent problems, we will not allow a global suspend if
+        * any ports are OC.
         *
-        * Some motherboards using the 82371AB/EB/MB (but not the USB portion)
-        * appear to hardwire the over current inputs active to disable
-        * the USB ports.
+        * Some motherboards using Intel's chipsets (but not using all
+        * the USB ports) appear to hardwire the over current inputs active
+        * to disable the USB ports.
         */
 
        /* check for over current condition on any port */



-------------------------------------------------------
This SF.net email is sponsored by: VM Ware
With VMware you can run multiple operating systems on a single machine.
WITHOUT REBOOTING! Mix Linux / Windows / Novell virtual machines at the
same time. Free trial click here: http://www.vmware.com/wl/offer/345/0
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to