Re: [PATCH] pinctrl: mcp23s08: Get rid of spurious level interrupts

[Linus Walleij <linus.walleij@xxxxxxxxxx>]

On Fri, Feb 7, 2025 at 9:36 PM Dmitry Mastykin <mastichi@xxxxxxxxx> wrote:

> I made more tests and think that this patch shouldn't be applied.
> It removes duplicated interrupts, but sometimes they increase the performance:
> a new interrupt may come during handling a spurious one, and spurious one will
> become valid (it's kind of a polling). I see the number of my touchscreen
> interrupts reduced from 200 to 130 per second with this patch. It may be a bigger
> problem for users, than duplicated interrupts. Sorry.

Don't be sorry about that, we code and learn by our mistakes.

So is this patch causing any regression for users? Like touch
events being slow to react? Also the expander could be used
for other things than touchscreens. If it's not causing any regression
for users it seems like a reasonable patch.

Yours,
Linus Walleij