On Fri, Feb 20, 2015 at 7:01 PM, Maxime Coquelin <mcoquelin.stm32@xxxxxxxxx> wrote: > This patch adds clocksource support for ARMv7-M's System timer, > also known as SysTick. > > Signed-off-by: Maxime Coquelin <mcoquelin.stm32@xxxxxxxxx> (...) > + /* If no clock found, try to get clock-frequency property */ > + if (!rate) { > + ret = of_property_read_u32(np, "clock-frequency", &rate); > + if (ret) > + goto out_unmap; > + } If this driver is only used for this one system, and if on this one system the clk subsystem will provide the clock rate, then there is no need to include this hackaround property. Alternatively there is no point including reading the frequency from the clk subsystem for this one system. So which one is it? Yours, Linus Walleij -- 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