Skip to content

Make Armada 375 use the reference clock when possible

Thomas Petazzoni linux-arm-kernel linux-watchdog 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest4 patches 0 repliesNew

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4clocksource: armada-370-xp: Add missing clock enable+0 -00 replies
2/4watchdog: orion: Use the reference clock on Armada 375 SoC+0 -00 replies
3/4clocksource: armada-370-xp: Use the reference clock on A375 SoC+0 -01 reply
4/4ARM: dts: Enable the reference clock for timer and watchdog on Armada 375 SoC+0 -01 reply

Reviews trailers per patch

1 of 4 patches has a Reviewed-by or Acked-by.

PatchThomas Petazzoni
1/4R-b
2/4
3/4
4/4

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/4] Make Armada 375 use the reference clock when possible

Dear Ezequiel Garcia,

Quoted 5 lines · Ezequiel Garcia

This series adds support for the 25 MHz reference clock available on Armada 375 SoC to use on the timer and watchdog drivers. It is similar to the one present in Armada XP SoC.

Given we initially had access to only a very early SoC revision (A375 Z0)

Actually: s/Z0/Z1/.

Quoted 9 lines

and due to a hardware issue, the timer and watchdog support was originally submitted to use the core clock.

Now that the A0 SoC revision is out, we can fix this and use the reference clock. The reason for this change is that the core clock is subject to the SSCG, so boards where SSCG is enabled exhibit a very large timer drift.

To prevent any compatibility issues when booting with an older devicetree, this series provides proper fall backs in each case.

You don't clearly state whether your patch series keep compatibility with the 375 Z1 or not. And in fact, it doesn't keep compatibility with Z1. I'm fine with that, but then it means we should officially declare the Z1 support in mainline as dead, and get rid of the workarounds that applied only to 375 Z1.

Best regards,

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com