Hi,
These patches will be further taken by Sarthak.
Thanks,
Satya
-----Original Message-----
From: Ulf Hansson <ulf.hansson@xxxxxxxxxx>
Sent: Wednesday, April 27, 2022 4:06 AM
To: quic_spathi <quic_spathi@xxxxxxxxxxx>
Cc: u.kleine-koenig@xxxxxxxxxxxxxx; YehezkelShB@xxxxxxxxx; rmk+kernel@xxxxxxxxxxxxxxx; t.scherer@xxxxxxxxxxxx; s.shtylyov@xxxxxx; sensor1010@xxxxxxx; sartgarg@xxxxxxxxxxxxxx; hns@xxxxxxxxxxxxx; uic_kamasali@xxxxxxxxxxx; tiantao6@xxxxxxxxxxxxx; linux-mmc@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Kamasali Satyanarayan (Consultant) (QUIC) <quic_kamasali@xxxxxxxxxxx>
Subject: Re: [PATCH V1] mmc: sdhci-msm: Add wakeup functionality support for sdio cards
On Tue, 26 Apr 2022 at 11:10, Srinivasarao Pathipati <quic_spathi@xxxxxxxxxxx> wrote:
From: Sarthak Garg <sartgarg@xxxxxxxxxxxxxx>There is not a single line changed in the sdhci-msm driver, so I am not sure what this is intended to fix.
This adds external GPIO wakeup support to sdhci-msm driver for sdio
cards.
I assume this is a downstream patch you want to upstream, which is a good thing that we all appreciate. However, before just posting a patch from downstream code, please have a look at the commit message and let it explain what and why you want to change things.
I am deferring to review this, until you post a version with a proper commit message. Sorry.
Kind regards
Uffe
Also enables clk gating only in system Suspend/Resume for SDIO card.
Also add the below fixes from 4.9 kernel :
c363224b: Fix wakeup functionality for SDIO
61fc5bf6: Remove flag MMC_PM_WAKE_SDIO_IRQ in mmc_resume_host
a7a2a82e: Set sdio_pending_processing default state to false.
Signed-off-by: Sarthak Garg <sartgarg@xxxxxxxxxxxxxx>
Signed-off-by: kamasali <quic_kamasali@xxxxxxxxxxx>
Signed-off-by: Srinivasarao Pathipati <quic_spathi@xxxxxxxxxxx>
---
drivers/mmc/core/bus.c | 7 +++++++
drivers/mmc/core/sdio.c | 1 +
2 files changed, 8 insertions(+)
diff --git a/drivers/mmc/core/bus.c b/drivers/mmc/core/bus.c index
58a60af..a475fe1 100644
--- a/drivers/mmc/core/bus.c
+++ b/drivers/mmc/core/bus.c
@@ -364,6 +364,13 @@ int mmc_add_card(struct mmc_card *card) #endif
card->dev.of_node = mmc_of_find_child_device(card->host, 0);
+ if (mmc_card_sdio(card)) {
+ ret = device_init_wakeup(&card->dev, true);
+ if (ret)
+ pr_err("%s: %s: failed to init wakeup: %d\n",
+ mmc_hostname(card->host), __func__, ret);
+ }
+
device_enable_async_suspend(&card->dev);
ret = device_add(&card->dev);
diff --git a/drivers/mmc/core/sdio.c b/drivers/mmc/core/sdio.c index
25799ac..9502318 100644
--- a/drivers/mmc/core/sdio.c
+++ b/drivers/mmc/core/sdio.c
@@ -1096,6 +1096,7 @@ static int mmc_sdio_resume(struct mmc_host *host)
mmc_release_host(host);
host->pm_flags &= ~MMC_PM_KEEP_POWER;
+ host->pm_flags &= ~MMC_PM_WAKE_SDIO_IRQ;
return err;
}
--
2.7.4