Thread (8 messages) 8 messages, 3 authors, 2014-09-03
STALE4345d
Revisions (6)
  1. v1 [diff vs current]
  2. v1 [diff vs current]
  3. v1 current
  4. v1 [diff vs current]
  5. v2 [diff vs current]
  6. v3 [diff vs current]

[PATCH 3/3] clocksource: sh_tmu: Document r8a7779 binding

From: Sergei Shtylyov <hidden>
Date: 2014-08-27 11:10:44
Also in: linux-sh

On 8/27/2014 9:00 AM, Simon Horman wrote:
In general Renesas hardware is not documented to the extent
where the relationship between IP blocks on different SoCs can be assumed
although they may appear to operate the same way. Furthermore the
documentation typically does not specify a version for individual
IP blocks. For these reasons a convention of using the SoC name in place
of a version and providing SoC-specific compat strings has been adopted.
Although not universally liked this convention is used in the bindings
for the drivers a number of drivers for Renesas hardware. The purpose
of this patch is to update the Renesas R-Car Timer Unit (TMU)
driver to follow this convention.
Signed-off-by: Simon Horman <redacted>
---
* I plan to follow up with a patch patch to use the new binding in the
   dtsi files for the r8a7779 SoC.
---
  Documentation/devicetree/bindings/timer/renesas,tmu.txt | 6 ++++--
  1 file changed, 4 insertions(+), 2 deletions(-)
quoted hunk ↗ jump to hunk
diff --git a/Documentation/devicetree/bindings/timer/renesas,tmu.txt b/Documentation/devicetree/bindings/timer/renesas,tmu.txt
index 425d0c5..712ddfa 100644
--- a/Documentation/devicetree/bindings/timer/renesas,tmu.txt
+++ b/Documentation/devicetree/bindings/timer/renesas,tmu.txt
@@ -8,7 +8,9 @@ are independent. The TMU hardware supports up to three channels.

  Required Properties:

-  - compatible: must contain "renesas,tmu"
+  - compatible: must contain one of the following.
    Likewise, the example has both.
quoted hunk ↗ jump to hunk
+    - "renesas,tmu" generic TMU
+    - "renesas,tmu-r8a7779" R8A7779 TMU

    - reg: base address and length of the registers block for the timer module.
@@ -27,7 +29,7 @@ Optional Properties:
  Example: R8A7779 (R-Car H1) TMU0 node

  	tmu0: timer at ffd80000 {
-		compatible = "renesas,tmu";
+		compatible = "renesas,tmu-r8a7779", "renesas,tmu";
  		reg = <0xffd80000 0x30>;
  		interrupts = <0 32 IRQ_TYPE_LEVEL_HIGH>,
  			     <0 33 IRQ_TYPE_LEVEL_HIGH>,
WBR, Sergei
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help