On Tue, Aug 02, 2011 at 11:44:46AM +0300, Felipe Balbi wrote: > On Tue, Aug 02, 2011 at 10:28:21AM +0300, Heikki Krogerus wrote: > > On Mon, Aug 01, 2011 at 05:35:46PM +0300, Felipe Balbi wrote: > > > I would start by splitting <linux/usb/otg.h> into things which are > > > really otg specific and things are only related to transceivers... > > > > Well.. I though I did, with the new struct otg and struct > > usb_transceiver. The phy utility does not have much, and can be part > > of <linux/usb/otg.h> if you like. > > my point is that you shouldn't add new stuff when there's already code > in place to do that. So instead of adding a new struct usb_transceiver, > just rename the one we have in place ;-) OK. I understand your point. I'll do it like that then. Just to be sure, do I still introduce a new structure for otg already? The one where we move all the otg specific stuff? Or do you think we should first only change the name and add the state machine to it? The state machine is the thing that I feel we really should have. If we only add it to the xceiv structure, how do we force everybody to start using it? > > > only then I would start adding the PHY layer. Also, keep in mind that > > > this will be a gigantic amount of work and all drivers in mainline need > > > to be changed :-( > > > > ..This is why I decided it's better to keep the existing > > otg_transceiver unchanged, so there will be plenty of time for all the > > drivers to be ported to the new utility. > > I don't think it's worth doing that, it won't push people to fix up > what's already there for ages. OK. I'm fine with doing it like this if every one agrees. I think your point also answers to my question above, right? Did you have time to review the code? Do you think the idea about USB charging is OK? I'm trying to avoid the need to rely on the notifiers. -- heikki -- 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
