Bug 201801 - SD card reader in Acer Aspire S5 broken after ASPM changes
Summary: SD card reader in Acer Aspire S5 broken after ASPM changes
Status: NEW
Alias: None
Product: Drivers
Classification: Unclassified
Component: PCI (show other bugs)
Hardware: All Linux
: P1 normal
Assignee: drivers_pci@kernel-bugs.osdl.org
URL: https://lore.kernel.org/lkml/2960808....
Depends on:
Reported: 2018-11-28 11:59 UTC by Rafael J. Wysocki
Modified: 2019-02-06 15:55 UTC (History)
1 user (show)

See Also:
Kernel Version: 4.20-rc[1-4]
Tree: Mainline
Regression: Yes

dmesg log with pci=earlydump (103.45 KB, text/plain)
2018-11-28 12:08 UTC, Rafael J. Wysocki
acpidump output (281.17 KB, application/octet-stream)
2018-11-28 12:09 UTC, Rafael J. Wysocki
Output of 'lspci -vv' from Aspire S5 (60.87 KB, text/plain)
2018-11-28 12:09 UTC, Rafael J. Wysocki
sample patch for RTS5209 (1.61 KB, patch)
2019-02-06 15:55 UTC, Patrick Talbert
Details | Diff

Description Rafael J. Wysocki 2018-11-28 11:59:39 UTC
The SD card reader in my Acer Aspire S5 doesn't work with 4.20-rc.

When it doesn't work, it doesn't generate any interrupts on device insertion
and removal and this seems to be reproducible 100% of the time.

Reverting commit

commit 17c91487364fb33797ed84022564ee7544ac4945
Author: Patrick Talbert <ptalbert@redhat.com>
Date:   Wed Sep 5 09:12:53 2018 +0200

    PCI/ASPM: Do not initialize link state when aspm_disabled is set

makes the problem go away.
Comment 1 Rafael J. Wysocki 2018-11-28 12:08:42 UTC
Created attachment 279695 [details]
dmesg log with pci=earlydump
Comment 2 Rafael J. Wysocki 2018-11-28 12:09:15 UTC
Created attachment 279697 [details]
acpidump output
Comment 3 Rafael J. Wysocki 2018-11-28 12:09:55 UTC
Created attachment 279699 [details]
Output of 'lspci -vv' from Aspire S5
Comment 4 Patrick Talbert 2019-02-06 15:55:12 UTC
Created attachment 281023 [details]
sample patch for RTS5209

This seems to possibly be an issue with the card reader or rtsx driver.

I take it if you boot with pcie_aspm=off the card reader ends up in the same bad state?

What about this silly quirk? Can you test if it works for you?

Note You need to log in before you can comment on or make changes to this bug.