[PATCH 3/3] clocksource: sh_tmu: Document r8a7779 binding
Simon Horman
horms at verge.net.au
Tue Sep 2 18:04:06 PDT 2014
On Wed, Aug 27, 2014 at 03:10:44PM +0400, Sergei Shtylyov wrote:
> 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 <horms+renesas at verge.net.au>
>
> >---
> >* 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(-)
>
> >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.
Thanks, as I will change "one of" to "one or more of".
> >+ - "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
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
More information about the linux-arm-kernel
mailing list