[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