[PATCH] staging:iio:tsl2563 rewrite probe error handling

Dan Carpenter dan.carpenter at oracle.com
Fri Mar 9 06:23:58 UTC 2012

On Thu, Mar 08, 2012 at 07:51:07PM -0800, Grant Grundler wrote:
> Hi Jonathan!
> On Wed, Mar 7, 2012 at 3:28 AM, Jonathan Cameron <jic23 at kernel.org> wrote:
> > Fundamentally an excellent patch fixing some real issues,
> Thank you! :)
> > but just
> > because I'm in that sort of mood I'll be fussy about the fact it is
> > one patch.
> >
> > To nitpick, I'd have prefered this to be two patches. First that fixes
> > the issues and the second that adds the additional output.  Actually
> > should probably have even had a third making the white space changes.
> Yup - I see/understand why. I need to improve my git foo and work
> habits so it's not hard to play with a series of smaller patches
> rather than one patch that does everything to one function.

If you use git citool then you can just highlight and right click on
the lines you want to merge.

dan carpenter

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/attachments/20120309/d04443e0/attachment.asc>

More information about the devel mailing list