On Sat, Jan 16, 2016 at 05:14:35PM +0100, Alexander Koch wrote: > This patch series aims at enabling IRQ-less operation for the TI OPT3001 light > sensor. Jon, I was checking to see if the DT bindings have been updated to reflect the interrupt-less operation however I can't seem to find it in the Kernel tree. The last activity I can find is this one... https://patchwork.ozlabs.org/patch/490819/ I guess it may have slipped through the cracks so I'm going to update the bindings document and re-submit. Regards, Andreas > > The current version of the driver requires an interrupt line to be connected to > the INT pin of the sensor, through which the IIO framework gets notified about > readout values being ready. In the datasheet this is described as optional (sec. > 8.1.1). > > In my use case of the OPT3001 I do not have any interrupt lines or GPIOs > available to connect the INT pin to, so I have implemented an interrupt-less > operation mode that simply sleeps for the specified worst-case readout time > instead of waiting for the interrupt. > > This change is transparent as interrupt-less operation mode is done only when no > valid interrupt no. is configured via device tree. > > Patches are taken against linux-next/master, tested by compilation, > checkpatch.pl and by running on an embedded developer board (both IRQ-enabled > and IRQ-less mode). > > > Changes from v1: > > - use type bool for introduced member 'use_irq' > - include trivial refactoring step that changes the types of two other members > to bool, in order to match 'use_irq' > > > Alexander Koch (3): > iio: light: opt3001: extract int. time constants > iio: light: opt3001: trivial type refactoring > iio: light: opt3001: enable operation w/o IRQ > > drivers/iio/light/opt3001.c | 156 ++++++++++++++++++++++++++++++-------------- > 1 file changed, 106 insertions(+), 50 deletions(-) > > -- > 2.7.0 > -- 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
