On 6/17/25 3:18 PM, Mikko Korhonen wrote:
> Asus ROG STRIX B550-F GAMING (WI-FI) motherboard has problems on some
> SATA ports with at least one hard drive model (WDC WD20EFAX-68FB5N0)
> when LPM is enabled. Disabling LPM solves the issue.
>
> Fixes: 7627a0edef54 ("ata: ahci: Drop low power policy board type")
> Signed-off-by: Mikko Korhonen <mjkorhon@xxxxxxxxx>
> ---
> I think we might include the comment about the non working drive model, as
> we have tested only one model and in principle this could also be an
> interoperation issue. If one would need to enable LPM for the motherboard
> and happens to have a working hard drive model, he/she would know
> why we have made the patch in the first place.
>
> drivers/ata/ahci.c | 16 ++++++++++++++++
> 1 file changed, 16 insertions(+)
>
> diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c
> index c8ad8ace7496..ae8c4072a5a3 100644
> --- a/drivers/ata/ahci.c
> +++ b/drivers/ata/ahci.c
> @@ -1454,6 +1454,22 @@ static bool ahci_broken_lpm(struct pci_dev *pdev)
> },
> /* 320 is broken, there is no known good version yet. */
> },
> + {
> + /*
> + * AMD 500 Series Chipset SATA Controller [1022:43eb]
> + * on this motherboard timouts on ports 5 and 6
> + * when LPM enabled at least with WDC WD20EFAX-68FB5N0
> + * hard drives.
> + */
> + .matches = {
> + DMI_MATCH(DMI_BOARD_VENDOR,
> + "ASUSTeK COMPUTER INC."),
> + DMI_MATCH(DMI_BOARD_NAME,
> + "ROG STRIX B550-F GAMING (WI-FI)"),
> + },
> + /* 3621 is broken, there is no known good version yet.
> + */
Please use the correct comment format:
/*
* 3621 is broken and there is no known good version.
*/
> + },
> { } /* terminate list */
> };
> const struct dmi_system_id *dmi = dmi_first_match(sysids);
--
Damien Le Moal
Western Digital Research
