[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <DC148C5AA1CEBA4E87973D432B1C2D8825E261E9@P3PWEX4MB008.ex4.secureserver.net>
Date: Wed, 30 Apr 2014 16:58:10 +0000
From: Hartley Sweeten <HartleyS@...ionengravers.com>
To: Chase Southwood <chase.southwood@...il.com>
CC: "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>,
"abbotti@....co.uk" <abbotti@....co.uk>,
"devel@...verdev.osuosl.org" <devel@...verdev.osuosl.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH 6/6 v2] staging: comedi: addi_apci_1564: remove use of
devpriv->s_EeParameters
On Wednesday, April 30, 2014 12:52 AM, Chase Southwood wrote:
> Thanks so much, I greatly appreciate the review. I'll fix the
> changelog for patch 4 and send once more (as I assume that's easier
> for Greg). Also, I should know better about the cover letter as
> well...I was once told not to send them for strictly cleanup patchsets
> (as Greg can't do anything with them and cleanups should be obvious)
> but I've gotten in the habit of not doing a cover letter for any
> patchsets. I will send cover letters (and be more careful about my
> other mistakes) in the future.
Chase,
Good job on the cleanup you have done so far.
The cover letter does not get committed as part of the patch set. But
when you are submitting a series it give a convenient place to add
the sign off tag for a series. Also, if you would have done a cover letter
for this series you would have probably spotted the subject line issues
in patches 5 and 6.
Keep up the good work!
Hartley
Powered by blists - more mailing lists