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

Grant Likely grant.likely at secretlab.ca
Thu May 7 08:46:42 EST 2009


On Wed, May 6, 2009 at 4:16 PM, Wolfgang Denk <wd at denx.de> wrote:
> 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?

Fair enough.  I was going through your series pretty quickly and this
one doesn't really need it.

However, in this particular case, I think I would rather see both .dts
rework patches put into the same patch so it can be reviewed all at
once.

g.


-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.



More information about the Linuxppc-dev mailing list