Hi Sylvain, On 09.02.2016 22:07, Sylvain Lemieux wrote: > On Tue, 2016-02-09 at 10:57 -0800, Stephen Boyd wrote: >> On 02/09, slemieux.tyco@xxxxxxxxx wrote: >>> From: Sylvain Lemieux <slemieux@xxxxxxxxxxx> >>> >>> This patch add the support to setup the clock frequency output >>> using an optional parameter from the device tree. >>> >>> The previous kernel version did not change the clock frequency >>> output setup by the kickstart and/or bootloader; >>> this version always setup the clock frequency output to 208MHz. ok, it makes perfect sense, and here I see two options, one is to recover the previous behaviour by removing clk_set_rate() of LPC32XX_CLK_HCLK_PLL completely, another one (and more preferable) is to add this clock to a list in include/dt-bindings/clock/lpc32xx-clock.h and set its rate in a board specific DTB file. >>> Signed-off-by: Sylvain Lemieux <slemieux@xxxxxxxxxxx> >>> --- >> >> We have assigned-clock-rates, can you use that instead? >> > OK, I will submit a new version of the patch tomorrow. > By the way if you have a cover letter or 2/2 patch in the series, please send it to linux-clk as well. -- With best wishes, Vladimir -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html