Re: [PATCH 00/14] usb: dwc2: Fix Partial Power down issues.

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

 



On Thu, Apr 08, 2021 at 05:57:57AM +0000, Artur Petrosyan wrote:
> Hi Greg,
> 
> On 4/7/2021 14:00, Artur Petrosyan wrote:
> > This patch set fixes and improves the Partial Power Down mode for
> > dwc2 core.
> > It adds support for the following cases
> >      1. Entering and exiting partial power down when a port is
> >         suspended, resumed, port reset is asserted.
> >      2. Exiting the partial power down mode before removing driver.
> >      3. Exiting partial power down in wakeup detected interrupt handler.
> >      4. Exiting from partial power down mode when connector ID.
> >         status changes to "connId B
> > 
> > It updates and fixes the implementation of dwc2 entering and
> > exiting partial power down mode when the system (PC) is suspended.
> > 
> > The patch set also improves the implementation of function handlers
> > for entering and exiting host or device partial power down.
> > 
> > NOTE: This is the second patch set in the power saving mode fixes
> > series.
> > This patch set is part of multiple series and is continuation
> > of the "usb: dwc2: Fix and improve power saving modes" patch set.
> > (Patch set link: https://marc.info/?l=linux-usb&m=160379622403975&w=2).
> > The patches that were included in the "usb: dwc2:
> > Fix and improve power saving modes" which was submitted
> > earlier was too large and needed to be split up into
> > smaller patch sets.
> > 
> > 
> > Artur Petrosyan (14):
> >    usb: dwc2: Add device partial power down functions
> >    usb: dwc2: Add host partial power down functions
> >    usb: dwc2: Update enter and exit partial power down functions
> >    usb: dwc2: Add partial power down exit flow in wakeup intr.
> >    usb: dwc2: Update port suspend/resume function definitions.
> >    usb: dwc2: Add enter partial power down when port is suspended
> >    usb: dwc2: Add exit partial power down when port is resumed
> >    usb: dwc2: Add exit partial power down when port reset is asserted
> >    usb: dwc2: Add part. power down exit from
> >      dwc2_conn_id_status_change().
> >    usb: dwc2: Allow exit partial power down in urb enqueue
> >    usb: dwc2: Fix session request interrupt handler
> >    usb: dwc2: Update partial power down entering by system suspend
> >    usb: dwc2: Fix partial power down exiting by system resume
> >    usb: dwc2: Add exit partial power down before removing driver
> > 
> >   drivers/usb/dwc2/core.c      | 113 ++-------
> >   drivers/usb/dwc2/core.h      |  27 ++-
> >   drivers/usb/dwc2/core_intr.c |  46 ++--
> >   drivers/usb/dwc2/gadget.c    | 148 ++++++++++-
> >   drivers/usb/dwc2/hcd.c       | 458 +++++++++++++++++++++++++----------
> >   drivers/usb/dwc2/hw.h        |   1 +
> >   drivers/usb/dwc2/platform.c  |  11 +-
> >   7 files changed, 558 insertions(+), 246 deletions(-)
> > 
> > 
> > base-commit: e9fcb07704fcef6fa6d0333fd2b3a62442eaf45b
> > 
> I have submitted this patch set yesterday. It contains 14 patches. But 
> only 2 of those patches were received by LKML only the cover letter and 
> the 13th patch. 
> (https://lore.kernel.org/linux-usb/cover.1617782102.git.Arthur.Petrosyan@xxxxxxxxxxxx/T/#t)
> 
> I checked here at Synopsys, Minas did receive all the patches as his 
> email is in To list. Could this be an issue of vger.kernel.org mailing 
> server?
> 
> Because I checked every local possibility that could result to such 
> behavior. The patch 13 which was received by LKML has the similar 
> content as the other patches.
> 
> The mailing tool that was used is ssmtp, checked all the configurations 
> everything is fine.
> 
> Could you please suggest what should I do in this situation?

Odd, I got them here, but lore seems to not have them :(

Can you just resend them as a "v2" series so we know which to review and
let's see if that works better...

thanks,

greg k-h



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux