diff options
Diffstat (limited to 'drivers/gpio/gpio-tc3589x.c')
| -rw-r--r-- | drivers/gpio/gpio-tc3589x.c | 152 | 
1 files changed, 34 insertions, 118 deletions
diff --git a/drivers/gpio/gpio-tc3589x.c b/drivers/gpio/gpio-tc3589x.c index 4a5de273c23..51f7cbd9ff7 100644 --- a/drivers/gpio/gpio-tc3589x.c +++ b/drivers/gpio/gpio-tc3589x.c @@ -12,8 +12,6 @@  #include <linux/slab.h>  #include <linux/gpio.h>  #include <linux/of.h> -#include <linux/irq.h> -#include <linux/irqdomain.h>  #include <linux/interrupt.h>  #include <linux/mfd/tc3589x.h> @@ -31,10 +29,6 @@ struct tc3589x_gpio {  	struct tc3589x *tc3589x;  	struct device *dev;  	struct mutex irq_lock; -	struct irq_domain *domain; - -	int irq_base; -  	/* Caches of interrupt control registers for bus_lock */  	u8 regs[CACHE_NR_REGS][CACHE_NR_BANKS];  	u8 oldregs[CACHE_NR_REGS][CACHE_NR_BANKS]; @@ -95,30 +89,6 @@ static int tc3589x_gpio_direction_input(struct gpio_chip *chip,  	return tc3589x_set_bits(tc3589x, reg, 1 << pos, 0);  } -/** - * tc3589x_gpio_irq_get_virq(): Map an interrupt on a chip to a virtual IRQ - * - * @tc3589x_gpio: tc3589x_gpio_irq controller to operate on. - * @irq: index of the interrupt requested in the chip IRQs - * - * Useful for drivers to request their own IRQs. - */ -static int tc3589x_gpio_irq_get_virq(struct tc3589x_gpio *tc3589x_gpio, -				     int irq) -{ -	if (!tc3589x_gpio) -		return -EINVAL; - -	return irq_create_mapping(tc3589x_gpio->domain, irq); -} - -static int tc3589x_gpio_to_irq(struct gpio_chip *chip, unsigned offset) -{ -	struct tc3589x_gpio *tc3589x_gpio = to_tc3589x_gpio(chip); - -	return tc3589x_gpio_irq_get_virq(tc3589x_gpio, offset); -} -  static struct gpio_chip template_chip = {  	.label			= "tc3589x",  	.owner			= THIS_MODULE, @@ -126,13 +96,13 @@ static struct gpio_chip template_chip = {  	.get			= tc3589x_gpio_get,  	.direction_output	= tc3589x_gpio_direction_output,  	.set			= tc3589x_gpio_set, -	.to_irq			= tc3589x_gpio_to_irq, -	.can_sleep		= 1, +	.can_sleep		= true,  };  static int tc3589x_gpio_irq_set_type(struct irq_data *d, unsigned int type)  { -	struct tc3589x_gpio *tc3589x_gpio = irq_data_get_irq_chip_data(d); +	struct gpio_chip *gc = irq_data_get_irq_chip_data(d); +	struct tc3589x_gpio *tc3589x_gpio = container_of(gc, struct tc3589x_gpio, chip);  	int offset = d->hwirq;  	int regoffset = offset / 8;  	int mask = 1 << (offset % 8); @@ -159,14 +129,16 @@ static int tc3589x_gpio_irq_set_type(struct irq_data *d, unsigned int type)  static void tc3589x_gpio_irq_lock(struct irq_data *d)  { -	struct tc3589x_gpio *tc3589x_gpio = irq_data_get_irq_chip_data(d); +	struct gpio_chip *gc = irq_data_get_irq_chip_data(d); +	struct tc3589x_gpio *tc3589x_gpio = container_of(gc, struct tc3589x_gpio, chip);  	mutex_lock(&tc3589x_gpio->irq_lock);  }  static void tc3589x_gpio_irq_sync_unlock(struct irq_data *d)  { -	struct tc3589x_gpio *tc3589x_gpio = irq_data_get_irq_chip_data(d); +	struct gpio_chip *gc = irq_data_get_irq_chip_data(d); +	struct tc3589x_gpio *tc3589x_gpio = container_of(gc, struct tc3589x_gpio, chip);  	struct tc3589x *tc3589x = tc3589x_gpio->tc3589x;  	static const u8 regmap[] = {  		[REG_IBE]	= TC3589x_GPIOIBE0, @@ -194,7 +166,8 @@ static void tc3589x_gpio_irq_sync_unlock(struct irq_data *d)  static void tc3589x_gpio_irq_mask(struct irq_data *d)  { -	struct tc3589x_gpio *tc3589x_gpio = irq_data_get_irq_chip_data(d); +	struct gpio_chip *gc = irq_data_get_irq_chip_data(d); +	struct tc3589x_gpio *tc3589x_gpio = container_of(gc, struct tc3589x_gpio, chip);  	int offset = d->hwirq;  	int regoffset = offset / 8;  	int mask = 1 << (offset % 8); @@ -204,7 +177,8 @@ static void tc3589x_gpio_irq_mask(struct irq_data *d)  static void tc3589x_gpio_irq_unmask(struct irq_data *d)  { -	struct tc3589x_gpio *tc3589x_gpio = irq_data_get_irq_chip_data(d); +	struct gpio_chip *gc = irq_data_get_irq_chip_data(d); +	struct tc3589x_gpio *tc3589x_gpio = container_of(gc, struct tc3589x_gpio, chip);  	int offset = d->hwirq;  	int regoffset = offset / 8;  	int mask = 1 << (offset % 8); @@ -242,9 +216,10 @@ static irqreturn_t tc3589x_gpio_irq(int irq, void *dev)  		while (stat) {  			int bit = __ffs(stat);  			int line = i * 8 + bit; -			int virq = tc3589x_gpio_irq_get_virq(tc3589x_gpio, line); +			int irq = irq_find_mapping(tc3589x_gpio->chip.irqdomain, +						   line); -			handle_nested_irq(virq); +			handle_nested_irq(irq);  			stat &= ~(1 << bit);  		} @@ -254,61 +229,6 @@ static irqreturn_t tc3589x_gpio_irq(int irq, void *dev)  	return IRQ_HANDLED;  } -static int tc3589x_gpio_irq_map(struct irq_domain *d, unsigned int virq, -				irq_hw_number_t hwirq) -{ -	struct tc3589x *tc3589x_gpio = d->host_data; - -	irq_set_chip_data(virq, tc3589x_gpio); -	irq_set_chip_and_handler(virq, &tc3589x_gpio_irq_chip, -				handle_simple_irq); -	irq_set_nested_thread(virq, 1); -#ifdef CONFIG_ARM -	set_irq_flags(virq, IRQF_VALID); -#else -	irq_set_noprobe(virq); -#endif - -	return 0; -} - -static void tc3589x_gpio_irq_unmap(struct irq_domain *d, unsigned int virq) -{ -#ifdef CONFIG_ARM -	set_irq_flags(virq, 0); -#endif -	irq_set_chip_and_handler(virq, NULL, NULL); -	irq_set_chip_data(virq, NULL); -} - -static struct irq_domain_ops tc3589x_irq_ops = { -	.map    = tc3589x_gpio_irq_map, -	.unmap  = tc3589x_gpio_irq_unmap, -	.xlate  = irq_domain_xlate_twocell, -}; - -static int tc3589x_gpio_irq_init(struct tc3589x_gpio *tc3589x_gpio, -				struct device_node *np) -{ -	int base = tc3589x_gpio->irq_base; - -	/* -	 * If this results in a linear domain, irq_create_mapping() will -	 * take care of allocating IRQ descriptors at runtime. When a base -	 * is provided, the IRQ descriptors will be allocated when the -	 * domain is instantiated. -	 */ -	tc3589x_gpio->domain = irq_domain_add_simple(np, -			tc3589x_gpio->chip.ngpio, base, &tc3589x_irq_ops, -			tc3589x_gpio); -	if (!tc3589x_gpio->domain) { -		dev_err(tc3589x_gpio->dev, "Failed to create irqdomain\n"); -		return -ENOSYS; -	} - -	return 0; -} -  static int tc3589x_gpio_probe(struct platform_device *pdev)  {  	struct tc3589x *tc3589x = dev_get_drvdata(pdev->dev.parent); @@ -329,7 +249,8 @@ static int tc3589x_gpio_probe(struct platform_device *pdev)  	if (irq < 0)  		return irq; -	tc3589x_gpio = kzalloc(sizeof(struct tc3589x_gpio), GFP_KERNEL); +	tc3589x_gpio = devm_kzalloc(&pdev->dev, sizeof(struct tc3589x_gpio), +				    GFP_KERNEL);  	if (!tc3589x_gpio)  		return -ENOMEM; @@ -347,30 +268,36 @@ static int tc3589x_gpio_probe(struct platform_device *pdev)  	tc3589x_gpio->chip.of_node = np;  #endif -	tc3589x_gpio->irq_base = tc3589x->irq_base ? -		tc3589x->irq_base + TC3589x_INT_GPIO(0) : 0; -  	/* Bring the GPIO module out of reset */  	ret = tc3589x_set_bits(tc3589x, TC3589x_RSTCTRL,  			       TC3589x_RSTCTRL_GPIRST, 0);  	if (ret < 0) -		goto out_free; - -	ret = tc3589x_gpio_irq_init(tc3589x_gpio, np); -	if (ret) -		goto out_free; +		return ret; -	ret = request_threaded_irq(irq, NULL, tc3589x_gpio_irq, IRQF_ONESHOT, -				   "tc3589x-gpio", tc3589x_gpio); +	ret = devm_request_threaded_irq(&pdev->dev, +					irq, NULL, tc3589x_gpio_irq, +					IRQF_ONESHOT, "tc3589x-gpio", +					tc3589x_gpio);  	if (ret) {  		dev_err(&pdev->dev, "unable to get irq: %d\n", ret); -		goto out_free; +		return ret;  	}  	ret = gpiochip_add(&tc3589x_gpio->chip);  	if (ret) {  		dev_err(&pdev->dev, "unable to add gpiochip: %d\n", ret); -		goto out_freeirq; +		return ret; +	} + +	ret =  gpiochip_irqchip_add(&tc3589x_gpio->chip, +				    &tc3589x_gpio_irq_chip, +				    0, +				    handle_simple_irq, +				    IRQ_TYPE_NONE); +	if (ret) { +		dev_err(&pdev->dev, +			"could not connect irqchip to gpiochip\n"); +		return ret;  	}  	if (pdata && pdata->setup) @@ -379,12 +306,6 @@ static int tc3589x_gpio_probe(struct platform_device *pdev)  	platform_set_drvdata(pdev, tc3589x_gpio);  	return 0; - -out_freeirq: -	free_irq(irq, tc3589x_gpio); -out_free: -	kfree(tc3589x_gpio); -	return ret;  }  static int tc3589x_gpio_remove(struct platform_device *pdev) @@ -392,7 +313,6 @@ static int tc3589x_gpio_remove(struct platform_device *pdev)  	struct tc3589x_gpio *tc3589x_gpio = platform_get_drvdata(pdev);  	struct tc3589x *tc3589x = tc3589x_gpio->tc3589x;  	struct tc3589x_gpio_platform_data *pdata = tc3589x->pdata->gpio; -	int irq = platform_get_irq(pdev, 0);  	int ret;  	if (pdata && pdata->remove) @@ -405,10 +325,6 @@ static int tc3589x_gpio_remove(struct platform_device *pdev)  		return ret;  	} -	free_irq(irq, tc3589x_gpio); - -	kfree(tc3589x_gpio); -  	return 0;  }  | 
