Re: [PATCH v4 8/8] Documentation: usb: chipidea: Update test procedure for HNP polling

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Apr 30, 2015 at 04:46:01PM +0800, Peter Chen wrote:
> On Tue, Apr 28, 2015 at 11:56:27AM -0500, Felipe Balbi wrote:
> > On Thu, Mar 19, 2015 at 11:11:25AM +0800, Li Jun wrote:
> > > From: Li Jun <b47624@xxxxxxxxxxxxx>
> > > 
> > > Update HNP test procedure as HNP polling is supported.
> > > 
> > > Signed-off-by: Li Jun <jun.li@xxxxxxxxxxxxx>
> > > ---
> > >  Documentation/usb/chipidea.txt |   11 ++++++-----
> > >  1 file changed, 6 insertions(+), 5 deletions(-)
> > > 
> > > diff --git a/Documentation/usb/chipidea.txt b/Documentation/usb/chipidea.txt
> > > index 3f848c1..ff965b2 100644
> > > --- a/Documentation/usb/chipidea.txt
> > > +++ b/Documentation/usb/chipidea.txt
> > > @@ -26,16 +26,17 @@ cat /sys/kernel/debug/ci_hdrc.0/registers
> > >     On B-device:
> > >     echo 1 > /sys/bus/platform/devices/ci_hdrc.0/inputs/b_bus_req
> > >  
> > > -   if HNP polling is not supported, also need:
> > > -   On A-device:
> > > -   echo 0 > /sys/bus/platform/devices/ci_hdrc.0/inputs/a_bus_req
> > > -
> > >     B-device should take host role and enumrate A-device.
> > >  
> > >  4) A-device switch back to host.
> > > -   On B-device:
> > > +   on B-device:
> > >     echo 0 > /sys/bus/platform/devices/ci_hdrc.0/inputs/b_bus_req
> > >  
> > > +   or, by introducing HNP polling, B-Host can know when A-peripheral wish to be
> > > +   host role, so this role switch also can be trigged in A-peripheral side by
> > > +   answering the polling from B-Host, this can be done on A-device:
> > > +   echo 1 > /sys/bus/platform/devices/ci_hdrc.0/inputs/a_bus_req
> > 
> > why is this still chipidea specific ? Why isn't this part on udc-core so
> > that all OTG UDCs can support this too ?
> > 
> 
> Currently, hnp polling is enabled during the otg fsm handling, but there
> is no otg framework (like udc core, Roger Quadros is doing that), so
> no common sys entry for otg fsm variables to control otg fsm.

then we wait for whatever Roger's doing, otherwise this will be another
ABI to support forever.

-- 
balbi

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux