Skip to content

mxs-lradc: Add support for current sources

Stefan Wahren linux-arm-kernel linux-devicetree linux-iio 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest0 of 3 patches 0 repliesNew

Patches in v1 No applies recorded yet

None of the 3 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.

Read the v1 thread

Cover letter [PATCH 0/3] mxs-lradc: Add support for current sources

Hi Harald,

Am 22.04.2016 um 15:52 schrieb Harald Geyer:

Quoted 10 lines

Patch 1/3 changes the driver and updates the binding documentation (I guess it is still in staging.)

Patches 2/3 and 3/3 add the devicetree nodes to imx23 and imx28 boards. I'd like to get input whether this is actually desired. On boards where these regulators would never be enabled this costs a few extra bytes of RAM for allocation of the device data, because the nodes can't be easily removed in .dts files which are including the .dtsi files. The alternative is to add the new nodes to many .dts files, which would be a lot code duplication.

if i get it right the real intention of this patch series is to make the mxs-lradc provide resistance values instead of voltages. So how about dropping the whole regulator stuff and provide the values as IIO_RESISTANCE via iio interface?

Btw this feature should be only added to dts files where is actually used.

Regards
Stefan

Quoted 15 lines

Harald Geyer (3):
iio: mxs-lradc: Add regulators for current sources
ARM: dts: imx23: Provide regulators for the current sources of the
LRADC
ARM: dts: imx28: Provide regulators for the current sources of the
LRADC

 .../bindings/staging/iio/adc/mxs-lradc.txt         |  29 ++++
 arch/arm/boot/dts/imx23.dtsi                       |   8 ++
 arch/arm/boot/dts/imx28.dtsi                       |   8 ++
 drivers/iio/adc/Kconfig                            |   1 +
 drivers/iio/adc/mxs-lradc.c                        | 152 +++++++++++++++++++++
 5 files changed, 198 insertions(+)