Hi, On 12-04-17 15:07, Adrian Hunter wrote:
On 05/04/17 16:00, Hans de Goede wrote:Commit e5bbf30733f9 ("mmc: sdhci-acpi: Ensure connected devices are powered when probing") introduced unconditional calling of acpi_device_fix_up_power() on the mmc controller and its children. This broke wifi on some systems because acpi_device_fix_up_power() was called even for disabled children sometimes leaving gpio-s in a state where wifi would not work, this was fixed in commit e1d070c3793a ("mmc: sdhci-acpi: Only powered up enabled acpi child devices"). Unfortunately on some devices calling acpi_device_fix_up_power() still causes issues. Specifically on the GPDwin mini clam-shell PC which has a pcie wifi module, it causes the wifi module to get turned off. This is a BIOS bug and I've tried to get the manufacturer to fix this but sofar they have not responded (and even if they do then we cannot assume all users will update their BIOS). This commit adds a new sdhci_acpi.fix_up_power_blacklist module option which can be set to an ACPI hid:uid pair, e.g. "80860F14:2" to disable the calling of acpi_device_fix_up_power() for the sdhci host matching the hid:uid pair, fixing the wifi not working on this machine.Sorry for the slow reply :-(.
No worries.
I would much prefer to blacklist the whole device i.e. if the firmware is broken then the device is broken.
So I'm in 2 minds about this, for the GPDwin disabling the entire sdio controller is the right thing to do, since the wifi is actually attached to the pcie. OTOH I've the feeling (but no proof) that not doing the fixup-power might actually help sdio attached wifi on some devices (this feeling is based on my patch where I added the _STA check before calling fixup power, which does fix some real world issues). Anyways you say you prefer blacklisting the entire sdhci controller, so I will send a v3 doing that. Regards, Hans -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
