Commit ca55f46e authored by Haavard Skinnemoen's avatar Haavard Skinnemoen

atmel-mci: Don't overwrite error bits when NOTBUSY is set

After a data error, we wait for the NOTBUSY bit to be set so that we can
be sure the data transfer is completely finished. However, when NOTBUSY
is set, the interrupt handler copies the contents of SR into
data_status, overwriting any error bits we may have detected earlier.

To avoid this, initialize data_status to 0 before starting a request, and
don't overwrite it unless it still contains 0.
Signed-off-by: default avatarHaavard Skinnemoen <haavard.skinnemoen@atmel.com>
parent 65e8b083
...@@ -694,6 +694,7 @@ static void atmci_start_request(struct atmel_mci *host, ...@@ -694,6 +694,7 @@ static void atmci_start_request(struct atmel_mci *host,
host->pending_events = 0; host->pending_events = 0;
host->completed_events = 0; host->completed_events = 0;
host->data_status = 0;
if (host->need_reset) { if (host->need_reset) {
mci_writel(host, CR, MCI_CR_SWRST); mci_writel(host, CR, MCI_CR_SWRST);
...@@ -1408,7 +1409,8 @@ static irqreturn_t atmci_interrupt(int irq, void *dev_id) ...@@ -1408,7 +1409,8 @@ static irqreturn_t atmci_interrupt(int irq, void *dev_id)
if (pending & MCI_NOTBUSY) { if (pending & MCI_NOTBUSY) {
mci_writel(host, IDR, mci_writel(host, IDR,
ATMCI_DATA_ERROR_FLAGS | MCI_NOTBUSY); ATMCI_DATA_ERROR_FLAGS | MCI_NOTBUSY);
host->data_status = status; if (!host->data_status)
host->data_status = status;
smp_wmb(); smp_wmb();
atmci_set_pending(host, EVENT_DATA_COMPLETE); atmci_set_pending(host, EVENT_DATA_COMPLETE);
tasklet_schedule(&host->tasklet); tasklet_schedule(&host->tasklet);
......
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