kernel-packages team mailing list archive

[Víctor Hernández <bartovictorh@xxxxxxxxx>]

** Changed in: linux (Ubuntu)
       Status: Confirmed => Fix Released

-- 
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1118657

Title:
  Lenovo m490/k4350/b580/L530/k29 - ACPI Warnings for
  \_SB_.PCI0.LPCB.H_EC.LED_

Status in HWE Next Project:
  Won't Fix
Status in linux package in Ubuntu:
  Fix Released

Bug description:
  Suspend and hibernate stress test got failed with following errors in logs:
  s3: HIGH Kernel message: [10512.177806] ACPI Warning: For \_SB_.PCI0.LPCB.H_EC.LED_: Excess arguments - needs 1, found 2 (20110623/nspredef-326)
  s4: HIGH Kernel message: [11728.328110] ACPI Warning: For \_SB_.PCI0.LPCB.H_EC.LED_: Excess arguments - needs 1, found 2 (20110623/nspredef-326)

  Steps:
  1. Log into desktop
  2. Run s3/s4 stress test for 30 times

  Expected results:
  Suspend and hibernate stress got passed without any error

  Actual results:
  s3/s4 got failures in log file

  Diagnosis:

  This warning is generated by the ACPI LED method expects 1 argument, but the kernel driver thinkpad_acpi.c function led_set_status will evaluate the ACPI LED method with two arguments (let and led_status):
                      Method (LED, 1, NotSerialized)
                      {
                          If (H8DR)
                          {
                              Store (Arg0, MLED)
                          }
                          Else
                          {
                              And (RBEC (0x0B), 0xFD, Local0)
                              Or (Local0, Arg0, Local0)
                              WBEC (0x0B, Local0)
                          }
                      }

  and I also checked with lenovo thiinkpad x220, x200, seems it accepts two arguments in ACPI LED method:
                      Method (LED, 2, NotSerialized)
                      {
                          Or (Arg0, Arg1, Local0)
                          If (\H8DR)
                          {
                              Store (Local0, HLCL)
                          }
                          Else
                          {
                              \WBEC (0x0C, Local0)
                          }
                      }

  It seems the spec change, if yes, we might need the spec.

To manage notifications about this bug go to:
https://bugs.launchpad.net/hwe-next/+bug/1118657/+subscriptions