spi: spi-tegra20-sflash: remove redundant irqsave and irqrestore in hardIRQ

Running in hardIRQ, disabling IRQ is redundant.

Signed-off-by: Barry Song <song.bao.hua@hisilicon.com>
Link: https://lore.kernel.org/r/20200926001616.21292-1-song.bao.hua@hisilicon.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/drivers/spi/spi-tegra20-sflash.c b/drivers/spi/spi-tegra20-sflash.c
index 02cf5f463ba676a378518646cc8ed64152a3c19c..b59015c7c8a80405f523efb1299b337bb1a3839a 100644 (file)
--- a/drivers/spi/spi-tegra20-sflash.c
+++ b/drivers/spi/spi-tegra20-sflash.c
@@ -359,9 +359,8 @@ exit:
 static irqreturn_t handle_cpu_based_xfer(struct tegra_sflash_data *tsd)
 {
        struct spi_transfer *t = tsd->curr_xfer;
-       unsigned long flags;
 
-       spin_lock_irqsave(&tsd->lock, flags);
+       spin_lock(&tsd->lock);
        if (tsd->tx_status || tsd->rx_status || (tsd->status_reg & SPI_BSY)) {
                dev_err(tsd->dev,
                        "CpuXfer ERROR bit set 0x%x\n", tsd->status_reg);
@@ -391,7 +390,7 @@ static irqreturn_t handle_cpu_based_xfer(struct tegra_sflash_data *tsd)
        tegra_sflash_calculate_curr_xfer_param(tsd->cur_spi, tsd, t);
        tegra_sflash_start_cpu_based_transfer(tsd, t);
 exit:
-       spin_unlock_irqrestore(&tsd->lock, flags);
+       spin_unlock(&tsd->lock);
        return IRQ_HANDLED;
 }