** Description changed:

  [Impact]
  
  On some platform designs the the power button wakeup from s0i3 doesn't
  work even with the existing changes to IRQ handling for the GPIO
  controller.
  
  In these designs the ACPI SCI and GPIO controller share an IRQ. Due to
- the the way the s2idle loop handles the IRQ for the ACPI SCI the GPIO
+ the way the s2idle loop handles the IRQ for the ACPI SCI the GPIO
  controller driver doesn't get a chance to wake the system in this
  circumstance.
- 
  
  [Fix]
  To fix this rework the existing IRQ handler function to function as a checker 
and an IRQ handler depending on the calling arguments.
  
  The patches were cherry-picked from 5.16rc3.
  
  [Test]
  This is requested by AMD. The test was done on two systems L15 AMD Gen2 and 
P14s AMD Gen2 and both can be waken up by power button
  
  [Where problems could occur]
  Risk is low. This only affects AMD platforms with s0i3 enabled. 
Suspends/resumes are not affect.

** Description changed:

  [Impact]
  
  On some platform designs the the power button wakeup from s0i3 doesn't
  work even with the existing changes to IRQ handling for the GPIO
  controller.
  
  In these designs the ACPI SCI and GPIO controller share an IRQ. Due to
  the way the s2idle loop handles the IRQ for the ACPI SCI the GPIO
  controller driver doesn't get a chance to wake the system in this
  circumstance.
  
  [Fix]
  To fix this rework the existing IRQ handler function to function as a checker 
and an IRQ handler depending on the calling arguments.
  
  The patches were cherry-picked from 5.16rc3.
  
  [Test]
- This is requested by AMD. The test was done on two systems L15 AMD Gen2 and 
P14s AMD Gen2 and both can be waken up by power button
+ This is requested by AMD. The test was done on two systems L15 AMD Gen2 and 
P14s AMD Gen2 and both can be waken up by power button presses.
  
  [Where problems could occur]
  Risk is low. This only affects AMD platforms with s0i3 enabled. 
Suspends/resumes are not affect.

-- 
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/1953540

Title:
   Fix power button wakeup with shared IRQs on AMD platforms

Status in linux package in Ubuntu:
  In Progress

Bug description:
  [Impact]

  On some platform designs the the power button wakeup from s0i3 doesn't
  work even with the existing changes to IRQ handling for the GPIO
  controller.

  In these designs the ACPI SCI and GPIO controller share an IRQ. Due to
  the way the s2idle loop handles the IRQ for the ACPI SCI the GPIO
  controller driver doesn't get a chance to wake the system in this
  circumstance.

  [Fix]
  To fix this rework the existing IRQ handler function to function as a checker 
and an IRQ handler depending on the calling arguments.

  The patches were cherry-picked from 5.16rc3.

  [Test]
  This is requested by AMD. The test was done on two systems L15 AMD Gen2 and 
P14s AMD Gen2 and both can be waken up by power button presses.

  [Where problems could occur]
  Risk is low. This only affects AMD platforms with s0i3 enabled. 
Suspends/resumes are not affect.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1953540/+subscriptions


-- 
Mailing list: https://launchpad.net/~kernel-packages
Post to     : kernel-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kernel-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to