[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAJZ5v0hnX512AheTwhpj09+eyiyv6txsgGQcT2afWiB2JjjCzw@mail.gmail.com>
Date: Wed, 28 Nov 2018 10:11:16 +0100
From: "Rafael J. Wysocki" <rafael@...nel.org>
To: Viresh Kumar <viresh.kumar@...aro.org>
Cc: "Rafael J. Wysocki" <rjw@...ysocki.net>,
Jonathan Corbet <corbet@....net>,
Linux PM <linux-pm@...r.kernel.org>,
Jacob Pan <jacob.jun.pan@...ux.intel.com>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Len Brown <len.brown@...el.com>,
ACPI Devel Maling List <linux-acpi@...r.kernel.org>,
"open list:DOCUMENTATION" <linux-doc@...r.kernel.org>,
Peter Zijlstra <peterz@...radead.org>,
Ulf Hansson <ulf.hansson@...aro.org>,
Daniel Lezcano <daniel.lezcano@...aro.org>,
Giovanni Gherdovich <ggherdovich@...e.cz>,
Sudeep Holla <sudeep.holla@....com>
Subject: Re: [PATCH] Documentation: admin-guide: PM: Add cpuidle document
On Wed, Nov 28, 2018 at 6:48 AM Viresh Kumar <viresh.kumar@...aro.org> wrote:
>
> On 26-11-18, 14:11, Rafael J. Wysocki wrote:
> > From: Rafael J. Wysocki <rafael.j.wysocki@...el.com>
> >
> > Important information is missing from user/admin cpuidle documentation
> > available today, so add a new user/admin document for cpuidle containing
> > current and comprehensive information to admin-guide and drop the old
> > .txt documents it is replacing.
> >
> > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@...el.com>
> > ---
> > Documentation/admin-guide/pm/cpuidle.rst | 603 +++++++++++++++++++++++++
> > Documentation/admin-guide/pm/working-state.rst | 1
> > Documentation/cpuidle/core.txt | 23
> > Documentation/cpuidle/sysfs.txt | 98 ----
> > 4 files changed, 604 insertions(+), 121 deletions(-)
>
> Nice work Rafael. Minor nits below..
Thanks for the typo fixes. The spellchecker I have here evidently doesn't work.
[cut]
> Maybe I missed, but I couldn't find any text that says what state 0, 1, ... N
> mean.
There is a paragraph on that above.
> Like which is the deepest idle state and which one is the shallowest.
But this part is missing, good catch!
Thanks,
Rafael
Powered by blists - more mailing lists