LKML Archive on
help / color / mirror / Atom feed
From: Kalle Valo <>
To: Eyal Reizer <>
	Eyal Reizer <>,
Subject: Re: [PATCH] wlcore: sdio: allow pm to handle sdio power
Date: Wed, 25 Apr 2018 16:58:15 +0300	[thread overview]
Message-ID: <> (raw)
In-Reply-To: <> (Eyal Reizer's message of "Wed, 25 Apr 2018 16:41:50 +0300")

Eyal Reizer <> writes:

> pm_runtime handles sdio power on and power off transitions.
> An old workaround for trying to control the power explicitly from the
> driver was in fact causing failures on suspend/resume as the mmc layer
> already power the module on resume.
> In case of resume pm_runtime_get sync returns a positive device's usage
> count causing the driver to try an re-initialize an already initialized
> device. This was causing sdio bus failure on resume.
> Remove this manual power on/off sequence as it is in-fact not needed.
> Signed-off-by: Eyal Reizer <>
> Cc:

I'll remove "net: wireless: ti:" from the title. The idea is not to put
the whole directory structure there, just the driver name.

Kalle Valo

      parent reply	other threads:[~2018-04-25 13:58 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-25 13:41 [PATCH] net: wireless: ti: " Eyal Reizer
2018-04-25 13:55 ` Tony Lindgren
2018-04-25 13:58 ` Kalle Valo [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \ \ \ \ \ \ \ \ \ \ \
    --subject='Re: [PATCH] wlcore: sdio: allow pm to handle sdio power' \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).