Re: [PATCH v2 04/18] clocksource: Add ARM System timer driver

From: Linus Walleij
Date: Mon Mar 09 2015 - 11:50:38 EST

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?

Linus Walleij
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at