Commit ff8dcfeb authored by Sebastian Andrzej Siewior's avatar Sebastian Andrzej Siewior Committed by Thomas Gleixner
Browse files

staging: greybus: gpio: Use generic_handle_irq_safe(). 



Instead of manually disabling interrupts before invoking use
generic_handle_irq_safe() which can be invoked with enabled and disabled
interrupts.

Signed-off-by: default avatarSebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
Acked-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Acked-by: default avatarJohan Hovold <johan@kernel.org>
Cc: Alex Elder <elder@kernel.org>
Cc: Rui Miguel Silva <rmfrfs@gmail.com>
Link: https://lore.kernel.org/r/20220211181500.1856198-8-bigeasy@linutronix.de
parent bfe6b967

drivers/staging/greybus/gpio.c

+1 −4
Original line number Diff line number Diff line
@@ -391,10 +391,7 @@ static int gb_gpio_request_handler(struct gb_operation *op) 
		return -EINVAL;

	}



	local_irq_disable();

	ret = generic_handle_irq(irq);

	local_irq_enable();



	ret = generic_handle_irq_safe(irq);

	if (ret)

		dev_err(dev, "failed to invoke irq handler\n");