linux-kernel - RE: [PATCH V1] mmc: sdhci-msm: Add wakeup functionality support for sdio cards

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 24 May 2022 11:56:16 +0000
From:   "Kamasali Satyanarayan (Consultant) (QUIC)" 
        <quic_kamasali@...cinc.com>
To:     'Ulf Hansson' <ulf.hansson@...aro.org>,
        "Sarthak Garg (QUIC)" <quic_sartgarg@...cinc.com>
CC:     "u.kleine-koenig@...gutronix.de" <u.kleine-koenig@...gutronix.de>,
        "YehezkelShB@...il.com" <YehezkelShB@...il.com>,
        "rmk+kernel@...linux.org.uk" <rmk+kernel@...linux.org.uk>,
        "t.scherer@...elmann.de" <t.scherer@...elmann.de>,
        "s.shtylyov@....ru" <s.shtylyov@....ru>,
        "sensor1010@....com" <sensor1010@....com>,
        "sartgarg@...eaurora.org" <sartgarg@...eaurora.org>,
        "hns@...delico.com" <hns@...delico.com>,
        "uic_kamasali@...cinc.com" <uic_kamasali@...cinc.com>,
        "tiantao6@...ilicon.com" <tiantao6@...ilicon.com>,
        "linux-mmc@...r.kernel.org" <linux-mmc@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "'quic_spathi@...cinc.com'" <quic_spathi@...cinc.com>
Subject: RE: [PATCH V1] mmc: sdhci-msm: Add wakeup functionality support for
 sdio cards

Hi,
These patches will be further taken by Sarthak.

Thanks,
Satya

-----Original Message-----
From: Ulf Hansson <ulf.hansson@...aro.org> 
Sent: Wednesday, April 27, 2022 4:06 AM
To: quic_spathi <quic_spathi@...cinc.com>
Cc: u.kleine-koenig@...gutronix.de; YehezkelShB@...il.com; rmk+kernel@...linux.org.uk; t.scherer@...elmann.de; s.shtylyov@....ru; sensor1010@....com; sartgarg@...eaurora.org; hns@...delico.com; uic_kamasali@...cinc.com; tiantao6@...ilicon.com; linux-mmc@...r.kernel.org; linux-kernel@...r.kernel.org; Kamasali Satyanarayan (Consultant) (QUIC) <quic_kamasali@...cinc.com>
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@...cinc.com> wrote:
>
> From: Sarthak Garg <sartgarg@...eaurora.org>
>
> This adds external GPIO wakeup support to sdhci-msm driver for sdio 
> cards.

There is not a single line changed in the sdhci-msm driver, so I am not sure what this is intended to fix.

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@...eaurora.org>
> Signed-off-by: kamasali <quic_kamasali@...cinc.com>
> Signed-off-by: Srinivasarao Pathipati <quic_spathi@...cinc.com>
> ---
>  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
>