Commit bb58a47a authored by Linus Walleij's avatar Linus Walleij
Browse files

gpio: max732x: Use irqchip template



This makes the driver use the irqchip template to assign
properties to the gpio_irq_chip instead of using the
explicit calls to gpiochip_irqchip_add_nested() and
gpiochip_set_nested_irqchip(). The irqchip is instead
added while adding the gpiochip.

Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Tested-by: default avatarSam Protsenko <semen.protsenko@linaro.org>
Reviewed-by: default avatarSam Protsenko <semen.protsenko@linaro.org>
Cc: Sam Protsenko <semen.protsenko@linaro.org>
Link: https://lore.kernel.org/r/20200726221259.133536-1-linus.walleij@linaro.org
parent beb3fb41
Loading
Loading
Loading
Loading
+15 −15
Original line number Diff line number Diff line
@@ -503,6 +503,8 @@ static int max732x_irq_setup(struct max732x_chip *chip,

	if (((pdata && pdata->irq_base) || client->irq)
			&& has_irq != INT_NONE) {
		struct gpio_irq_chip *girq;

		if (pdata)
			irq_base = pdata->irq_base;
		chip->irq_features = has_irq;
@@ -517,19 +519,17 @@ static int max732x_irq_setup(struct max732x_chip *chip,
				client->irq);
			return ret;
		}
		ret =  gpiochip_irqchip_add_nested(&chip->gpio_chip,
						   &max732x_irq_chip,
						   irq_base,
						   handle_simple_irq,
						   IRQ_TYPE_NONE);
		if (ret) {
			dev_err(&client->dev,
				"could not connect irqchip to gpiochip\n");
			return ret;
		}
		gpiochip_set_nested_irqchip(&chip->gpio_chip,
					    &max732x_irq_chip,
					    client->irq);

		girq = &chip->gpio_chip.irq;
		girq->chip = &max732x_irq_chip;
		/* This will let us handle the parent IRQ in the driver */
		girq->parent_handler = NULL;
		girq->num_parents = 0;
		girq->parents = NULL;
		girq->default_type = IRQ_TYPE_NONE;
		girq->handler = handle_simple_irq;
		girq->threaded = true;
		girq->first = irq_base; /* FIXME: get rid of this */
	}

	return 0;
@@ -695,11 +695,11 @@ static int max732x_probe(struct i2c_client *client,
			return ret;
	}

	ret = devm_gpiochip_add_data(&client->dev, &chip->gpio_chip, chip);
	ret = max732x_irq_setup(chip, id);
	if (ret)
		return ret;

	ret = max732x_irq_setup(chip, id);
	ret = devm_gpiochip_add_data(&client->dev, &chip->gpio_chip, chip);
	if (ret)
		return ret;