[lkml]   [2009]   [Feb]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    Subject[PATCH RFT] MMC: core/core.c: mmc_rescan detects card change in one run
    With this patch, mmc_rescan can detect the removal of an mmc card and
    the insertion of (possibly another) card in the same run. This means
    that a card change can be detected without having to call
    mmc_detect_change multiple times.

    This change should generalise the core such that it can be easily used
    by hosts which provide a mechanism to detect only the presence of a card
    reader cover, which has to be taken off in order to insert a card. Other
    hosts ("card detect" or "MMC_CAP_NEEDS_POLL") each receive an event when
    a card is removed and when a card is inserted, so it is sufficient for
    them if mmc_rescan handles only one event at a time. "Cover detect"
    hosts, however, only receive events about the cover status. This means
    that between 2 subsequent events, both a card removal and a card
    insertion can occur. In this case, the current version of mmc_rescan
    would only detect the removal of the previous card but not the insertion
    of the new card.

    This patch has been tested with the "cover detect" mechanism. If you
    can, please test it with "card detect" and/or "polling" card readers.

    Signed-off-by: Jorg Schummer <>
    drivers/mmc/core/core.c | 22 +++++++++++++++++-----
    1 files changed, 17 insertions(+), 5 deletions(-)

    diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c
    index df6ce4a..cd2e29f 100644
    --- a/drivers/mmc/core/core.c
    +++ b/drivers/mmc/core/core.c
    @@ -740,6 +740,22 @@ void mmc_rescan(struct work_struct *work)


    + /* if there is a card registered */
    + if (host->bus_ops != NULL) {
    + if (host->bus_ops->detect && !host->bus_dead) {
    + /* check whether the card is still present */
    + host->bus_ops->detect(host);
    + /* release the bus and update bus status in case
    + the card was removed */
    + mmc_bus_put(host);
    + mmc_bus_get(host);
    + }
    + }
    + /* if there is no card registered */
    if (host->bus_ops == NULL) {
    * Only we can add a new handler, so it's safe to
    @@ -789,12 +805,8 @@ void mmc_rescan(struct work_struct *work)

    - } else {
    - if (host->bus_ops->detect && !host->bus_dead)
    - host->bus_ops->detect(host);
    + } else
    - }
    if (host->caps & MMC_CAP_NEEDS_POLL)
    mmc_schedule_delayed_work(&host->detect, HZ);

     \ /
      Last update: 2009-02-19 16:29    [W:0.023 / U:0.672 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site