Re: [PATCH] lib: update DEBUG_SHIRQ docs to match reality

From: Andy Shevchenko
Date: Mon Jun 22 2020 - 12:02:12 EST


On Mon, Jun 22, 2020 at 6:56 PM Wolfram Sang <wsa@xxxxxxxxxxxxx> wrote:

> > > Maybe your case was like Krzysztof's case where the issue turned out to
> > > be the extra interrupt on deregistering after a deferred probe? He
> > > thought it was the initial interrupt but it wasn't.
> >
> > Commit
> > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/iio/pressure/bmp280-core.c?id=97b31a6f5fb95b1ec6575b78a7240baddba34384
> >
> > The relevant IRQ core code
> > https://elixir.bootlin.com/linux/latest/source/kernel/irq/manage.c#L1774
> >
> > It runs it at deregistering, right.
>
> So, can I read this as an Acked-by?

Yes. It means I agree that text should be fixed. Alas, I'm not native
speaker, so I can't check the text for (stylistic, spelling, etc)
correctness.

--
With Best Regards,
Andy Shevchenko