From patchwork Mon Feb 14 05:50:23 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Philip Rakity X-Patchwork-Id: 553881 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter1.kernel.org (8.14.4/8.14.3) with ESMTP id p1E5wDu9010128 for ; Mon, 14 Feb 2011 05:58:13 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751000Ab1BNF6M (ORCPT ); Mon, 14 Feb 2011 00:58:12 -0500 Received: from na3sys009aog109.obsmtp.com ([74.125.149.201]:41963 "EHLO na3sys009aog109.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750764Ab1BNF6M convert rfc822-to-8bit (ORCPT ); Mon, 14 Feb 2011 00:58:12 -0500 Received: from source ([65.219.4.130]) (using TLSv1) by na3sys009aob109.postini.com ([74.125.148.12]) with SMTP ID DSNKTVjEcip9B2hpm5liPhia0+oRfh8CVRHS@postini.com; Sun, 13 Feb 2011 21:58:12 PST Received: from SC-vEXCH3.marvell.com ([10.93.76.133]) by sc-owa02.marvell.com ([10.93.76.22]) with mapi; Sun, 13 Feb 2011 21:50:24 -0800 From: Philip Rakity To: Pierre Tardy CC: "linux-mmc@vger.kernel.org" Date: Sun, 13 Feb 2011 21:50:23 -0800 Subject: Re: [RFC} sdhci.c set_ios -- disable global interrupts and Question on Power Management Thread-Topic: [RFC} sdhci.c set_ios -- disable global interrupts and Question on Power Management Thread-Index: AcvMCxKR/Z7xGU32R9i5R3Htcg3qUQ== Message-ID: References: <7953F5A4-7BFA-4E40-8202-C682C0314F49@marvell.com> <447896B1-8836-4DFE-A00D-D41C395C79FC@marvell.com> In-Reply-To: <447896B1-8836-4DFE-A00D-D41C395C79FC@marvell.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US MIME-Version: 1.0 Sender: linux-mmc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-mmc@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter1.kernel.org [140.211.167.41]); Mon, 14 Feb 2011 05:58:19 +0000 (UTC) diff --git a/drivers/mmc/core/core.h b/drivers/mmc/core/core.h index 20b1c08..146b6f6 100644 --- a/drivers/mmc/core/core.h +++ b/drivers/mmc/core/core.h @@ -11,8 +11,6 @@ #ifndef _MMC_CORE_CORE_H #define _MMC_CORE_CORE_H -#include - #define MMC_CMD_RETRIES 3 struct mmc_bus_ops { @@ -43,16 +41,6 @@ void mmc_set_bus_width_ddr(struct mmc_host *host, unsigned int width, u32 mmc_select_voltage(struct mmc_host *host, u32 ocr); void mmc_set_timing(struct mmc_host *host, unsigned int timing); -static inline void mmc_delay(unsigned int ms) -{ - if (ms < 1000 / HZ) { - cond_resched(); - mdelay(ms); - } else { - msleep(ms); - } -} - void mmc_rescan(struct work_struct *work); void mmc_start_host(struct mmc_host *host); void mmc_stop_host(struct mmc_host *host); diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 655617c..2d7fbd1 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/mmc/host/sdhci.c @@ -1043,7 +1043,7 @@ static void sdhci_set_clock(struct sdhci_host *host, unsigned int clock) return; } timeout--; - mdelay(1); + mmc_delay(1); } clk |= SDHCI_CLOCK_CARD_EN; @@ -1108,7 +1108,7 @@ static void sdhci_set_power(struct sdhci_host *host, unsigned short power) * can apply clock after applying power */ if (host->quirks & SDHCI_QUIRK_DELAY_AFTER_POWER) - mdelay(10); + mmc_delay(10); } /*****************************************************************************\ @@ -1161,22 +1161,21 @@ static void sdhci_request(struct mmc_host *mmc, struct mmc_request *mrq) static void sdhci_set_ios(struct mmc_host *mmc, struct mmc_ios *ios) { struct sdhci_host *host; - unsigned long flags; unsigned int lastclock; + unsigned long flags; u8 ctrl; host = mmc_priv(mmc); - spin_lock_irqsave(&host->lock, flags); - if (host->flags & SDHCI_DEVICE_DEAD) - goto out; + return; /* * get/put runtime_pm usage counter at ios->clock transitions * We need to do it before any other chip access, as sdhci could * be power gated */ + spin_lock_irqsave(&host->lock, flags); lastclock = host->iosclock; host->iosclock = ios->clock; if (lastclock == 0 && ios->clock != 0) { @@ -1189,9 +1188,18 @@ static void sdhci_set_ios(struct mmc_host *mmc, struct mmc_ios *ios) pm_runtime_put_autosuspend(host->mmc->parent); spin_lock_irqsave(&host->lock, flags); } + spin_unlock_irqrestore(&host->lock, flags); + /* no need to configure the rest.. */ if (host->iosclock == 0) - goto out; + return; + + /* disable local interrupts and lock structures so + * that mmc_delay can be scheduled. This cannot happen + * if interrupts are globally disabled + */ + disable_irq(host->irq); + spin_lock(&host->lock); /* * Reset the chip on each power off. @@ -1255,9 +1263,9 @@ static void sdhci_set_ios(struct mmc_host *mmc, struct mmc_ios *ios) if(host->quirks & SDHCI_QUIRK_RESET_CMD_DATA_ON_IOS) sdhci_reset(host, SDHCI_RESET_CMD | SDHCI_RESET_DATA); -out: mmiowb(); - spin_unlock_irqrestore(&host->lock, flags); + spin_unlock(&host->lock); + enable_irq(host->irq); } static int sdhci_get_ro(struct mmc_host *mmc) diff --git a/include/linux/mmc/host.h b/include/linux/mmc/host.h index bcb793e..d9edc06 100644 --- a/include/linux/mmc/host.h +++ b/include/linux/mmc/host.h @@ -15,6 +15,7 @@ #include #include +#include struct mmc_ios { unsigned int clock; /* clock rate */ @@ -326,5 +327,15 @@ static inline int mmc_card_is_powered_resumed(struct mmc_host *host) return host->pm_flags & MMC_PM_KEEP_POWER; } +static inline void mmc_delay(unsigned int ms) +{ + if (ms < 1000 / HZ) { + cond_resched(); + mdelay(ms); + } else { + msleep(ms); + } +} + #endif