diff mbox

ARM: tegra: Add clocks property to TWD node

Message ID 1360684611-11152-1-git-send-email-thierry.reding@avionic-design.de
State Changes Requested, archived
Headers show

Commit Message

Thierry Reding Feb. 12, 2013, 3:56 p.m. UTC
The generic ARM TWD code has recently gained support for looking up the
clock from DT. Adding the appropriate clocks property allows the driver
to successfully register the clock event device.

Signed-off-by: Thierry Reding <thierry.reding@avionic-design.de>
---
Note that the binding documentation says that the twd clock is 132 on
Tegra20, but according to drivers/clk/tegra/clk-tegra20.c it is in fact
130. The patch doesn't update the binding documentation because some
other offsets are probably also wrong and I didn't have the time to go
through all of them.
---
 arch/arm/boot/dts/tegra20.dtsi | 1 +
 arch/arm/boot/dts/tegra30.dtsi | 1 +
 2 files changed, 2 insertions(+)

Comments

Stephen Warren Feb. 12, 2013, 7:29 p.m. UTC | #1
On 02/12/2013 08:56 AM, Thierry Reding wrote:
> The generic ARM TWD code has recently gained support for looking up the
> clock from DT. Adding the appropriate clocks property allows the driver
> to successfully register the clock event device.
> 
> Signed-off-by: Thierry Reding <thierry.reding@avionic-design.de>
> ---
> Note that the binding documentation says that the twd clock is 132 on
> Tegra20, but according to drivers/clk/tegra/clk-tegra20.c it is in fact
> 130. The patch doesn't update the binding documentation because some
> other offsets are probably also wrong and I didn't have the time to go
> through all of them.

The driver is buggy in that it doesn't match the DT bindings. I'll send
a patch to fix that up very soon. Then this patch can be rebased.
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/boot/dts/tegra20.dtsi b/arch/arm/boot/dts/tegra20.dtsi
index cdb8da0..d6642b2 100644
--- a/arch/arm/boot/dts/tegra20.dtsi
+++ b/arch/arm/boot/dts/tegra20.dtsi
@@ -118,6 +118,7 @@ 
 		compatible = "arm,cortex-a9-twd-timer";
 		reg = <0x50040600 0x20>;
 		interrupts = <1 13 0x304>;
+		clocks = <&tegra_car 130>;
 	};
 
 	intc: interrupt-controller {
diff --git a/arch/arm/boot/dts/tegra30.dtsi b/arch/arm/boot/dts/tegra30.dtsi
index 572a45b..922c8cf 100644
--- a/arch/arm/boot/dts/tegra30.dtsi
+++ b/arch/arm/boot/dts/tegra30.dtsi
@@ -119,6 +119,7 @@ 
 		compatible = "arm,cortex-a9-twd-timer";
 		reg = <0x50040600 0x20>;
 		interrupts = <1 13 0xf04>;
+		clocks = <&tegra_car 214>;
 	};
 
 	intc: interrupt-controller {