Messages in this thread | | | Date | Thu, 12 Jan 2017 09:47:38 +0100 | From | Johan Hovold <> | Subject | Re: [PATCH v3] staging: greybus: checkpatch: Change parameter type unsigned to unsigned int |
| |
On Thu, Jan 12, 2017 at 10:05:10AM +0300, Dan Carpenter wrote: > On Wed, Jan 11, 2017 at 04:00:35PM +0100, Roman Sommer wrote: > > Note that this patch does not fix all checkpatch warnings for the > > affected files. > > > > A lot of ways that display git commits put the changelog and the summary > far apart. The changelog is pretty useless by itself in this case. > > https://lkml.org/lkml/2017/1/11/308 > > See how it puts the date in between? marc.info is even worse but google > hasn't indexed it yet and I'm too lazy to look it up. So people read > the summary or the changelog but not necessarily both.
Yeah, let's do this right.
Roman, could you resend this with a self-contained commit message, and a more compact summary (Subject) as I suggested earlier?
Thanks, Johan
| |