Re: [PATCH 2/5] mfd: tps65217: Specify the IRQ name

From: Lee Jones
Date: Tue Nov 22 2016 - 10:44:01 EST


On Tue, 15 Nov 2016, Milo Kim wrote:

> TPS65217 MFD is an interrupt controller and MFD slave devices like
> tps65217-charger and tps65217-pwrbutton request an interrupt to handle
> each HW event.
>
> Currently, TPS65217 IRQ name is not defined, so the result is as below.
>
> root@arm:~# cat /proc/interrupts
> ...
> 182: 0 INTC 7 Level tps65217-irq
> 183: 0 - 1 Edge tps65217-charger
> 185: 0 - 2 Edge tps65217_pwrbutton
>
> This patch specifies the name of the interrupt controller.
>
> 182: 0 INTC 7 Level tps65217-irq
> 183: 0 tps65217 1 Edge tps65217-charger
> 185: 0 tps65217 2 Edge tps65217_pwrbutton
>
> Signed-off-by: Milo Kim <woogyom.kim@xxxxxxxxx>
> ---
> drivers/mfd/tps65217.c | 1 +
> 1 file changed, 1 insertion(+)

Applied, thanks.

> diff --git a/drivers/mfd/tps65217.c b/drivers/mfd/tps65217.c
> index df2e775..77fb812 100644
> --- a/drivers/mfd/tps65217.c
> +++ b/drivers/mfd/tps65217.c
> @@ -105,6 +105,7 @@ static void tps65217_irq_disable(struct irq_data *data)
> }
>
> static struct irq_chip tps65217_irq_chip = {
> + .name = "tps65217",
> .irq_bus_lock = tps65217_irq_lock,
> .irq_bus_sync_unlock = tps65217_irq_sync_unlock,
> .irq_enable = tps65217_irq_enable,

--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog