[Bug 214899] New: ideapad-laptop: platform backlight device not working

[Bug 214899] New: ideapad-laptop: platform backlight device not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

            Bug ID: 214899
           Summary: ideapad-laptop: platform backlight device not working
           Product: Drivers
           Version: 2.5
    Kernel Version: 5.10.27-5.14.15
          Hardware: All
                OS: Linux
              Tree: Mainline
            Status: NEW
          Severity: normal
          Priority: P1
         Component: Platform_x86
          Assignee: drivers_platform_x86@kernel-bugs.osdl.org
          Reporter: johannes.penssel@gmail.com
        Regression: No

Created attachment 299379
  --> https://bugzilla.kernel.org/attachment.cgi?id=299379&action=edit
DSDT.dsl

Hardware: Lenovo IdeaPad 5 15ITL05, latest BIOS version
Issue: Platform backlight device provided by ideapad-laptop has no effect on
actual brightness.
Pressing brightness keys on my keyboard causes changes in
/sys/class/backlight/ideapad/actual_brightness, but no uevent is generated (as
outlined in drivers/video/backlight.c source code comments), and
/sys/class/backlight/ideapad/brightness is unaffected. 
When manipulating /sys/class/backlight/ideapad/brightness directly, uevents are
being generated, but no change in real screen brightness occurs. 
The only way that screen brightness can actually be changed is by using i915's
intel_backlight sysfs interface, which does not respond to brightness keyboard
keys.

attached is my decompiled DSDT and dmesg.

ls -la /sys/class/backlight/ideapad/
total 0
drwxr-xr-x 3 root root     0 Oct 31 19:40 .
drwxr-xr-x 3 root root     0 Oct 31 19:40 ..
-r--r--r-- 1 root root  4096 Oct 31 19:40 actual_brightness
-rw-r--r-- 1 root root  4096 Oct 31 19:40 bl_power
-rw-rw-r-- 1 root video 4096 Oct 31 19:43 brightness
lrwxrwxrwx 1 root root     0 Oct 31 19:40 device -> ../../../VPC2004:00
-r--r--r-- 1 root root  4096 Oct 31 19:40 max_brightness
drwxr-xr-x 2 root root     0 Oct 31 19:40 power
-r--r--r-- 1 root root  4096 Oct 31 19:40 scale
lrwxrwxrwx 1 root root     0 Oct 31 19:40 subsystem ->
../../../../../../../class/backlight
-r--r--r-- 1 root root  4096 Oct 31 19:40 type
-rw-r--r-- 1 root root  4096 Oct 31 19:40 uevent

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: platform backlight device not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

Hans de Goede (jwrdegoede@fedoraproject.org) changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jwrdegoede@fedoraproject.or
                   |                            |g

--- Comment #2 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
What is your kernel commandline?  According to your attached dmesg it is empty,
but that seems unlikely... ?

The ideapad-laptop code has this:

        if (acpi_video_get_backlight_type() == acpi_backlight_vendor) {
                err = ideapad_backlight_init(priv);
                if (err && err != -ENODEV)
                        goto backlight_failed;
        }

And acpi_video_get_backlight_type() is:

enum acpi_backlight_type acpi_video_get_backlight_type(void)
{
...
        if (acpi_backlight_cmdline != acpi_backlight_undef)
                return acpi_backlight_cmdline;

        if (acpi_backlight_dmi != acpi_backlight_undef)
                return acpi_backlight_dmi;

        if (!(video_caps & ACPI_VIDEO_BACKLIGHT))
                return acpi_backlight_vendor;

        if (acpi_osi_is_win8() && backlight_device_get_by_type(BACKLIGHT_RAW))
                return acpi_backlight_native;

        return acpi_backlight_video;
}

So the only way that the ideapad backlight can get registered is if:

1. You have acpi_backlight=vendor on your kernel cmdline
2. There is a DMI quirk setting for your model in the kernel which there is
not.
3. Your ACPI tables do not implement the ACPI_VIDEO_BACKLIGHT interface, but
your dmesg says:

[    2.237329] ACPI: video: Video Device [GFX0] (multi-head: yes  rom: no 
post: no)

So this is unlikely, it is still possible though that your DSDT implements the
ACPI video interface but not the backlight part (that would be a first). I
cannot tell since the ACPI video interface is not in the DSDT, it is likely
defined in one of the SSDT tables. Please attach a full acpidump.

Can you try specifying: "acpi_backlight=video" on the kernel commandline and
then do:

ls /sys/class/backlight

And copy and paste the output here. This cmdline option not only helps me debug
this, but it should also workaround your backlight issue :)

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: platform backlight device not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #3 from Johannes P (johannes.penssel@gmail.com) ---
I'm specifying my command line via kernel config, so for some reason, it shows
up in a separate message somewhat late in dmesg:

[    0.017237] Kernel command line:
crypt_root=UUID=2cf18008-b55c-4f5f-926b-5f92a588348a
root=UUID=c890d077-9357-4996-94a3-811d6f477c89 i915.enable_guc=2
iwlwifi.power_save=1 iwlmvm.power_scheme=3
snd-hda-intel.power_save_controller=1 acpi_backlight=vendor

Possibly of note as well is that I am able to write custom values into
/sys/class/backlight/ideapad/actual_brightness, which none of the other
interfaces allows. 

ls -la /sys/class/backlight (with acpi_backlight=video)
total 0
drwxr-xr-x  2 root root 0 Nov  1 21:28 .
drwxr-xr-x 79 root root 0 Nov  1 21:28 ..
lrwxrwxrwx  1 root root 0 Nov  1 21:28 acpi_video0 ->
../../devices/pci0000:00/0000:00:02.0/backlight/acpi_video0
lrwxrwxrwx  1 root root 0 Nov  1 21:28 acpi_video1 ->
../../devices/pci0000:00/0000:00:02.0/backlight/acpi_video1
lrwxrwxrwx  1 root root 0 Nov  1 21:28 intel_backlight ->
../../devices/pci0000:00/0000:00:02.0/drm/card0/card0-eDP-1/intel_backlight

I am afraid to report that booting with acpi_backlight=video did not help with
the brightness keys :(

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: platform backlight device not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #5 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
Ah, ok so you indeed have "acpi_backlight=vendor" on the kernel commandline,
please don't do that, the old Ideapad specific backlight interface enabled by
that is only for very old laptop models.

Where as your laptop is very new! If you omit this, then you should only have
the intel_backlight under /sys/class/backlight and brightness control to e.g.
the slider in the GNOME system (top right) menu should work.

The brightness-keys directly changing the brightness is something from the same
era as the Ideapad specific backlight interface. Now a days almost without
exception the brightness keys will simply generate key-press events which are
processed by userspace to actually change the brightness. This allows userspace
to apply various policies to the brightness setting, for e.g. interaction with
auto-brightness control on devices with an ambient-light-sensor (ALS).

Almost all "integrated" desktop environments like GNOME, KDE, XFCE, MATE, etc.
will automatically do the right thing with the brightness-key events. If you
are however using e.g. i3 as window-manager then you will need to script this
yourself, there are plenty of examples how to do this.

###

So as said if you omit "acpi_backlight=vendor" on the kernel commandline, you
should only get the intel_backlight in /sys/class/backlight, which you have
already indicated works as it should.

All that remains then is to check that key-press events are properly generated
and if they are then this is "not a kernel bug". Please install evemu or evtest
and then run (as root) evemu-record or evtest and test for each input-device if
it perhaps sends brightness key-press events when pressing the brightness
hotkeys.

The following devices are a good starting point to test:
"AT Translated Set 2 keyboard"
"Video Bus"

But if neither of those works then please test them all 1 by 1. Note it is
possible that the "AT Translated Set 2 keyboard" generates only EV_MSC events
or the wrong EV_KEY codes for the brightness keys (so not
KEY_BRIGHTNESS[DOWN|UP]). In this case we need to add a mapping for your laptop
to fix this.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: platform backlight device not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #6 from Johannes P (johannes.penssel@gmail.com) ---
Getting those brightness keys to actually produce key events is something I
have tried unsuccessfully for quite a while now. Having just re-checked all 14
available input devices for good measure with evtest, there is still nothing to
report. Same goes for ACPI and WMI events as far as I can tell. Under Windows,
the brightness keys work without issues.

I have noticed that this is actually very similar to how the microphone mute
key behaves without the ideapad-laptop driver loaded. (i.e. it is virtually
undetectable) The driver exposes this key via the keymap it creates for the
"Ideapad extra buttons" input device. Maybe my brightness keys need to be added
to this keymap as well?

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: platform backlight device not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #7 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
The ideapad-laptop code seems to log to dmesg on most unknown events, I assume
nothing shows up in dmesg?

I think based on your other experiments that it would be interesting to log the
value of now in this function in ideapad-laptop.c:


static void ideapad_backlight_notify_brightness(struct ideapad_private *priv)
{
        unsigned long now;

        /* if we control brightness via acpi video driver */
        if (!priv->blightdev)
                read_ec_data(priv->adev->handle, VPCCMD_R_BL, &now);
        else
                backlight_force_update(priv->blightdev,
BACKLIGHT_UPDATE_HOTKEY);
}

Changing it to:

static void ideapad_backlight_notify_brightness(struct ideapad_private *priv)
{
        unsigned long now;

        /* if we control brightness via acpi video driver */
        if (!priv->blightdev) {
                read_ec_data(priv->adev->handle, VPCCMD_R_BL, &now);
                pr_info("EC's LCD backlight value is now: %ld\n", now);
        } else
                backlight_force_update(priv->blightdev,
BACKLIGHT_UPDATE_HOTKEY);
}


I suspect that we will see that value change (note this assumes you do not
specify an acpi_backlight= kernel cmdline option).

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: platform backlight device not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #8 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
Note to self, find _DOS ACPI video method, see what setting bit 2 does (this
should disable automatic brightness adjustment by the EC).

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: platform backlight device not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #10 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
Created attachment 299401
  --> https://bugzilla.kernel.org/attachment.cgi?id=299401&action=edit
ideapad-laptop event debug patch.

Can you give this patch a try please; also make sure you have CONFIG_ACPI_WMI
enabled please.

With this patch pressing your micmute button should definitely show an event in
the "dmesg -w" output; if the micmute does not show events your not running the
patched code.

Once you have verified that the micmute button logs events in dmesg, please try
the brightness buttons.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: platform backlight device not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #11 from Johannes P (johannes.penssel@gmail.com) ---
[  150.562836] ideapad_laptop: ideapad_acpi_notify vpc: 100
[  152.094907] ideapad_laptop: ideapad_acpi_notify vpc: 2000
[  154.742909] ideapad_laptop: ideapad_wmi_notify value: d0

These are the events picked up with your ideapad-laptop patch.

ideapad_acpi_notify vpc: 100 
this is the mic mute key

ideapad_acpi_notify vpc: 2000 
this is the rfkill/airplane mode key

ideapad_wmi_notify value: d0
this is a custom "open app" key. Although it generates no key code, it produces
this ACPI event: (picked up with acpi_listen)
8FC0DE0C-B4E4- 000000d0 00000000

Still nothing but grave silence from the brightness keys though.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: platform backlight device not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

Andy Shevchenko (andy.shevchenko@gmail.com) changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |andy.shevchenko@gmail.com

--- Comment #12 from Andy Shevchenko (andy.shevchenko@gmail.com) ---
(In reply to Johannes P from comment #11)
> Still nothing but grave silence from the brightness keys though.

Which might be a sign that the key presses are handled by some microcontroller
(EC?) with a custom firmware and being delivered to the OS in different way.
It's also possible that the WMI can be used but these events have to be enabled
somehow (obviously with ACPI methods or so).

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: platform backlight device not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #13 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
> Still nothing but grave silence from the brightness keys though.

Ok, I suspect that we need to enable reporting of events from them by some ACPI
video call; and that we will then get events for them on the "Video Bus" input
device.

I need to do a deep dive into your acpidump to see if I can find anything
there. Mondays are my "look into odd bugs which need more then a quick 5 minute
bugzilla reply" day, so I hope to make some time to look into this next Monday.

In the mean time I have one last 5 minutes idea :)  Let me attach another patch
with some video-bus debugging added.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: platform backlight device not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #14 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
Created attachment 299405
  --> https://bugzilla.kernel.org/attachment.cgi?id=299405&action=edit
acpi-video event debug patch

Please give this a test spin and see if any events are printed to dmesg when
pressing the brightness keys.

p.s.

We should really also hookup that WMI 0xd0 event to e.g. KEY_OPEN. Johannes,
that would potentially make a good first kernel patch for you (assuming you
haven't written kernel patches before).

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: platform backlight device not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #15 from Johannes P (johannes.penssel@gmail.com) ---
Unfortunately, your latest patch did not yield any results either. :(

In the meantime, I have discovered another peculiarity: the ACPI/WMI events
generated by this "open app" key are identical to the ones created by pressing
Fn + ESC. (i.e. toggling Fn lock) In fact, the WMI 0xd0 event is already
handled by ideapad driver in the ideapad_wmi_notify function. The only apparent
difference between pressing "open app" and pressing Fn + ESC is that the latter
also toggles the LED on the ESC key. This might be a firmware issue however, as
this key does nothing in Windows either.

Thanks a lot for your support!

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: platform backlight device not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #16 from Johannes P (johannes.penssel@gmail.com) ---
(In reply to Andy Shevchenko from comment #12)
> (In reply to Johannes P from comment #11)
> > Still nothing but grave silence from the brightness keys though.
> 
> Which might be a sign that the key presses are handled by some
> microcontroller (EC?) with a custom firmware and being delivered to the OS
> in different way. It's also possible that the WMI can be used but these
> events have to be enabled somehow (obviously with ACPI methods or so).

This is something that came to my mind as well. The keyboard backlight on this
machine works in a similar way: It seems to be entirely handled by the EC,
because it is in no way exposed to the OS, but can be toggled at any time, even
during boot/shutdown/restart.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: platform backlight device not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #17 from Johannes P (johannes.penssel@gmail.com) ---
Looks like the mic mute key and the brightness keys do not behave identically
with the ideapad-laptop module unloaded after all.

While playing around with ACPI EC debugging today, I was able to produce output
for everything (including the otherwise "invisible" KB backlight toggle) except
the brightness keys. This included the mic mute key and other keys reliant on
ideapad-laptop. Without the module, pressing these keys still triggered EC
debug output.

Just as a correction to my above claims.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: platform backlight device not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #18 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
Just a quick update that this is still on my radar, but unfortunately so far
I've not found the time to do a deep dive into the DSDT.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

Johannes P (johannes.penssel@gmail.com) changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|ideapad-laptop: platform    |ideapad-laptop: brightness
                   |backlight device not        |hotkeys not working
                   |working                     |

--- Comment #19 from Johannes P (johannes.penssel@gmail.com) ---
After doing some further testing, I am pretty sure that these keys are not
handled by ACPI at all. Setting acpi.debug_level to maximum and going through
the debug layers one by one, I was not able to produce any sort of response
from them. I would really like to know what is going on here. 

Curiously, uninstalling the Lenovo hotkey driver in Windows does not break them
there, so I guess they nonetheless must be using some kind of standardized
method to communicate with the OS?

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

Johannes P (johannes.penssel@gmail.com) changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Kernel Version|5.10.27-5.14.15             |5.10.27-

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #20 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
So I just hit a similar problem on another laptop what helped there (it got the
keys to send events on the "AT Translated Set 2 keyboard" device) is adding
i8042.dritek=Y to the kernel commandline.

I think doing this is worth a try on your ideapad too.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #21 from Johannes P (johannes.penssel@gmail.com) ---
Thank you for your advice, I really appreciate that you still keep my issue in
mind. Unfortunately, still no success with i8042.dritek=1. 

After digging through Windows Event Viewer, I was able to figure out that the
keys actually do trigger ACPI events there. When pressing brightness up/down,
the corresponding EC query methods _Q12/_Q11 are executed, resulting in an ACPI
brightness event.

            Method (_Q11, 0, NotSerialized)  // _Qxx: EC Query, xx=0x00-0xFF
            {
                P80B = 0x11
                Notify (^^^GFX0.DD1F, 0x87) // Device-Specific
                Notify (VPC0, 0x80) // Status Change
            }

            Method (_Q12, 0, NotSerialized)  // _Qxx: EC Query, xx=0x00-0xFF
            {
                P80B = 0x12
                Notify (^^^GFX0.DD1F, 0x86) // Device-Specific
                Notify (VPC0, 0x80) // Status Change
            }

When calling the methods on Linux through the ACPI kernel debugger, the same
thing happens. A brightness event is generated and the EC debug module prints
this:

[ 1101.374238] ACPI: EC: 2: Increase command
[ 1101.374244] ACPI: EC: Command(WR_EC) started
[ 1101.374247] ACPI: EC: TASK (0)
[ 1101.374271] ACPI: EC: EC_SC(R) = 0x08 SCI_EVT=0 BURST=0 CMD=1 IBF=0 OBF=0
[ 1101.374275] ACPI: EC: EC_SC(W) = 0x81
[ 1101.374497] ACPI: EC: IRQ (1)
[ 1101.374532] ACPI: EC: EC_SC(R) = 0x08 SCI_EVT=0 BURST=0 CMD=1 IBF=0 OBF=0
[ 1101.374537] ACPI: EC: EC_DATA(W) = 0x00
[ 1101.374837] ACPI: EC: IRQ (1)
[ 1101.374872] ACPI: EC: EC_SC(R) = 0x00 SCI_EVT=0 BURST=0 CMD=0 IBF=0 OBF=0
[ 1101.374876] ACPI: EC: EC_DATA(W) = 0x10
[ 1101.375113] ACPI: EC: IRQ (1)
[ 1101.375145] ACPI: EC: EC_SC(R) = 0x00 SCI_EVT=0 BURST=0 CMD=0 IBF=0 OBF=0
[ 1101.375163] ACPI: EC: Command(WR_EC) stopped
[ 1101.375169] ACPI: EC: 1: Decrease command
[ 1101.375301] ACPI: EC: 2: Increase command
[ 1101.375304] ACPI: EC: Command(RD_EC) started
[ 1101.375307] ACPI: EC: TASK (0)
[ 1101.375331] ACPI: EC: EC_SC(R) = 0x00 SCI_EVT=0 BURST=0 CMD=0 IBF=0 OBF=0
[ 1101.375337] ACPI: EC: EC_SC(W) = 0x80
[ 1101.375564] ACPI: EC: IRQ (1)
[ 1101.375599] ACPI: EC: EC_SC(R) = 0x08 SCI_EVT=0 BURST=0 CMD=1 IBF=0 OBF=0
[ 1101.375603] ACPI: EC: EC_DATA(W) = 0x00
[ 1101.375853] ACPI: EC: IRQ (1)
[ 1101.375888] ACPI: EC: EC_SC(R) = 0x01 SCI_EVT=0 BURST=0 CMD=0 IBF=0 OBF=1
[ 1101.375908] ACPI: EC: EC_DATA(R) = 0x00
[ 1101.375922] ACPI: EC: Command(RD_EC) stopped
[ 1101.375926] ACPI: EC: 1: Decrease command
[ 1101.376178] ACPI: EC: 2: Increase command
[ 1101.376182] ACPI: EC: Command(WR_EC) started
[ 1101.376185] ACPI: EC: TASK (0)
[ 1101.376209] ACPI: EC: EC_SC(R) = 0x00 SCI_EVT=0 BURST=0 CMD=0 IBF=0 OBF=0
[ 1101.376214] ACPI: EC: EC_SC(W) = 0x81
[ 1101.376601] ACPI: EC: IRQ (1)
[ 1101.376636] ACPI: EC: EC_SC(R) = 0x08 SCI_EVT=0 BURST=0 CMD=1 IBF=0 OBF=0
[ 1101.376641] ACPI: EC: EC_DATA(W) = 0x00
[ 1101.377350] ACPI: EC: IRQ (1)
[ 1101.377385] ACPI: EC: EC_SC(R) = 0x00 SCI_EVT=0 BURST=0 CMD=0 IBF=0 OBF=0
[ 1101.377389] ACPI: EC: EC_DATA(W) = 0x1a
[ 1101.377671] ACPI: EC: IRQ (1)
[ 1101.377697] ACPI: EC: EC_SC(R) = 0x00 SCI_EVT=0 BURST=0 CMD=0 IBF=0 OBF=0
[ 1101.377712] ACPI: EC: Command(WR_EC) stopped
[ 1101.377715] ACPI: EC: 1: Decrease command
[ 1101.377802] ACPI: EC: 2: Increase command
[ 1101.377805] ACPI: EC: Command(RD_EC) started
[ 1101.377808] ACPI: EC: TASK (0)
[ 1101.377831] ACPI: EC: EC_SC(R) = 0x00 SCI_EVT=0 BURST=0 CMD=0 IBF=0 OBF=0
[ 1101.377836] ACPI: EC: EC_SC(W) = 0x80
[ 1101.378089] ACPI: EC: IRQ (1)
[ 1101.378124] ACPI: EC: EC_SC(R) = 0x08 SCI_EVT=0 BURST=0 CMD=1 IBF=0 OBF=0
[ 1101.378128] ACPI: EC: EC_DATA(W) = 0x00
[ 1101.378445] ACPI: EC: IRQ (1)
[ 1101.378469] ACPI: EC: EC_SC(R) = 0x01 SCI_EVT=0 BURST=0 CMD=0 IBF=0 OBF=1
[ 1101.378489] ACPI: EC: EC_DATA(R) = 0x00
[ 1101.378498] ACPI: EC: Command(RD_EC) stopped
[ 1101.378501] ACPI: EC: 1: Decrease command

But why doesn't this happen when pressing the actual key like on Windows?

As far as I can tell, Windows registers keycodes on the exact same keys as
evtest, i.e. there are no codes registered when I press the brightness keys.
(this also applies to mic mute, rfkill, etc...)

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #22 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
Sorry for the slow reply (note I'm afraid my next reply will likely also be a
bit slow).

Good find on the methods, so this means that the key-press events are supposed
to come from the "Video Bus" device and as you point out for some reason this
is not happening. I guess that the drivers/acpi/acpi_video.c is not making some
call to enable the events which your laptop requires; or it is making a call
which it should not make which disables the events...

One thing which I just noticed is that when passing acpi_backlight=video you
get 2 acpi_video# backlight devices. Given that AFAICT your laptop has only a
single GPU,  may be an indication of a problem.

I wonder what happens when you pass:

video.only_lcd=1 acpi_backlight=video

on the kernel cmdline, I would expect there to only be 1 acpi_video# backlight
device under /sys/class/backlight then ?    And maybe this also fixes the
brightness keys?

I assume that without any "acpi_backlight=xxx" on the kernel cmdline you only
get the intel_backlight device, right? You could also check to see if adding
just "video.only_lcd=1" to the kernel cmdline helps.

If that does not help, you may want to play with the acpi_video_bus_DOS() call
from 
drivers/acpi/acpi_video.c .

This gets called from the acpi_video_bus_start_devices() function:

static int acpi_video_bus_start_devices(struct acpi_video_bus *video)
{
        return acpi_video_bus_DOS(video, 0,
                                  acpi_osi_is_win8() ? 1 : 0);
}

You may want to modify this trying the following:

        return acpi_video_bus_DOS(video, 3,
                                  acpi_osi_is_win8() ? 1 : 0);

Looking at your acpidump it seems the second parameter is ignored, so changing
that will likely not matter.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #23 from Johannes P (johannes.penssel@gmail.com) ---
No problem, this isn't exactly the most pressing of issues.

Looks like neither any combination of kernel parameters, nor the code change
you suggested, leads to any difference in behaviour.

My laptop does have a 2nd GPU actually, a GeForce MX450 which is for all
practical purposes disabled at runtime by the nouveau driver. But no matter if
enabled or disabled in BIOS, there is only the intel_backlight device in sysfs
without any kernel parameters, as expected. By the way, enabling the GPU
results in this warning message at boot:

[    3.894320] ACPI: video: [Firmware Bug]: ACPI(PEGP) defines _DOD but not
_DOS

Interestingly, I have noticed that booting with acpi_backlight=video results in
just one acpi_video device appearing in sysfs now. (regardless if the GPU is
activated or not) I'm not sure if this is due to a change in BIOS or the
kernel, so just to be sure, I am uploading an acpi dump from the most recent
BIOS release. 

Here is some additional info about the input devices registered by acpi video
that might be helpful:

/proc/bus/input/devices

I: Bus=0019 Vendor=0000 Product=0006 Version=0000
N: Name="Video Bus"
P: Phys=LNXVIDEO/video/input0
S:
Sysfs=/devices/LNXSYSTM:00/LNXSYBUS:00/PNP0A08:00/device:00/LNXVIDEO:00/input/input8
U: Uniq=
H: Handlers=kbd event6 
B: PROP=0
B: EV=3
B: KEY=3e000b00000000 0 0 0

I: Bus=0019 Vendor=0000 Product=0006 Version=0000
N: Name="Video Bus"
P: Phys=LNXVIDEO/video/input0
S: Sysfs=/devices/LNXSYSTM:00/LNXSYBUS:00/PNP0A08:00/LNXVIDEO:01/input/input9
U: Uniq=
H: Handlers=kbd event7 
B: PROP=0
B: EV=3
B: KEY=3e000b00000000 0 0 0

Not sure if relevant, but something else I have noticed is that the brightness
value recorded by the ideapad backlight device, which as mentioned does change
when the brightness keys are pressed, corresponds to value B9 on the register
map dumped by the kernel ec tool:

     00  01  02  03  04  05  06  07  08  09  0A  0B  0C  0D  0E  0F
00:  00  01  00  00  00  00  00  01  00  00  10  00  00  00  29  00 
10:  00  00  80  00  01  00  65  04  00  01  9a  20  00  02  00  00 
20:  02  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00 
30:  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00 
40:  00  00  00  00  00  00  00  00  00  00  00  00  8d  52  00  00 
50:  00  00  00  33  00  00  2d  00  00  00  00  00  00  00  00  00 
60:  00  80  02  03  02  00  00  08  ef  17  85  43  01  00  20  01 
70:  00  02  10  01  00  00  00  00  00  00  00  00  00  00  00  00 
80:  00  00  00  00  08  00  00  00  c0  00  93  00  20  01  00  4c 
90:  47  43  00  32  30  31  39  00  4c  31  39  4c  34  50  46  31 
A0:  00  01  00  80  00  00  c0  00  00  00  00  00  00  15  00  00 
B0:  2d  2f  00  2a  00  24  00  28  02  11  04  00  00  00  01  00 
C0:  30  00  ce  14  33  05  97  3e  10  3b  58  1b  9b  1b  4b  00 
D0:  00  00  00  00  00  00  00  23  2f  0c  00  00  00  00  00  00 
E0:  00  00  00  00  00  00  00  00  00  00  00  00  01  00  43  00 
F0:  01  00  00  40  90  51  80  43  00  00  00  00  00  00  00  00

This value persists across reboots, but does not change when pressing the
brightness keys in Windows.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #24 from Johannes P (johannes.penssel@gmail.com) ---
Created attachment 300805
  --> https://bugzilla.kernel.org/attachment.cgi?id=300805&action=edit
acpidump BIOS version FHCN64WW, GPU enabled

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #25 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
Hmm, your last comments make me wonder if that maybe the ideapad-laptop module
is actually causing this problem and if iy t maybe somehow causes the EC to
stop sending the events needed for the "Video bus" input device.

Can you try blacklisting the ideapad-laptop module and see if that makes a
change? I guess you may loose functionality of some other hotkeys then, but
lets worry about that latter and for now just see if this makes a change?

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #26 from Johannes P (johannes.penssel@gmail.com) ---
Blacklisting ideapad-laptop seems to have no impact on the brightness keys.
Pressing them still results in the same register change outlined above, and
nothing else. 

Note for sanity:
For debugging purposes, I am running video.ko with report_key_events=3. Please
let me know in case that may be a source of trouble.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #27 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
(In reply to Johannes P from comment #26)
> Blacklisting ideapad-laptop seems to have no impact on the brightness keys.
> Pressing them still results in the same register change outlined above, and
> nothing else. 

Bummer, I'm afraid I'm all out of ideas then.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #28 from Johannes P (johannes.penssel@gmail.com) ---
Finally, I have managed to get the keys working. 

Using ec-dump.exe, I figured out which EC registers hold different values on
Windows compared to Linux. After writing 0x86 to offset 0xA3, (which is 0x80 on
Linux by default) the keys started working instantly, with proper ACPI events
and everything.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #29 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
(In reply to Johannes Penßel from comment #28)
> Finally, I have managed to get the keys working. 
> 
> Using ec-dump.exe, I figured out which EC registers hold different values on
> Windows compared to Linux. After writing 0x86 to offset 0xA3, (which is 0x80
> on Linux by default) the keys started working instantly, with proper ACPI
> events and everything.

That is some good detective work on your side!

May I ask how you are writing 0x86 to offset 0xA3 under Linux ?

I think this part of the DSDT is interesting:

        Device (EC0)
        {
            ...

            OperationRegion (ERAX, SystemMemory, 0xFE0B0400, 0xFF)
            Field (ERAX, ByteAcc, Lock, Preserve)
            {
                ...
                Offset (0xA3), 
                OSTY,   3, 
                    ,   1, 
                ADPI,   2, 
                    ,   1, 
                ADPT,   1, 
                ...
            }
            ...

            Method (_REG, 2, NotSerialized)  // _REG: Region Availability
            {
                If ((Arg0 == 0x03))
                {
                    ECAV = Arg1
                }

                If (((Arg0 == 0x03) && (Arg1 == One)))
                {
                    ...
                    If ((OSYS == 0x07DF))
                    {
                        Local0 = 0x06
                    }

                    If ((Acquire (LFCM, 0xA000) == Zero))
                    {
                        OSTY = Local0


Under Linux OSYS should be 0x06. So this in essence writes 0x06 to the lower 4
bits of the byte at offset 0xA3 of the MMIO region at 0xFE0B0400...

So maybe this is the path which we need to hit, but which for some reason is
not being hit under Linux ... ?

We could try to confirm this with a DSDT override. where we add an extra
unconditional (sow without all the if-s):


                       OSTY = 0x06 

To the top of the _REG method above and then build a new DSDT and use the
initrd override method to inject this. See:

https://docs.kernel.org/admin-guide/acpi/initrd_table_override.html

If you can give this a try that would be great. If the issue turns out to be
that the _REG code is somehow not hitting the desired code-path then that
likely is a generic issue caused by subtle differences between how Linux runs
ACPI code vs how Windows does it. And figuring that out and fixing it will
likely also help on other devices.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #30 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
If it turns out that the issue is that for some reason the "Local0 = 0x06" part
of:

                    If ((OSYS == 0x07DF))
                    {
                        Local0 = 0x06
                    }

Does not execute then this is similar to:
https://bugzilla.redhat.com/show_bug.cgi?id=1842039

Which is also caused by an OSYS check for some reason not working. Note OSYS
gets set to 0x07DF this bit of DSDT code:

                If (_OSI ("Windows 2015"))
                {
                    OSYS = 0x07DF
                }

Which should run early on, but on the 1842039 this for some reason is not
working.

For some more background info, Linux will answer true for _OSI ("Windows 2015")
which is asking the OS if it is "Windows 10", or IOW if it can handle modern
ACPI features, which Linux can.

So assuming that this is the cause of 0xA3 is not being set to 0x86 (*), then
getting to the bottom of this is definitely interesting.

*) Note I'm not even sure that the OSTY reference to address 0xFE0B04A3 is the
A3 we are looking for ...

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #31 from Johannes Penßel (johannes.penssel@gmail.com) ---
Thank you for your help!

The kernel ships with a tool to read / write to the EC. It is built by running
"make ec" inside tools/power/acpi and relies on the module ec_sys (enabled by
CONFIG_ACPI_EC_DEBUGFS). To enable EC write support, this module must be loaded
with the parameter "write_support=1". Writing to a register works by running
"./ec -w [offset] -v [value]".

Replacing all the OSYS if-conditionals inside _REG with just "Local0 = 0x06"
indeed fixes the issue. Looks like _REG somehow does not receive a correct
value for OSYS. I've tried changing _SB.PC00._INI so it unconditionally sets
OSYS to 0x07DF, but this does not fix _REG. When executed in step-by-step mode
with the kernel AML debugger, _SB.PC00._INI seems to function correctly by
default anyway.

Here comes the interesting part: When running _REG (with arguments 0x03 and
0x01) inside the kernel AML debugger with unmodified ACPI tables, the method
executes correctly and the keys start working.

My understanding of ACPI is limited, but my guess would be that for some
reason, \_SB.PC00.LPCB.EC0._REG gets evaluated before _SB.PC00._INI, thus
leaving the former with an incorrect value for OSYS. AFAICT, this does not look
too dissimilar from the issue in the bugzilla thread you linked.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #32 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
(In reply to Johannes Penßel from comment #31)
> Thank you for your help!

You're welcome and thank you for your work on this, your APCI debugging skills
are impressive!

> My understanding of ACPI is limited, but my guess would be that for some
> reason, \_SB.PC00.LPCB.EC0._REG gets evaluated before _SB.PC00._INI, thus
> leaving the former with an incorrect value for OSYS. AFAICT, this does not
> look too dissimilar from the issue in the bugzilla thread you linked.

Yes, I believe that that is what is happening too.

Looking at the code I have an idea how to fix this, I'm testing a patch for
this now (to make sure it does not regress things on other hw, or at least on
my laptop).

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #33 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
Created attachment 301163
  --> https://bugzilla.kernel.org/attachment.cgi?id=301163&action=edit
ACPICA: eval \_SB.PC00._INI and \_SB.PCI0._INI before running  _REG methods

As promised here is a kernel patch which will hopefully fix this, please give
this a try.

If this works, please let me know if it is ok to add the following to the
commit msg:

Reported-and-tested-by: Johannes Penßel <johannes.xxxxxxx@domain.tld>

But then when your real email address. Note this will expose your email address
to the entire world (where as here in bugzilla it is only visible to users with
a bugzilla account).

Alternatively I can also give you credit using just your name:

Reported-and-tested-by: Johannes Penßel

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #34 from Johannes Penßel (johannes.penssel@gmail.com) ---
I am afraid to report that the patch does not work on my system, even after a
full kernel rebuild:(

I have also tried simply replacing \\_SB with \\_SB.PC00 at line 166 of
nsinit.c:

                status = acpi_get_handle(NULL, "\\_SB", &handle);
                if (ACPI_SUCCESS(status)) {
                        memset(info.evaluate_info, 0,
                               sizeof(struct acpi_evaluate_info));

But that did not work either. Neither did removing the code block that
evaluates all _REG methods.



Glad I can be of help! Regarding the mail address, I'd prefer
<johannespenssel@posteo.net> :)

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #35 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
I think I know what is going on here:

1. acpi_bus_init() calls acpi_ec_ecdt_probe()
2. acpi_ec_ecdt_probe() calls acpi_ec_setup()
3. acpi_ec_setup() calls ec_install_handlers()
4. ec_install_handlers() calls acpi_install_address_space_handler()
5. acpi_install_address_space_handler() calls _REG for Device (EC0)
6. acpi_bus_init() calls acpi_initialize_objects(), but _REG has already run

I've a plan for fixing this. I hope to have a set of patches for you to test in
a couple of hours.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #36 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
Created attachment 301177
  --> https://bugzilla.kernel.org/attachment.cgi?id=301177&action=edit
Attempt 2 at fixing this (3 patches in a tarbal)

Here is attempt 2 at fixing this. I just noticed looking at the tables of my
X1C8 that this series will not work entirely correct here, since those depend
on having the EC OpRegion working from \_SB.PCI0._INI. So this needs more work.

Still if you've time it would be good if you can give this a shot to see if
this early version does fix things for you.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #37 from Johannes Penßel (johannes.penssel@gmail.com) ---
Works like a charm! Kernel log looks normal as well.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #38 from Andy Shevchenko (andy.shevchenko@gmail.com) ---
(In reply to Hans de Goede from comment #36)
> Created attachment 301177 [details]
> Attempt 2 at fixing this (3 patches in a tarbal)

I would propose, if it's feasible, to evaluate _HID/_CID (to be PCI host bridge
compatible) of the first level of \_SB and evaluate _INI of all of them. The
listing of the names seems a bit fragile, while I don't believe there will be
many new entries. That said, IIRC, \_SB.PCI is also present in some DSDTs.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #39 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
(In reply to Andy Shevchenko from comment #38)
> (In reply to Hans de Goede from comment #36)
> > Created attachment 301177 [details]
> > Attempt 2 at fixing this (3 patches in a tarbal)
> 
> I would propose, if it's feasible, to evaluate _HID/_CID (to be PCI host
> bridge compatible) of the first level of \_SB and evaluate _INI of all of
> them. The listing of the names seems a bit fragile, while I don't believe
> there will be many new entries. That said, IIRC, \_SB.PCI is also present in
> some DSDTs.

That is an interesting proposal, but this happens really early on and for some
devices _HID / _CID is not just an ACPI Name() field but an actual Method which
does a bunch of things, like e.g. checking GPIOs to see which touchscreen/pad
model is installed. 

Although it is unlikely to encounter such _CID / _HID methods on devices
directly under _SB, it cannot be ruled out, so calling _CID + _HID for all
devices under _SB seems unwise.

Also note that these patches still need to change a bit, as I mentioned they
cause trouble on a X1C8. I've a plan for this which I plan to implement today
and then I'll send out a RFC series to the linux-acpi list (and also attach it
here for Johannes to test the new version).

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

Hans de Goede (jwrdegoede@fedoraproject.org) changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #301163|0                           |1
        is obsolete|                            |
 Attachment #301177|0                           |1
        is obsolete|                            |

--- Comment #40 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
Created attachment 301181
  --> https://bugzilla.kernel.org/attachment.cgi?id=301181&action=edit
Attempt 3 at fixing this (4 patches in a tarbal)

Ok, this patch-set should still fix things on your IdeaPad while also not
causing any ACPI errors on my X1C8.

So this should be it, assuming it passes review.

Please give this a try and let me know how it goes.

I'll also submit this upstream as a RFC to get some discussion going about the
chosen approach, I'll put you in the Cc of that.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #41 from Johannes Penßel (johannes.penssel@gmail.com) ---
I'm happy to report that on my end, there are still zero issues and working
brightness keys with the latest patch set.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

tinozzo123@tutanota.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tinozzo123@tutanota.com

--- Comment #42 from tinozzo123@tutanota.com ---
I have zero issues on an IdeaPad 5 14ARE05.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

Philipp Jungkamp (p.jungkamp@gmx.net) changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |p.jungkamp@gmx.net

--- Comment #43 from Philipp Jungkamp (p.jungkamp@gmx.net) ---
I can report that this also fixes the brightness keys on the 'Lenovo Yoga 9
14IAP7'.

I was just trying to trace the ACPI events on Windows when I found this thread
while browsing the Mailing list in the train.

(In reply to Johannes Penßel from comment #15)
> Unfortunately, your latest patch did not yield any results either. :(
> 
> In the meantime, I have discovered another peculiarity: the ACPI/WMI events
> generated by this "open app" key are identical to the ones created by
> pressing Fn + ESC. (i.e. toggling Fn lock) In fact, the WMI 0xd0 event is
> already handled by ideapad driver in the ideapad_wmi_notify function. The
> only apparent difference between pressing "open app" and pressing Fn + ESC
> is that the latter also toggles the LED on the ESC key. This might be a
> firmware issue however, as this key does nothing in Windows either.
> 
> Thanks a lot for your support!

My device has multiple of these open app keys (Support, Favorite App, Virtual
Background, Sound Profile, Dark Mode Toggle), I just wrote a short WMI driver
to get them working. Have you figured out how to get these working for
yourself? I presume the open application key you were talking about is the
"star with an S inside", which is also present on my device.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

Hans de Goede (jwrdegoede@fedoraproject.org) changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #301181|0                           |1
        is obsolete|                            |

--- Comment #44 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
Created attachment 301329
  --> https://bugzilla.kernel.org/attachment.cgi?id=301329&action=edit
[PATCH 1/2] ACPICA: Make address-space-handler install and _REG execution 2
separate steps

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #45 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
Created attachment 301330
  --> https://bugzilla.kernel.org/attachment.cgi?id=301330&action=edit
[PATCH 2/2] ACPI: EC: fix ECDT probe ordering issues

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #46 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
The Linux kernel ACPI subsystem maintainer has requested to solve this in a
different way.

Johannes, can you please test the 2 patches which I've just attached?  (these
replace the 4 previous patches).

Philipp, if you can test these 2 patches too that would be great.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #47 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
(In reply to Philipp Jungkamp from comment #43)
> I can report that this also fixes the brightness keys on the 'Lenovo Yoga 9
> 14IAP7'.

That is great news.

> My device has multiple of these open app keys (Support, Favorite App,
> Virtual Background, Sound Profile, Dark Mode Toggle), I just wrote a short
> WMI driver to get them working.

Cool, please submit the new driver upstream to:
platform-driver-x86@vger.kernel.org
and then I can review it and once it passes review merge it into the mainline
kernel.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #48 from Johannes Penßel (johannes.penssel@gmail.com) ---
(In reply to Hans de Goede from comment #46)
> Johannes, can you please test the 2 patches which I've just attached? 
> (these replace the 4 previous patches).

On my system, the two new patches are working just as well as the four older
ones did.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #49 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
(In reply to Johannes Penßel from comment #48)
> (In reply to Hans de Goede from comment #46)
> > Johannes, can you please test the 2 patches which I've just attached? 
> > (these replace the 4 previous patches).
> 
> On my system, the two new patches are working just as well as the four older
> ones did.

Thank you for testing we are discussing how to move forward with this upstream.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #50 from Philipp Jungkamp (p.jungkamp@gmx.net) ---
A late response from me: Your new patches are also working wonderfully, I just
applied and tested them!

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

cris223 (cristina223@gmail.com) changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |cristina223@gmail.com

--- Comment #51 from cris223 (cristina223@gmail.com) ---
I tested the 2 patches and i can confirm works fine on my Lenovo Yoga 7i 16IAP7
Philipp's patch for favorites keys works good too!
Thank you guys.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #52 from cris223 (cristina223@gmail.com) ---
I've compiled the new ver 5.18.5 linux-xanmod-edge with the patches given by
Phillip's github
https://github.com/PJungkamp/yoga9-linux/tree/main/kernel-patches

on my github
https://github.com/322sirc/linux-xanmod-edge

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

Hans de Goede (jwrdegoede@fedoraproject.org) changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #301329|0                           |1
        is obsolete|                            |
 Attachment #301330|0                           |1
        is obsolete|                            |

--- Comment #53 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
Created attachment 302925
  --> https://bugzilla.kernel.org/attachment.cgi?id=302925&action=edit
[PATCH 1/4] ACPICA: include/acpi/acpixf.h: Fix indentation

Sorry for the long silence, I'm afraid my workload is causing me to sometimes
be a bit slow with following up on some bugs.

I have prepared a new set of patches (with only minor changes) based on
upstream feedback which I will send upstream soon.

If possible please give this new patch series a try. It is based on top of the
6.0 kernel.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #54 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
Created attachment 302926
  --> https://bugzilla.kernel.org/attachment.cgi?id=302926&action=edit
[PATCH 2/4] ACPICA: Allow address_space_handler Install and _REG execution as 2
separate steps

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #55 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
Created attachment 302927
  --> https://bugzilla.kernel.org/attachment.cgi?id=302927&action=edit
[PATCH 3/4] ACPI: EC: Fix EC address space handler unregistration

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #56 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
Created attachment 302928
  --> https://bugzilla.kernel.org/attachment.cgi?id=302928&action=edit
[PATCH 4/4] ACPI: EC: fix ECDT probe ordering issues

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #57 from Johannes Penßel (johannes.penssel@gmail.com) ---
On my machine, the new patch set is working flawlessly with Linux 6.0.

No problem, thank you very much for your time and effort working on this!

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #58 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
(In reply to Johannes Penßel from comment #57)
> On my machine, the new patch set is working flawlessly with Linux 6.0.

Great, thank you for testing!

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

Gerald Nunn (gerald.b.nunn@gmail.com) changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |gerald.b.nunn@gmail.com

--- Comment #59 from Gerald Nunn (gerald.b.nunn@gmail.com) ---
Tested the patches on a Lenovo Slim 7 (i7-1260p) which is the updated version
of the Ideapad Slim 7 Pro and it works great. Using Arch with kernel 6.0.5

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

--- Comment #60 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
(In reply to Gerald Nunn from comment #59)
> Tested the patches on a Lenovo Slim 7 (i7-1260p) which is the updated
> version of the Ideapad Slim 7 Pro and it works great. Using Arch with kernel
> 6.0.5

That is good to hear, we are waiting for:

https://github.com/acpica/acpica/pull/786

to get merged and the fixes should land in the mainline kernel pretty quickly
after that.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

Takashi Iwai (tiwai@suse.de) changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tuxuser@quantentunnel.de

--- Comment #61 from Takashi Iwai (tiwai@suse.de) ---
*** Bug 216933 has been marked as a duplicate of this bug. ***

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

[Bug 214899] ideapad-laptop: brightness hotkeys not working

[bugzilla-daemon]

https://bugzilla.kernel.org/show_bug.cgi?id=214899

Hans de Goede (jwrdegoede@fedoraproject.org) changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |CODE_FIX

--- Comment #62 from Hans de Goede (jwrdegoede@fedoraproject.org) ---
(In reply to Hans de Goede from comment #60)
> That is good to hear, we are waiting for:
> 
> https://github.com/acpica/acpica/pull/786
> 
> to get merged and the fixes should land in the mainline kernel pretty
> quickly after that.

The fixes for this have been merged into the 6.2 series, closing.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.