Hi Jean,

Jean Pihet <[email protected]> writes:

> On Thu, Mar 24, 2011 at 1:00 AM, Kevin Hilman <[email protected]> wrote:
>> Start cleaning up the voltage layer to have a voltage domain layer
>> that resembles thae structure of the existing clock and power domain
> s/thae/the
>
>> layers.  To that end:
> Extra space
>

Thanks for the review of this series.

When commenting on a patch (especially large ones) it helps if you
remove context that is not relevant to the patch.

For example, your two comments above are the only ones on this patch,
yet below you still have the entire patch context, which requires the
author to look through the whole patch again to see if there are other
comments. 

It is a great help to the author (and other reviewers) if the reply only
keeps the relevant context so it's obvious what the reply is to.

Thanks,

Kevin

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to