On Wed, Jan 11, 2017 at 02:51:11PM +0800, Phil Reid wrote: > This adds TI's tlc4541 16-bit ADC driver. Which is a single channel > ADC. Supports raw and trigger buffer access. > Also supports the tlc3541 14-bit device, which has not been tested. > Implementation of the tlc3541 is fairly straight forward thou. > > Signed-off-by: Phil Reid <preid@xxxxxxxxxxxxxxxxx> > --- > > Notes: > Changes from v1: > - Add tlc3541 support and chan spec. > - remove fields that where already 0 from TLC4541_V_CHAN macro > - Increase rx_buf size in tlc4541_state to avoid copy in tlc4541_trigger_handle > - Remove erroneous be16_to_cpu in tlc4541_trigger_handle > - Docs/binding: spi -> SPI & add ti,tlc3541 > > I haven't add Rob's Ack due to adding a new compatible string. > > I tried to ".index = 1" from the spec as suggested by Peter, but that didn't > seem to work. Perhaps remove of .channel was the intended target. > > Example output from iio_readdev > > with ".index = 1" > root@cyclone5:~# mkdir /sys/kernel/config/iio/triggers/hrtimer/hr1 > root@cyclone5:~# iio_readdev -t hr1 -b 32 -s 10 tlc4541 | hexdump > WARNING: High-speed mode not enabled > 0000000 af00 0000 0000 0000 b922 ca99 93da 1492 > 0000010 a800 00ff 0000 0000 b246 cb30 93da 1492 > 0000020 a900 0000 0000 0000 4f9c cbc9 93da 1492 > 0000030 aa00 00ff 0000 0000 bd2c cc61 93da 1492 > 0000040 aa00 00ff 0000 0000 544c ccfa 93da 1492 > 0000050 ab00 00ff 0000 0000 e806 cd92 93da 1492 > 0000060 a900 00ff 0000 0000 846c ce2b 93da 1492 > 0000070 ab00 0000 0000 0000 2efc cec8 93da 1492 > 0000080 a800 00ff 0000 0000 b090 cf5c 93da 1492 > 0000090 a900 00ff 0000 0000 476a cff5 93da 1492 > > without .index > root@cyclone5:~# mkdir /sys/kernel/config/iio/triggers/hrtimer/hr1 > root@cyclone5:~# iio_readdev -t hr1 -b 32 -s 10 tlc4541 | hexdump > WARNING: High-speed mode not enabled > 0000000 6db0 eeb6 93e3 1492 35e0 ef4f 93e3 1492 > 0000010 4b34 efe5 93e3 1492 e9f2 f07d 93e3 1492 > 0000020 6182 f116 93e3 1492 090a f1af 93e3 1492 > 0000030 409c f249 93e3 1492 6c1a f2e0 93e3 1492 > 0000040 cd02 f378 93e3 1492 9582 f411 93e3 1492 > > .../devicetree/bindings/iio/adc/ti-tlc4541.txt | 17 ++ Acked-by: Rob Herring <robh@xxxxxxxxxx> > drivers/iio/adc/Kconfig | 11 + > drivers/iio/adc/Makefile | 1 + > drivers/iio/adc/ti-tlc4541.c | 276 +++++++++++++++++++++ > 4 files changed, 305 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iio/adc/ti-tlc4541.txt > create mode 100644 drivers/iio/adc/ti-tlc4541.c -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html