Forgot to add some ccs. Adding them here. Going very close to my ISPs' email limit per hour Antheas On Mon, 17 Mar 2025 at 16:54, Antheas Kapenekakis <lkml@xxxxxxxxxxx> wrote: > > This four part series updates the oxpsensors module to bring it in line > with its Windows OneXPlayer counterpart. First, it adds support for all > 2024, 2025 OneXPlayer handhelds and their special variants. Then, it moves > the module to platform/x86 to allow for including more EC features. > > Then, it adds the new charge limiting and bypass features that were first > introduced in the X1 and retrofit to older OneXFly variants and for > controlling the turbo led found in the X1 models. For Bypass, it adds a new > charge_behaviour variant called inhibit-charge-s0. > > Finally, it performs a minor refactor by moving around switch statements > into their own functions, in order to allow for fixing the pwm1_enable ABI > in the final patch. Currently, pwm1_enable sets the fan to auto with the > value 0 and allows manual control with the value 1. This patch makes it > so 0 sets the fan to full speed, 1 sets the fan to manual control, and > 2 sets the fan to auto. This requires both setting enable and the fan > speed when the enable sysfs is written to as 0, hence the refactor. > > As this is a minor ABI break and there is userspace software relying > on this previous behavior, the last patch also changes the /name of the > hwmon endpoint to "oxp_ec" from "oxpec" (mirroring WMI module conventions) > such that userspace software that relied on the previous behavior can be > retrofit to the new kernel while enabling correct functionality on old > and new kernels. Failing that, software that is not updated will just > stop controlling the fans, ensuring no malignant behavior. > > --- > V4: https://lore.kernel.org/all/20250311165406.331046-1-lkml@xxxxxxxxxxx/ > V3: https://lore.kernel.org/all/20250309112114.1177361-1-lkml@xxxxxxxxxxx/ > > Changes since V4: > - Fix nits by Hans > - change inhibit-charge-s0 to inhibit-charge-awake > - use devm_battery_hook_register and power_supply_unregister_extension > (based on cros driver) > - move charge behavior patches to the end to make the rest of the series > easier to merge > - CC platform-x86 and power maintainers > > Changes since V3: > - Fix nits by Derek > - Remove the hwmon documentation as it is not required for platform > drivers (suggested by Guenter) > - Add ACPI_BATTERY and HWMON depends to Kconfig > (reported by kernel robot) > - Homogenize driver into following reverse xmas convention > > Changes since V2: > - Add ack by Guenter, move platform move patch to be third (not first > to allow for device support backport to lts kernels) > - Rework patch text, especially in the refactor patches as per Derek > - Change bypass to use charge_behaviour instead of charge_type, as that > ABI supports capability detection and is more appropriate > - Move battery attach to probe instead of init > - Fix bug where reading tt_led would instead use the turbo register > > Changes since V1: > - Add X1 Pro, F1 Pro variants > - Fix minor typo in initial patches > - Convert oxp-sensors into a platform driver, as it is no longer > considered a hwmon driver. > - Add sysfs documentation and myself to the MAINTAINERS file > - Update documentation to state that this is the OneXPlayer/AOKZOE > platform driver, and that support for Ayaneo/OPI is provided until > they gain their own platform driver. > > Antheas Kapenekakis (13): > hwmon: (oxp-sensors) Distinguish the X1 variants > hwmon: (oxp-sensors) Add all OneXFly variants > platform/x86: oxpec: Move hwmon/oxp-sensors to platform/x86 > ABI: testing: add tt_toggle and tt_led entries > platform/x86: oxpec: Rename ec group to tt_toggle > platform/x86: oxpec: Add turbo led support to X1 devices > platform/x86: oxpec: Move pwm_enable read to its own function > platform/x86: oxpec: Move pwm value read/write to separate functions > platform/x86: oxpec: Move fan speed read to separate function > platform/x86: oxpec: Adhere to sysfs-class-hwmon and enable pwm on 2 > platform/x86: oxpec: Follow reverse xmas convention for tt_toggle > power: supply: add inhibit-charge-awake to charge_behaviour > platform/x86: oxpec: Add charge threshold and behaviour to OneXPlayer > > Documentation/ABI/testing/sysfs-class-power | 11 +- > Documentation/ABI/testing/sysfs-platform-oxp | 26 + > Documentation/hwmon/index.rst | 2 +- > Documentation/hwmon/oxp-sensors.rst | 89 --- > MAINTAINERS | 7 +- > drivers/hwmon/Kconfig | 11 - > drivers/hwmon/Makefile | 1 - > drivers/platform/x86/Kconfig | 13 + > drivers/platform/x86/Makefile | 3 + > .../oxp-sensors.c => platform/x86/oxpec.c} | 628 ++++++++++++++---- > drivers/power/supply/power_supply_sysfs.c | 1 + > drivers/power/supply/test_power.c | 1 + > include/linux/power_supply.h | 1 + > 13 files changed, 542 insertions(+), 252 deletions(-) > create mode 100644 Documentation/ABI/testing/sysfs-platform-oxp > delete mode 100644 Documentation/hwmon/oxp-sensors.rst > rename drivers/{hwmon/oxp-sensors.c => platform/x86/oxpec.c} (52%) > > > base-commit: 4701f33a10702d5fc577c32434eb62adde0a1ae1 > -- > 2.48.1 >
