diff options
author | Roman Moracik <roman.moravcik@gmail.com> | 2008-06-30 17:40:28 -0400 |
---|---|---|
committer | Pierre Ossman <drzeus@drzeus.cx> | 2008-07-15 08:14:47 -0400 |
commit | d643b5f7e0793ef7828a35a5ea049d675ad2ad8c (patch) | |
tree | 7614a54010082800e97807834ef035f748a6f5f2 /drivers/mmc/host | |
parent | bdbc9c3a8f7a7956611c970e262693faa95081a5 (diff) |
MMC: Fix S3C24XX IRQ enable during PIO transfers
Fix Bug #677 - I/O errors on heavy microSD writes for 2.6.22.x.
Signed-off-by: Ben Dooks <ben-linux@fluff.org>
Signed-off-by: Pierre Ossman <drzeus@drzeus.cx>
Diffstat (limited to 'drivers/mmc/host')
-rw-r--r-- | drivers/mmc/host/s3cmci.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/drivers/mmc/host/s3cmci.c b/drivers/mmc/host/s3cmci.c index 8c68b2e0dd6e..774af3d7218f 100644 --- a/drivers/mmc/host/s3cmci.c +++ b/drivers/mmc/host/s3cmci.c | |||
@@ -335,6 +335,8 @@ static void pio_tasklet(unsigned long data) | |||
335 | struct s3cmci_host *host = (struct s3cmci_host *) data; | 335 | struct s3cmci_host *host = (struct s3cmci_host *) data; |
336 | 336 | ||
337 | 337 | ||
338 | disable_irq(host->irq); | ||
339 | |||
338 | if (host->pio_active == XFER_WRITE) | 340 | if (host->pio_active == XFER_WRITE) |
339 | do_pio_write(host); | 341 | do_pio_write(host); |
340 | 342 | ||
@@ -352,9 +354,9 @@ static void pio_tasklet(unsigned long data) | |||
352 | host->mrq->data->error = -EINVAL; | 354 | host->mrq->data->error = -EINVAL; |
353 | } | 355 | } |
354 | 356 | ||
355 | disable_irq(host->irq); | ||
356 | finalize_request(host); | 357 | finalize_request(host); |
357 | } | 358 | } else |
359 | enable_irq(host->irq); | ||
358 | } | 360 | } |
359 | 361 | ||
360 | /* | 362 | /* |
@@ -630,7 +632,6 @@ out: | |||
630 | spin_unlock_irqrestore(&host->complete_lock, iflags); | 632 | spin_unlock_irqrestore(&host->complete_lock, iflags); |
631 | return; | 633 | return; |
632 | 634 | ||
633 | |||
634 | fail_request: | 635 | fail_request: |
635 | host->mrq->data->error = -EINVAL; | 636 | host->mrq->data->error = -EINVAL; |
636 | host->complete_what = COMPLETION_FINALIZE; | 637 | host->complete_what = COMPLETION_FINALIZE; |