summaryrefslogtreecommitdiffstats
path: root/drivers/mmc
diff options
context:
space:
mode:
authorAdrian Hunter <adrian.hunter@intel.com>2016-06-29 16:24:21 +0300
committerUlf Hansson <ulf.hansson@linaro.org>2016-07-25 10:34:37 +0200
commit64ed8dd46ba69a7f03f9019758ef35604f0cdb04 (patch)
tree04aa7dfb1c18f3aa1493e6f725991f053598b31f /drivers/mmc
parent6bde868181dd98aef14ac3ba02a3216b40466867 (diff)
downloadblackbird-obmc-linux-64ed8dd46ba69a7f03f9019758ef35604f0cdb04.tar.gz
blackbird-obmc-linux-64ed8dd46ba69a7f03f9019758ef35604f0cdb04.zip
mmc: sdhci: Get rid of redundant BUG_ONs
BUG is never the right thing for SDHCI to do. Get rid of BUG_ON in cases it will oops anyway if the pointer is NULL, or if the condition is logically impossible. Signed-off-by: Adrian Hunter <adrian.hunter@intel.com> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
Diffstat (limited to 'drivers/mmc')
-rw-r--r--drivers/mmc/host/sdhci.c9
1 files changed, 0 insertions, 9 deletions
diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c
index 9ca4253b0ab7..f4d6c0460818 100644
--- a/drivers/mmc/host/sdhci.c
+++ b/drivers/mmc/host/sdhci.c
@@ -428,8 +428,6 @@ static void sdhci_transfer_pio(struct sdhci_host *host)
{
u32 mask;
- BUG_ON(!host->data);
-
if (host->blocks == 0)
return;
@@ -929,8 +927,6 @@ static void sdhci_finish_data(struct sdhci_host *host)
{
struct mmc_data *data;
- BUG_ON(!host->data);
-
data = host->data;
host->data = NULL;
@@ -1060,8 +1056,6 @@ static void sdhci_finish_command(struct sdhci_host *host)
{
int i;
- BUG_ON(host->cmd == NULL);
-
if (host->cmd->flags & MMC_RSP_PRESENT) {
if (host->cmd->flags & MMC_RSP_136) {
/* CRC is stripped so we need to do some shifting. */
@@ -2279,8 +2273,6 @@ static void sdhci_timeout_timer(unsigned long data)
static void sdhci_cmd_irq(struct sdhci_host *host, u32 intmask, u32 *mask)
{
- BUG_ON(intmask == 0);
-
if (!host->cmd) {
pr_err("%s: Got command interrupt 0x%08x even though no command operation was in progress.\n",
mmc_hostname(host->mmc), (unsigned)intmask);
@@ -2361,7 +2353,6 @@ static void sdhci_adma_show_error(struct sdhci_host *host) { }
static void sdhci_data_irq(struct sdhci_host *host, u32 intmask)
{
u32 command;
- BUG_ON(intmask == 0);
/* CMD19 generates _only_ Buffer Read Ready interrupt */
if (intmask & SDHCI_INT_DATA_AVAIL) {
OpenPOWER on IntegriCloud