Re: [PATCH 1/2] iio: adc: qcom-spmi-adc5-gen3: Share SDAM0 IRQ with ADC_TM auxiliary driver

[Daniel Lezcano]

Hi Andy,

On 5/17/26 09:08, Andy Shevchenko wrote:
> On Fri, May 15, 2026 at 02:23:44PM +0530, Jishnu Prakash wrote:
> > The SDAM0 IRQ can be triggered for both EOC (end of conversion) events for
> > immediate ADC reads done in this driver and for threshold violation events,
> > based on ADC_TM thresholds configured from the auxiliary ADC_TM driver on
> > TM channels on the first SDAM.
> >
> > At present, this interrupt is handled only in the ISR in the main ADC driver.
> > When the ISR is triggered for an ADC_TM event, this driver notifies the ADC_TM
> > driver by calling a notifier callback exposed from it for this purpose.
> >
> > To simplify the interrupt handling in both drivers, share the interrupt between
> > the drivers. With this, ADC_TM interrupts on SDAM0 will be handled directly in
>
> "will"?! Do you mean this patch deliberately breaks the working setups?

No, the interrupt is definitively shared between the iio and the thermal 
sensor. So when submitting the previous series, all IIO patches were 
picked up but I asked Jishnu to revisit the interrupt handling in order 
to make it shared. That is what he did here by reworking the ISR in the 
IIO and resending the thermal sensor I did not pick for this reason before.

This patch does not break working setups.