On 09/11/2018 09:35 PM, Sergei Shtylyov wrote:

>>>>> Describe TMUs in the R8A779{7|8}0 device trees.
>>>>>
>>>>> Based on the original (and large) patches by Vladimir Barinov.
>>>>>
>>>>> Signed-off-by: Vladimir Barinov <vladimir.bari...@cogentembedded.com>
>>>>> Signed-off-by: Sergei Shtylyov <sergei.shtyl...@cogentembedded.com>
>>>>>
>>>>> ---
>>>>> This patch is against the 'renesas-devel-20180906-v4.19-rc2' branch of
>>>>> Simon Horman's 'renesas.git' repo plus the R8A779{7|8}0 DT patch adding
>>>>> the CMT support).
>>>>>
>>>>> The R8A779{7|8}0 TMU DT binding update have been just posted...
>>>>>
>>>>>  arch/arm64/boot/dts/renesas/r8a77970.dtsi |   66 
>>>>> ++++++++++++++++++++++++++++++
>>>>>  arch/arm64/boot/dts/renesas/r8a77980.dtsi |   66 
>>>>> ++++++++++++++++++++++++++++++
>>>>>  2 files changed, 132 insertions(+)
>>>>>
>>>>> Index: renesas/arch/arm64/boot/dts/renesas/r8a77970.dtsi
>>>>> ===================================================================
>>>>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970.dtsi
>>>>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970.dtsi
>>>>> @@ -316,6 +316,72 @@
>>>>>                   resets = <&cpg 407>;
>>>>>           };
>>>>>  
>>>>> +         tmu0: timer@e61e0000 {
>>>>> +                 compatible = "renesas,tmu-r8a77970", "renesas,tmu";
>>>>> +                 reg = <0 0xe61e0000 0 0x30>;
>>>>> +                 interrupts = <GIC_SPI 136 IRQ_TYPE_LEVEL_HIGH>,
>>>>> +                              <GIC_SPI 137 IRQ_TYPE_LEVEL_HIGH>,
>>>>> +                              <GIC_SPI 138 IRQ_TYPE_LEVEL_HIGH>;
>>>>> +                 clocks = <&cpg CPG_MOD 125>;
>>>>> +                 clock-names = "fck";
>>>>> +                 power-domains = <&sysc R8A77970_PD_ALWAYS_ON>;
>>>>> +                 resets = <&cpg 125>;
>>>>> +                 status = "disabled";
>>>>> +         };
>>>>> +
>>>>> +         tmu1: timer@e6fc0000 {
>>>>> +                 compatible = "renesas,tmu-r8a77970", "renesas,tmu";
>>>>> +                 reg = <0 0xe6fc0000 0 0x30>;
>>>>> +                 interrupts = <GIC_SPI 128 IRQ_TYPE_LEVEL_HIGH>,
>>>>> +                              <GIC_SPI 129 IRQ_TYPE_LEVEL_HIGH>,
>>>>> +                              <GIC_SPI 130 IRQ_TYPE_LEVEL_HIGH>;
>>>>> +                 clocks = <&cpg CPG_MOD 124>;
>>>>> +                 clock-names = "fck";
>>>>> +                 power-domains = <&sysc R8A77970_PD_ALWAYS_ON>;
>>>>> +                 resets = <&cpg 124>;
>>>>> +                 status = "disabled";
>>>>> +         };
>>>>> +
>>>>> +         tmu2: timer@e6fd0000 {
>>>>> +                 compatible = "renesas,tmu-r8a77970", "renesas,tmu";
>>>>> +                 reg = <0 0xe6fd0000 0 0x30>;
>>>>> +                 interrupts = <GIC_SPI 303 IRQ_TYPE_LEVEL_HIGH>,
>>>>> +                              <GIC_SPI 304 IRQ_TYPE_LEVEL_HIGH>,
>>>>> +                              <GIC_SPI 305 IRQ_TYPE_LEVEL_HIGH>;
>>>>
>>>> Should GIC_SPI 306 also be here for TMU 2 channel 3?> 
>>>> And likewise for the r8a77980 (V3H)
>>>
>>>    There are only 3 channels per TMU according to the beginning of the TMU 
>>> chapter.
>>>
>>>> The documentation seems inconsistent as I see this listed in the
>>>> interrupt controller documentation. But I do not see that channel
>>>> documented in the TMU documentation.
>>>
>>>    Right!
>>>
>>>>> +                 clocks = <&cpg CPG_MOD 123>;
>>>>> +                 clock-names = "fck";
>>>>> +                 power-domains = <&sysc R8A77970_PD_ALWAYS_ON>;
>>>>> +                 resets = <&cpg 123>;
>>>>> +                 status = "disabled";
>>>>> +         };
>>>>> +
>>>>> +         tmu3: timer@e6fe0000 {
>>>>> +                 compatible = "renesas,tmu-r8a7797", "renesas,tmu";
>>>>> +                 reg = <0 0xe6fe0000 0 0x30>;
>>>>> +                 interrupts = <GIC_SPI 131 IRQ_TYPE_LEVEL_HIGH>,
>>>>> +                              <GIC_SPI 132 IRQ_TYPE_LEVEL_HIGH>,
>>>>> +                              <GIC_SPI 133 IRQ_TYPE_LEVEL_HIGH>;
>>>>> +                 clocks = <&cpg CPG_MOD 122>;
>>>>> +                 clock-names = "fck";
>>>>> +                 power-domains = <&sysc R8A77970_PD_ALWAYS_ON>;
>>>>> +                 resets = <&cpg 122>;
>>>>> +                 status = "disabled";
>>>>> +         };
>>>>> +
>>>>> +         tmu4: timer@ffc00000 {
>>>>> +                 compatible = "renesas,tmu-r8a7797", "renesas,tmu";
>>>>> +                 reg = <0 0xffc00000 0 0x30>;
>>>>> +                 interrupts = <GIC_SPI 116 IRQ_TYPE_LEVEL_HIGH>,
>>>>> +                              <GIC_SPI 117 IRQ_TYPE_LEVEL_HIGH>,
>>>>> +                              <GIC_SPI 118 IRQ_TYPE_LEVEL_HIGH>,
>>>>> +                              <GIC_SPI 369 IRQ_TYPE_LEVEL_HIGH>;
>>>>
>>>> Should GIC_SPI 369 for TMU 4 channel 3 be present not here for
>>>> the r8a77970 (V3M) but rather below for the r8a77980 (V3H) ?
>>>
>>>    I don't think it should be pesent in either place, and I thought I had 
>>> removed
>>> the 4th IRQ from every node before posting... :-/
>>>
>>>> As per my note above, the documentation seems inconsistent here.
>>>
>>>    Yes.
>>
>> Lets go with no 4th IRQ anywhere :)
> 
>    After having studied the manual, 4th IRQ might have sometging to do with 
> the input capture channel capability which uses an extra IRQ output.
> 
>> Could you send an updated patch?
> 
>    Sure. I'll verify and repost.

   No, the extra IRQ doesn't match the existing of the input capture hardware.

MBR, Sergei

Reply via email to