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/93c10d38-718c-459d-84a5-4d87680b4da7@debian.org/

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.

Fixes: cc77e2ce187d ("ata: libata-core: Add ATA_QUIRK_NOLPM for Samsung SSD 870 QVO drives")
Reported-by: Dieter Mummenschanz <dmummenschanz@web.de>
Closes: https://bugzilla.kernel.org/show_bug.cgi?id=219747
Link: https://lore.kernel.org/r/20250228122603.91814-2-cassel@kernel.org
Signed-off-by: Niklas Cassel <cassel@kernel.org>

diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c
index 63ec2f218431900ba2e130eb0efda2bcc9a9a283..c085dd81ebe7f646c061cd813fdcb3f4428b16cb 100644 (file)
--- a/drivers/ata/libata-core.c
+++ b/drivers/ata/libata-core.c
@@ -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 },