From: Ulf Hansson <ulf.hansson@linaro.org>
To: linux-mmc@vger.kernel.org, Ulf Hansson <ulf.hansson@linaro.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>,
Wolfram Sang <wsa+renesas@sang-engineering.com>,
Ludovic Barre <ludovic.barre@st.com>,
Baolin Wang <baolin.wang7@gmail.com>,
Linus Walleij <linus.walleij@linaro.org>,
Chaotian Jing <chaotian.jing@mediatek.com>,
Shawn Lin <shawn.lin@rock-chips.com>,
mirq-linux@rere.qmqm.pl
Subject: [PATCH 00/12] mmc: core: Improve code for polling and HW busy detect
Date: Tue, 4 Feb 2020 09:54:37 +0100 [thread overview]
Message-ID: <20200204085449.32585-1-ulf.hansson@linaro.org> (raw)
There exists several separate variants of polling loops, used to detect when
the card stop signals busy for various operations, in the mmc core. All of them
have different issues that needs to be fixed.
The intent with this series, is to address some of these problems, via first
improving the mmc_poll_for_busy() function, then consolidate code by moving
more users to it.
While I was working on this, I stumbled over some code here and there, that
deserved some cleanup, hence I also folded in a couple of patches for this.
So far, I have only managed to extensively test the updated mmc_poll_for_busy()
function for CMD6 commands. Some tests for erase/trim/discard and for
HPI+sanitize are needed.
Note that, there are still separate polling loops in the mmc block layer, but
moving that to mmc_poll_for_busy() involves some additional work. I am looking
into that as a next step.
Please help review and test!
Kind regards
Ulf Hansson
Ulf Hansson (12):
mmc: core: Throttle polling rate for CMD6
mmc: core: Drop unused define
mmc: core: Extend mmc_switch_status() to rid of __mmc_switch_status()
mmc: core: Drop redundant in-parameter to __mmc_switch()
mmc: core: Split up mmc_poll_for_busy()
mmc: core: Enable re-use of mmc_blk_in_tran_state()
mmc: core: Update CMD13 busy check for CMD6 commands
mmc: core: Convert to mmc_poll_for_busy() for erase/trim/discard
mmc: core: Drop redundant out-parameter to mmc_send_hpi_cmd()
mmc: core: Convert to mmc_poll_for_busy() for HPI commands
mmc: core: Fixup support for HW busy detection for HPI commands
mmc: core: Re-work the error path for the eMMC sanitize command
drivers/mmc/core/block.c | 55 +++++--------
drivers/mmc/core/core.c | 53 +------------
drivers/mmc/core/mmc.c | 38 ++++-----
drivers/mmc/core/mmc_ops.c | 159 ++++++++++++++++++++++---------------
drivers/mmc/core/mmc_ops.h | 13 ++-
include/linux/mmc/core.h | 3 -
include/linux/mmc/mmc.h | 10 +++
7 files changed, 157 insertions(+), 174 deletions(-)
--
2.17.1
next reply other threads:[~2020-02-04 8:55 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-04 8:54 Ulf Hansson [this message]
2020-02-04 8:54 ` [PATCH 01/12] mmc: core: Throttle polling rate for CMD6 Ulf Hansson
2020-02-12 13:51 ` Ludovic BARRE
2020-02-12 14:18 ` Ulf Hansson
2020-02-12 14:24 ` Ludovic BARRE
2020-02-04 8:54 ` [PATCH 02/12] mmc: core: Drop unused define Ulf Hansson
2020-02-04 8:54 ` [PATCH 03/12] mmc: core: Extend mmc_switch_status() to rid of __mmc_switch_status() Ulf Hansson
2020-02-04 8:54 ` [PATCH 04/12] mmc: core: Drop redundant in-parameter to __mmc_switch() Ulf Hansson
2020-02-04 8:54 ` [PATCH 05/12] mmc: core: Split up mmc_poll_for_busy() Ulf Hansson
2020-02-04 8:54 ` [PATCH 06/12] mmc: core: Enable re-use of mmc_blk_in_tran_state() Ulf Hansson
2020-02-04 8:54 ` [PATCH 07/12] mmc: core: Update CMD13 busy check for CMD6 commands Ulf Hansson
2020-02-04 8:54 ` [PATCH 08/12] mmc: core: Convert to mmc_poll_for_busy() for erase/trim/discard Ulf Hansson
2020-02-04 8:54 ` [PATCH 09/12] mmc: core: Drop redundant out-parameter to mmc_send_hpi_cmd() Ulf Hansson
2020-02-04 8:54 ` [PATCH 10/12] mmc: core: Convert to mmc_poll_for_busy() for HPI commands Ulf Hansson
2020-02-04 8:54 ` [PATCH 11/12] mmc: core: Fixup support for HW busy detection " Ulf Hansson
2020-02-04 8:54 ` [PATCH 12/12] mmc: core: Re-work the error path for the eMMC sanitize command Ulf Hansson
2020-02-11 22:29 ` kbuild test robot
2020-02-11 13:17 ` [PATCH 00/12] mmc: core: Improve code for polling and HW busy detect Baolin Wang
2020-02-13 6:23 ` Baolin Wang
2020-02-13 11:08 ` Ulf Hansson
2020-02-13 14:42 ` Ludovic BARRE
2020-02-14 14:21 ` Ulf Hansson
2020-02-18 23:38 ` Ulf Hansson
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:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200204085449.32585-1-ulf.hansson@linaro.org \
--to=ulf.hansson@linaro.org \
--cc=adrian.hunter@intel.com \
--cc=baolin.wang7@gmail.com \
--cc=chaotian.jing@mediatek.com \
--cc=linus.walleij@linaro.org \
--cc=linux-mmc@vger.kernel.org \
--cc=ludovic.barre@st.com \
--cc=mirq-linux@rere.qmqm.pl \
--cc=shawn.lin@rock-chips.com \
--cc=wsa+renesas@sang-engineering.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).