Re: [RFC v1 02/25] avr32, irq: Use irq_desc_get_xxx() to avoid redundant lookup of irq_desc

Hans-Christian Egtvedt
2015-05-20T10:16:23Z

Around Wed 20 May 2015 17:59:50 +0800 or thereabout, Jiang Liu wrote:
> Use irq_desc_get_xxx() to avoid redundant lookup of irq_desc while we
> already have a pointer to corresponding irq_desc.
>
> Signed-off-by: Jiang Liu <jiang.liu@linux.intel.com>

Acked-by: Hans-Christian Egtvedt <egtvedt@samfundet.no>

> ---
> arch/avr32/mach-at32ap/pio.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/avr32/mach-at32ap/pio.c b/arch/avr32/mach-at32ap/pio.c
> index 903c7d81d0d5..2583e6cc44e9 100644
> --- a/arch/avr32/mach-at32ap/pio.c
> +++ b/arch/avr32/mach-at32ap/pio.c
> @@ -286,7 +286,7 @@ static void gpio_irq_handler(unsigned irq, struct irq_desc *desc)
> struct pio_device *pio = irq_desc_get_chip_data(desc);
> unsigned gpio_irq;
>
> - gpio_irq = (unsigned) irq_get_handler_data(irq);
> + gpio_irq = (unsigned) irq_desc_get_handler_data(desc);
> for (;;) {
> u32 isr;
>
--
mvh
Hans-Christian Egtvedt