Hi, On Thu, Oct 29, 2009 at 08:03:21AM +0100, ext Gupta, Ajay Kumar wrote: > > I was thinking on adding a musb_hdrc_board_data which would group > > board-specific data such as this one. > > > > Musb's init phase is quite messy as of today so we would need to clean > > that up. Anyways, the main idea is: > > > > board will call usb_musb_init() with a musb_hdrc_board_data * as > > parameter. usb-musb would still hold a static struct > > musb_hdrc_platform_data for the (in our case) OMAP-specific init, those > > would be passed down to driver and init phase would be done as > > following: > > > > musb_init() > > -> musb_platform_init() > > -> musb_board_init() > > > > we could also have board_ops and platform_ops structures for the > > function pointers to be passed to musb_core.c Then all init could be > > done there. > > > > What do you say ??? > > This looks really good to isolate board specific settings. Do you have any > ready patch on this ? no, currently I'm working on refactoring the otg support. I could work on this after I finish otg refactoring unless you want to take this task :-) -- balbi -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html