RE: [PATCH v6 00/10] add HNP polling support for usb otg fsm

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

 



Hi Felipe

I didn't get any response for this request, if you need a resend
of the whole patchset, please let me know.

Li Jun

> -----Original Message-----
> From: linux-usb-owner@xxxxxxxxxxxxxxx [mailto:linux-usb-
> owner@xxxxxxxxxxxxxxx] On Behalf Of Jun Li
> Sent: Monday, February 15, 2016 10:20 AM
> To: Felipe Balbi <balbi@xxxxxxxxxx>
> Cc: linux-usb@xxxxxxxxxxxxxxx; Peter Chen <peter.chen@xxxxxxx>;
> hzpeterchen@xxxxxxxxx; gregkh@xxxxxxxxxxxxxxxxxxx
> Subject: RE: [PATCH v6 00/10] add HNP polling support for usb otg fsm
> 
> Resend with Felipe's new email address
> 
> > -----Original Message-----
> > From: Jun Li
> > Sent: Monday, February 15, 2016 10:16 AM
> > To: balbi@xxxxxx
> > Cc: linux-usb@xxxxxxxxxxxxxxx; Peter Chen <peter.chen@xxxxxxx>;
> > hzpeterchen@xxxxxxxxx; gregkh@xxxxxxxxxxxxxxxxxxx; Jun Li
> > <jun.li@xxxxxxx>
> > Subject: RE: [PATCH v6 00/10] add HNP polling support for usb otg fsm
> >
> > Ping...
> >
> > > -----Original Message-----
> > > From: linux-usb-owner@xxxxxxxxxxxxxxx [mailto:linux-usb-
> > > owner@xxxxxxxxxxxxxxx] On Behalf Of Jun Li
> > > Sent: Thursday, January 28, 2016 2:50 PM
> > > To: balbi@xxxxxx
> > > Cc: linux-usb@xxxxxxxxxxxxxxx; Peter Chen <peter.chen@xxxxxxx>;
> > > hzpeterchen@xxxxxxxxx; gregkh@xxxxxxxxxxxxxxxxxxx
> > > Subject: RE: [PATCH v6 00/10] add HNP polling support for usb otg
> > > fsm
> > >
> > > Hi Felipe,
> > >
> > > > -----Original Message-----
> > > > From: Jun Li
> > > > Sent: Tuesday, January 26, 2016 3:19 PM
> > > > To: Peter Chen <peter.chen@xxxxxxx>; hzpeterchen@xxxxxxxxx;
> > > > balbi@xxxxxx; gregkh@xxxxxxxxxxxxxxxxxxx
> > > > Cc: linux-usb@xxxxxxxxxxxxxxx; Jun Li <jun.li@xxxxxxx>
> > > > Subject: [PATCH v6 00/10] add HNP polling support for usb otg fsm
> > > >
> > > > HNP polling is a mechanism which allows the OTG device currently
> > > > acting as host to determine when the other attached OTG device
> > > > wishes to take the host role. When an OTG host, which supports
> > > > HNP, is connected to an OTG peripheral which also supports HNP it
> > > > shall poll the peripheral regularly to determine whether it
> > > > requires a role-swap and grant this at the earliest opportunity.
> > > > This patchset adds OTG HNP polling support, and enable for
> > > > chipidea usb otg fsm driver, more patches for pass OTG
> > > > certification will come
> > > later.
> > > >
> > > > changes for v6:
> > > > - Remove patch of disable irq while stop host role.
> > > > - Split B_AIDL_BDIS timer patch to be 2 patches(9/10 and 10/10).
> > > > - Add Peter's ack for patch 3, 7, 8/10.
> > > >
> > > > changes for v5:
> > > > - Instead of use stack memory, use kmalloc to allocate memory for
> host
> > > >   request flag one byte buffer(DMA read), and add host_req_flag
> > pointer
> > > >   in struct otg_fsm for reference to that flag buffer.
> > > > - Add one patch to disable irq while stop host role(7/10).
> > > > - Add one patch to fix B_AIDL_BDIS timing issue(10/10).
> > > >
> > > > changes for v4:
> > > > - Add OTG HNP capable check for connected device before sending
> > > > HNP polling
> > > >   in patch 3/8.
> > > > - Add comment to explain HNP test update in chipidea.txt in patch
> 8/8.
> > > > - Fix some typo.
> > > > - Add Peter's Ack in patch 1,2,4,5,6,7/8 of the series.
> > > >
> > > > Li Jun (10):
> > > >   usb: gadget: add hnp_polling_support and host_request_flag in
> > > >     usb_gadget
> > > >   usb: add OTG status selector definition for HNP polling
> > > >   usb: common: otg-fsm: add HNP polling support
> > > >   usb: chipidea: udc: bypass otg status selector handling to gadget
> > > >     driver
> > > >   usb: gadget: composite: handle otg status selector request from
> OTG
> > > >     host
> > > >   usb: chipidea: otg: set host_request_flag for gadget
> > > >   usb: chipidea: otg: enable HNP polling support for gadget and host
> > > >   Documentation: usb: chipidea: Update test procedure for HNP
> polling
> > > >   usb: otg-fsm: add B_AIDL_BDIS timer
> > > >   usb: chipidea: otg: add A idle to B disconnect timer
> > > >
> > > >  Documentation/usb/chipidea.txt   |  9 +++--
> > > >  drivers/usb/chipidea/otg_fsm.c   | 29 ++++++++++++--
> > > >  drivers/usb/chipidea/otg_fsm.h   |  2 +
> > > >  drivers/usb/chipidea/udc.c       |  3 +-
> > > >  drivers/usb/common/usb-otg-fsm.c | 87
> > > > ++++++++++++++++++++++++++++++++++++++++
> > > >  drivers/usb/gadget/composite.c   | 25 ++++++++----
> > > >  include/linux/usb/gadget.h       |  6 +++
> > > >  include/linux/usb/otg-fsm.h      | 15 +++++++
> > > >  include/uapi/linux/usb/ch9.h     |  1 +
> > > >  9 files changed, 161 insertions(+), 16 deletions(-)
> > > >
> > > > --
> > > > 1.9.1
> > >
> > > Could you please take look this series again? Basically this is a
> > > updated version based on v4 you reviewed, for common and gadget
> > > part, the only major change is use dedicated buffer through kmalloc
> > > instead of use a stack variable for host request flag(usb control
> > > transfer, DMA
> > read).
> > > Other changes are for chipidea driver.
> > > The common HNP polling implementation is generic so does not depend
> > > on any specific controller drivers, also Roger's OTG core. If you
> > > still think the chipidea driver ABI change is not in right direction
> > > since Roger will introduce OTG core concept, I can drop that part.
> > >
> > > Thanks!
> > > 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
> --
> 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
--
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



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

  Powered by Linux