diff options
| author | Sebastian Andrzej Siewior <bigeasy@linutronix.de> | 2019-08-13 14:55:54 +0300 |
|---|---|---|
| committer | Wolfram Sang <wsa@the-dreams.de> | 2019-09-05 00:13:16 +0300 |
| commit | 41d529d6227c443a5827cb8b8f040402dedcf3d2 (patch) | |
| tree | 9ee63f1965e1c9921eb19dc54d93a39144aaccb4 /drivers/i2c/i2c-slave-eeprom.c | |
| parent | 2252c3172cc5ecfab5aef1057f7c57b39e485f21 (diff) | |
| download | linux-41d529d6227c443a5827cb8b8f040402dedcf3d2.tar.xz | |
i2c: exynos5: Remove IRQF_ONESHOT
The drivers sets IRQF_ONESHOT and passes only a primary handler. The IRQ
is masked while the primary is handler is invoked independently of
IRQF_ONESHOT.
With IRQF_ONESHOT the core code will not force-thread the interrupt and
this is probably not intended. I *assume* that the original author copied
the IRQ registration from another driver which passed a primary and
secondary handler and removed the secondary handler but keeping the
ONESHOT flag.
Remove IRQF_ONESHOT.
Reported-by: Benjamin Rouxel <benjamin.rouxel@uva.nl>
Tested-by: Benjamin Rouxel <benjamin.rouxel@uva.nl>
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Tested-by: Krzysztof Kozlowski <krzk@kernel.org>
Acked-by: Krzysztof Kozlowski <krzk@kernel.org>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
Diffstat (limited to 'drivers/i2c/i2c-slave-eeprom.c')
0 files changed, 0 insertions, 0 deletions