RE: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-25 Thread Jun Li
Hi Roger > >> > >> Here, we should be checking if user needs to disable any OTG > >> features. So, > >> > >>if (dev->of_node) > >>of_usb_update_otg_caps(dev->of_node, >caps); > >> > >> Do you agree? > >> This means we need to change otg->caps from 'struct usb_otg_caps > *caps;' >

RE: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-25 Thread Jun Li
Hi Roger > >> > >> Here, we should be checking if user needs to disable any OTG > >> features. So, > >> > >>if (dev->of_node) > >>of_usb_update_otg_caps(dev->of_node, >caps); > >> > >> Do you agree? > >> This means we need to change otg->caps from 'struct usb_otg_caps > *caps;' >

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-25 Thread Roger Quadros
org; nsek...@ti.com; b-...@ti.com; linux-...@vger.kernel.org; >> linux-o...@vger.kernel.org; linux-kernel@vger.kernel.org; >> devicet...@vger.kernel.org >> Subject: Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core >> >> On Tue, May 24, 2016 at 12:45:46PM +0300, Roge

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-25 Thread Roger Quadros
sek...@ti.com; b-...@ti.com; linux-...@vger.kernel.org; >> linux-o...@vger.kernel.org; linux-kernel@vger.kernel.org; >> devicet...@vger.kernel.org >> Subject: Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core >> >> On Tue, May 24, 2016 at 12:45:46PM +0300, Roger Quadros wrote: &g

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-25 Thread Roger Quadros
On 25/05/16 05:44, Peter Chen wrote: > On Tue, May 24, 2016 at 12:45:46PM +0300, Roger Quadros wrote: >> Hi Peter, >> >> I have one question here. Please see below. >> >> On 13/05/16 13:03, Roger Quadros wrote: >>> It provides APIs for the following tasks >>> >>> - Registering an OTG/dual-role

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-25 Thread Roger Quadros
On 25/05/16 05:44, Peter Chen wrote: > On Tue, May 24, 2016 at 12:45:46PM +0300, Roger Quadros wrote: >> Hi Peter, >> >> I have one question here. Please see below. >> >> On 13/05/16 13:03, Roger Quadros wrote: >>> It provides APIs for the following tasks >>> >>> - Registering an OTG/dual-role

RE: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-24 Thread Jun Li
ernel.org; linux-kernel@vger.kernel.org; > devicet...@vger.kernel.org > Subject: Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core > > On Tue, May 24, 2016 at 12:45:46PM +0300, Roger Quadros wrote: > > Hi Peter, > > > > I have one question here. Please see below. &

RE: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-24 Thread Jun Li
kernel@vger.kernel.org; > devicet...@vger.kernel.org > Subject: Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core > > On Tue, May 24, 2016 at 12:45:46PM +0300, Roger Quadros wrote: > > Hi Peter, > > > > I have one question here. Please see below. > &g

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-24 Thread Peter Chen
On Tue, May 24, 2016 at 12:45:46PM +0300, Roger Quadros wrote: > Hi Peter, > > I have one question here. Please see below. > > On 13/05/16 13:03, Roger Quadros wrote: > > It provides APIs for the following tasks > > > > - Registering an OTG/dual-role capable controller > > - Registering Host

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-24 Thread Peter Chen
On Tue, May 24, 2016 at 12:45:46PM +0300, Roger Quadros wrote: > Hi Peter, > > I have one question here. Please see below. > > On 13/05/16 13:03, Roger Quadros wrote: > > It provides APIs for the following tasks > > > > - Registering an OTG/dual-role capable controller > > - Registering Host

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-24 Thread Roger Quadros
Hi Peter, I have one question here. Please see below. On 13/05/16 13:03, Roger Quadros wrote: > It provides APIs for the following tasks > > - Registering an OTG/dual-role capable controller > - Registering Host and Gadget controllers to OTG core > - Providing inputs to and kicking the OTG

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-24 Thread Roger Quadros
Hi Peter, I have one question here. Please see below. On 13/05/16 13:03, Roger Quadros wrote: > It provides APIs for the following tasks > > - Registering an OTG/dual-role capable controller > - Registering Host and Gadget controllers to OTG core > - Providing inputs to and kicking the OTG

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-23 Thread Roger Quadros
On 20/05/16 12:53, Peter Chen wrote: > On Fri, May 20, 2016 at 12:19:07PM +0300, Roger Quadros wrote: >> On 20/05/16 11:31, Roger Quadros wrote: >>> On 18/05/16 15:59, Roger Quadros wrote: Hi Peter, On 18/05/16 10:45, Peter Chen wrote: > > > On Mon, May 16, 2016 at 5:00

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-23 Thread Roger Quadros
On 20/05/16 12:53, Peter Chen wrote: > On Fri, May 20, 2016 at 12:19:07PM +0300, Roger Quadros wrote: >> On 20/05/16 11:31, Roger Quadros wrote: >>> On 18/05/16 15:59, Roger Quadros wrote: Hi Peter, On 18/05/16 10:45, Peter Chen wrote: > > > On Mon, May 16, 2016 at 5:00

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-20 Thread Peter Chen
On Fri, May 20, 2016 at 12:19:07PM +0300, Roger Quadros wrote: > On 20/05/16 11:31, Roger Quadros wrote: > > On 18/05/16 15:59, Roger Quadros wrote: > >> Hi Peter, > >> > >> On 18/05/16 10:45, Peter Chen wrote: > >>> > >>> > >>> On Mon, May 16, 2016 at 5:00 PM, Roger Quadros >>>

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-20 Thread Peter Chen
On Fri, May 20, 2016 at 12:19:07PM +0300, Roger Quadros wrote: > On 20/05/16 11:31, Roger Quadros wrote: > > On 18/05/16 15:59, Roger Quadros wrote: > >> Hi Peter, > >> > >> On 18/05/16 10:45, Peter Chen wrote: > >>> > >>> > >>> On Mon, May 16, 2016 at 5:00 PM, Roger Quadros >>>

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-20 Thread Roger Quadros
On 20/05/16 11:31, Roger Quadros wrote: > On 18/05/16 15:59, Roger Quadros wrote: >> Hi Peter, >> >> On 18/05/16 10:45, Peter Chen wrote: >>> >>> >>> On Mon, May 16, 2016 at 5:00 PM, Roger Quadros >> > wrote: >>> >>> On 13/05/16 13:03, Roger Quadros

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-20 Thread Roger Quadros
On 20/05/16 11:31, Roger Quadros wrote: > On 18/05/16 15:59, Roger Quadros wrote: >> Hi Peter, >> >> On 18/05/16 10:45, Peter Chen wrote: >>> >>> >>> On Mon, May 16, 2016 at 5:00 PM, Roger Quadros >> > wrote: >>> >>> On 13/05/16 13:03, Roger Quadros wrote: >>> > It

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-20 Thread Roger Quadros
On 18/05/16 15:59, Roger Quadros wrote: > Hi Peter, > > On 18/05/16 10:45, Peter Chen wrote: >> >> >> On Mon, May 16, 2016 at 5:00 PM, Roger Quadros > > wrote: >> >> On 13/05/16 13:03, Roger Quadros wrote: >> > It provides APIs for the following

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-20 Thread Roger Quadros
On 18/05/16 15:59, Roger Quadros wrote: > Hi Peter, > > On 18/05/16 10:45, Peter Chen wrote: >> >> >> On Mon, May 16, 2016 at 5:00 PM, Roger Quadros > > wrote: >> >> On 13/05/16 13:03, Roger Quadros wrote: >> > It provides APIs for the following tasks >> > >>

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-18 Thread Roger Quadros
Hi Peter, On 18/05/16 10:45, Peter Chen wrote: > > > On Mon, May 16, 2016 at 5:00 PM, Roger Quadros > wrote: > > On 13/05/16 13:03, Roger Quadros wrote: > > It provides APIs for the following tasks > > > > - Registering an OTG/dual-role

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-18 Thread Roger Quadros
Hi Peter, On 18/05/16 10:45, Peter Chen wrote: > > > On Mon, May 16, 2016 at 5:00 PM, Roger Quadros > wrote: > > On 13/05/16 13:03, Roger Quadros wrote: > > It provides APIs for the following tasks > > > > - Registering an OTG/dual-role capable controller

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-16 Thread Roger Quadros
On 13/05/16 13:03, Roger Quadros wrote: > It provides APIs for the following tasks > > - Registering an OTG/dual-role capable controller > - Registering Host and Gadget controllers to OTG core > - Providing inputs to and kicking the OTG state machine > > Provide a dual-role device (DRD) state

Re: [PATCH v8 08/14] usb: otg: add OTG/dual-role core

2016-05-16 Thread Roger Quadros
On 13/05/16 13:03, Roger Quadros wrote: > It provides APIs for the following tasks > > - Registering an OTG/dual-role capable controller > - Registering Host and Gadget controllers to OTG core > - Providing inputs to and kicking the OTG state machine > > Provide a dual-role device (DRD) state