On Sun, Dec 19, 2010 at 10:58:59PM +0800, Ming Lei wrote:
2010/12/13 Felipe Balbi <balbi@xxxxxx>:
Hi,
On Mon, Dec 13, 2010 at 01:01:52AM +0800, tom.leiming@xxxxxxxxx wrote:
From: Ming Lei <tom.leiming@xxxxxxxxx>
This patch introduces MUSB_GLUE_TUSB_STYLE flag, which
can be used to handle tusb dependent things by runtime
method instead of previous static method.
This patch handles tusb fifo style according to the flag
of MUSB_GLUE_TUSB_STYLE passed from hw glue drivers.
I'm not sure I like this approach. I didn't want musb to have to know
_where_ it's running. As long as platform sets up things correctly.
We need to remove CONFIG_USB_MUSB_TUSB6010 first, so I think
the method taken by the patch is doable for the purpose.
Once we have better one, we can convert to it, but I find no better
one than it at least now.
I think we can, instead, move all of those "the following field is only
used by tusb" to tusb glue layer. Put those on struct tusb_glue and
access them only from there.
I didn't check the code thoroughly but looks like it's possible.
--
balbi
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
