Hi all,

I have touched ScreenPad handling code based on latest Luke's work
and in doing so I caused some regressions in the screenpad backlight
handling.

This patchset attempts to amend those regressions while also
improving the support based on feedbacks and suggestions
I have received.

The screenpad backlight regression introduced by 130d29c5627c
("platform/x86: asus-wmi: adjust screenpad power/brightness handling")
left the ScreenPad panel unusable on several models: the panel is
powered off at probe time or bl_power writes have inverted effects.
Patches 1 and 2 fix the regression, patch 3 makes the screenpad
backlight honor the core blank and suspend states and patch 4 removes
dead code left behind by the same rework.

I would ask everybody involved to test these changes again and
provide feedback.

Thanks to everyone who provided feedback and helped
identify the regressions, especially Hugo Baigue that also
provided values straight from the hardware and suggestions.

The use of AI here is to review the content and to research
relevant links, putting the correct tags at the end of
each patch and referencing the correct people,
the rest comes either from me or from Hugo.

Honestly I am a bit unsure on what older DUO models might
make of these changes, but it's impossible for them to work
worse than they are doing now due to me inverting
the backlight logic, so I'm willing to hear any brightness
misbehavior and to resolve them on a per-model basis if
this works well in newer laptops.

Best regards,
Denis Benato

Link: https://lore.kernel.org/regressions/178362762638.911488.8564892548331679884@eldamar.lan/ (X532FA regression, Debian #1141604)
Link: https://bugs.debian.org/1141604
Link: https://lore.kernel.org/all/CAO84+xJLVW_9_D9-8cwJYv_Vz+LK7nJqRaG-1P4_eeLrQmSiWw@mail.gmail.com/ (UX5400EA power state report)
Link: https://lore.kernel.org/all/CAO84+x+P2_xyHP89+nGVSMV6bL+dy0P9=vyEEfZGnFhv0hBNWw@mail.gmail.com/ (UX5400EA tests, brightness fix and DSDT analysis)
Link: https://lore.kernel.org/all/985b98f7-04a3-449b-ab6e-7f23e29e4b43@gmail.com/ (X532FA confirming patch 1 works)

Denis Benato (4):
  platform/x86: asus-wmi: fix unclear usage of bd->props.power
  platform/x86: asus-wmi: fix screenpad power state detection
  platform/x86: asus-wmi: use backlight_is_blank() for screenpad power
  platform/x86: asus-wmi: remove unused screenpad_brightness

 drivers/platform/x86/asus-wmi.c | 50 ++++++++++++++++-----------------
 drivers/platform/x86/asus-wmi.h |  1 -
 2 files changed, 24 insertions(+), 27 deletions(-)

-- 
2.47.3

On Wed, 16 Sep 2026 14:38:24 +0000, Denis Benato wrote:

> I have touched ScreenPad handling code based on latest Luke's work
> and in doing so I caused some regressions in the screenpad backlight
> handling.
> 
> This patchset attempts to amend those regressions while also
> improving the support based on feedbacks and suggestions
> I have received.
> 
> [...]

Thank you for your contribution, it has been applied to my local
review-ilpo-next branch. Note it will show up in the public
platform-drivers-x86/review-ilpo-next branch only once I've pushed my
local branch there, which might take a while.

FYI [if applicable to your patch], as per Linus' policy change, also
fixes are mostly routed through for-next unless the fix is for a
commit introduced in the most recent cycle or is clearly a regression
fix.

The list of commits applied:
[1/4] platform/x86: asus-wmi: fix unclear usage of bd->props.power
      commit: 159e956fd4f9cf863b2759cffb4f01fd4d6c2742
[2/4] platform/x86: asus-wmi: fix screenpad power state detection
      commit: 341b4769f5cf2836d1d3ee7ea2259ffa7a477546
[3/4] platform/x86: asus-wmi: use backlight_is_blank() for screenpad power
      commit: 99215e618ff209ccfc01ec62ee58c4b3179a7435
[4/4] platform/x86: asus-wmi: remove unused screenpad_brightness
      commit: 854be60ed0e9f2b7729be3972da14e63d1058e95

--
 i.