[PATCH 05/12] mpc5121ads: Added Reset Module node to DTS.

Wolfgang Denk wd at denx.de
Thu May 7 08:16:48 EST 2009


Dear Grant Likely,

In message <fa686aa40905061340s212385b1r8be4e2f7505bd0a0 at mail.gmail.com> you wrote:
> On Wed, May 6, 2009 at 2:15 PM, Wolfgang Denk <wd at denx.de> wrote:
> > From: Piotr Ziecik <kosmo at semihalf.com>
> >
> > Signed-off-by: Piotr Ziecik <kosmo at semihalf.com>
> > Signed-off-by: Wolfgang Denk <wd at denx.de>
> > Cc: Grant Likely <grant.likely at secretlab.ca>
> > Cc: John Rigby <jcrigby at gmail.com>
> 
> Missing patch description.  Otherwise looks okay.

This is a repeating complaint. Do I really need an extra  description
for a trivial patch that does exactly what the Subject: says?

What is recommended practice in such a case?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
 The software required `Windows 95 or better', so I installed Linux.



More information about the Linuxppc-dev mailing list