Linux-USB Archive on lore.kernel.org
 help / color / Atom feed
From: Artur Petrosyan <Arthur.Petrosyan@synopsys.com>
To: Artur Petrosyan <Arthur.Petrosyan@synopsys.com>,
	John Youn <John.Youn@synopsys.com>,
	Felipe Balbi <balbi@kernel.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.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>
Cc: 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 10:09:20 +0000
Message-ID: <af45ed18-2ce4-43da-f28c-d5cda0710b9f@synopsys.com> (raw)
In-Reply-To: <3625b740-b362-5ec6-8fba-cd7babcab35b@synopsys.com>

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,

Working example.
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, 
linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org

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.

Regards,
Artur

  reply index

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-07 10:00 [PATCH " 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 [this message]
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=af45ed18-2ce4-43da-f28c-d5cda0710b9f@synopsys.com \
    --to=arthur.petrosyan@synopsys.com \
    --cc=John.Youn@synopsys.com \
    --cc=Minas.Harutyunyan@synopsys.com \
    --cc=balbi@kernel.org \
    --cc=dianders@chromium.org \
    --cc=gregkh@linuxfoundation.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