Re: [PATCH] ata: ahci: Add Intel Alder Lake-P AHCI controller to low power chipsets list

[Mika Westerberg <mika.westerberg@linux.intel.com>]

Hi,

On Mon, Sep 25, 2023 at 11:09:01AM +0200, Damien Le Moal wrote:
> On 2023/09/25 10:05, Mika Westerberg wrote:
> > Intel Alder Lake-P AHCI controller needs to be added to the mobile
> > chipsets list in order to have link power management enabled. Without
> > this the CPU cannot enter lower power C-states making idle power
> > consumption high.
> > 
> > Cc: Koba Ko <koba.ko@canonical.com>
> > Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com>
> 
> Looks OK, but given that there is a tendency of the low power stuff to be buggy,
> was this well tested ?

Yes it was tested (Koba Cc'd can confirm this). We also confirmed from
Intel AHCI folks that the ADL (and RPL) AHCI controllers fully support
this configuration.

> Also, does this need a Fixes/CC stable tag ? If not, I
> will queue this for 6.7.

Up to you :) Typically PCI ID additions can go to stable as well. No
fixes tag needed, though (there is no commit that this one fixes).

Thanks!