Fwd: Fwd: [PATCH 0/2] iio: ti_am335x_adc: Add optional DT properties for tscadc
From: Hannes Petermaier
Date: Tue Apr 14 2015 - 07:32:08 EST
did you recognize my last email ? whats your opinion around that =
----- Forwarded by Hannes Petermaier/Eggelsberg/AT/B&R on 14.04.2015 13:14
Hannes Petermaier/Eggelsberg/AT/B&R schrieb am 07.04.2015 14:30:19:
> From: Hannes Petermaier/Eggelsberg/AT/B&R
> To: vigneshr@xxxxxx
> Cc: linux-iio@xxxxxxxxxxxxxxx
> Date: 07.04.2015 14:30
> Subject: WG: Fwd: [PATCH 0/2] iio: ti_am335x_adc: Add optional DT
properties for tscadc
> Hi Vignesh,
> would it be possible to add some more channel-specific settings ?
> It would be nice to have allmost full control to the STEPCONFIGx
> At least we need to write the bits
> In the current mainline version only (SEL_INP_SWC_3_0) is written.
> So for the other bits "0" is value is used, for my point of view this is
> For example if we want to read a value from AIN5 the negative pin from
> muxed allways to AIN0.
> In fact i can readout heavy jitter even if AIN5 is connected to ground -
> setting up negative adc pin within code (to use REFN) the readout value
> as expected without nameable jitter.
> If i short AIN0 also to ground, jitter is also eliminated.
> Maybe this is also some fault of TI SoC ... in normal case somebody
> expect, that negative adc pin is equal even the Diff_CNTRL bit isn't set
> in practice it isn't.
> Also actually it isn't possible to make some accurate measurement due to
> fact that allways VDDA_ADC is used as positive reference.
> So it would be nice to have control around this bits.
> Whats your opinion around that?
> best regards,
> ----- Forwarded by Hannes Petermaier/Eggelsberg/AT/B&R on 07.04.2015
> From: Johannes Pointner <johannes.pointner@xxxxxxxxx>
> To: hannes.petermaier@xxxxxxxxxxxxxxxxx
> Date: 07.04.2015 13:58
> Subject: Fwd: [PATCH 0/2] iio: ti_am335x_adc: Add optional DT properties
> ---------- Forwarded message ----------
> From: Vignesh R <vigneshr@xxxxxx>
> Date: 2015-03-31 13:12 GMT+02:00
> Subject: [PATCH 0/2] iio: ti_am335x_adc: Add optional DT properties for
> To: Rob Herring <robh+dt@xxxxxxxxxx>, Pawel Moll <pawel.moll@xxxxxxx>,
> Mark Rutland <mark.rutland@xxxxxxx>, Ian Campbell
> <ijc+devicetree@xxxxxxxxxxxxxx>, Kumar Gala <galak@xxxxxxxxxxxxxx>,
> Jonathan Cameron <jic23@xxxxxxxxxx>
> Cc: Hartmut Knaack <knaack.h@xxxxxx>, Lars-Peter Clausen
> <lars@xxxxxxxxxx>, Peter Meerwald <pmeerw@xxxxxxxxxx>, Dmitry Torokhov
> <dmitry.torokhov@xxxxxxxxx>, Karol Wrona <k.wrona@xxxxxxxxxxx>, Jan
> Kardell <jan.kardell@xxxxxxxxxx>, devicetree@xxxxxxxxxxxxxxx,
> linux-kernel@xxxxxxxxxxxxxxx, linux-iio@xxxxxxxxxxxxxxx,
> fcooper@xxxxxx, Vignesh R <vigneshr@xxxxxx>
> This patch adds optional DT properties for tscadc to set open delay,
> sample delay and number of averages per sample for each adc channel.
> Open delay, sample delay and averaging are some of the parameters that
> affect the sampling rate and accuracy of the tscadc. Decreasing delays
> and averaging helps to achieve higher sampling rates, while increasing
> this parameters provides greater accuracy. Hence, this patch provides DT
> properties to set the initial values for delays and number of averages
> per sample. User space control via sysfs can be added later.
> Vignesh R (2):
> iio: adc: ti_am335x_adc: refactor DT parsing into a function
> iio: adc: ti_am335x_adc: make sample delay, open delay, averaging DT
> .../bindings/input/touchscreen/ti-tsc-adc.txt | 24 +++++++
> drivers/iio/adc/ti_am335x_adc.c | 83
> 2 files changed, 91 insertions(+), 16 deletions(-)
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/