On Fri, Mar 25, 2011 at 4:48 PM, Kevin Hilman <khilman@xxxxxx> wrote: > Hi Jean, > > Jean Pihet <jean.pihet@xxxxxxxxxxxxxx> writes: > >> On Thu, Mar 24, 2011 at 1:00 AM, Kevin Hilman <khilman@xxxxxx> 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. Ok got the point. In fact my e-mail app (gmail web) does automatically hide the similar portions of text. > > 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 Thanks, Jean > > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html