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>
This commit is contained in:
parent
41e2a48935
commit
0b38c4ebf0
|
@ -343,18 +343,14 @@ unsigned int mmc_queue_map_sg(struct mmc_queue *mq)
|
||||||
*/
|
*/
|
||||||
void mmc_queue_bounce_pre(struct mmc_queue *mq)
|
void mmc_queue_bounce_pre(struct mmc_queue *mq)
|
||||||
{
|
{
|
||||||
unsigned long flags;
|
|
||||||
|
|
||||||
if (!mq->bounce_buf)
|
if (!mq->bounce_buf)
|
||||||
return;
|
return;
|
||||||
|
|
||||||
if (rq_data_dir(mq->req) != WRITE)
|
if (rq_data_dir(mq->req) != WRITE)
|
||||||
return;
|
return;
|
||||||
|
|
||||||
local_irq_save(flags);
|
|
||||||
sg_copy_to_buffer(mq->bounce_sg, mq->bounce_sg_len,
|
sg_copy_to_buffer(mq->bounce_sg, mq->bounce_sg_len,
|
||||||
mq->bounce_buf, mq->sg[0].length);
|
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)
|
void mmc_queue_bounce_post(struct mmc_queue *mq)
|
||||||
{
|
{
|
||||||
unsigned long flags;
|
|
||||||
|
|
||||||
if (!mq->bounce_buf)
|
if (!mq->bounce_buf)
|
||||||
return;
|
return;
|
||||||
|
|
||||||
if (rq_data_dir(mq->req) != READ)
|
if (rq_data_dir(mq->req) != READ)
|
||||||
return;
|
return;
|
||||||
|
|
||||||
local_irq_save(flags);
|
|
||||||
sg_copy_from_buffer(mq->bounce_sg, mq->bounce_sg_len,
|
sg_copy_from_buffer(mq->bounce_sg, mq->bounce_sg_len,
|
||||||
mq->bounce_buf, mq->sg[0].length);
|
mq->bounce_buf, mq->sg[0].length);
|
||||||
local_irq_restore(flags);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue