[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Y5FTHGsezkdzwvUa@debian.me>
Date: Thu, 8 Dec 2022 09:59:40 +0700
From: Bagas Sanjaya <bagasdotme@...il.com>
To: Wyes Karny <wyes.karny@....com>, linux-doc@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-pm@...r.kernel.org
Cc: Jonathan Corbet <corbet@....net>, Huang Rui <ray.huang@....com>,
"Rafael J . Wysocki" <rafael@...nel.org>,
Viresh Kumar <viresh.kumar@...aro.org>,
Mario.Limonciello@....com, Perry.Yuan@....com,
Ananth Narayan <ananth.narayan@....com>, gautham.shenoy@....com
Subject: Re: [PATCH 4/4] Documentation: amd_pstate: Add amd_pstate state
sysfs file
On Wed, Dec 07, 2022 at 03:46:48PM +0000, Wyes Karny wrote:
> +``state``
> +
> +``amd_pstate`` also exposes a sysfs interface to view and control the driver state.
> +The driver state can be one of the following:
> +``disable`` : indicates driver is in unloaded state.
> +``passive`` : indicates driver is loaded and currently in passive mode.
> +``guided`` : indicates driver is loaded and currenlty in guided autonomous mode.
Use bullet lists for above:
---- >8 ----
diff --git a/Documentation/admin-guide/pm/amd-pstate.rst b/Documentation/admin-guide/pm/amd-pstate.rst
index 4d3783516ddc2c..0d0e0affa3adb2 100644
--- a/Documentation/admin-guide/pm/amd-pstate.rst
+++ b/Documentation/admin-guide/pm/amd-pstate.rst
@@ -269,9 +269,12 @@ Other performance and frequency values can be read back from
``amd_pstate`` also exposes a sysfs interface to view and control the driver state.
The driver state can be one of the following:
-``disable`` : indicates driver is in unloaded state.
-``passive`` : indicates driver is loaded and currently in passive mode.
-``guided`` : indicates driver is loaded and currenlty in guided autonomous mode.
+
+ - ``disable`` : indicates driver is in unloaded state.
+ - ``passive`` : indicates driver is loaded and currently in passive mode.
+ - ``guided`` : indicates driver is loaded and currenlty in guided
+ autonomous mode.
+
This file can be found here: ``/sys/devices/system/cpu/amd_pstate/state``.
To switch to passive mode: ``echo passive > /sys/devices/system/cpu/amd_pstate/state``
> +This file can be found here: ``/sys/devices/system/cpu/amd_pstate/state``.
> +
> +To switch to passive mode: ``echo passive > /sys/devices/system/cpu/amd_pstate/state``
> +To switch to guided mode: ``echo guided > /sys/devices/system/cpu/amd_pstate/state``
>
What about these wordings instead?
---- >8 ----
diff --git a/Documentation/admin-guide/pm/amd-pstate.rst b/Documentation/admin-guide/pm/amd-pstate.rst
index 4d3783516ddc2c..6465bd39b7dcbc 100644
--- a/Documentation/admin-guide/pm/amd-pstate.rst
+++ b/Documentation/admin-guide/pm/amd-pstate.rst
@@ -267,15 +267,16 @@ Other performance and frequency values can be read back from
``state``
-``amd_pstate`` also exposes a sysfs interface to view and control the driver state.
-The driver state can be one of the following:
-``disable`` : indicates driver is in unloaded state.
-``passive`` : indicates driver is loaded and currently in passive mode.
-``guided`` : indicates driver is loaded and currenlty in guided autonomous mode.
-This file can be found here: ``/sys/devices/system/cpu/amd_pstate/state``.
+``amd_pstate`` also exposes a sysfs interface to view and control the driver
+state, named ``/sys/devices/system/cpu/amd_pstate/state``. The driver state
+can be one of the following:
-To switch to passive mode: ``echo passive > /sys/devices/system/cpu/amd_pstate/state``
-To switch to guided mode: ``echo guided > /sys/devices/system/cpu/amd_pstate/state``
+ - ``disable`` : the driver is disabled
+ - ``passive`` : the driver is in passive mode.
+ - ``guided`` : the driver is in guided autonomous mode.
+
+To switch between these modes above, write the appropriate value to the
+aforementioned sysfs file.
``amd-pstate`` vs ``acpi-cpufreq``
======================================
Thanks.
--
An old man doll... just what I always wanted! - Clara
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists