Commit ff8c804f authored by Ben Dooks's avatar Ben Dooks Committed by Pierre Ossman

MMC: S3C24XX: Update error debugging.

Add better debugging to show where errors are being
generated, as some error codes can come from several
different code paths.

Also fix the error return path from s3cmci_setup_data()
to return the error it returned to the request.
Signed-off-by: default avatarBen Dooks <ben-linux@fluff.org>
Signed-off-by: default avatarPierre Ossman <drzeus@drzeus.cx>
parent 50a84570
...@@ -450,6 +450,7 @@ static irqreturn_t s3cmci_irq(int irq, void *dev_id) ...@@ -450,6 +450,7 @@ static irqreturn_t s3cmci_irq(int irq, void *dev_id)
} }
if (mci_csta & S3C2410_SDICMDSTAT_CMDTIMEOUT) { if (mci_csta & S3C2410_SDICMDSTAT_CMDTIMEOUT) {
dbg(host, dbg_err, "CMDSTAT: error CMDTIMEOUT\n");
cmd->error = -ETIMEDOUT; cmd->error = -ETIMEDOUT;
host->status = "error: command timeout"; host->status = "error: command timeout";
goto fail_transfer; goto fail_transfer;
...@@ -505,12 +506,14 @@ static irqreturn_t s3cmci_irq(int irq, void *dev_id) ...@@ -505,12 +506,14 @@ static irqreturn_t s3cmci_irq(int irq, void *dev_id)
/* Check for FIFO failure */ /* Check for FIFO failure */
if (host->is2440) { if (host->is2440) {
if (mci_fsta & S3C2440_SDIFSTA_FIFOFAIL) { if (mci_fsta & S3C2440_SDIFSTA_FIFOFAIL) {
dbg(host, dbg_err, "FIFO failure\n");
host->mrq->data->error = -EILSEQ; host->mrq->data->error = -EILSEQ;
host->status = "error: 2440 fifo failure"; host->status = "error: 2440 fifo failure";
goto fail_transfer; goto fail_transfer;
} }
} else { } else {
if (mci_dsta & S3C2410_SDIDSTA_FIFOFAIL) { if (mci_dsta & S3C2410_SDIDSTA_FIFOFAIL) {
dbg(host, dbg_err, "FIFO failure\n");
cmd->data->error = -EILSEQ; cmd->data->error = -EILSEQ;
host->status = "error: fifo failure"; host->status = "error: fifo failure";
goto fail_transfer; goto fail_transfer;
...@@ -518,18 +521,21 @@ static irqreturn_t s3cmci_irq(int irq, void *dev_id) ...@@ -518,18 +521,21 @@ static irqreturn_t s3cmci_irq(int irq, void *dev_id)
} }
if (mci_dsta & S3C2410_SDIDSTA_RXCRCFAIL) { if (mci_dsta & S3C2410_SDIDSTA_RXCRCFAIL) {
dbg(host, dbg_err, "bad data crc (outgoing)\n");
cmd->data->error = -EILSEQ; cmd->data->error = -EILSEQ;
host->status = "error: bad data crc (outgoing)"; host->status = "error: bad data crc (outgoing)";
goto fail_transfer; goto fail_transfer;
} }
if (mci_dsta & S3C2410_SDIDSTA_CRCFAIL) { if (mci_dsta & S3C2410_SDIDSTA_CRCFAIL) {
dbg(host, dbg_err, "bad data crc (incoming)\n");
cmd->data->error = -EILSEQ; cmd->data->error = -EILSEQ;
host->status = "error: bad data crc (incoming)"; host->status = "error: bad data crc (incoming)";
goto fail_transfer; goto fail_transfer;
} }
if (mci_dsta & S3C2410_SDIDSTA_DATATIMEOUT) { if (mci_dsta & S3C2410_SDIDSTA_DATATIMEOUT) {
dbg(host, dbg_err, "data timeout\n");
cmd->data->error = -ETIMEDOUT; cmd->data->error = -ETIMEDOUT;
host->status = "error: data timeout"; host->status = "error: data timeout";
goto fail_transfer; goto fail_transfer;
...@@ -956,8 +962,9 @@ static void s3cmci_send_request(struct mmc_host *mmc) ...@@ -956,8 +962,9 @@ static void s3cmci_send_request(struct mmc_host *mmc)
host->dcnt++; host->dcnt++;
if (res) { if (res) {
cmd->error = -EINVAL; dbg(host, dbg_err, "setup data error %d\n", res);
cmd->data->error = -EINVAL; cmd->error = res;
cmd->data->error = res;
mmc_request_done(mmc, mrq); mmc_request_done(mmc, mrq);
return; return;
...@@ -969,6 +976,7 @@ static void s3cmci_send_request(struct mmc_host *mmc) ...@@ -969,6 +976,7 @@ static void s3cmci_send_request(struct mmc_host *mmc)
res = s3cmci_prepare_pio(host, cmd->data); res = s3cmci_prepare_pio(host, cmd->data);
if (res) { if (res) {
dbg(host, dbg_err, "data prepare error %d\n", res);
cmd->error = res; cmd->error = res;
cmd->data->error = res; cmd->data->error = res;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment