Re: [PATCH] Fix IRQ issue by setting IRQ_DISABLE_UNLAZY flag

From: Masahiro Honda
Date: Mon Mar 27 2023 - 05:04:21 EST


Hi, Nuno

> On Mon, Mar 6, 2023 at 10:30 PM Nuno Sá <noname.nuno@xxxxxxxxx> wrote:
> > Another thing that came to my mind is if the data is totally
> > garbage/wrong or is it just some outstanding sample?

I'm sorry for not answering your question. The data is the same as
the previous conversion even if the input voltage is changed. At this
time, a logic analyzer shows that the read operation is performed
immediately after the conversion is performed. The read operation
returns the previous conversion result because it is performed before
the completion of the conversion.

> > Some research on this also seems to point that we should (need?) call
> > irq_clear_status_flags(irq, IRQ_DISABLE_UNLAZY) when freeing the IRQ.

I have understood that I need to call irq_clear_status_flags. However,
I cannot find a code to free the IRQ in ad_sigma_delta.c and other
Sigma-Delta ADC driver source files. So, I would like to implement
only irq_set_status_flags.