linux-kernel - Re: [PATCH 00/15] iio: ti_am335x_adc: Add continuous mode take 2

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20130720124731.GA10834@gmail.com>
Date:	Sat, 20 Jul 2013 13:47:33 +0100
From:	"Zubair Lutfullah :" <zubair.lutfullah@...il.com>
To:	Jonathan Cameron <jic23@...nel.org>
Cc:	Zubair Lutfullah <zubair.lutfullah@...il.com>, 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

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.

Thank-you
Zubair Lutfullah
--
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/