Re: [PATCH] Documentation: admin-guide: PM: Add cpuidle document

From: Rafael J. Wysocki
Date: Wed Nov 28 2018 - 05:01:36 EST


On Wed, Nov 28, 2018 at 10:31 AM Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
>
> On 28-11-18, 10:11, Rafael J. Wysocki wrote:
> > On Wed, Nov 28, 2018 at 6:48 AM Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> > >
> > > On 26-11-18, 14:11, Rafael J. Wysocki wrote:
> > > > From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
> > > >
> > > > 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@xxxxxxxxx>
> > > > ---
> > > > 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.
>
> I have this in my .vimrc and I am shown these spelling mistakes somewhat
> forcefully :)
>
> set spell spelllang=en_us

Interestingly enough, it appears to work when I turn the automatic
spellchecking, which I don't do for code as a rule, because it
distracts me. I will need to do that for docs going forward it seems,
though.

BTW, I didn't respond to the remark about the ladder governor. I have
no plans to describe it at this time and that can be done at any time
later easily enough if anyone wants to do it.

Thanks,
Rafael