Am 07.11.2013 um 08:55 schrieb Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>: > On Thu, 2013-11-07 at 08:52 +0100, Alexander Graf wrote: >> Am 06.11.2013 um 20:58 schrieb Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>: >> >>> On Wed, 2013-11-06 at 12:24 +0100, Alexander Graf wrote: >>>> On 05.11.2013, at 08:42, Liu Ping Fan <kernelfans@xxxxxxxxx> wrote: >>>> >>>>> Signed-off-by: Liu Ping Fan <pingfank@xxxxxxxxxxxxxxxxxx> >>>> >>>> Patch description missing. >>> >>> Do you really need a description for trivial one-lines whose subject >>> is a perfectly complete description already ? >> >> Would I ask for it otherwise? It's also not 100% obvious that the assignment is redundant. > > And ? An explanation isn't going to be clearer than the code in that > case ... It's pretty non-obvious when you do a git show on that patch in 1 year from now, as the redundancy is out of scope of what the diff shows. Alex -- To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
