Re: [PATCH] usb host: Faraday FUSBH200 HCD driver.

From: Alan Stern
Date: Tue Mar 19 2013 - 11:48:19 EST


On Tue, 19 Mar 2013, Yuan-Hsin Chen wrote:

> > What about the port_status registers? They're not between command and
> > async_next. If they aren't consistent with EHCI, it makes things a lot
> > more complicated.
>
> fusbh200 has only one port_status register with different offset,
> 0x30, and the position of some bits are different from EHCI.

That's pretty nasty. Integrating that with the standard EHCI driver
would be considerably more difficult.

Why was the FUSBH200 designed in this strange way? Why doesn't it use
the standard EHCI register layout? Were the engineers at Faraday
deliberately trying to make life harder for driver writers?

> Also, usbmode_ex, hostpc, and txfill_tuning other than configured_flag
> are non-existent in fusbh200. They are used in both ehci-hcd.c and
> ehci-hub.c for several times.

They are used only if the hardware supports them, that is, only if the
ehci->has_hostpc flag is set.

Alan Stern

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/