diff options
| author | John Ogness <john.ogness@linutronix.de> | 2011-03-25 09:47:37 +0100 |
|---|---|---|
| committer | Chris Ball <cjb@laptop.org> | 2011-05-25 02:59:17 +0200 |
| commit | 0b38c4ebf037290eef27b31bdc37cacf804c7a48 (patch) | |
| tree | 97a947537da85f161b2862d2d26221402d637194 /drivers | |
| parent | mmc: Ensure linux starts in eMMC user partition (diff) | |
| download | linux-0b38c4ebf037290eef27b31bdc37cacf804c7a48.tar.xz linux-0b38c4ebf037290eef27b31bdc37cacf804c7a48.zip | |
mmc: remove redundant irq disabling
There is no need to disable irq's when using the sg_copy_*_buffer()
functions because those functions do that already. There are also
no races for the mm_queue struct here that would require the irq's
to be disabled before calling sg_copy_*_buffer().
Signed-off-by: John Ogness <john.ogness@linutronix.de>
Signed-off-by: Chris Ball <cjb@laptop.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/mmc/card/queue.c | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/drivers/mmc/card/queue.c b/drivers/mmc/card/queue.c index 2ae727568df9..c07322c2658c 100644 --- a/drivers/mmc/card/queue.c +++ b/drivers/mmc/card/queue.c @@ -343,18 +343,14 @@ unsigned int mmc_queue_map_sg(struct mmc_queue *mq) */ void mmc_queue_bounce_pre(struct mmc_queue *mq) { - unsigned long flags; - if (!mq->bounce_buf) return; if (rq_data_dir(mq->req) != WRITE) return; - local_irq_save(flags); sg_copy_to_buffer(mq->bounce_sg, mq->bounce_sg_len, mq->bounce_buf, mq->sg[0].length); - local_irq_restore(flags); } /* @@ -363,17 +359,13 @@ void mmc_queue_bounce_pre(struct mmc_queue *mq) */ void mmc_queue_bounce_post(struct mmc_queue *mq) { - unsigned long flags; - if (!mq->bounce_buf) return; if (rq_data_dir(mq->req) != READ) return; - local_irq_save(flags); sg_copy_from_buffer(mq->bounce_sg, mq->bounce_sg_len, mq->bounce_buf, mq->sg[0].length); - local_irq_restore(flags); } |