]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commit
libata: add ATA_HORKAGE_NOLPM for Pioneer BDR-207M and BDR-205
authorNiklas Cassel <niklas.cassel@wdc.com>
Mon, 26 Sep 2022 18:38:09 +0000 (18:38 +0000)
committerStefan Bader <stefan.bader@canonical.com>
Mon, 14 Nov 2022 10:25:26 +0000 (11:25 +0100)
commitb994627ee665285d79f3661946274f7420cfb65f
treeca6d1d59bfa7a9dba5cd1e9549fcbff17e50ea2a
parent6820514d49e77658a17d0a96c100192c50d303db
libata: add ATA_HORKAGE_NOLPM for Pioneer BDR-207M and BDR-205

BugLink: https://bugs.launchpad.net/bugs/1995517
commit ea08aec7e77bfd6599489ec430f9f859ab84575a upstream.

Commit 1527f69204fe ("ata: ahci: Add Green Sardine vendor ID as
board_ahci_mobile") added an explicit entry for AMD Green Sardine
AHCI controller using the board_ahci_mobile configuration (this
configuration has later been renamed to board_ahci_low_power).

The board_ahci_low_power configuration enables support for low power
modes.

This explicit entry takes precedence over the generic AHCI controller
entry, which does not enable support for low power modes.

Therefore, when commit 1527f69204fe ("ata: ahci: Add Green Sardine
vendor ID as board_ahci_mobile") was backported to stable kernels,
it make some Pioneer optical drives, which was working perfectly fine
before the commit was backported, stop working.

The real problem is that the Pioneer optical drives do not handle low
power modes correctly. If these optical drives would have been tested
on another AHCI controller using the board_ahci_low_power configuration,
this issue would have been detected earlier.

Unfortunately, the board_ahci_low_power configuration is only used in
less than 15% of the total AHCI controller entries, so many devices
have never been tested with an AHCI controller with low power modes.

Fixes: 1527f69204fe ("ata: ahci: Add Green Sardine vendor ID as board_ahci_mobile")
Cc: stable@vger.kernel.org
Reported-by: Jaap Berkhout <j.j.berkhout@staalenberk.nl>
Signed-off-by: Niklas Cassel <niklas.cassel@wdc.com>
Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>
Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Kamal Mostafa <kamal@canonical.com>
Signed-off-by: Stefan Bader <stefan.bader@canonical.com>
drivers/ata/libata-core.c