From: Jeremie Samuel <jeremie.samuel.ext@parrot.com>
To: Chris Ball <cjb@laptop.org>
Cc: linux-mmc@vger.kernel.org,
"Grégor Boirie" <gregor.boirie@parrot.com>,
"Matthieu Castet" <matthieu.castet@parrot.com>,
"Jeremie Samuel" <jeremie.samuel.ext@parrot.com>
Subject: [PATCH v2 0/8] sdhci: Move real work out of an atomic context
Date: Mon, 21 Oct 2013 17:25:49 +0200 [thread overview]
Message-ID: <1382369157-16129-1-git-send-email-jeremie.samuel.ext@parrot.com> (raw)
Hi all,
Currently the sdhci driver does everything in the atomic context.
And what is worse, PIO transfers are made from the IRQ handler.
Some patches were already submitted to solve this issue. But there were
rejected because they involved new issues.
This set of patches is an evolution of an old patch from Anton Vorontsov.
I tried to fix all the problems involved by the patches. I tested it for
several time now with SD cards and SDIO.
So, this patch set reworks sdhci code to avoid atomic context,
almost completely.
Thanks,
Jeremie Samuel
Jeremie Samuel (8):
sdhci: Turn timeout timer into delayed work
sdhci: Turn tuning timeout timer into delayed work
sdhci: Use work structs instead of tasklets
sdhci: Use threaded IRQ handler
sdhci: Delay led blinking
sdhci: Turn host->lock into a mutex
sdhci: Get rid of mdelay()s where it is safe and makes sense
sdhci: Use jiffies instead of a timeout counter
drivers/mmc/host/sdhci-bcm-kona.c | 2 +-
drivers/mmc/host/sdhci-dove.c | 6 +-
drivers/mmc/host/sdhci-esdhc-imx.c | 4 +-
drivers/mmc/host/sdhci-pxav3.c | 10 +-
drivers/mmc/host/sdhci-s3c.c | 7 +-
drivers/mmc/host/sdhci-sirf.c | 4 +-
drivers/mmc/host/sdhci-spear.c | 2 +-
drivers/mmc/host/sdhci.c | 329 ++++++++++++++++++------------------
include/linux/mmc/sdhci.h | 13 +-
9 files changed, 185 insertions(+), 192 deletions(-)
--
1.7.10.4
next reply other threads:[~2013-10-21 15:27 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-21 15:25 Jeremie Samuel [this message]
2013-10-21 15:25 ` [PATCH v2 1/8] sdhci: Turn timeout timer into delayed work Jeremie Samuel
2013-10-21 15:25 ` [PATCH v2 2/8] sdhci: Turn tuning " Jeremie Samuel
2013-10-21 15:25 ` [PATCH v2 3/8] sdhci: Use work structs instead of tasklets Jeremie Samuel
2013-10-21 15:25 ` [PATCH v2 4/8] sdhci: Use threaded IRQ handler Jeremie Samuel
2013-10-30 19:30 ` Olof Johansson
2013-10-21 15:25 ` [PATCH v2 5/8] sdhci: Delay led blinking Jeremie Samuel
2013-10-21 15:25 ` [PATCH v2 6/8] sdhci: Turn host->lock into a mutex Jeremie Samuel
2013-10-21 15:25 ` [PATCH v2 7/8] sdhci: Get rid of mdelay()s where it is safe and makes sense Jeremie Samuel
2013-10-21 15:25 ` [PATCH v2 8/8] sdhci: Use jiffies instead of a timeout counter Jeremie Samuel
2013-10-21 15:31 ` [PATCH v2 0/8] sdhci: Move real work out of an atomic context Chris Ball
2013-10-21 15:52 ` Jeremie Samuel
2013-10-21 20:02 ` Chris Ball
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=1382369157-16129-1-git-send-email-jeremie.samuel.ext@parrot.com \
--to=jeremie.samuel.ext@parrot.com \
--cc=cjb@laptop.org \
--cc=gregor.boirie@parrot.com \
--cc=linux-mmc@vger.kernel.org \
--cc=matthieu.castet@parrot.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).