[PATCH] ARM: mmc: fix NONREMOVABLE test in sdhci-bcm-kona

From
Christian Daudt <hidden>
Date
2013-08-19 23:30:10
Also in
linux-mmc, lkml
Reply to
Christian Daudt, 2013-08-07
On 13-08-07 11:59 AM, Christian Daudt wrote:
quoted hunk ↗ jump to hunk
sdhci-bcm-kona driver is incorrectly doing "|" to bit-test
NONREMOVABLE. Switch to "&"

Reviewed-by: Markus Mayer <redacted>
Reviewed-by: Matt Porter <redacted>
Signed-off-by: Christian Daudt <redacted>
diff --git a/drivers/mmc/host/sdhci-bcm-kona.c b/drivers/mmc/host/sdhci-bcm-kona.c
index 87175f9..9ffac0b 100644
--- a/drivers/mmc/host/sdhci-bcm-kona.c
+++ b/drivers/mmc/host/sdhci-bcm-kona.c
@@ -263,7 +263,7 @@ static int __init sdhci_bcm_kona_probe(struct platform_device *pdev)
  		(mmc_gpio_get_cd(host->mmc) != -ENOSYS) ? 'Y' : 'N',
  		(mmc_gpio_get_ro(host->mmc) != -ENOSYS) ? 'Y' : 'N');
  
-	if (host->mmc->caps | MMC_CAP_NONREMOVABLE)
+	if (host->mmc->caps & MMC_CAP_NONREMOVABLE)
  		host->quirks |= SDHCI_QUIRK_BROKEN_CARD_DETECTION;
  
  	dev_dbg(dev, "is_8bit=%c\n",
@@ -282,7 +282,7 @@ static int __init sdhci_bcm_kona_probe(struct platform_device *pdev)
  	}
  
  	/* if device is eMMC, emulate card insert right here */
-	if (host->mmc->caps | MMC_CAP_NONREMOVABLE) {
+	if (host->mmc->caps & MMC_CAP_NONREMOVABLE) {
  		ret = sdhci_bcm_kona_sd_card_emulate(host, 1);
  		if (ret) {
  			dev_err(dev,
applied to bcm/for-3.12/soc

  thanks,
    csd
Thread (2 messages) 2 messages, 1 author, last reply 2013-08-19
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help