Commit f9261eb7 authored by Xiaofei Tan's avatar Xiaofei Tan Committed by Ulf Hansson
Browse files

mmc: cavium: Replace spin_lock_irqsave with spin_lock in hard IRQ 



It is redundant to do irqsave and irqrestore in hardIRQ context, where
it has been in a irq-disabled context.

Signed-off-by: default avatarXiaofei Tan <tanxiaofei@huawei.com>
Link: https://lore.kernel.org/r/1612422438-32525-1-git-send-email-tanxiaofei@huawei.com


Signed-off-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
parent 1791557f

drivers/mmc/host/cavium.c

+2 −3
Original line number Diff line number Diff line
@@ -436,12 +436,11 @@ irqreturn_t cvm_mmc_interrupt(int irq, void *dev_id) 
{

	struct cvm_mmc_host *host = dev_id;

	struct mmc_request *req;

	unsigned long flags = 0;

	u64 emm_int, rsp_sts;

	bool host_done;



	if (host->need_irq_handler_lock)

		spin_lock_irqsave(&host->irq_handler_lock, flags);

		spin_lock(&host->irq_handler_lock);

	else

		__acquire(&host->irq_handler_lock);
@@ -504,7 +503,7 @@ irqreturn_t cvm_mmc_interrupt(int irq, void *dev_id) 
		host->release_bus(host);

out:

	if (host->need_irq_handler_lock)

		spin_unlock_irqrestore(&host->irq_handler_lock, flags);

		spin_unlock(&host->irq_handler_lock);

	else

		__release(&host->irq_handler_lock);

	return IRQ_RETVAL(emm_int != 0);