Naveen, On Mon, Oct 14, 2013 at 10:15 PM, Naveen Krishna Chatradhi <ch.naveen@xxxxxxxxxxx> wrote: > This patch does the following > 1. The irq routine is so simple (just one register read) shouldn't be long > Hence, reduce the timeout to 100milli secs, > 2. With 100ms of wait time, interruptible is very much unnecessary. > Hence, use wait_for_completion_timeout instead of > wait_for_completion_interruptible_timeout > 3. Reset software if a timeout happens. > 4. Add INIT_COMPLETION before the wait_for_completion_timeout in raw_read() > > Note: submitted for review at https://patchwork.kernel.org/patch/2279591/ > > Signed-off-by: Naveen Krishna Chatradhi <ch.naveen@xxxxxxxxxxx> > Cc: Doug Anderson <dianders@xxxxxxxxxxxx> > Cc: Lars-Peter Clausen <lars@xxxxxxxxxx> > --- > Changes since v1: > As per discussion at > http://marc.info/?l=linux-kernel&m=136517637228869&w=3 > > Changes since v2: > None. > Rebased and reposting. > > Changes since v3: > 1. commit message change and > 2. removed an unncessary assignment > > drivers/iio/adc/exynos_adc.c | 69 +++++++++++++++++++++++------------------- > 1 file changed, 38 insertions(+), 31 deletions(-) Please spin to match the changes that Grant requested on our gerrit at <https://chromium-review.googlesource.com/172724>. Thanks! :) -Doug -- 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
