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

2017-01-20 Thread Vivek Gautam
On 01/20/2017 02:00 PM, Roger Quadros wrote: Vivek, On 19/01/17 17:15, vivek.gau...@codeaurora.org wrote: Hi Roger, On 2017-01-19 17:45, Roger Quadros wrote: Vivek, On 19/01/17 13:56, Vivek Gautam wrote: Hi, On Wed, Jun 22, 2016 at 2:00 PM, Roger Quadros wrote: Luckily

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

2017-01-20 Thread Vivek Gautam
On 01/20/2017 02:00 PM, Roger Quadros wrote: Vivek, On 19/01/17 17:15, vivek.gau...@codeaurora.org wrote: Hi Roger, On 2017-01-19 17:45, Roger Quadros wrote: Vivek, On 19/01/17 13:56, Vivek Gautam wrote: Hi, On Wed, Jun 22, 2016 at 2:00 PM, Roger Quadros wrote: Luckily hit this

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

2017-01-20 Thread Roger Quadros
Vivek, On 19/01/17 17:15, vivek.gau...@codeaurora.org wrote: > Hi Roger, > > On 2017-01-19 17:45, Roger Quadros wrote: >> Vivek, >> >> On 19/01/17 13:56, Vivek Gautam wrote: >>> Hi, >>> >>> >>> On Wed, Jun 22, 2016 at 2:00 PM, Roger Quadros wrote: >>> >>> Luckily hit this thread

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

2017-01-20 Thread Roger Quadros
Vivek, On 19/01/17 17:15, vivek.gau...@codeaurora.org wrote: > Hi Roger, > > On 2017-01-19 17:45, Roger Quadros wrote: >> Vivek, >> >> On 19/01/17 13:56, Vivek Gautam wrote: >>> Hi, >>> >>> >>> On Wed, Jun 22, 2016 at 2:00 PM, Roger Quadros wrote: >>> >>> Luckily hit this thread while checking

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

2017-01-19 Thread vivek . gautam
Hi Roger, On 2017-01-19 17:45, Roger Quadros wrote: Vivek, On 19/01/17 13:56, Vivek Gautam wrote: Hi, On Wed, Jun 22, 2016 at 2:00 PM, Roger Quadros wrote: Luckily hit this thread while checking about DRD role functionality for DWC3. On 22/06/16 11:14, Felipe Balbi

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

2017-01-19 Thread vivek . gautam
Hi Roger, On 2017-01-19 17:45, Roger Quadros wrote: Vivek, On 19/01/17 13:56, Vivek Gautam wrote: Hi, On Wed, Jun 22, 2016 at 2:00 PM, Roger Quadros wrote: Luckily hit this thread while checking about DRD role functionality for DWC3. On 22/06/16 11:14, Felipe Balbi wrote: Hi, Roger

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

2017-01-19 Thread Roger Quadros
Vivek, On 19/01/17 13:56, Vivek Gautam wrote: > Hi, > > > On Wed, Jun 22, 2016 at 2:00 PM, Roger Quadros wrote: > > Luckily hit this thread while checking about DRD role functionality for DWC3. > >> On 22/06/16 11:14, Felipe Balbi wrote: >>> >>> Hi, >>> >>> Roger Quadros

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

2017-01-19 Thread Roger Quadros
Vivek, On 19/01/17 13:56, Vivek Gautam wrote: > Hi, > > > On Wed, Jun 22, 2016 at 2:00 PM, Roger Quadros wrote: > > Luckily hit this thread while checking about DRD role functionality for DWC3. > >> On 22/06/16 11:14, Felipe Balbi wrote: >>> >>> Hi, >>> >>> Roger Quadros writes:

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

2017-01-19 Thread Vivek Gautam
Hi, On Wed, Jun 22, 2016 at 2:00 PM, Roger Quadros wrote: Luckily hit this thread while checking about DRD role functionality for DWC3. > On 22/06/16 11:14, Felipe Balbi wrote: >> >> Hi, >> >> Roger Quadros writes: >>> For the real use case, some Carplay

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

2017-01-19 Thread Vivek Gautam
Hi, On Wed, Jun 22, 2016 at 2:00 PM, Roger Quadros wrote: Luckily hit this thread while checking about DRD role functionality for DWC3. > On 22/06/16 11:14, Felipe Balbi wrote: >> >> Hi, >> >> Roger Quadros writes: >>> For the real use case, some Carplay platforms need it. >>

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

2016-06-23 Thread Yoshihiro Shimoda
Hi Roger-san, < snip > > commit 11c011a5e777c83819078a18672543f04482b3ec > Author: Srinivas Kandagatla > Date: Thu May 19 11:12:56 2016 +0100 > > usb: echi-hcd: Add ehci_setup check before echi_shutdown > > > > In

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

2016-06-23 Thread Yoshihiro Shimoda
Hi Roger-san, < snip > > commit 11c011a5e777c83819078a18672543f04482b3ec > Author: Srinivas Kandagatla > Date: Thu May 19 11:12:56 2016 +0100 > > usb: echi-hcd: Add ehci_setup check before echi_shutdown > > > > In some cases, the USB code

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

2016-06-23 Thread Yoshihiro Shimoda
Hi, > From: Peter Chen > Sent: Wednesday, June 22, 2016 12:34 PM > > On Tue, Jun 21, 2016 at 05:47:47PM +0300, Felipe Balbi wrote: > > > > Hi, > > > > Peter Chen writes: > > >> >> >> >> >>> + * @otg_dev: OTG controller device, if needs to be used > > >> >> >> >> >>> with

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

2016-06-23 Thread Yoshihiro Shimoda
Hi, > From: Peter Chen > Sent: Wednesday, June 22, 2016 12:34 PM > > On Tue, Jun 21, 2016 at 05:47:47PM +0300, Felipe Balbi wrote: > > > > Hi, > > > > Peter Chen writes: > > >> >> >> >> >>> + * @otg_dev: OTG controller device, if needs to be used > > >> >> >> >> >>> with OTG core. > > >> >> >>

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

2016-06-22 Thread Roger Quadros
On 22/06/16 11:14, Felipe Balbi wrote: > > Hi, > > Roger Quadros writes: >> For the real use case, some Carplay platforms need it. > > Carplay does *NOT* rely on OTG. Apple has its own proprietary and > closed > specification which is not

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

2016-06-22 Thread Roger Quadros
On 22/06/16 11:14, Felipe Balbi wrote: > > Hi, > > Roger Quadros writes: >> For the real use case, some Carplay platforms need it. > > Carplay does *NOT* rely on OTG. Apple has its own proprietary and > closed > specification which is not OTG-compliant.

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

2016-06-22 Thread Felipe Balbi
Hi, Roger Quadros writes: > For the real use case, some Carplay platforms need it. Carplay does *NOT* rely on OTG. Apple has its own proprietary and closed specification which is not OTG-compliant. >>> >>> Yes, it is

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

2016-06-22 Thread Felipe Balbi
Hi, Roger Quadros writes: > For the real use case, some Carplay platforms need it. Carplay does *NOT* rely on OTG. Apple has its own proprietary and closed specification which is not OTG-compliant. >>> >>> Yes, it is not

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

2016-06-22 Thread Felipe Balbi
Hi, Peter Chen writes: >> >> >> >> So far, I haven't seen anybody talking about real USB OTG (the spec) >> >> >> >> when they say OTG. Usually they just mean "a method for swapping >> >> >> >> between >> >> >> >> host and peripheral roles, but we really don't want all

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

2016-06-22 Thread Felipe Balbi
Hi, Peter Chen writes: >> >> >> >> So far, I haven't seen anybody talking about real USB OTG (the spec) >> >> >> >> when they say OTG. Usually they just mean "a method for swapping >> >> >> >> between >> >> >> >> host and peripheral roles, but we really don't want all the extra >> >> >> >>

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

2016-06-22 Thread Felipe Balbi
Hi, Peter Chen writes: >> >> >> >> > So, unless we use OTG FSM defined in OTG spec, we should not >> >> >> >> > mention >> >> >> >> > "OTG" in Linux, right? >> >> >> >> >> >> >> >> to avoid confusion with the terminology, yes. With that settled, >> >> >> >> let's >> >>

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

2016-06-22 Thread Felipe Balbi
Hi, Peter Chen writes: >> >> >> >> > So, unless we use OTG FSM defined in OTG spec, we should not >> >> >> >> > mention >> >> >> >> > "OTG" in Linux, right? >> >> >> >> >> >> >> >> to avoid confusion with the terminology, yes. With that settled, >> >> >> >> let's >> >> >> >> figure out how

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

2016-06-22 Thread Peter Chen
On Wed, Jun 22, 2016 at 09:56:22AM +0300, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: > >> Peter Chen writes: > >> >> >> So far, I haven't seen anybody talking about real USB OTG (the spec) > >> >> >> when they say OTG. Usually they just

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

2016-06-22 Thread Peter Chen
On Wed, Jun 22, 2016 at 09:56:22AM +0300, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: > >> Peter Chen writes: > >> >> >> So far, I haven't seen anybody talking about real USB OTG (the spec) > >> >> >> when they say OTG. Usually they just mean "a method for swapping > >> >> >> between >

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

2016-06-22 Thread Roger Quadros
Hi Felipe, On 22/06/16 09:56, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: >>> Peter Chen writes: >>> So far, I haven't seen anybody talking about real USB OTG (the spec) >>> when they say OTG. Usually they just mean "a method for

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

2016-06-22 Thread Roger Quadros
Hi Felipe, On 22/06/16 09:56, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: >>> Peter Chen writes: >>> So far, I haven't seen anybody talking about real USB OTG (the spec) >>> when they say OTG. Usually they just mean "a method for swapping between >>> host and peripheral

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

2016-06-22 Thread Peter Chen
On Wed, Jun 22, 2016 at 09:51:19AM +0300, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: > >> >> >> > So, unless we use OTG FSM defined in OTG spec, we should not > >> >> >> > mention > >> >> >> > "OTG" in Linux, right? > >> >> >> > >> >> >> to avoid confusion with

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

2016-06-22 Thread Peter Chen
On Wed, Jun 22, 2016 at 09:51:19AM +0300, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: > >> >> >> > So, unless we use OTG FSM defined in OTG spec, we should not > >> >> >> > mention > >> >> >> > "OTG" in Linux, right? > >> >> >> > >> >> >> to avoid confusion with the terminology, yes.

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

2016-06-22 Thread Felipe Balbi
Hi, Peter Chen writes: >> Peter Chen writes: >> >> >> So far, I haven't seen anybody talking about real USB OTG (the spec) >> >> >> when they say OTG. Usually they just mean "a method for swapping >> >> >> between >> >> >> host and peripheral

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

2016-06-22 Thread Felipe Balbi
Hi, Peter Chen writes: >> Peter Chen writes: >> >> >> So far, I haven't seen anybody talking about real USB OTG (the spec) >> >> >> when they say OTG. Usually they just mean "a method for swapping >> >> >> between >> >> >> host and peripheral roles, but we really don't want all the extra cost

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

2016-06-22 Thread Felipe Balbi
Hi, Peter Chen writes: >> >> >> > So, unless we use OTG FSM defined in OTG spec, we should not mention >> >> >> > "OTG" in Linux, right? >> >> >> >> >> >> to avoid confusion with the terminology, yes. With that settled, let's >> >> >> figure out how you can deliver what

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

2016-06-22 Thread Felipe Balbi
Hi, Peter Chen writes: >> >> >> > So, unless we use OTG FSM defined in OTG spec, we should not mention >> >> >> > "OTG" in Linux, right? >> >> >> >> >> >> to avoid confusion with the terminology, yes. With that settled, let's >> >> >> figure out how you can deliver what your marketting guys

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

2016-06-21 Thread Peter Chen
On Tue, Jun 21, 2016 at 05:47:47PM +0300, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: > >> >> >> >> >>> + * @otg_dev: OTG controller device, if needs to be used with > >> >> >> >> >>> OTG core. > >> >> >> >> >> > >> >> >> >> >> do you really know of any platform

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

2016-06-21 Thread Peter Chen
On Tue, Jun 21, 2016 at 05:47:47PM +0300, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: > >> >> >> >> >>> + * @otg_dev: OTG controller device, if needs to be used with > >> >> >> >> >>> OTG core. > >> >> >> >> >> > >> >> >> >> >> do you really know of any platform which has a separate

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

2016-06-21 Thread Peter Chen
On Tue, Jun 21, 2016 at 01:02:59PM +0300, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: > >> >> So far, I haven't seen anybody talking about real USB OTG (the spec) > >> >> when they say OTG. Usually they just mean "a method for swapping between > >> >> host and

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

2016-06-21 Thread Peter Chen
On Tue, Jun 21, 2016 at 01:02:59PM +0300, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: > >> >> So far, I haven't seen anybody talking about real USB OTG (the spec) > >> >> when they say OTG. Usually they just mean "a method for swapping between > >> >> host and peripheral roles, but we

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

2016-06-21 Thread Felipe Balbi
Hi, Peter Chen writes: >> >> >> >> >>> + * @otg_dev: OTG controller device, if needs to be used with >> >> >> >> >>> OTG core. >> >> >> >> >> >> >> >> >> >> do you really know of any platform which has a separate OTG >> >> >> >> >> controller? >> >> >> >> >> >> >> >>

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

2016-06-21 Thread Felipe Balbi
Hi, Peter Chen writes: >> >> >> >> >>> + * @otg_dev: OTG controller device, if needs to be used with >> >> >> >> >>> OTG core. >> >> >> >> >> >> >> >> >> >> do you really know of any platform which has a separate OTG >> >> >> >> >> controller? >> >> >> >> >> >> >> >> >> > >> >> >> >> >

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

2016-06-21 Thread Peter Chen
On Tue, Jun 21, 2016 at 03:35:00PM +0300, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: > >> >> >> >>> + * @otg_dev: OTG controller device, if needs to be used with > >> >> >> >>> OTG core. > >> >> >> >> > >> >> >> >> do you really know of any platform which has a

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

2016-06-21 Thread Peter Chen
On Tue, Jun 21, 2016 at 03:35:00PM +0300, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: > >> >> >> >>> + * @otg_dev: OTG controller device, if needs to be used with > >> >> >> >>> OTG core. > >> >> >> >> > >> >> >> >> do you really know of any platform which has a separate OTG > >> >>

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

2016-06-21 Thread Felipe Balbi
Hi, Peter Chen writes: >> >> >> >>> + * @otg_dev: OTG controller device, if needs to be used with OTG >> >> >> >>> core. >> >> >> >> >> >> >> >> do you really know of any platform which has a separate OTG >> >> >> >> controller? >> >> >> >> >> >> >> > >> >> >> >

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

2016-06-21 Thread Felipe Balbi
Hi, Peter Chen writes: >> >> >> >>> + * @otg_dev: OTG controller device, if needs to be used with OTG >> >> >> >>> core. >> >> >> >> >> >> >> >> do you really know of any platform which has a separate OTG >> >> >> >> controller? >> >> >> >> >> >> >> > >> >> >> > Andrew had pointed out in

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

2016-06-21 Thread Peter Chen
On Tue, Jun 21, 2016 at 11:18:21AM +0300, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: > >> Peter Chen writes: > >> >> >>> + > >> >> >>> + /* start host */ > >> >> >>> + ret = hcd_ops->add(otg->primary_hcd.hcd, > >> >> >>>

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

2016-06-21 Thread Peter Chen
On Tue, Jun 21, 2016 at 11:18:21AM +0300, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: > >> Peter Chen writes: > >> >> >>> + > >> >> >>> + /* start host */ > >> >> >>> + ret = hcd_ops->add(otg->primary_hcd.hcd, > >> >> >>> +

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

2016-06-21 Thread Felipe Balbi
Hi, Tony Lindgren writes: > * Felipe Balbi [160621 03:06]: >> 8<-- >> vbus = read(VBUS_STATE); /* could be a gpio_get_value() */ >> id = read(ID_STATE); /* could be a

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

2016-06-21 Thread Felipe Balbi
Hi, Tony Lindgren writes: > * Felipe Balbi [160621 03:06]: >> 8<-- >> vbus = read(VBUS_STATE); /* could be a gpio_get_value() */ >> id = read(ID_STATE); /* could be a gpio_get_value() */ >> >>

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

2016-06-21 Thread Tony Lindgren
* Felipe Balbi [160621 03:06]: > 8<-- > vbus = read(VBUS_STATE); /* could be a gpio_get_value() */ > id = read(ID_STATE); /* could be a gpio_get_value() */ > > set_role(id); >

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

2016-06-21 Thread Tony Lindgren
* Felipe Balbi [160621 03:06]: > 8<-- > vbus = read(VBUS_STATE); /* could be a gpio_get_value() */ > id = read(ID_STATE); /* could be a gpio_get_value() */ > > set_role(id); > set_vbus(vbus); We

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

2016-06-21 Thread Felipe Balbi
Hi, Peter Chen writes: >> >> So far, I haven't seen anybody talking about real USB OTG (the spec) >> >> when they say OTG. Usually they just mean "a method for swapping between >> >> host and peripheral roles, but we really don't want all the extra cost >> >> of the OTG

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

2016-06-21 Thread Felipe Balbi
Hi, Peter Chen writes: >> >> So far, I haven't seen anybody talking about real USB OTG (the spec) >> >> when they say OTG. Usually they just mean "a method for swapping between >> >> host and peripheral roles, but we really don't want all the extra cost >> >> of the OTG specification". >> >>

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

2016-06-21 Thread Peter Chen
On Tue, Jun 21, 2016 at 10:26:00AM +0300, Felipe Balbi wrote: > > Hi, > > >> > >> So far, I haven't seen anybody talking about real USB OTG (the spec) > >> when they say OTG. Usually they just mean "a method for swapping between > >> host and peripheral roles, but we really don't want all the

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

2016-06-21 Thread Peter Chen
On Tue, Jun 21, 2016 at 10:26:00AM +0300, Felipe Balbi wrote: > > Hi, > > >> > >> So far, I haven't seen anybody talking about real USB OTG (the spec) > >> when they say OTG. Usually they just mean "a method for swapping between > >> host and peripheral roles, but we really don't want all the

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

2016-06-21 Thread Felipe Balbi
Hi, Peter Chen writes: >> Peter Chen writes: >> >> >>> + >> >> >>> + /* start host */ >> >> >>> + ret = hcd_ops->add(otg->primary_hcd.hcd, >> >> >>> +otg->primary_hcd.irqnum, >> >> >>> +

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

2016-06-21 Thread Felipe Balbi
Hi, Peter Chen writes: >> Peter Chen writes: >> >> >>> + >> >> >>> + /* start host */ >> >> >>> + ret = hcd_ops->add(otg->primary_hcd.hcd, >> >> >>> +otg->primary_hcd.irqnum, >> >> >>> +

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

2016-06-21 Thread Peter Chen
On Tue, Jun 21, 2016 at 10:19:32AM +0300, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: > >> >>> + > >> >>> + /* start host */ > >> >>> + ret = hcd_ops->add(otg->primary_hcd.hcd, > >> >>> +

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

2016-06-21 Thread Peter Chen
On Tue, Jun 21, 2016 at 10:19:32AM +0300, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: > >> >>> + > >> >>> + /* start host */ > >> >>> + ret = hcd_ops->add(otg->primary_hcd.hcd, > >> >>> + otg->primary_hcd.irqnum, > >> >>> +

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

2016-06-21 Thread Felipe Balbi
Hi, Yoshihiro Shimoda writes: > Hi Roger, > >> From: Roger Quadros >> Sent: Monday, June 20, 2016 7:13 PM >> >> Hi, >> >> On 20/06/16 10:45, Felipe Balbi wrote: > < snip > >> >> diff --git a/include/linux/usb/gadget.h b/include/linux/usb/gadget.h >> >> index

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

2016-06-21 Thread Felipe Balbi
Hi, Yoshihiro Shimoda writes: > Hi Roger, > >> From: Roger Quadros >> Sent: Monday, June 20, 2016 7:13 PM >> >> Hi, >> >> On 20/06/16 10:45, Felipe Balbi wrote: > < snip > >> >> diff --git a/include/linux/usb/gadget.h b/include/linux/usb/gadget.h >> >> index f4fc0aa..1d74fb8 100644 >> >> ---

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

2016-06-21 Thread Felipe Balbi
Hi, Peter Chen writes: >> >> > 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 >> >>

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

2016-06-21 Thread Felipe Balbi
Hi, Peter Chen writes: >> >> > 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 >> >> >> >> I think I have

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

2016-06-21 Thread Felipe Balbi
Hi, Peter Chen writes: >> >>> + >> >>> +/* start host */ >> >>> +ret = hcd_ops->add(otg->primary_hcd.hcd, >> >>> + otg->primary_hcd.irqnum, >> >>> +

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

2016-06-21 Thread Felipe Balbi
Hi, Peter Chen writes: >> >>> + >> >>> +/* start host */ >> >>> +ret = hcd_ops->add(otg->primary_hcd.hcd, >> >>> + otg->primary_hcd.irqnum, >> >>> + otg->primary_hcd.irqflags); >> >> >> >> this

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

2016-06-21 Thread Peter Chen
On Mon, Jun 20, 2016 at 03:03:37PM +0300, Felipe Balbi wrote: > > Hi, > > >>> + > >>> + /* start host */ > >>> + ret = hcd_ops->add(otg->primary_hcd.hcd, > >>> +otg->primary_hcd.irqnum, > >>> +otg->primary_hcd.irqflags); >

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

2016-06-21 Thread Peter Chen
On Mon, Jun 20, 2016 at 03:03:37PM +0300, Felipe Balbi wrote: > > Hi, > > >>> + > >>> + /* start host */ > >>> + ret = hcd_ops->add(otg->primary_hcd.hcd, > >>> +otg->primary_hcd.irqnum, > >>> +otg->primary_hcd.irqflags); >

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

2016-06-21 Thread Peter Chen
On Mon, Jun 20, 2016 at 03:08:15PM +0300, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: > >> Roger Quadros writes: > >> > It provides APIs for the following tasks > >> > > >> > - Registering an OTG/dual-role capable controller > >> > - Registering

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

2016-06-21 Thread Peter Chen
On Mon, Jun 20, 2016 at 03:08:15PM +0300, Felipe Balbi wrote: > > Hi, > > Peter Chen writes: > >> Roger Quadros writes: > >> > It provides APIs for the following tasks > >> > > >> > - Registering an OTG/dual-role capable controller > >> > - Registering Host and Gadget controllers to OTG core >

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

2016-06-20 Thread Yoshihiro Shimoda
Hi Roger, > From: Roger Quadros > Sent: Monday, June 20, 2016 7:13 PM > > Hi, > > On 20/06/16 10:45, Felipe Balbi wrote: < snip > > >> diff --git a/include/linux/usb/gadget.h b/include/linux/usb/gadget.h > >> index f4fc0aa..1d74fb8 100644 > >> --- a/include/linux/usb/gadget.h > >> +++

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

2016-06-20 Thread Yoshihiro Shimoda
Hi Roger, > From: Roger Quadros > Sent: Monday, June 20, 2016 7:13 PM > > Hi, > > On 20/06/16 10:45, Felipe Balbi wrote: < snip > > >> diff --git a/include/linux/usb/gadget.h b/include/linux/usb/gadget.h > >> index f4fc0aa..1d74fb8 100644 > >> --- a/include/linux/usb/gadget.h > >> +++

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

2016-06-20 Thread Felipe Balbi
Hi, Roger Quadros writes: > diff --git a/drivers/usb/Kconfig b/drivers/usb/Kconfig > index 8689dcb..ed596ec 100644 > --- a/drivers/usb/Kconfig > +++ b/drivers/usb/Kconfig > @@ -32,6 +32,23 @@ if USB_SUPPORT > config USB_COMMON > tristate >

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

2016-06-20 Thread Felipe Balbi
Hi, Roger Quadros writes: > diff --git a/drivers/usb/Kconfig b/drivers/usb/Kconfig > index 8689dcb..ed596ec 100644 > --- a/drivers/usb/Kconfig > +++ b/drivers/usb/Kconfig > @@ -32,6 +32,23 @@ if USB_SUPPORT > config USB_COMMON > tristate > > +config

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

2016-06-20 Thread Roger Quadros
On 20/06/16 15:03, Felipe Balbi wrote: > > Hi, > > Roger Quadros writes: >>> Roger Quadros writes: It provides APIs for the following tasks - Registering an OTG/dual-role capable controller - Registering Host and Gadget controllers to OTG core

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

2016-06-20 Thread Roger Quadros
On 20/06/16 15:03, Felipe Balbi wrote: > > Hi, > > Roger Quadros writes: >>> Roger Quadros writes: 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

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

2016-06-20 Thread Felipe Balbi
Hi, Roger Quadros writes: >> Roger Quadros writes: >>> 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

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

2016-06-20 Thread Felipe Balbi
Hi, Roger Quadros writes: >> Roger Quadros writes: >>> 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 >> >> I think I have

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

2016-06-20 Thread Felipe Balbi
Hi, Peter Chen writes: >> Roger Quadros writes: >> > 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

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

2016-06-20 Thread Felipe Balbi
Hi, Peter Chen writes: >> Roger Quadros writes: >> > 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 >> >> I think I

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

2016-06-20 Thread Peter Chen
On Mon, Jun 20, 2016 at 10:45:31AM +0300, Felipe Balbi wrote: > > Hi, > > Roger Quadros writes: > > It provides APIs for the following tasks > > > > - Registering an OTG/dual-role capable controller > > - Registering Host and Gadget controllers to OTG core > > - Providing inputs

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

2016-06-20 Thread Peter Chen
On Mon, Jun 20, 2016 at 10:45:31AM +0300, Felipe Balbi wrote: > > Hi, > > Roger Quadros writes: > > 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

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

2016-06-20 Thread Roger Quadros
Hi, On 20/06/16 10:45, Felipe Balbi wrote: > > Hi, > > Roger Quadros writes: >> 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 v11 08/14] usb: otg: add OTG/dual-role core

2016-06-20 Thread Roger Quadros
Hi, On 20/06/16 10:45, Felipe Balbi wrote: > > Hi, > > Roger Quadros writes: >> 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 >

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

2016-06-20 Thread Felipe Balbi
Hi, Roger Quadros writes: > 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 I think I have already mentioned this, but

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

2016-06-20 Thread Felipe Balbi
Hi, Roger Quadros writes: > 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 I think I have already mentioned this, but after over 10

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

2016-06-13 Thread Peter Chen
On Mon, Jun 13, 2016 at 10:56:30AM +0300, 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

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

2016-06-13 Thread Peter Chen
On Mon, Jun 13, 2016 at 10:56:30AM +0300, 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

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

2016-06-13 Thread Roger Quadros
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 machine. DRD mode is a reduced functionality OTG mode.

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

2016-06-13 Thread Roger Quadros
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 machine. DRD mode is a reduced functionality OTG mode.