Revert "ata: libata-core: Add ATA_QUIRK_NOLPM for Samsung SSD 870 QVO drives"
Niklas Cassel
maintainerPATCH 1/1 Revert "ata: libata-core: Add ATA_QUIRK_NOLPM for Samsung SSD 870 QVO drives"
This reverts commit cc77e2ce187d26cc66af3577bf896d7410eb25ab.
It was reported that adding ATA_QUIRK_NOLPM for Samsung SSD 870 QVO drives breaks entering lower package states for certain systems.
It turns out that Samsung SSD 870 QVO actually has working LPM when using a recent SSD firmware version.
The author of commit cc77e2ce187d ("ata: libata-core: Add ATA_QUIRK_NOLPM for Samsung SSD 870 QVO drives") reported himself that only older SSD firmware versions have broken LPM:
https://lore.kernel.org/stable/[email protected]/
Unfortunately, he did not specify which older firmware version he was using which had broken LPM.
Let's revert this quirk, which has FW version field specified as NULL (which means that it applies for all Samsung SSD 870 QVO firmware versions) for now. Once the author reports which older firmware version(s) that are broken, we can create a more fine grained quirk, which populates the FW version field accordingly.
--- drivers/ata/libata-core.c | 4 ---- 1 file changed, 4 deletions(-)
drivers/ata/libata-core.c +0 -4
@@ -4143,10 +4143,6 @@ static const struct ata_dev_quirks_entry __ata_dev_quirks[] = { {"Samsung SSD 860*" ,NULL ,ATA_QUIRK_NO_NCQ_TRIM |ATA_QUIRK_ZERO_AFTER_TRIM | ATA_QUIRK_NO_NCQ_ON_ATI }, - { "Samsung SSD 870 QVO*", NULL, ATA_QUIRK_NO_NCQ_TRIM | -ATA_QUIRK_ZERO_AFTER_TRIM | -ATA_QUIRK_NO_NCQ_ON_ATI | - ATA_QUIRK_NOLPM }, { "Samsung SSD 870*", NULL, ATA_QUIRK_NO_NCQ_TRIM |ATA_QUIRK_ZERO_AFTER_TRIM | ATA_QUIRK_NO_NCQ_ON_ATI },
Signature · 1 line
2.48.1
Quoted 9 lines · Niklas Cassel
This reverts commit
cc77e2ce187d26cc66af3577bf896d7410eb25ab.It was reported that adding
ATA_QUIRK_NOLPMfor Samsung SSD 870 QVO drives breaks entering lower package states for certain systems.It turns out that Samsung SSD 870 QVO actually has working LPM when using a recent SSD firmware version.
[...]
Applied to libata/linux.git (for-6.14), thanks!
[1/1] Revert "ata: libata-core: Add ATA_QUIRK_NOLPM for Samsung SSD 870 QVO drives"
https://git.kernel.org/libata/linux/c/a2f925a2
Kind regards,
Niklas