On Fri, Jan 08, 2016 at 01:45:13PM -0500, Rhyland Klein wrote: > From: Mark Kuo <mkuo@xxxxxxxxxx> > > Fix PLLE spread spectrum configuration so it aligns with downstream > kernel. > > Signed-off-by: Mark Kuo <mkuo@xxxxxxxxxx> > Signed-off-by: Rhyland Klein <rklein@xxxxxxxxxx> > --- > drivers/clk/tegra/clk-pll.c | 22 ++++++++++++++-------- > 1 file changed, 14 insertions(+), 8 deletions(-) Can we have a better description for this. Merely saying that this is meant to align things with downstream isn't really helpful for anyone who doesn't have the downstream kernel to check against. I also see some potential for splitting this up into several logical changes, since for example it seems to fix a typo (?) where some bits were being set that should really be cleared, and in other hunks it splits up the SS coefficients by SoC generation. Splitting it up into logical changes might make it easier to write accurate commit descriptions rather than this kind of blanket commit message. Thierry
Attachment:
signature.asc
Description: PGP signature
