On 07/18/2013 11:21 PM, Zubair Lutfullah wrote: > Patches now give correct authorship. > > and checkpatch.pl issues are checked for each patch. > > I hope the actual code bashing can begin now. > > A series of patches that add continuous sampling support > for the adc drivers for the am335x. > > These apply on top of mfd-next after the recent set of patches > on this driver by Sebastian Andrzej Siewior > > Tested on the Beaglebone Black running 3.11 > > Patil, Rachna (5): > MFD: ti_tscadc: disable TSC config registers in adc mode > iio: ti_am335x_adc: Fix wrong samples received on 1st read > input: ti_tsc: Enable shared IRQ for TSC > iio: mfd: input: ti_am335x_adc:Add support for continuous mode > MFD: ti_tscadc: ADC Clock check not required > > Russ Dill (10): > iio: ti_am335x_adc: Handle set to clear IRQENABLE > iio: ti_am335x_adc: Handle set to clear IRQSTATUS > iio: ti_am335x_adc: Handle overrun before threshold event > iio: ti_am335x_adc: Avoid double threshold event > iio: ti_am335x_adc: Also clear threshold event when clearing overrun > event > iio: ti_am335x_adc: Print error and handle short FIFO events > iio: ti_am335x_adc: Fix allocation count of FIFO buffer. > iio: ti_am335x_adc: Fix capture operation during resume > iio: ti_am335x_adc: Reset and clear overrun status before capture > iio: ti_am335x_adc: Properly handle out of memory situation I am a little irritated by the mess we have here. Russ has clearly done a lot of fine work cleaning up the earlier patches. As a result we have a series of initial buggy patches and then a series of patches fixing them again. As you are submitting these for mainline I would really like it all merged down into a clean series of clear patches. Part 1. Any fixes that are unconnected to the rest of the series. These will then get sent upstream within this cycle. Part 2. Feature add patch. Here that is basically a single patch adding the continous mode support. It is nice to maintain history and all but I would much rather have something that is easy to review, with appropriate comments or if people will give them, sign offs to reflect the various contributions. Jonathan > > drivers/iio/adc/ti_am335x_adc.c | 354 +++++++++++++++++++++++++---- > drivers/input/touchscreen/ti_am335x_tsc.c | 17 +- > drivers/mfd/ti_am335x_tscadc.c | 30 ++- > include/linux/mfd/ti_am335x_tscadc.h | 27 ++- > 4 files changed, 366 insertions(+), 62 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