Re: [PATCH v6] gpio/mpc8xxx: change irq handler from chained to normal

From: Linus Walleij
Date: Thu Oct 03 2019 - 06:19:58 EST


On Fri, Sep 27, 2019 at 5:26 AM Hui Song <hui.song_1@xxxxxxx> wrote:

> From: Song Hui <hui.song_1@xxxxxxx>
>
> More than one gpio controllers can share one interrupt, change the
> driver to request shared irq.
>
> While this will work, it will mess up userspace accounting of the number
> of interrupts per second in tools such as vmstat. The reason is that
> for every GPIO interrupt, /proc/interrupts records the count against GIC
> interrupt 68 or 69, as well as the GPIO itself. So, for every GPIO
> interrupt, the total number of interrupts that the system has seen
> increments by two
>
> Signed-off-by: Laurentiu Tudor <Laurentiu.Tudor@xxxxxxx>
> Signed-off-by: Alex Marginean <alexandru.marginean@xxxxxxx>
> Signed-off-by: Song Hui <hui.song_1@xxxxxxx>
> ---
> Changes in v6:
> - change request_irq to devm_request_irq and add commit message

There is some build error on the patch, can you look into it?

Yours,
Linus Walleij