All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Rafael J. Wysocki" <rafael@kernel.org>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: "Rafael J. Wysocki" <rafael@kernel.org>,
	Ulf Hansson <ulf.hansson@linaro.org>,
	"Rafael J. Wysocki" <rjw@rjwysocki.net>,
	Linux PM <linux-pm@vger.kernel.org>,
	Kevin Hilman <khilman@kernel.org>,
	Maulik Shah <mkshah@codeaurora.org>,
	Linux ARM <linux-arm-kernel@lists.infradead.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] PM: runtime: Allow rpm_resume() to succeed when runtime PM is disabled
Date: Thu, 2 Dec 2021 19:01:11 +0100	[thread overview]
Message-ID: <CAJZ5v0ix2Xi0Tnt=ZgCLT4Q0K=1+kqey2QKNDR3y7=uVsFf-nA@mail.gmail.com> (raw)
In-Reply-To: <Yaj5Vur0fTFXvhwE@rowland.harvard.edu>

On Thu, Dec 2, 2021 at 5:50 PM Alan Stern <stern@rowland.harvard.edu> wrote:
>
> On Thu, Dec 02, 2021 at 05:18:26PM +0100, Rafael J. Wysocki wrote:
> > On Thu, Dec 2, 2021 at 12:29 PM Ulf Hansson <ulf.hansson@linaro.org> wrote:
> > >
> > > Alright, this sounds reasonable to me. I have also looked at the code
> > > below and it looks good to me.
> > >
> > > Do you intend to post a formal patch? In any case, feel free to add my
> > > reviewed-by tag.
> >
> > I will, thank you!
>
> Don't forget to check for spelling errors (I noticed one in the first version of
> the patch you posted).  And please include a documentation update.

I will.

> On the whole, the idea seems like a good improvement.

Thanks!

WARNING: multiple messages have this Message-ID (diff)
From: "Rafael J. Wysocki" <rafael@kernel.org>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: "Rafael J. Wysocki" <rafael@kernel.org>,
	Ulf Hansson <ulf.hansson@linaro.org>,
	 "Rafael J. Wysocki" <rjw@rjwysocki.net>,
	Linux PM <linux-pm@vger.kernel.org>,
	 Kevin Hilman <khilman@kernel.org>,
	Maulik Shah <mkshah@codeaurora.org>,
	 Linux ARM <linux-arm-kernel@lists.infradead.org>,
	 Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] PM: runtime: Allow rpm_resume() to succeed when runtime PM is disabled
Date: Thu, 2 Dec 2021 19:01:11 +0100	[thread overview]
Message-ID: <CAJZ5v0ix2Xi0Tnt=ZgCLT4Q0K=1+kqey2QKNDR3y7=uVsFf-nA@mail.gmail.com> (raw)
In-Reply-To: <Yaj5Vur0fTFXvhwE@rowland.harvard.edu>

On Thu, Dec 2, 2021 at 5:50 PM Alan Stern <stern@rowland.harvard.edu> wrote:
>
> On Thu, Dec 02, 2021 at 05:18:26PM +0100, Rafael J. Wysocki wrote:
> > On Thu, Dec 2, 2021 at 12:29 PM Ulf Hansson <ulf.hansson@linaro.org> wrote:
> > >
> > > Alright, this sounds reasonable to me. I have also looked at the code
> > > below and it looks good to me.
> > >
> > > Do you intend to post a formal patch? In any case, feel free to add my
> > > reviewed-by tag.
> >
> > I will, thank you!
>
> Don't forget to check for spelling errors (I noticed one in the first version of
> the patch you posted).  And please include a documentation update.

I will.

> On the whole, the idea seems like a good improvement.

Thanks!

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2021-12-02 18:01 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-26 22:26 [PATCH] PM: runtime: Allow rpm_resume() to succeed when runtime PM is disabled Ulf Hansson
2021-10-26 22:26 ` Ulf Hansson
2021-10-27  2:02 ` Alan Stern
2021-10-27  2:02   ` Alan Stern
2021-10-27 10:55   ` Ulf Hansson
2021-10-27 10:55     ` Ulf Hansson
2021-10-27 14:33     ` Alan Stern
2021-10-27 14:33       ` Alan Stern
2021-10-28 22:20       ` Ulf Hansson
2021-10-28 22:20         ` Ulf Hansson
2021-10-29 18:26         ` Rafael J. Wysocki
2021-10-29 18:26           ` Rafael J. Wysocki
2021-11-01  9:27           ` Ulf Hansson
2021-11-01  9:27             ` Ulf Hansson
2021-11-01 14:41             ` Grygorii Strashko
2021-11-01 14:41               ` Grygorii Strashko
2021-11-05 16:03               ` Ulf Hansson
2021-11-05 16:03                 ` Ulf Hansson
2021-11-26 12:19             ` Ulf Hansson
2021-11-26 12:19               ` Ulf Hansson
2021-11-26 13:30               ` Rafael J. Wysocki
2021-11-26 13:30                 ` Rafael J. Wysocki
2021-11-26 13:46                 ` Ulf Hansson
2021-11-26 13:46                   ` Ulf Hansson
2021-11-26 17:58                   ` Rafael J. Wysocki
2021-11-26 17:58                     ` Rafael J. Wysocki
2021-11-26 18:29                     ` Rafael J. Wysocki
2021-11-26 18:29                       ` Rafael J. Wysocki
2021-11-30 11:57                     ` Ulf Hansson
2021-11-30 11:57                       ` Ulf Hansson
2021-11-30 13:01                       ` Rafael J. Wysocki
2021-11-30 13:01                         ` Rafael J. Wysocki
2021-11-30 16:41                         ` Ulf Hansson
2021-11-30 16:41                           ` Ulf Hansson
2021-11-30 17:26                           ` Rafael J. Wysocki
2021-11-30 17:26                             ` Rafael J. Wysocki
2021-12-01  9:02                             ` Ulf Hansson
2021-12-01  9:02                               ` Ulf Hansson
2021-12-01 13:49                               ` Rafael J. Wysocki
2021-12-01 13:49                                 ` Rafael J. Wysocki
2021-12-01 15:22                                 ` Ulf Hansson
2021-12-01 15:22                                   ` Ulf Hansson
2021-12-01 17:44                                   ` Rafael J. Wysocki
2021-12-01 17:44                                     ` Rafael J. Wysocki
2021-12-01 20:11                                     ` Rafael J. Wysocki
2021-12-01 20:11                                       ` Rafael J. Wysocki
2021-12-02 11:28                                       ` Ulf Hansson
2021-12-02 11:28                                         ` Ulf Hansson
2021-12-02 16:18                                         ` Rafael J. Wysocki
2021-12-02 16:18                                           ` Rafael J. Wysocki
2021-12-02 16:50                                           ` Alan Stern
2021-12-02 16:50                                             ` Alan Stern
2021-12-02 18:01                                             ` Rafael J. Wysocki [this message]
2021-12-02 18:01                                               ` Rafael J. Wysocki

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='CAJZ5v0ix2Xi0Tnt=ZgCLT4Q0K=1+kqey2QKNDR3y7=uVsFf-nA@mail.gmail.com' \
    --to=rafael@kernel.org \
    --cc=khilman@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=mkshah@codeaurora.org \
    --cc=rjw@rjwysocki.net \
    --cc=stern@rowland.harvard.edu \
    --cc=ulf.hansson@linaro.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.