diff options
| author | Barry Song <song.bao.hua@hisilicon.com> | 2020-09-09 06:10:01 +0200 |
|---|---|---|
| committer | Wolfram Sang <wsa@kernel.org> | 2020-09-29 21:40:03 +0200 |
| commit | 06856269d43ab3ce6609c90a6e42bcd9564eaa02 (patch) | |
| tree | aa039b3c043c0ac8a8d5835d7cc878da374ba93e /drivers/i2c/busses/i2c-owl.c | |
| parent | i2c: efm32: Use devm_platform_get_and_ioremap_resource() (diff) | |
| download | linux-06856269d43ab3ce6609c90a6e42bcd9564eaa02.tar.xz linux-06856269d43ab3ce6609c90a6e42bcd9564eaa02.zip | |
i2c: busses: replace spin_lock_irqsave by spin_lock in hard IRQ
The code has been in a irq-disabled context since it is hard IRQ. There
is no necessity to do it again.
Signed-off-by: Barry Song <song.bao.hua@hisilicon.com>
Reviewed-by: Akash Asthana <akashast@codeaurora.org>
Reviewed-by: Mukesh Kumar Savaliya <msavaliy@codeaurora.org>
Signed-off-by: Wolfram Sang <wsa@kernel.org>
Diffstat (limited to 'drivers/i2c/busses/i2c-owl.c')
| -rw-r--r-- | drivers/i2c/busses/i2c-owl.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/i2c/busses/i2c-owl.c b/drivers/i2c/busses/i2c-owl.c index 672f1f239bd6..618d3013d0b6 100644 --- a/drivers/i2c/busses/i2c-owl.c +++ b/drivers/i2c/busses/i2c-owl.c @@ -165,10 +165,9 @@ static irqreturn_t owl_i2c_interrupt(int irq, void *_dev) { struct owl_i2c_dev *i2c_dev = _dev; struct i2c_msg *msg = i2c_dev->msg; - unsigned long flags; unsigned int stat, fifostat; - spin_lock_irqsave(&i2c_dev->lock, flags); + spin_lock(&i2c_dev->lock); i2c_dev->err = 0; @@ -208,7 +207,7 @@ stop: OWL_I2C_STAT_IRQP, true); complete_all(&i2c_dev->msg_complete); - spin_unlock_irqrestore(&i2c_dev->lock, flags); + spin_unlock(&i2c_dev->lock); return IRQ_HANDLED; } |