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
