Re: [PATCH 4/4] docs: Fix typos, improve grammar in Userspace API

From: Mark Pearson
Date: Sun May 18 2025 - 21:49:58 EST


Hi

On Sat, May 17, 2025, at 9:27 AM, Hanne-Lotta Mäenpää wrote:
> Fix a typo and improve wording and punctuation in
> the documentation for Userspace API.
>
> Signed-off-by: Hanne-Lotta Mäenpää <hannelotta@xxxxxxxxx>
> ---
> Documentation/userspace-api/sysfs-platform_profile.rst | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/Documentation/userspace-api/sysfs-platform_profile.rst
> b/Documentation/userspace-api/sysfs-platform_profile.rst
> index 7f013356118a..6613e188242a 100644
> --- a/Documentation/userspace-api/sysfs-platform_profile.rst
> +++ b/Documentation/userspace-api/sysfs-platform_profile.rst
> @@ -18,9 +18,9 @@ API for selecting the platform profile of these
> automatic mechanisms.
> Note that this API is only for selecting the platform profile, it is
> NOT a goal of this API to allow monitoring the resulting performance
> characteristics. Monitoring performance is best done with device/vendor
> -specific tools such as e.g. turbostat.
> +specific tools, e.g. turbostat.
>
> -Specifically when selecting a high performance profile the actual achieved
> +Specifically, when selecting a high performance profile the actual achieved
> performance may be limited by various factors such as: the heat generated
> by other components, room temperature, free air flow at the bottom of a
> laptop, etc. It is explicitly NOT a goal of this API to let userspace know
> @@ -44,7 +44,7 @@ added. Drivers which wish to introduce new profile names must:
> "Custom" profile support
> ========================
> The platform_profile class also supports profiles advertising a "custom"
> -profile. This is intended to be set by drivers when the setttings in the
> +profile. This is intended to be set by drivers when the settings in the
> driver have been modified in a way that a standard profile doesn't represent
> the current state.
>
> --
> 2.39.5

Looks good. Thanks.
Mark

Reviewed-by: Mark Pearson <mpearson-lenovo@xxxxxxxxx>