Hi Baolu > -----Original Message----- > From: Lu Baolu [mailto:baolu.lu@xxxxxxxxxxxxxxx] > Sent: Tuesday, June 07, 2016 2:27 PM > To: Jun Li <jun.li@xxxxxxx>; Roger Quadros <rogerq@xxxxxx>; Peter Chen > <hzpeterchen@xxxxxxxxx> > Cc: felipe.balbi@xxxxxxxxxxxxxxx; Mathias Nyman <mathias.nyman@xxxxxxxxx>; > Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>; Lee Jones > <lee.jones@xxxxxxxxxx>; Heikki Krogerus <heikki.krogerus@xxxxxxxxxxxxxxx>; > Liam Girdwood <lgirdwood@xxxxxxxxx>; Mark Brown <broonie@xxxxxxxxxx>; > linux-usb@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx > Subject: Re: [PATCH v10 2/7] usb: mux: add generic code for dual role port > mux > > Hi Jun, > > On 06/07/2016 11:03 AM, Jun Li wrote: > > Hi Roger > > > >> > >> For Mux devices implementing dual-role, the mux device driver _must_ > >> use OTG/dual-role core API so that a common ABI is presented to user > >> space for OTG/dual-role. > > That's the only point we have concern, do dual role switch through > > OTG/dual-role core, not do it by itself. > > That really depends on how do you define "dual role". Can you please > provide an unambiguous definition of "dual role" used in OTG/dual-role > framework? Host and peripheral. > > Best regards, > Lu Baolu > > > > >> I haven't yet looked at the mux framework but if we take care of the > >> above point then we are not introducing any redundancy. > >> > > Roger, actually this is my worry on OTG core: those dual role switch > > users just tends to do it simply by itself(straightforward and easy), > > not through the OTG core(some complicated in first look), this is just > > an example for us to convince people to select a better > > way:) > > > > Li Jun > > -- 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
