| Message ID | 1588235987-12300-7-git-send-email-guohanjun@huawei.com (mailing list archive) |
|---|---|
| State | Superseded, archived |
| Headers | show |
| Series | cpuidle: Make cpuidle governor switchable to be the default behaviour | expand |
On 30/04/2020 10:39, Hanjun Guo wrote: > Since both current_governor and current_governor_ro co-exist under > /sys/devices/te system/cpu/cpuidle/ file, and it's duplicate, > make current_governer_ro as a candidate for removal. > > Signed-off-by: Hanjun Guo <guohanjun@huawei.com> Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org> > --- > Documentation/ABI/obsolete/sysfs-cpuidle | 9 +++++++++ > 1 file changed, 9 insertions(+) > create mode 100644 Documentation/ABI/obsolete/sysfs-cpuidle > > diff --git a/Documentation/ABI/obsolete/sysfs-cpuidle b/Documentation/ABI/obsolete/sysfs-cpuidle > new file mode 100644 > index 00000000..f984b9c > --- /dev/null > +++ b/Documentation/ABI/obsolete/sysfs-cpuidle > @@ -0,0 +1,9 @@ > +What: /sys/devices/system/cpu/cpuidle/current_governor_ro > +Date: April, 2020 > +Contact: linux-pm@vger.kernel.org > +Description: > + current_governor_ro shows current using cpuidle governor, but read only. > + with the update that cpuidle governor can be changed at runtime in default, > + both current_governor and current_governor_ro co-exist under > + /sys/devices/te system/cpu/cpuidle/ file, it's duplicate so make > + current_governor_ro obselete. >
On 2020/4/30 16:39, Hanjun Guo wrote: > Since both current_governor and current_governor_ro co-exist under > /sys/devices/te system/cpu/cpuidle/ file, and it's duplicate, > make current_governer_ro as a candidate for removal. > > Signed-off-by: Hanjun Guo<guohanjun@huawei.com> > --- > Documentation/ABI/obsolete/sysfs-cpuidle | 9 +++++++++ > 1 file changed, 9 insertions(+) > create mode 100644 Documentation/ABI/obsolete/sysfs-cpuidle > > diff --git a/Documentation/ABI/obsolete/sysfs-cpuidle b/Documentation/ABI/obsolete/sysfs-cpuidle > new file mode 100644 > index 00000000..f984b9c > --- /dev/null > +++ b/Documentation/ABI/obsolete/sysfs-cpuidle > @@ -0,0 +1,9 @@ > +What: /sys/devices/system/cpu/cpuidle/current_governor_ro > +Date: April, 2020 > +Contact: linux-pm@vger.kernel.org > +Description: > + current_governor_ro shows current using cpuidle governor, but read only. > + with the update that cpuidle governor can be changed at runtime in default, > + both current_governor and current_governor_ro co-exist under > + /sys/devices/te system/cpu/cpuidle/ file, it's duplicate so make Another typo here, s/te system/system, I will update it, since it minor, I think Doug's reivew and Daniel's ack are still valid. Thanks Hanjun
diff --git a/Documentation/ABI/obsolete/sysfs-cpuidle b/Documentation/ABI/obsolete/sysfs-cpuidle new file mode 100644 index 00000000..f984b9c --- /dev/null +++ b/Documentation/ABI/obsolete/sysfs-cpuidle @@ -0,0 +1,9 @@ +What: /sys/devices/system/cpu/cpuidle/current_governor_ro +Date: April, 2020 +Contact: linux-pm@vger.kernel.org +Description: + current_governor_ro shows current using cpuidle governor, but read only. + with the update that cpuidle governor can be changed at runtime in default, + both current_governor and current_governor_ro co-exist under + /sys/devices/te system/cpu/cpuidle/ file, it's duplicate so make + current_governor_ro obselete.
Since both current_governor and current_governor_ro co-exist under /sys/devices/te system/cpu/cpuidle/ file, and it's duplicate, make current_governer_ro as a candidate for removal. Signed-off-by: Hanjun Guo <guohanjun@huawei.com> --- Documentation/ABI/obsolete/sysfs-cpuidle | 9 +++++++++ 1 file changed, 9 insertions(+) create mode 100644 Documentation/ABI/obsolete/sysfs-cpuidle