Re: [PATCH] UWB: i1480: disable MAC/PHY information checking
Iñaky Pérez-González <[email protected]> Tue, 24 Jun 2008 09:45:18 -0700
| Newsgroups | gmane.linux.usb.general,gmane.linux.usb.devel |
|---|---|
| Message-ID | <[email protected]> |
On Tuesday 24 June 2008, David Vrabel wrote: > Anderson Lizardo wrote: > > Some hardware/firmware combinations (most notably an IOGear HWA using the i1480 > > firmware) kill the host controller after issuing a GET_MAC_PHY_INFO command. > > Disabling this check seems harmless otherwise. > > > > The patch fixes the issue where the HC is killed, showing the message: > > > > ehci_hcd 0000:00:1d.7: HC died; cleaning up > > > > After this error, USB comes back only after reloading the ehci_hcd module. > > If this is going to always be disabled, why not remove the call to > GET_MAC_PHY_INFO completely? > > I'm also not keen on adding workarounds for old, beta, or release > candidate firmware. I should be safe to remove this; one of the only reasons I was doing it (if memory serves) was as a kind of an "ack" of the firmware being up and running (if the command worked, we were gold). Maybe it could be replaced with an WUSB1.0 standard command for performing the same kind of check, some command that has no secondary effects, like DEV_ADDR. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html