Linux-USB Archive on lore.kernel.org
 help / color / Atom feed
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Artur Petrosyan <Arthur.Petrosyan@synopsys.com>
Cc: John Youn <John.Youn@synopsys.com>,
	Felipe Balbi <balbi@kernel.org>,
	Minas Harutyunyan <Minas.Harutyunyan@synopsys.com>,
	"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Mian Yousaf Kaukab <yousaf.kaukab@intel.com>,
	Gregory Herrero <gregory.herrero@intel.com>,
	Douglas Anderson <dianders@chromium.org>,
	Paul Zimmerman <paulz@synopsys.com>,
	"stable@vger.kernel.org" <stable@vger.kernel.org>,
	Robert Baldyga <r.baldyga@samsung.com>,
	Kever Yang <kever.yang@rock-chips.com>
Subject: Re: [PATCH 00/14] usb: dwc2: Fix Partial Power down issues.
Date: Thu, 8 Apr 2021 08:09:24 +0200
Message-ID: <YG6eFEv8JfE3qSdV@kroah.com> (raw)
In-Reply-To: <60f356b2-e465-48e7-ad37-f1021e3581ff@synopsys.com>

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@synopsys.com/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

  reply index

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-07 10:00 Artur Petrosyan
2021-04-07 10:08 ` [PATCH 13/14] usb: dwc2: Fix partial power down exiting by system resume Artur Petrosyan
2021-04-08  5:57 ` [PATCH 00/14] usb: dwc2: Fix Partial Power down issues Artur Petrosyan
2021-04-08  6:09   ` Greg Kroah-Hartman [this message]
2021-04-08  7:28 ` [PATCH v2 " Artur Petrosyan
2021-04-08  9:17   ` Artur Petrosyan
2021-04-08 10:09     ` Artur Petrosyan
2021-04-08 11:00       ` Greg Kroah-Hartman
2021-04-08  7:30 ` [PATCH v2 13/14] usb: dwc2: Fix partial power down exiting by system resume Artur Petrosyan
2021-04-08  9:44 ` [PATCH v3 00/14] usb: dwc2: Fix Partial Power down issues Artur Petrosyan
2021-04-08  9:44 ` [PATCH v3 01/14] usb: dwc2: Add device partial power down functions Artur Petrosyan
2021-04-08  9:44 ` [PATCH v3 02/14] usb: dwc2: Add host " Artur Petrosyan
2021-04-08  9:44 ` [PATCH v3 03/14] usb: dwc2: Update enter and exit " Artur Petrosyan
2021-04-08  9:44 ` [PATCH v3 04/14] usb: dwc2: Add partial power down exit flow in wakeup intr Artur Petrosyan
2021-04-08  9:45 ` [PATCH v3 05/14] usb: dwc2: Update port suspend/resume function definitions Artur Petrosyan
2021-04-08 13:39   ` Minas Harutyunyan
2021-04-08  9:45 ` [PATCH v3 06/14] usb: dwc2: Add enter partial power down when port is suspended Artur Petrosyan
2021-04-08 13:39   ` Minas Harutyunyan
2021-04-08  9:45 ` [PATCH v3 07/14] usb: dwc2: Add exit partial power down when port is resumed Artur Petrosyan
2021-04-08 13:39   ` Minas Harutyunyan
2021-04-08  9:45 ` [PATCH v3 08/14] usb: dwc2: Add exit partial power down when port reset is asserted Artur Petrosyan
2021-04-08 13:39   ` Minas Harutyunyan
2021-04-08  9:45 ` [PATCH v3 09/14] usb: dwc2: Add part. power down exit from dwc2_conn_id_status_change() Artur Petrosyan
2021-04-08  9:45 ` [PATCH v3 10/14] usb: dwc2: Allow exit partial power down in urb enqueue Artur Petrosyan
2021-04-08 13:40   ` Minas Harutyunyan
2021-04-08  9:45 ` [PATCH v3 11/14] usb: dwc2: Fix session request interrupt handler Artur Petrosyan
2021-04-08  9:45 ` [PATCH v3 12/14] usb: dwc2: Update partial power down entering by system suspend Artur Petrosyan
2021-04-08 13:40   ` Minas Harutyunyan
2021-04-08  9:46 ` [PATCH v3 13/14] usb: dwc2: Fix partial power down exiting by system resume Artur Petrosyan
2021-04-08 13:40   ` Minas Harutyunyan
2021-04-08  9:46 ` [PATCH v3 14/14] usb: dwc2: Add exit partial power down before removing driver Artur Petrosyan
2021-04-08 13:41   ` Minas Harutyunyan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=YG6eFEv8JfE3qSdV@kroah.com \
    --to=gregkh@linuxfoundation.org \
    --cc=Arthur.Petrosyan@synopsys.com \
    --cc=John.Youn@synopsys.com \
    --cc=Minas.Harutyunyan@synopsys.com \
    --cc=balbi@kernel.org \
    --cc=dianders@chromium.org \
    --cc=gregory.herrero@intel.com \
    --cc=kever.yang@rock-chips.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=paulz@synopsys.com \
    --cc=r.baldyga@samsung.com \
    --cc=stable@vger.kernel.org \
    --cc=yousaf.kaukab@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Linux-USB Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-usb/0 linux-usb/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-usb linux-usb/ https://lore.kernel.org/linux-usb \
		linux-usb@vger.kernel.org
	public-inbox-index linux-usb

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-usb


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git