| Message ID | 20250617062055.784827-1-mjkorhon@gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | ata: ahci: Disallow LPM for Asus B550-F motherboard | expand |
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@gmail.com> > --- > 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);
On Tue, Jun 17, 2025 at 03:32:17PM +0900, Damien Le Moal wrote: > 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@gmail.com> > > --- > > 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 s/timouts/timeouts/ > > + * when LPM enabled at least with WDC WD20EFAX-68FB5N0 > > + * hard drives. > > + */ > > + .matches = { > > + DMI_MATCH(DMI_BOARD_VENDOR, > > + "ASUSTeK COMPUTER INC."), align to open parenthesis > > + DMI_MATCH(DMI_BOARD_NAME, > > + "ROG STRIX B550-F GAMING (WI-FI)"), align to open parenthesis > > + }, > > + /* 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. > */ I think having it a single line as: /* 3621 is broken and there is no known good version. */ is fine, as it will be less than 80 columns, and matches the existing single line comments in the same table. Anyway, I can fix these nits up when applying, no need to send a V2. Mikko, thank you for your efforts in debugging and helping out with reviews. Kind regards, Niklas
On Tue, 17 Jun 2025 09:18:41 +0300, 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. > > Applied to libata/linux.git (for-6.16-fixes), thanks! [1/1] ata: ahci: Disallow LPM for Asus B550-F motherboard https://git.kernel.org/libata/linux/c/a7b3b77f Kind regards, Niklas
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. + */ + }, { } /* terminate list */ }; const struct dmi_system_id *dmi = dmi_first_match(sysids);
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@gmail.com> --- 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(+)