12345678910111213141516171819202122232425 |
- From ffec7861dda17a3da5f6337fbf89114f86437d7e Mon Sep 17 00:00:00 2001
- From: Phil Elwell <[email protected]>
- Date: Mon, 6 Jan 2020 16:04:30 +0000
- Subject: [PATCH] pinctrl: bcm2835: Remove gpiochip on error
- A failure in gpiochip_irqchip_add leads to a leak of a gpiochip. Fix
- the leak with the use of devm_gpiochip_add_data.
- Fixes: 85ae9e512f43 ("pinctrl: bcm2835: switch to GPIOLIB_IRQCHIP")
- Signed-off-by: Phil Elwell <[email protected]>
- ---
- drivers/pinctrl/bcm/pinctrl-bcm2835.c | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
- --- a/drivers/pinctrl/bcm/pinctrl-bcm2835.c
- +++ b/drivers/pinctrl/bcm/pinctrl-bcm2835.c
- @@ -1337,7 +1337,7 @@ static int bcm2835_pinctrl_probe(struct
- girq->default_type = IRQ_TYPE_NONE;
- girq->handler = handle_level_irq;
-
- - err = gpiochip_add_data(&pc->gpio_chip, pc);
- + err = devm_gpiochip_add_data(dev, &pc->gpio_chip, pc);
- if (err) {
- dev_err(dev, "could not add GPIO chip\n");
- goto out_remove;
|