diff options
author | Venkatraman S <svenkatr@ti.com> | 2012-11-19 11:30:00 -0500 |
---|---|---|
committer | Chris Ball <cjb@laptop.org> | 2012-12-06 13:54:58 -0500 |
commit | b1e056ae4bfaa12dda5c24bd9b9cf902c5c62033 (patch) | |
tree | 1b6fc21417c0666a10d83484d253a76fcf819e0a /drivers/mmc/host | |
parent | a48ce884d5819d5df2cf1139ab3c43f8e9e419b3 (diff) |
mmc: omap_hsmmc: Convert critical failure reports to dev_err
Fatal errors for the driver are not reported when just error debug
is enabled. Convert selected dev_dbg to dev_err for accurate error
reporting.
Reported-by: Benoit Cousson <b-cousson@ti.com>
Signed-off-by: Venkatraman S <svenkatr@ti.com>
Signed-off-by: Chris Ball <cjb@laptop.org>
Diffstat (limited to 'drivers/mmc/host')
-rw-r--r-- | drivers/mmc/host/omap_hsmmc.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c index fad9250e4bc..9de0b9eb562 100644 --- a/drivers/mmc/host/omap_hsmmc.c +++ b/drivers/mmc/host/omap_hsmmc.c | |||
@@ -305,7 +305,7 @@ static int omap_hsmmc_reg_get(struct omap_hsmmc_host *host) | |||
305 | 305 | ||
306 | reg = regulator_get(host->dev, "vmmc"); | 306 | reg = regulator_get(host->dev, "vmmc"); |
307 | if (IS_ERR(reg)) { | 307 | if (IS_ERR(reg)) { |
308 | dev_dbg(host->dev, "vmmc regulator missing\n"); | 308 | dev_err(host->dev, "vmmc regulator missing\n"); |
309 | return PTR_ERR(reg); | 309 | return PTR_ERR(reg); |
310 | } else { | 310 | } else { |
311 | mmc_slot(host).set_power = omap_hsmmc_set_power; | 311 | mmc_slot(host).set_power = omap_hsmmc_set_power; |
@@ -1133,7 +1133,7 @@ static int omap_hsmmc_switch_opcond(struct omap_hsmmc_host *host, int vdd) | |||
1133 | 1133 | ||
1134 | return 0; | 1134 | return 0; |
1135 | err: | 1135 | err: |
1136 | dev_dbg(mmc_dev(host->mmc), "Unable to switch operating voltage\n"); | 1136 | dev_err(mmc_dev(host->mmc), "Unable to switch operating voltage\n"); |
1137 | return ret; | 1137 | return ret; |
1138 | } | 1138 | } |
1139 | 1139 | ||
@@ -1392,7 +1392,7 @@ omap_hsmmc_prepare_data(struct omap_hsmmc_host *host, struct mmc_request *req) | |||
1392 | if (host->use_dma) { | 1392 | if (host->use_dma) { |
1393 | ret = omap_hsmmc_start_dma_transfer(host, req); | 1393 | ret = omap_hsmmc_start_dma_transfer(host, req); |
1394 | if (ret != 0) { | 1394 | if (ret != 0) { |
1395 | dev_dbg(mmc_dev(host->mmc), "MMC start dma failure\n"); | 1395 | dev_err(mmc_dev(host->mmc), "MMC start dma failure\n"); |
1396 | return ret; | 1396 | return ret; |
1397 | } | 1397 | } |
1398 | } | 1398 | } |
@@ -1927,13 +1927,13 @@ static int __devinit omap_hsmmc_probe(struct platform_device *pdev) | |||
1927 | ret = request_irq(host->irq, omap_hsmmc_irq, 0, | 1927 | ret = request_irq(host->irq, omap_hsmmc_irq, 0, |
1928 | mmc_hostname(mmc), host); | 1928 | mmc_hostname(mmc), host); |
1929 | if (ret) { | 1929 | if (ret) { |
1930 | dev_dbg(mmc_dev(host->mmc), "Unable to grab HSMMC IRQ\n"); | 1930 | dev_err(mmc_dev(host->mmc), "Unable to grab HSMMC IRQ\n"); |
1931 | goto err_irq; | 1931 | goto err_irq; |
1932 | } | 1932 | } |
1933 | 1933 | ||
1934 | if (pdata->init != NULL) { | 1934 | if (pdata->init != NULL) { |
1935 | if (pdata->init(&pdev->dev) != 0) { | 1935 | if (pdata->init(&pdev->dev) != 0) { |
1936 | dev_dbg(mmc_dev(host->mmc), | 1936 | dev_err(mmc_dev(host->mmc), |
1937 | "Unable to configure MMC IRQs\n"); | 1937 | "Unable to configure MMC IRQs\n"); |
1938 | goto err_irq_cd_init; | 1938 | goto err_irq_cd_init; |
1939 | } | 1939 | } |
@@ -1956,7 +1956,7 @@ static int __devinit omap_hsmmc_probe(struct platform_device *pdev) | |||
1956 | IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING | IRQF_ONESHOT, | 1956 | IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING | IRQF_ONESHOT, |
1957 | mmc_hostname(mmc), host); | 1957 | mmc_hostname(mmc), host); |
1958 | if (ret) { | 1958 | if (ret) { |
1959 | dev_dbg(mmc_dev(host->mmc), | 1959 | dev_err(mmc_dev(host->mmc), |
1960 | "Unable to grab MMC CD IRQ\n"); | 1960 | "Unable to grab MMC CD IRQ\n"); |
1961 | goto err_irq_cd; | 1961 | goto err_irq_cd; |
1962 | } | 1962 | } |