RE: [PATCH/RFC v2 2/2] mmc: core: Call mmc_power_off_nofity() if pm_suspend_via_firmware()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello!

Thank you for your review!

> From: Sergei Shtylyov, Sent: Friday, June 19, 2020 10:41 PM
> 
> Hello!
> 
> On 19.06.2020 14:29, Yoshihiro Shimoda wrote:
> 
> > If pm_suspend_via_firmware() returns true, the system will be able
> > to cut both vcc and vccq in the suspend. So, call
> > mmc_power_off_nofity() if pm_suspend_via_firmware() returns true.
> 
>     mmc_poweroff_notify()? :-)

Oops! I'll fix it.

Best regards,
Yoshihiro Shimoda

> > Note that we should not update the MMC_CAP2_FULL_PWR_CYCLE caps
> > because the mmc_select_voltage() checks the caps when attaches
> > a mmc/sd.
> >
> > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx>
> > ---
> >   drivers/mmc/core/mmc.c | 4 +++-
> >   1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
> > index 4203303..81941fd 100644
> > --- a/drivers/mmc/core/mmc.c
> > +++ b/drivers/mmc/core/mmc.c
> [...]
> > @@ -2038,7 +2039,8 @@ static int _mmc_suspend(struct mmc_host *host, bool is_suspend)
> >   		goto out;
> >
> >   	if (mmc_can_poweroff_notify(host->card) &&
> > -		((host->caps2 & MMC_CAP2_FULL_PWR_CYCLE) || !is_suspend))
> > +	    ((host->caps2 & MMC_CAP2_FULL_PWR_CYCLE) || !is_suspend ||
> > +	     pm_suspend_via_firmware()))
> >   		err = mmc_poweroff_notify(host->card, notify_type);
> >   	else if (mmc_can_sleep(host->card))
> >   		err = mmc_sleep(host);
> >
> 
> MBR, Sergei




[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux