On Sat, Nov 13, 2021 at 7:17 PM Yauhen Kharuzhy <jekhor@xxxxxxxxx> wrote:
>
> On Sat, Nov 13, 2021 at 02:52:06PM +0100, Hans de Goede wrote:
> > It turns out that there is a WMI object which controls the PWM2 device
> > used for the keyboard backlight and that WMI object also provides some
> > other useful functionality.
> >
> > The upcoming lenovo-yogabook-wmi driver will offer both backlight
> > control and the other functionality, so there no longer is a need
> > to have the lpss-pwm driver binding to PWM2 for backlight control;
> > and this is now actually undesirable because this will cause both
> > the WMI code and the lpss-pwm driver to poke at the same PWM
> > controller.
>
> Acked-by: Yauhen Kharuzhy <jekhor@xxxxxxxxx>
>
> >
> > Revert commit ff6cdfd71495 ("ACPI / x86: Make PWM2 device always present
> > at Lenovo Yoga Book"), removing the always-present quirk for the PWM2
> > ACPI-device, so that the lpss-pwm controller will no longer bind to it.
> >
> > Cc: Yauhen Kharuzhy <jekhor@xxxxxxxxx>
> > Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx>
> > ---
> > drivers/acpi/x86/utils.c | 4 ----
> > 1 file changed, 4 deletions(-)
> >
> > diff --git a/drivers/acpi/x86/utils.c b/drivers/acpi/x86/utils.c
> > index cb988f9b23a1..bfcb76888ca7 100644
> > --- a/drivers/acpi/x86/utils.c
> > +++ b/drivers/acpi/x86/utils.c
> > @@ -54,10 +54,6 @@ static const struct always_present_id always_present_ids[] = {
> > ENTRY("80860F09", "1", X86_MATCH(ATOM_SILVERMONT), {}),
> > ENTRY("80862288", "1", X86_MATCH(ATOM_AIRMONT), {}),
> >
> > - /* Lenovo Yoga Book uses PWM2 for keyboard backlight control */
> > - ENTRY("80862289", "2", X86_MATCH(ATOM_AIRMONT), {
> > - DMI_MATCH(DMI_PRODUCT_NAME, "Lenovo YB1-X9"),
> > - }),
> > /* The Xiaomi Mi Pad 2 uses PWM2 for touchkeys backlight control */
> > ENTRY("80862289", "2", X86_MATCH(ATOM_AIRMONT), {
> > DMI_MATCH(DMI_SYS_VENDOR, "Xiaomi Inc"),
> > --
Applied as 5.16-rc2 material, thanks!
