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 v2 00/14] usb: dwc2: Fix Partial Power down issues.
Date: Thu, 8 Apr 2021 13:00:33 +0200 [thread overview]
Message-ID: <YG7iUey+Rtof+3Up@kroah.com> (raw)
In-Reply-To: <af45ed18-2ce4-43da-f28c-d5cda0710b9f@synopsys.com>
On Thu, Apr 08, 2021 at 10:09:20AM +0000, Artur Petrosyan wrote:
> Hi Greg,
>
> On 4/8/2021 13:17, Artur Petrosyan wrote:
> > Hi Greg,
> >
> > On 4/8/2021 11:28, 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://urldefense.com/v3/__https://marc.info/?l=linux-usb&m=160379622403975&w=2__;!!A4F2R9G_pg!IJ-Xl1ZwQU2kmqHB3ITyWyno9BgpWUsC647AqK7GIlgzJu9BzT6VN7jt--__fGdMtgWF69M$ ).
> >> 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.
> >>
> >> Changes since V1:
> >> No changes in the patches or the source code.
> >> Sending the second version of the patch set because the first version
> >> was not received by vger.kernel.org.
> >>
> >>
> >>
> >> 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
> >>
> >
> > Re sending as a "v2" did not work :(.
> > The patches are not in lore again.
> >
> > Could the issue be with a comma in the end of To: or Cc: list?
> > Let me remove the comma in the end of those lists and try sending as "v3".
> >
> > Regards,
> > Artur
> >
>
> I just removed the comma in the end of those lists and resent the patch
> set as a "v3" and they are already seen in lore.
> There is one strange thing though on lore. Some patch titles are not
> fully visible.
>
> For sure the issue was comma in the end of To: or Cc: lists.
> Not working example.
> To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
> linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org,
That's an invalid To: line for email.
> Working example.
> To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
> linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org
That's a correct line.
> If the comma is at least in the end of one of those lists (To: or Cc:)
> vger.kernel.org mailing server will not accept them.
I recommend using 'git send-email' with the --to="foo@bar.com" type
options so that you don't have to hand-edit the lines to try to get
stuff like this correct, as it is easy to get wrong.
thanks,
greg k-h
next prev parent reply other threads:[~2021-04-08 11:00 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-04-07 10:00 [PATCH 00/14] usb: dwc2: Fix Partial Power down issues 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
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 [this message]
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=YG7iUey+Rtof+3Up@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
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).