Re: [PATCH v3 3/7] iio: adc: ad7625: fix sampling frequency handling
From: Nicolae-Daniel Deaconescu
Date: Mon Nov 09 2026 - 09:14:55 EST
Hi Jonathan,
On Sat, 7 Nov 2026 01:50:08 +0000, Jonathan Cameron wrote:
>
> > Fix sampling frequency handling issues:
> > - Fix issue where writing 0 to sampling_frequency caused a division by
> > zero. Return -EINVAL.
> > - Fix issue where a waveform rounded up by the PWM went unreported.
> > pwm_round_waveform_might_sleep() returns 1 in that case and
> > ad7625_set_sampling_freq() passed it on, so the sampling_frequency write
> > returned 1 and the new waveforms were dropped. Return -EINVAL.
> > - Fix issue where probe() continued after failing to set the initial
> > sampling frequency. The dev_err_probe() result was not returned, leaving
> > the PWMs unconfigured. Return the error.
> > Fixes: b7ffd0fa65e9 ("iio: adc: ad7625: add driver")
> > Signed-off-by: Nicolae-Daniel Deaconescu <Nicolae-daniel.Deaconescu@xxxxxxxxxx>
>
> These are all clearly bad. Are they affecting known boards
> out there? I'm wondering whether to try and take this ahead
> of the rest of the series. For now I'm assuming it can wait
No reports from boards. I found them while working on this driver, so
it can wait for the series.
Thanks,
Daniel