Hi Tony, On 17/07/2018 07:55, Tony Lindgren wrote: > Hi, > > * Neil Armstrong <narmstrong@xxxxxxxxxxxx> [180716 20:42]: >> Add the missing fck clock to all timer nodes of the AM33XX dtsi. >> >> Signed-off-by: Neil Armstrong <narmstrong@xxxxxxxxxxxx> >> --- >> arch/arm/boot/dts/am33xx.dtsi | 10 ++++++++++ >> 1 file changed, 10 insertions(+) >> >> diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi >> index 9cd62bc2..e8e665d 100644 >> --- a/arch/arm/boot/dts/am33xx.dtsi >> +++ b/arch/arm/boot/dts/am33xx.dtsi >> @@ -541,6 +541,8 @@ >> reg = <0x48042000 0x400>; >> interrupts = <69>; >> ti,hwmods = "timer3"; >> + clocks = <&timer3_fck>; >> + clock-names = "fck"; >> }; > > These should all use the clkctrl clock nodes, see: Even the already configured timer1 & timer2 ? > > $ git grep CLKCTRL include/dt-bindings/clock/ | grep TIMER > > The module clock is bit 0, and on some SoCs bit 24 is the > timer fck. But I don't think these SoS use separate bit 24 > clock. > > Regards, > > Tony > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html