| Message ID | 20190618153448.27145-1-ulf.hansson@linaro.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | mmc: sdio: Various fixes/improvements for SDIO PM | expand |
On Tue, 18 Jun 2019 at 17:34, Ulf Hansson <ulf.hansson@linaro.org> wrote: > > The power management support implemented via system suspend/resume and runtime > suspend/resume for SDIO cards is rather messy, but also fragile. > > This series makes some improvement to this code. In particular the so called > powered-on re-initialization of SDIO card is quite questionable, I suspect > it may never really worked well. Therefore this series removes this code, which > helps to prepare for additional improvements on top in a later series. > > So far the series has only been compile tested, so any help in testing on HW for > regressions is greatly appreciated. > > Kind regards > Uffe > > Ulf Hansson (7): > mmc: sdio: Turn sdio_run_irqs() into static > mmc: sdio: Drop mmc_claim|release_host() in mmc_sdio_power_restore() > mmc: sdio: Move comment about re-initialization to > mmc_sdio_reinit_card() > mmc: sdio: Drop powered-on re-init at runtime resume and HW reset > mmc: sdio: Don't re-initialize powered-on removable SDIO cards at > resume > mmc: sdio: Drop unused in-parameter to mmc_sdio_reinit_card() > mmc: sdio: Drop unused in-parameter from mmc_sdio_init_card() > > drivers/mmc/core/sdio.c | 92 +++++++++++++++---------------------- > drivers/mmc/core/sdio_irq.c | 3 +- > include/linux/mmc/host.h | 1 - > 3 files changed, 38 insertions(+), 58 deletions(-) > > -- > 2.17.1 > I decided to queue this up, to see what tests from linux-next and kernelCI reports. Still, that doesn't mean I am appreciating test done on HW. I can also apply tested-by tags by amending patches after this point. Kind regards Uffe
The power management support implemented via system suspend/resume and runtime suspend/resume for SDIO cards is rather messy, but also fragile. This series makes some improvement to this code. In particular the so called powered-on re-initialization of SDIO card is quite questionable, I suspect it may never really worked well. Therefore this series removes this code, which helps to prepare for additional improvements on top in a later series. So far the series has only been compile tested, so any help in testing on HW for regressions is greatly appreciated. Kind regards Uffe Ulf Hansson (7): mmc: sdio: Turn sdio_run_irqs() into static mmc: sdio: Drop mmc_claim|release_host() in mmc_sdio_power_restore() mmc: sdio: Move comment about re-initialization to mmc_sdio_reinit_card() mmc: sdio: Drop powered-on re-init at runtime resume and HW reset mmc: sdio: Don't re-initialize powered-on removable SDIO cards at resume mmc: sdio: Drop unused in-parameter to mmc_sdio_reinit_card() mmc: sdio: Drop unused in-parameter from mmc_sdio_init_card() drivers/mmc/core/sdio.c | 92 +++++++++++++++---------------------- drivers/mmc/core/sdio_irq.c | 3 +- include/linux/mmc/host.h | 1 - 3 files changed, 38 insertions(+), 58 deletions(-)