meson_saradc fixes and minor improvements
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest0 of 5 patches 0 repliesNew
Patches in v1 No applies recorded yet
None of the 5 patches the cover letter announces are in this archive.
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [PATCH 0/5] meson_saradc fixes and minor improvements
Hi Jonathan,
Quoted 41 lines · Jonathan Cameron
Quoted 38 lines · On Tue, 31 Oct 2017 21:01:42 +0100 Martin Blumenstingl…
Meson8 and Meson8b require that the driver initializes the registers correctly, while on GXBB and newer this is done by the firmware. Thus the changes from this series are only relevant for Meson8 and Meson8b.
The first two patches are bugfixes:
- the first patch fixes an issue where the SAR ADC would not work at all
- the second patch initializes the bandgrap register on Meson8 and Meson8b (which is required for the SAR ADC to operate)The third patch is mainly a cosmetic fix because we don't want to read/write registers that are not documented.
Patch four is purely cosmetic so the mainline driver uses the same settings as Amlogic's vendor kernel driver.
The last patch initializes the channel muxes. There are sane defaults for these in the hardware itself. However, some bootloaders are setting strange values - the result is that reading the ADC gives garbage values.
I did not tag this as "fix" since in my opinion it's a feature for fixing broken bootloaders.Tested on:
- a Meson8m2 board (compatible with Meson8)
- a Meson8b EC-100
- on a Khadas VIM to ensure that the newer SoCs are still workingMartin Blumenstingl (5):
iio: adc: meson-saradc: fix thebit_idxof theadc_enclock
iio: adc: meson-saradc: initialize the bandgap correctly on older
SoCs iio: adc: meson-saradc: Meson8 and Meson8b do not have REG11 and REG13 iio: adc: meson-saradc: fix the clock frequency on Meson8 and Meson8b iio: adc: meson-saradc: program the channel muxes during
initialization
Quoted 6 lines
Quoted 4 lines
drivers/iio/adc/meson_saradc.c | 92+++++++++++++++++++++++++++++++++++------- 1 file changed, 78 insertions(+), 14 deletions(-)
Series seems fine to me. If no one else raises anything I'll pick this up when I'm am back in the UK the week after next.
Just a gentle ping on the status of this series as I'm not seeing it in linux-next.
Thanks,
Kevin