[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aa0a6b0b-7c2b-4a3e-bc47-81c59e776068@amd.com>
Date: Mon, 17 Mar 2025 16:58:36 -0500
From: Mario Limonciello <mario.limonciello@....com>
To: Antheas Kapenekakis <lkml@...heas.dev>,
platform-driver-x86@...r.kernel.org
Cc: linux-hwmon@...r.kernel.org, linux-doc@...r.kernel.org,
linux-pm@...r.kernel.org, Guenter Roeck <linux@...ck-us.net>,
Jean Delvare <jdelvare@...e.com>, Jonathan Corbet <corbet@....net>,
Joaquin Ignacio Aramendia <samsagax@...il.com>,
Derek J Clark <derekjohn.clark@...il.com>,
Kevin Greenberg <kdgreenberg234@...tonmail.com>,
Joshua Tam <csinaction@...me>, Parth Menon <parthasarathymenon@...il.com>,
Eileen <eileen@...-netbook.com>, linux-kernel@...r.kernel.org,
sre@...nel.org, linux@...ssschuh.net
Subject: Re: [PATCH v5 04/13] ABI: testing: add tt_toggle and tt_led entries
On 3/17/2025 10:53, Antheas Kapenekakis wrote:
> When tt_toggle was introduced, it was not added to the platform sysfs.
> Add it, then add documentation for tt_led.
>
Ideally I think this commit should split into two and there should be a
fixes tag for the commit when tt_toggle was introduced.
> Signed-off-by: Antheas Kapenekakis <lkml@...heas.dev>
> ---
> Documentation/ABI/testing/sysfs-platform-oxp | 26 ++++++++++++++++++++
> 1 file changed, 26 insertions(+)
> create mode 100644 Documentation/ABI/testing/sysfs-platform-oxp
>
> diff --git a/Documentation/ABI/testing/sysfs-platform-oxp b/Documentation/ABI/testing/sysfs-platform-oxp
> new file mode 100644
> index 0000000000000..776b7dee3d595
> --- /dev/null
> +++ b/Documentation/ABI/testing/sysfs-platform-oxp
> @@ -0,0 +1,26 @@
> +What: /sys/devices/platform/<platform>/tt_toggle
> +Date: Jun 2023
> +KernelVersion: 6.5
> +Contact: "Antheas Kapenekakis" <lkml@...heas.dev>
> +Description:
> + Takeover TDP controls from the device. OneXPlayer devices have a
> + turbo button that can be used to switch between two TDP modes
> + (usually 15W and 25W). By setting this attribute to 1, this
> + functionality is disabled, handing TDP control over to (Windows)
> + userspace software and the Turbo button turns into a keyboard
> + shortcut over the AT keyboard of the device. In addition,
> + using this setting is a prerequisite for PWM control for most
> + newer models (otherwise it NOOPs).
> +
> +What: /sys/devices/platform/<platform>/tt_led
> +Date: Feb 2025
> +KernelVersion: 6.15
> +Contact: "Antheas Kapenekakis" <lkml@...heas.dev>
> +Description:
> + Some OneXPlayer devices (e.g., X1 series) feature a little LED
> + nested in the Turbo button. This LED is illuminated when the
> + device is in the higher TDP mode (e.g., 25W). Once tt_toggle
> + is engaged, this LED is left dangling to its last state. This
> + attribute allows userspace to control the LED state manually
> + (either with 1 or 0). Only a subset of devices contain this LED.
> +
Powered by blists - more mailing lists