* [PATCH v3] ata: ahci: Do not apply Intel PCS quirk on Intel Alder Lake [not found] <20240513135302.1869084-1-dev@kayoway.com> @ 2024-05-21 13:36 ` Jason Nader 2024-05-27 8:12 ` Niklas Cassel 0 siblings, 1 reply; 3+ messages in thread From: Jason Nader @ 2024-05-21 13:36 UTC (permalink / raw) To: dlemoal, cassel; +Cc: linux-ide, Jason Nader, stable Commit b8b8b4e0c052 ("ata: ahci: Add Intel Alder Lake-P AHCI controller to low power chipsets list") added Intel Alder Lake to the ahci_pci_tbl. Because of the way that the Intel PCS quirk was implemented, having an explicit entry in the ahci_pci_tbl caused the Intel PCS quirk to be applied. (The quirk was not being applied if there was no explict entry.) Thus, entries that were added to the ahci_pci_tbl also got the Intel PCS quirk applied. The quirk was cleaned up in commit 7edbb6059274 ("ahci: clean up intel_pcs_quirk"), such that it is clear which entries that actually applies the Intel PCS quirk. Newer Intel AHCI controllers do not need the Intel PCS quirk, and applying it when not needed actually breaks some platforms. Do not apply the Intel PCS quirk for Intel Alder Lake. This is in line with how things worked before commit b8b8b4e0c052 ("ata: ahci: Add Intel Alder Lake-P AHCI controller to low power chipsets list"), such that certain platforms using Intel Alder Lake will work once again. Cc: stable@vger.kernel.org # 6.7 Fixes: b8b8b4e0c052 ("ata: ahci: Add Intel Alder Lake-P AHCI controller to low power chipsets list") Signed-off-by: Jason Nader <dev@kayoway.com> --- drivers/ata/ahci.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c index 6548f10e61d9..07d66d2c5f0d 100644 --- a/drivers/ata/ahci.c +++ b/drivers/ata/ahci.c @@ -429,7 +429,6 @@ static const struct pci_device_id ahci_pci_tbl[] = { { PCI_VDEVICE(INTEL, 0x02d7), board_ahci_pcs_quirk }, /* Comet Lake PCH RAID */ /* Elkhart Lake IDs 0x4b60 & 0x4b62 https://sata-io.org/product/8803 not tested yet */ { PCI_VDEVICE(INTEL, 0x4b63), board_ahci_pcs_quirk }, /* Elkhart Lake AHCI */ - { PCI_VDEVICE(INTEL, 0x7ae2), board_ahci_pcs_quirk }, /* Alder Lake-P AHCI */ /* JMicron 360/1/3/5/6, match class to avoid IDE function */ { PCI_VENDOR_ID_JMICRON, PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID, -- 2.45.1 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v3] ata: ahci: Do not apply Intel PCS quirk on Intel Alder Lake 2024-05-21 13:36 ` [PATCH v3] ata: ahci: Do not apply Intel PCS quirk on Intel Alder Lake Jason Nader @ 2024-05-27 8:12 ` Niklas Cassel 2024-05-30 14:11 ` Alex 0 siblings, 1 reply; 3+ messages in thread From: Niklas Cassel @ 2024-05-27 8:12 UTC (permalink / raw) To: Jason Nader; +Cc: dlemoal, linux-ide, stable On Tue, May 21, 2024 at 10:36:24PM +0900, Jason Nader wrote: > Commit b8b8b4e0c052 ("ata: ahci: Add Intel Alder Lake-P AHCI controller > to low power chipsets list") added Intel Alder Lake to the ahci_pci_tbl. > > Because of the way that the Intel PCS quirk was implemented, having > an explicit entry in the ahci_pci_tbl caused the Intel PCS quirk to > be applied. (The quirk was not being applied if there was no explict > entry.) > > Thus, entries that were added to the ahci_pci_tbl also got the Intel > PCS quirk applied. > > The quirk was cleaned up in commit 7edbb6059274 ("ahci: clean up > intel_pcs_quirk"), such that it is clear which entries that actually > applies the Intel PCS quirk. > > Newer Intel AHCI controllers do not need the Intel PCS quirk, > and applying it when not needed actually breaks some platforms. > > Do not apply the Intel PCS quirk for Intel Alder Lake. > This is in line with how things worked before commit b8b8b4e0c052 ("ata: > ahci: Add Intel Alder Lake-P AHCI controller to low power chipsets list"), > such that certain platforms using Intel Alder Lake will work once again. > > Cc: stable@vger.kernel.org # 6.7 > Fixes: b8b8b4e0c052 ("ata: ahci: Add Intel Alder Lake-P AHCI controller to low power chipsets list") > Signed-off-by: Jason Nader <dev@kayoway.com> > --- > drivers/ata/ahci.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c > index 6548f10e61d9..07d66d2c5f0d 100644 > --- a/drivers/ata/ahci.c > +++ b/drivers/ata/ahci.c > @@ -429,7 +429,6 @@ static const struct pci_device_id ahci_pci_tbl[] = { > { PCI_VDEVICE(INTEL, 0x02d7), board_ahci_pcs_quirk }, /* Comet Lake PCH RAID */ > /* Elkhart Lake IDs 0x4b60 & 0x4b62 https://sata-io.org/product/8803 not tested yet */ > { PCI_VDEVICE(INTEL, 0x4b63), board_ahci_pcs_quirk }, /* Elkhart Lake AHCI */ > - { PCI_VDEVICE(INTEL, 0x7ae2), board_ahci_pcs_quirk }, /* Alder Lake-P AHCI */ > > /* JMicron 360/1/3/5/6, match class to avoid IDE function */ > { PCI_VENDOR_ID_JMICRON, PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID, > -- > 2.45.1 > Applied: https://git.kernel.org/pub/scm/linux/kernel/git/libata/linux.git/log/?h=for-6.10-fixes ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v3] ata: ahci: Do not apply Intel PCS quirk on Intel Alder Lake 2024-05-27 8:12 ` Niklas Cassel @ 2024-05-30 14:11 ` Alex 0 siblings, 0 replies; 3+ messages in thread From: Alex @ 2024-05-30 14:11 UTC (permalink / raw) To: Niklas Cassel, Jason Nader; +Cc: dlemoal, linux-ide, stable On 27/05/2024 10:12, Niklas Cassel wrote: > > Applied: > https://git.kernel.org/pub/scm/linux/kernel/git/libata/linux.git/log/?h=for-6.10-fixes Thank you a lot for the fix, at last, i will see again my ssd :) ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-05-30 14:11 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20240513135302.1869084-1-dev@kayoway.com>
2024-05-21 13:36 ` [PATCH v3] ata: ahci: Do not apply Intel PCS quirk on Intel Alder Lake Jason Nader
2024-05-27 8:12 ` Niklas Cassel
2024-05-30 14:11 ` Alex
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox