diff options
author | Ezequiel Garcia <ezequiel.garcia@free-electrons.com> | 2013-08-13 10:43:14 -0400 |
---|---|---|
committer | Daniel Lezcano <daniel.lezcano@linaro.org> | 2013-09-02 15:44:06 -0400 |
commit | 8ad31e66c150aeb44927b44bfb1ce7a3e23ec463 (patch) | |
tree | f6b19e94e61687642e41fed0af07908c827b5fa5 | |
parent | 7cd6392c9bf5da6986103fcf5ca1b6fd0489d9b4 (diff) |
clocksource: armada-370-xp: Fix device-tree binding
This commit fixes the DT binding for the Armada 370/XP SoC timer.
The previous "marvell,armada-370-xp-timer" compatible is removed and
two new compatible strings are introduced: "marvell,armada-xp-timer"
and "marvell,armada-370-timer".
The rationale behind this change is that the Armada 370 SoC and the
Armada XP SoC timers are not really compatible:
* Armada 370 has no 25 MHz fixed timer.
* Armada XP cannot work properly without such 25 MHz fixed timer
as doing otherwise leads to using a clocksource whose frequency
varies when doing cpufreq frequency changes.
This commit also removes the "marvell,timer-25Mhz" property, given
it's now meaningless.
Cc: devicetree@vger.kernel.org
Acked-by: Jason Cooper <jason@lakedaemon.net>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
-rw-r--r-- | Documentation/devicetree/bindings/timer/marvell,armada-370-xp-timer.txt | 27 |
1 files changed, 22 insertions, 5 deletions
diff --git a/Documentation/devicetree/bindings/timer/marvell,armada-370-xp-timer.txt b/Documentation/devicetree/bindings/timer/marvell,armada-370-xp-timer.txt index 36381129d141..4c453b22b2b4 100644 --- a/Documentation/devicetree/bindings/timer/marvell,armada-370-xp-timer.txt +++ b/Documentation/devicetree/bindings/timer/marvell,armada-370-xp-timer.txt | |||
@@ -2,14 +2,31 @@ Marvell Armada 370 and Armada XP Timers | |||
2 | --------------------------------------- | 2 | --------------------------------------- |
3 | 3 | ||
4 | Required properties: | 4 | Required properties: |
5 | - compatible: Should be "marvell,armada-370-xp-timer" | 5 | - compatible: Should be either "marvell,armada-370-timer" or |
6 | "marvell,armada-xp-timer" as appropriate. | ||
6 | - interrupts: Should contain the list of Global Timer interrupts and | 7 | - interrupts: Should contain the list of Global Timer interrupts and |
7 | then local timer interrupts | 8 | then local timer interrupts |
8 | - reg: Should contain location and length for timers register. First | 9 | - reg: Should contain location and length for timers register. First |
9 | pair for the Global Timer registers, second pair for the | 10 | pair for the Global Timer registers, second pair for the |
10 | local/private timers. | 11 | local/private timers. |
11 | - clocks: clock driving the timer hardware | 12 | - clocks: clock driving the timer hardware, only required for |
13 | "marvell,armada-370-timer"; | ||
12 | 14 | ||
13 | Optional properties: | 15 | Examples: |
14 | - marvell,timer-25Mhz: Tells whether the Global timer supports the 25 | 16 | |
15 | Mhz fixed mode (available on Armada XP and not on Armada 370) | 17 | - Armada 370: |
18 | |||
19 | timer { | ||
20 | compatible = "marvell,armada-370-timer"; | ||
21 | reg = <0x20300 0x30>, <0x21040 0x30>; | ||
22 | interrupts = <37>, <38>, <39>, <40>, <5>, <6>; | ||
23 | clocks = <&coreclk 2>; | ||
24 | }; | ||
25 | |||
26 | - Armada XP: | ||
27 | |||
28 | timer { | ||
29 | compatible = "marvell,armada-xp-timer"; | ||
30 | reg = <0x20300 0x30>, <0x21040 0x30>; | ||
31 | interrupts = <37>, <38>, <39>, <40>, <5>, <6>; | ||
32 | }; | ||