On 05/22/2015 02:56 PM, Andrew Bresticker wrote: > On Thu, May 21, 2015 at 4:57 PM, Ezequiel Garcia > <[email protected]> wrote: >> From: Damien Horsley <[email protected]> >> >> Correct the critical clock list. The current one is wrong, and may >> fail under some circumstances. >> >> Signed-off-by: Damien Horsley <[email protected]> >> Signed-off-by: Ezequiel Garcia <[email protected]> > > The commit message could be more descriptive, e.g. explaining which > additional clocks must be enabled at all times and why, especially > since forcing on clocks form the clock driver is very much frowned > upon unless absolutely necessary. Otherwise, > > Reviewed-by: Andrew Bresticker <[email protected]> >
OK, I'll explain this in detail. Thanks for the review, -- Ezequiel -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
