Date: Sun, 9 Mar 2008 09:04:59 GMT From: Hans Petter Selasky <hselasky@FreeBSD.org> To: Perforce Change Reviews <perforce@FreeBSD.org> Subject: PERFORCE change 137211 for review Message-ID: <200803090904.m2994x1S072375@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=137211 Change 137211 by hselasky@hselasky_laptop001 on 2008/03/09 09:04:40 EHCI bugfix from Ivan Voras. See "usb/120017: CS5536 (AMD Geode) USB 2.0 quirk". PR: usb/120017 Affected files ... .. //depot/projects/usb/src/sys/dev/usb/ehci_pci.c#30 edit Differences ... ==== //depot/projects/usb/src/sys/dev/usb/ehci_pci.c#30 (text+ko) ==== @@ -150,6 +150,9 @@ if (device_id == 0x10227463) return "AMD 8111 USB 2.0 controller"; + if (device_id == 0x20951022) + return ("AMD CS5536 (Geode) USB 2.0 controller"); + if (device_id == 0x43451002) return "ATI SB200 USB 2.0 controller"; if (device_id == 0x43731002) @@ -235,9 +238,14 @@ case PCI_USBREV_PRE_1_0: case PCI_USBREV_1_0: case PCI_USBREV_1_1: - sc->sc_bus.usbrev = USBREV_UNKNOWN; - device_printf(self, "pre-2.0 USB rev\n"); - return (ENXIO); + /* + * NOTE: some EHCI USB controllers have the wrong USB + * revision number. It appears those controllers are + * fully compliant so we just ignore this value in + * some common cases. + */ + device_printf(self, "pre-2.0 USB revision (ignored)\n"); + /* fallthrough */ case PCI_USBREV_2_0: sc->sc_bus.usbrev = USBREV_2_0; break;
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200803090904.m2994x1S072375>