Re: [PATCH v4] iio: adc: stm32: add oversampling support

[Jonathan Cameron <jic23@kernel.org>]

On Thu, 10 Apr 2025 15:58:35 +0200
Olivier Moysan <olivier.moysan@foss.st.com> wrote:

> Add oversampling support for STM32H7, STM32MP15 & STM32MP13.
> STM32F4 ADC has no oversampling feature.
> 
> The current support of the oversampling feature aims at increasing
> the data SNR, without changing the data resolution.
> As the oversampling by itself increases data resolution,
> a right shift is applied to keep initial resolution.
> Only the oversampling ratio corresponding to a power of two are
> supported here, to get a direct link between right shift and
> oversampling ratio. (2exp(n) ratio <=> n right shift)
I rewrapped this description to 75 chars  (which is roughly the maximum
we should see in a commit message).

In general I'd prefer to keep comments in code to 80 char, but in this
particular case I left the formatting you have alone as it was really fiddly
to get it looking nice at a shorter length.

I had applied this but then noticed....

> 
> The oversampling ratio is shared by all channels, whatever channel type.
> (e.g. single ended or differential).
> 
> Oversampling can be configured using IIO ABI:
> - oversampling_ratio_available
> - oversampling_ratio
> 
> Signed-off-by: Fabrice Gasnier <fabrice.gasnier@foss.st.com>
> Signed-off-by: Olivier Moysan <olivier.moysan@foss.st.com>
This SoB sequence isn't right.  What was Fabrice's role
in this patch?   Either I'd expect the from to be set to Fabrice
(on basis Olivier was posting some work of Fabrice) or
a Co-developed-by for Fabrice to indicate it was joint work
and Olivier was the one posting it.