• Niklas Cassel's avatar
    ata: libata-core: Add ATA_HORKAGE_NOLPM for AMD Radeon S3 SSD · 47388036
    Niklas Cassel authored
    Commit 7627a0ed ("ata: ahci: Drop low power policy board type")
    dropped the board_ahci_low_power board type, and instead enables LPM if:
    -The AHCI controller reports that it supports LPM (Partial/Slumber), and
    -CONFIG_SATA_MOBILE_LPM_POLICY != 0, and
    -The port is not defined as external in the per port PxCMD register, and
    -The port is not defined as hotplug capable in the per port PxCMD
     register.
    
    Partial and Slumber LPM states can either be initiated by HIPM or DIPM.
    
    For HIPM (host initiated power management) to get enabled, both the AHCI
    controller and the drive have to report that they support HIPM.
    
    For DIPM (device initiated power management) to get enabled, only the
    drive has to report that it supports DIPM. However, the HBA will reject
    device requests to enter LPM states which the HBA does not support.
    
    The problem is that AMD Radeon S3 SSD drives do not handle low power modes
    correctly. The problem was most likely not seen before because no one
    had used this drive with a AHCI controller with LPM enabled.
    
    Add a quirk so that we do not enable LPM for this drive, since we see
    command timeouts if we do (even though the drive claims to support both
    HIPM and DIPM).
    
    Fixes: 7627a0ed ("ata: ahci: Drop low power policy board type")
    Cc: stable@vger.kernel.org
    Reported-by: default avatarDoru Iorgulescu <doru.iorgulescu1@gmail.com>
    Closes: https://bugzilla.kernel.org/show_bug.cgi?id=218832Reviewed-by: default avatarMika Westerberg <mika.westerberg@linux.intel.com>
    Reviewed-by: default avatarDamien Le Moal <dlemoal@kernel.org>
    Signed-off-by: default avatarNiklas Cassel <cassel@kernel.org>
    47388036
libata-core.c 171 KB