Re: [PATCH 0/3] iio: adc: Fix flags in sigma-delta drivers

From: Jonathan Cameron
Date: Tue Aug 03 2021 - 08:48:08 EST


On Tue, 3 Aug 2021 12:17:41 +0300
<alexandru.tachici@xxxxxxxxxx> wrote:

> In Sigma-Delta devices the SDO line is also used as an interrupt.
> Leaving IRQ on level instead of falling might trigger a sample read
> when the IRQ is enabled, as the SDO line is already low. Not sure
> if SDO line will always imediately go high in ad_sd_buffer_postenable
> before the IRQ is enabled.

ok. That last bit sounds like a good justification. It might do the
wrong thing on initial startup. Please resend with that in the patch
descriptions for those you are modifying.

For those where it is being introduced, we probably want to leave the
interrupt type to firmware (i.e. patch 1)

>
> Also the datasheets seem to explicitly say the falling edge of the SDO
> should be used as an interrupt.

Reference?

>
> -Alexandru