From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matt Fleming Subject: Re: [PATCH 2/2] sdhci: disable MMC_CAP_NEEDS_POLL in nonremovable case Date: Wed, 15 Sep 2010 16:14:21 +0100 Message-ID: <20100915151421.GC21538@console-pimps.org> References: <4d1a88f196fa7e7eaf4c4a4af13d91ec81fb084e.1283003070.git.matt@console-pimps.org> <7e892c443f11d51a30029ddfeef27af09a9a970d.1283003070.git.matt@console-pimps.org> <4C7F65A1.60105@samsung.com> <20100902091424.GA12456@console-pimps.org> <4C86E672.1030807@samsung.com> <20100915151142.GB21538@console-pimps.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from arkanian.console-pimps.org ([212.110.184.194]:54631 "EHLO arkanian.console-pimps.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753925Ab0IOPOW (ORCPT ); Wed, 15 Sep 2010 11:14:22 -0400 Content-Disposition: inline In-Reply-To: <20100915151142.GB21538@console-pimps.org> Sender: linux-mmc-owner@vger.kernel.org List-Id: linux-mmc@vger.kernel.org To: Chris Ball Cc: linux-mmc@vger.kernel.org, Kyungmin Park , Marek Szyprowski , Andrew Morton , Ben Dooks , Ben Hutchings , Yunpeng Gao , Jaehoon Chung On Wed, Sep 15, 2010 at 04:11:42PM +0100, Matt Fleming wrote: > On Wed, Sep 08, 2010 at 10:27:14AM +0900, Jaehoon Chung wrote: > > Matt Fleming wrote: > > > On Thu, Sep 02, 2010 at 05:51:45PM +0900, Jaehoon Chung wrote: > > >> Matt Fleming wrote: > > >>> From: Jaehoon Chung > > >>> > > >>> If controller use SDHCI_QUIRK_BROKEN_CARD_DETECTION quirk, > > >>> controller need always polling detect > > >>> > > >>> In this case, always generated interrupt.Because controller checked card status. > > >>> I think that is not efficiently. > > >>> > > >>> But if card is nonremovable, we need not always polling. > > >>> So i added the check-point which is nonremovable or not > > >>> > > >>> Signed-off-by: Jaehoon Chung > > >>> --- > > >>> drivers/mmc/host/sdhci.c | 5 +++-- > > >>> 1 files changed, 3 insertions(+), 2 deletions(-) > > >>> > > >>> diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c > > >>> index f6be963..3111859 100644 > > >>> --- a/drivers/mmc/host/sdhci.c > > >>> +++ b/drivers/mmc/host/sdhci.c > > >>> @@ -1827,8 +1827,9 @@ int sdhci_add_host(struct sdhci_host *host) > > >>> if (caps & SDHCI_CAN_DO_HISPD) > > >>> mmc->caps |= MMC_CAP_SD_HIGHSPEED | MMC_CAP_MMC_HIGHSPEED; > > >>> > > >>> - if (host->quirks & SDHCI_QUIRK_BROKEN_CARD_DETECTION) > > >>> - mmc->caps |= MMC_CAP_NEEDS_POLL; > > >>> + if ((host->quirks & SDHCI_QUIRK_BROKEN_CARD_DETECTION) && > > >>> + !mmc_card_is_removable(mmc)) > > >>> + mmc->caps |= MMC_CAP_NEEDS_POLL; > > >>> > > >>> mmc->ocr_avail = 0; > > >>> if (caps & SDHCI_CAN_VDD_330) > > >>> > > >> !mmc_card_is_removable(mmc) is right? > > >> i think that is mmc_card_is_removable.. > > >> because when card is removable, host controller needs polling. > > >> > > >> + if ((host->quirks & SDHCI_QUIRK_BROKEN_CARD_DETECTION) && > > >> + *mmc_card_is_removable(mmc))* > > >> + mmc->caps |= MMC_CAP_NEEDS_POLL; > > >> > > > > > > Sorry, I messed up the logic here. You're completely correct. > > > > > > With this change and patch 2/2 does your controller function > > > correctly? > > > > > yes. i changed and applied this patch, then i tested in my controller. > > it's working well. > > Chris, are you OK to pick this up (including Jaehoon's change)? Or > would you prefer me to resubmit? Argh, sorry! I hit send before changing the "To:" field.