[FOR-UPSTREAM] platform/x86: oxpec: Add ONEXPLAYER 3 fan monitoring - #100
Open
Grippy98 wants to merge 5 commits into
Open
[FOR-UPSTREAM] platform/x86: oxpec: Add ONEXPLAYER 3 fan monitoring#100Grippy98 wants to merge 5 commits into
Grippy98 wants to merge 5 commits into
Conversation
This is deliberately an OGC-carried patch and must not be sent upstream. Squatting on another device's LED name is not acceptable in mainline; the real fix belongs in the Steam client, which should match the "*:rgb:joystick_rings" suffix rather than a per-device prefix.
OneXPlayer X2 Mini Pro is a new Strix Halo handheld. It ships the same system board as the OneXPlayer APEX and uses the same registers as the OneXPlayer Fly devices. Add a quirk for it to the oxpec driver. Signed-off-by: Jeff Hagadorn <jeff@aletheia.io>
The Super X currently shares the oxp_g1_a board identifier because its existing EC functionality matches the G1 AMD. Give it a dedicated identifier so model-specific functionality can be selected without also enabling it on the G1. Preserve the existing G1 AMD behavior for the new identifier. Assisted-by: Codex:gpt-5.6 Signed-off-by: Alexander Egorov <begeebe@gmail.com>
…hables The OneXPlayer X1 family and Super X have detachable pogo-pin keyboards. The firmware does not expose a tablet-mode input switch. The Super X keyboard enumerates as 1a86:1305. The X1 family folio keyboard enumerates as 258a:001e. Register an input device on these boards and report SW_TABLET_MODE from the model-specific pogo keyboard USB hotplug state. Other USB and Bluetooth keyboards do not affect the switch. A complete dump of the 256-byte Super X EC register map was compared with the keyboard attached, detached, and reattached. No register changed predictably with the attachment state. Tested on a OneXPlayer Super X. Mutter enables accelerometer-driven display rotation only while the pogo keyboard is absent. Assisted-by: Codex:gpt-5.6 Signed-off-by: Alexander Egorov <begeebe@gmail.com>
Match the ONEXPLAYER 3 DMI identity and read its fan tachometer through the EC register. Signed-off-by: Andrei Aldea <andrei1998@gmail.com>
Member
|
Seems incomplete. Why is pwm control not enabled in this patch? |
KyleGospo
force-pushed
the
features/onexplayer
branch
from
August 27, 2026 17:12
6a25dac to
89c89f4
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Match the ONEXPLAYER 3 DMI identity and read its fan tachometer through the EC register.
Verified on real hardware w/Bazzite 44 and MangoHud
Assisted-by: Codex:gpt-5.6