[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <a8f2c435-027a-473f-9ea0-71bfe4276c2b@email.android.com>
Date: Sat, 20 Jul 2013 14:50:39 +0100
From: Jonathan Cameron <jic23@...nel.org>
To: "Zubair Lutfullah :" <zubair.lutfullah@...il.com>
CC: jic23@....ac.uk, gregkh@...uxfoundation.org,
linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org,
koen@...inion.thruhere.net
Subject: Re: [PATCH 00/15] iio: ti_am335x_adc: Add continuous mode take 2
"Zubair Lutfullah :" <zubair.lutfullah@...il.com> wrote:
>On Sat, Jul 20, 2013 at 12:25:48PM +0100, Jonathan Cameron wrote:
>> 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.
>
>Indeed.
>
>I had already merged a lot of minor bug fixes from Rachil into the main
>patch that adds continuous mode.
>
>But I kept Russ's ones separate because of authorship and history.
>
>
>> Part 1. Any fixes that are unconnected to the rest of the series.
>These will
>> then get sent upstream within this cycle.
>
>Understood. I'll send a small series that is disjoint and contains
>fixes for the existing driver.
>
>> 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
>
>And then one patch for continuous mode with appropriate
>comments/sign-offs.
>
Excellent. Thanks for sorting these out.
>Thank-you
>Zubair Lutfullah
>--
>To unsubscribe from this list: send the line "unsubscribe linux-iio" in
>the body of a message to majordomo@...r.kernel.org
>More majordomo info at http://vger.kernel.org/majordomo-info.html
--
Sent from my Android phone with K-9 Mail. Please excuse my brevity.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists