[lkml]   [2018]   [Mar]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH v4 14/16] powerpc: Use generic free_initrd_mem.
Shea Levy <> writes:

> Joe Perches <> writes:
>> On Wed, 2018-03-28 at 16:36 -0400, Shea Levy wrote:
>>> Signed-off-by: Shea Levy <>
>> Most people seem to want some form of commit message
>> and not just your sign-off.
> Ah, if the subject is insufficient I can add some more detail.

Yeah please do.

Seeing this patch in isolation, with no change log, I might think it's
safe for me to just apply it.

But that would break the build because I don't have patch 1.

So for starters you need to explain that part, eg something like:

A previous patch in the series added a weak definition of
free_initrd_mem() in init/initramfs.c.

The powerpc implementation is identical, so it can be removed allowing
the generic version to be used.

Then you could also tell me if you did/didn't build/boot test it.


 \ /
  Last update: 2018-03-29 15:21    [W:0.116 / U:2.888 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site