Re: [PATCH 04/21] usb: chipidea: Only read/write OTGSC from one place

2016-06-28 Thread Peter Chen
On Mon, Jun 27, 2016 at 12:07:54PM -0700, Stephen Boyd wrote: > Quoting Jun Li (2016-06-27 01:04:39) > > > diff --git a/drivers/usb/chipidea/otg.c b/drivers/usb/chipidea/otg.c index > > > 03b6743461d1..763a8332b009 100644 > > > --- a/drivers/usb/chipidea/otg.c > > > +++ b/drivers/usb/chipidea/otg.c

RE: [PATCH 04/21] usb: chipidea: Only read/write OTGSC from one place

2016-06-27 Thread Stephen Boyd
Quoting Jun Li (2016-06-27 01:04:39) > > diff --git a/drivers/usb/chipidea/otg.c b/drivers/usb/chipidea/otg.c index > > 03b6743461d1..763a8332b009 100644 > > --- a/drivers/usb/chipidea/otg.c > > +++ b/drivers/usb/chipidea/otg.c > > @@ -104,7 +104,32 @@ void ci_handle_vbus_change(struct ci_hdrc *ci)

RE: [PATCH 04/21] usb: chipidea: Only read/write OTGSC from one place

2016-06-27 Thread Jun Li
l.org; > linux-arm-...@vger.kernel.org; Andy Gross ; Bjorn > Andersson ; Neil Armstrong > ; Arnd Bergmann ; Felipe Balbi > ; Peter Chen ; Greg Kroah-Hartman > ; Ivan T. Ivanov > Subject: [PATCH 04/21] usb: chipidea: Only read/write OTGSC from one place > > With the id and

[PATCH 04/21] usb: chipidea: Only read/write OTGSC from one place

2016-06-26 Thread Stephen Boyd
With the id and vbus detection done via extcon we need to make sure we poll the status of OTGSC properly by considering what the extcon is saying, and not just what the register is saying. Let's move this hw_wait_reg() function to the only place it's used and simplify it for polling the OTGSC regis