Re: [PATCH] Documentation/ABI: update cpuidle sysfs documentation

[Sudeep Holla]

On 22/01/18 16:09, Aishwarya Pant wrote:
> Update cpuidle documentation using git logs and existing documentation
> in Documentation/cpuidle/sysfs.txt. This might be useful for scripting
> and tracking changes in the ABI.
> 
> Signed-off-by: Aishwarya Pant <aishpant@xxxxxxxxx>
> ---
>  Documentation/ABI/testing/sysfs-devices-system-cpu | 79 ++++++++++++++++++++++
>  1 file changed, 79 insertions(+)
> 
> diff --git a/Documentation/ABI/testing/sysfs-devices-system-cpu b/Documentation/ABI/testing/sysfs-devices-system-cpu
> index bfd29bc8d37a..f38e52d08be1 100644
> --- a/Documentation/ABI/testing/sysfs-devices-system-cpu
> +++ b/Documentation/ABI/testing/sysfs-devices-system-cpu
> @@ -126,6 +126,85 @@ Description:	Discover cpuidle policy and mechanism
>  		See files in Documentation/cpuidle/ for more information.
>  
>  
> +What:		/sys/devices/system/cpu/cpuidle/current_driver
> +What:		/sys/devices/system/cpu/cpuidle/available_governors
> +What:		/sys/devices/system/cpu/cpuidle/current_governor
> +Date:		September 2007
> +KernelVersion:	v2.6.24
> +Contact:	Linux kernel mailing list <linux-kernel@xxxxxxxxxxxxxxx>
> +Description:
> +		With the cpuidle_sysfs_switch boot option enabled (meant for
> +		developer testing), the following three attributes are visible
> +		instead:
> +
> +		current_driver: (RO) Display the current idle mechanism.
> +
> +		available_governors: (RO) Display a space separated list of
> +		available governors
> +
> +		current_governor: (RW) Display the current idle policy. Users
> +		can switch the governor at runtime by writing to this file.
> +

May be you can merge above with the previous entry which already has
/sys/devices/system/cpu/cpuidle/current_driver
/sys/devices/system/cpu/cpuidle/current_governer_ro

Otherwise looks good.

-- 
Regards,
Sudeep