mbox series

[v3,0/5] Add RZ/G2UL MTU3a support

Message ID 20230727081848.100834-1-biju.das.jz@bp.renesas.com
Headers show
Series Add RZ/G2UL MTU3a support | expand

Message

Biju Das July 27, 2023, 8:18 a.m. UTC
This patch series aims to add MTU3a support for RZ/G2UL SMARC EVK.
Also it fixes overflow/underflow interrupt names.

v2->v3:
 * Dropped patch#4, as it accepted for 6.5 fixes.
 * Moved patch#2 to patch#1 as it is fixes patch.
 * Added Rb tag from Geert for patch#1 and patch#3.
 * Updated the link to lore for Closes tag for patch#2.
 * Documented RZ/Five SoC as the same IP used in RZ/G2UL SoC.

v1->v2:
 * Added Ack tags from Conor Dooley for binding patches
 * Updated commit description RZ/G2UL->RZ/{G2UL,Five} for patch#5.
 * Fixed build error reported by kernel test robot by replacing
   GIC_SPI x ->SOC_PERIPHERAL_IRQ(x) for patch#5.

Biju Das (5):
  dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt
    names
  dt-bindings: timer: renesas,rz-mtu3: Improve documentation
  dt-bindings: timer: renesas,rz-mtu3: Document RZ/{G2UL,Five} SoCs
  arm64: dts: renesas: r9a07g043: Add MTU3a node
  arm64: dts: renesas: rzg2ul-smarc: Add support for enabling MTU3

 .../bindings/timer/renesas,rz-mtu3.yaml       | 67 +++++++++---------
 arch/arm64/boot/dts/renesas/r9a07g043.dtsi    | 70 +++++++++++++++++++
 .../boot/dts/renesas/r9a07g043u11-smarc.dts   | 11 +++
 .../dts/renesas/rzg2ul-smarc-pinfunction.dtsi |  6 ++
 arch/arm64/boot/dts/renesas/rzg2ul-smarc.dtsi | 13 ++++
 5 files changed, 134 insertions(+), 33 deletions(-)

Comments

Geert Uytterhoeven July 27, 2023, 12:22 p.m. UTC | #1
On Thu, Jul 27, 2023 at 10:19 AM Biju Das <biju.das.jz@bp.renesas.com> wrote:
> Add MTU3a node to R9A07G043 (RZ/{G2UL,Five}) SoC DTSI.
>
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> ---
> v2->v3:
>  * No change

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in renesas-devel for v6.6.

Gr{oetje,eeting}s,

                        Geert
Geert Uytterhoeven July 27, 2023, 12:22 p.m. UTC | #2
On Thu, Jul 27, 2023 at 10:19 AM Biju Das <biju.das.jz@bp.renesas.com> wrote:
> Add support for PMOD_MTU3 macro to enable MTU3 node on RZ/G2UL SMARC
> EVK.
>
> The MTU3a PWM pins on PMOD0 are muxed with SPI1. Disable SPI1, when
> PMOD_MTU3 macro is enabled.
>
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> ---
> v2->v3:
>  * No change

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in renesas-devel for v6.6.

Gr{oetje,eeting}s,

                        Geert
Conor Dooley Aug. 31, 2023, 4:08 p.m. UTC | #3
On Thu, Jul 27, 2023 at 09:18:43AM +0100, Biju Das wrote:
> This patch series aims to add MTU3a support for RZ/G2UL SMARC EVK.
> Also it fixes overflow/underflow interrupt names.
> 
> v2->v3:
>  * Dropped patch#4, as it accepted for 6.5 fixes.
>  * Moved patch#2 to patch#1 as it is fixes patch.
>  * Added Rb tag from Geert for patch#1 and patch#3.
>  * Updated the link to lore for Closes tag for patch#2.
>  * Documented RZ/Five SoC as the same IP used in RZ/G2UL SoC.
> 
> v1->v2:
>  * Added Ack tags from Conor Dooley for binding patches
>  * Updated commit description RZ/G2UL->RZ/{G2UL,Five} for patch#5.
>  * Fixed build error reported by kernel test robot by replacing
>    GIC_SPI x ->SOC_PERIPHERAL_IRQ(x) for patch#5.
> 
> Biju Das (5):
>   dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt
>     names
>   dt-bindings: timer: renesas,rz-mtu3: Improve documentation
>   dt-bindings: timer: renesas,rz-mtu3: Document RZ/{G2UL,Five} SoCs
>   arm64: dts: renesas: r9a07g043: Add MTU3a node
>   arm64: dts: renesas: rzg2ul-smarc: Add support for enabling MTU3

I'm seeing dtbs_check issues in next & Linus' tree as the binding
patches for this don't seem to have landed.
What's the craic with getting them applied?

Thanks,
Conor.
Biju Das Sept. 18, 2023, 8:04 a.m. UTC | #4
Hi All,

Gentle ping. Are we happy with timer documentation patches
as dts patches hit on 6.6-rc1?

Cheers,
Biju
 
> On Thu, Jul 27, 2023 at 09:18:43AM +0100, Biju Das wrote:
> > This patch series aims to add MTU3a support for RZ/G2UL SMARC EVK.
> > Also it fixes overflow/underflow interrupt names.
> >
> > v2->v3:
> >  * Dropped patch#4, as it accepted for 6.5 fixes.
> >  * Moved patch#2 to patch#1 as it is fixes patch.
> >  * Added Rb tag from Geert for patch#1 and patch#3.
> >  * Updated the link to lore for Closes tag for patch#2.
> >  * Documented RZ/Five SoC as the same IP used in RZ/G2UL SoC.
> >
> > v1->v2:
> >  * Added Ack tags from Conor Dooley for binding patches
> >  * Updated commit description RZ/G2UL->RZ/{G2UL,Five} for patch#5.
> >  * Fixed build error reported by kernel test robot by replacing
> >    GIC_SPI x ->SOC_PERIPHERAL_IRQ(x) for patch#5.
> >
> > Biju Das (5):
> >   dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt
> >     names
> >   dt-bindings: timer: renesas,rz-mtu3: Improve documentation
> >   dt-bindings: timer: renesas,rz-mtu3: Document RZ/{G2UL,Five} SoCs
> >   arm64: dts: renesas: r9a07g043: Add MTU3a node
> >   arm64: dts: renesas: rzg2ul-smarc: Add support for enabling MTU3
> 
> I'm seeing dtbs_check issues in next & Linus' tree as the binding patches
> for this don't seem to have landed.
> What's the craic with getting them applied?
> 
> Thanks,
> Conor.
Biju Das Oct. 9, 2023, 6:54 a.m. UTC | #5
Hi all,

Gentle ping. This patch is in the patch work for a while.
It is acked/reviewed by Conor Dooley and Geert Uytterhoeven.

Can we apply to mainline if you are happy? Or do you want me
to RESEND the patches? Please let me know.

Cheers,
Biju
> 
> > On Thu, Jul 27, 2023 at 09:18:43AM +0100, Biju Das wrote:
> > > This patch series aims to add MTU3a support for RZ/G2UL SMARC EVK.
> > > Also it fixes overflow/underflow interrupt names.
> > >
> > > v2->v3:
> > >  * Dropped patch#4, as it accepted for 6.5 fixes.
> > >  * Moved patch#2 to patch#1 as it is fixes patch.
> > >  * Added Rb tag from Geert for patch#1 and patch#3.
> > >  * Updated the link to lore for Closes tag for patch#2.
> > >  * Documented RZ/Five SoC as the same IP used in RZ/G2UL SoC.
> > >
> > > v1->v2:
> > >  * Added Ack tags from Conor Dooley for binding patches
> > >  * Updated commit description RZ/G2UL->RZ/{G2UL,Five} for patch#5.
> > >  * Fixed build error reported by kernel test robot by replacing
> > >    GIC_SPI x ->SOC_PERIPHERAL_IRQ(x) for patch#5.
> > >
> > > Biju Das (5):
> > >   dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt
> > >     names
> > >   dt-bindings: timer: renesas,rz-mtu3: Improve documentation
> > >   dt-bindings: timer: renesas,rz-mtu3: Document RZ/{G2UL,Five} SoCs
> > >   arm64: dts: renesas: r9a07g043: Add MTU3a node
> > >   arm64: dts: renesas: rzg2ul-smarc: Add support for enabling MTU3
Daniel Lezcano Oct. 9, 2023, 9:52 a.m. UTC | #6
On 09/10/2023 08:54, Biju Das wrote:
> Hi all,
> 
> Gentle ping. This patch is in the patch work for a while.
> It is acked/reviewed by Conor Dooley and Geert Uytterhoeven.
> 
> Can we apply to mainline if you are happy? Or do you want me
> to RESEND the patches? Please let me know.

Krzysztof ?


>>> On Thu, Jul 27, 2023 at 09:18:43AM +0100, Biju Das wrote:
>>>> This patch series aims to add MTU3a support for RZ/G2UL SMARC EVK.
>>>> Also it fixes overflow/underflow interrupt names.
>>>>
>>>> v2->v3:
>>>>   * Dropped patch#4, as it accepted for 6.5 fixes.
>>>>   * Moved patch#2 to patch#1 as it is fixes patch.
>>>>   * Added Rb tag from Geert for patch#1 and patch#3.
>>>>   * Updated the link to lore for Closes tag for patch#2.
>>>>   * Documented RZ/Five SoC as the same IP used in RZ/G2UL SoC.
>>>>
>>>> v1->v2:
>>>>   * Added Ack tags from Conor Dooley for binding patches
>>>>   * Updated commit description RZ/G2UL->RZ/{G2UL,Five} for patch#5.
>>>>   * Fixed build error reported by kernel test robot by replacing
>>>>     GIC_SPI x ->SOC_PERIPHERAL_IRQ(x) for patch#5.
>>>>
>>>> Biju Das (5):
>>>>    dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt
>>>>      names
>>>>    dt-bindings: timer: renesas,rz-mtu3: Improve documentation
>>>>    dt-bindings: timer: renesas,rz-mtu3: Document RZ/{G2UL,Five} SoCs
>>>>    arm64: dts: renesas: r9a07g043: Add MTU3a node
>>>>    arm64: dts: renesas: rzg2ul-smarc: Add support for enabling MTU3
>
Conor Dooley Oct. 9, 2023, 1:59 p.m. UTC | #7
On Mon, Oct 09, 2023 at 11:52:13AM +0200, Daniel Lezcano wrote:
> On 09/10/2023 08:54, Biju Das wrote:
> > Hi all,
> > 
> > Gentle ping. This patch is in the patch work for a while.
> > It is acked/reviewed by Conor Dooley and Geert Uytterhoeven.
> > 
> > Can we apply to mainline if you are happy? Or do you want me
> > to RESEND the patches? Please let me know.
> 
> Krzysztof ?

Daniel ?

(Or for the non-telepathic, what is "Krzysztof ?" supposed to mean?)

Cheers,
Conor.

> > > > On Thu, Jul 27, 2023 at 09:18:43AM +0100, Biju Das wrote:
> > > > > This patch series aims to add MTU3a support for RZ/G2UL SMARC EVK.
> > > > > Also it fixes overflow/underflow interrupt names.
> > > > > 
> > > > > v2->v3:
> > > > >   * Dropped patch#4, as it accepted for 6.5 fixes.
> > > > >   * Moved patch#2 to patch#1 as it is fixes patch.
> > > > >   * Added Rb tag from Geert for patch#1 and patch#3.
> > > > >   * Updated the link to lore for Closes tag for patch#2.
> > > > >   * Documented RZ/Five SoC as the same IP used in RZ/G2UL SoC.
> > > > > 
> > > > > v1->v2:
> > > > >   * Added Ack tags from Conor Dooley for binding patches
> > > > >   * Updated commit description RZ/G2UL->RZ/{G2UL,Five} for patch#5.
> > > > >   * Fixed build error reported by kernel test robot by replacing
> > > > >     GIC_SPI x ->SOC_PERIPHERAL_IRQ(x) for patch#5.
> > > > > 
> > > > > Biju Das (5):
> > > > >    dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt
> > > > >      names
> > > > >    dt-bindings: timer: renesas,rz-mtu3: Improve documentation
> > > > >    dt-bindings: timer: renesas,rz-mtu3: Document RZ/{G2UL,Five} SoCs
> > > > >    arm64: dts: renesas: r9a07g043: Add MTU3a node
> > > > >    arm64: dts: renesas: rzg2ul-smarc: Add support for enabling MTU3
> > 
> 
> -- 
> <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
> 
> Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
> <http://twitter.com/#!/linaroorg> Twitter |
> <http://www.linaro.org/linaro-blog/> Blog
>
Daniel Lezcano Oct. 9, 2023, 2:18 p.m. UTC | #8
On 09/10/2023 15:59, Conor Dooley wrote:
> On Mon, Oct 09, 2023 at 11:52:13AM +0200, Daniel Lezcano wrote:
>> On 09/10/2023 08:54, Biju Das wrote:
>>> Hi all,
>>>
>>> Gentle ping. This patch is in the patch work for a while.
>>> It is acked/reviewed by Conor Dooley and Geert Uytterhoeven.
>>>
>>> Can we apply to mainline if you are happy? Or do you want me
>>> to RESEND the patches? Please let me know.
>>
>> Krzysztof ?
> 
> Daniel ?
> 
> (Or for the non-telepathic, what is "Krzysztof ?" supposed to mean?)

Sorry I missed you were in the DT bindings maintainer list. I was 
expecting Krzysztof tag.

I'll pick the patches now.

Thanks

>>>>> On Thu, Jul 27, 2023 at 09:18:43AM +0100, Biju Das wrote:
>>>>>> This patch series aims to add MTU3a support for RZ/G2UL SMARC EVK.
>>>>>> Also it fixes overflow/underflow interrupt names.
>>>>>>
>>>>>> v2->v3:
>>>>>>    * Dropped patch#4, as it accepted for 6.5 fixes.
>>>>>>    * Moved patch#2 to patch#1 as it is fixes patch.
>>>>>>    * Added Rb tag from Geert for patch#1 and patch#3.
>>>>>>    * Updated the link to lore for Closes tag for patch#2.
>>>>>>    * Documented RZ/Five SoC as the same IP used in RZ/G2UL SoC.
>>>>>>
>>>>>> v1->v2:
>>>>>>    * Added Ack tags from Conor Dooley for binding patches
>>>>>>    * Updated commit description RZ/G2UL->RZ/{G2UL,Five} for patch#5.
>>>>>>    * Fixed build error reported by kernel test robot by replacing
>>>>>>      GIC_SPI x ->SOC_PERIPHERAL_IRQ(x) for patch#5.
>>>>>>
>>>>>> Biju Das (5):
>>>>>>     dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt
>>>>>>       names
>>>>>>     dt-bindings: timer: renesas,rz-mtu3: Improve documentation
>>>>>>     dt-bindings: timer: renesas,rz-mtu3: Document RZ/{G2UL,Five} SoCs
>>>>>>     arm64: dts: renesas: r9a07g043: Add MTU3a node
>>>>>>     arm64: dts: renesas: rzg2ul-smarc: Add support for enabling MTU3
>>>
>>
>> -- 
>> <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
>>
>> Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
>> <http://twitter.com/#!/linaroorg> Twitter |
>> <http://www.linaro.org/linaro-blog/> Blog
>>
Daniel Lezcano Oct. 9, 2023, 2:28 p.m. UTC | #9
On 27/07/2023 10:18, Biju Das wrote:
> This patch series aims to add MTU3a support for RZ/G2UL SMARC EVK.
> Also it fixes overflow/underflow interrupt names.
> 
> v2->v3:
>   * Dropped patch#4, as it accepted for 6.5 fixes.
>   * Moved patch#2 to patch#1 as it is fixes patch.
>   * Added Rb tag from Geert for patch#1 and patch#3.
>   * Updated the link to lore for Closes tag for patch#2.
>   * Documented RZ/Five SoC as the same IP used in RZ/G2UL SoC.
> 
> v1->v2:
>   * Added Ack tags from Conor Dooley for binding patches
>   * Updated commit description RZ/G2UL->RZ/{G2UL,Five} for patch#5.
>   * Fixed build error reported by kernel test robot by replacing
>     GIC_SPI x ->SOC_PERIPHERAL_IRQ(x) for patch#5.
> 
> Biju Das (5):
>    dt-bindings: timer: renesas,rz-mtu3: Fix overflow/underflow interrupt
>      names
>    dt-bindings: timer: renesas,rz-mtu3: Improve documentation
>    dt-bindings: timer: renesas,rz-mtu3: Document RZ/{G2UL,Five} SoCs
>    arm64: dts: renesas: r9a07g043: Add MTU3a node
>    arm64: dts: renesas: rzg2ul-smarc: Add support for enabling MTU3
> 
>   .../bindings/timer/renesas,rz-mtu3.yaml       | 67 +++++++++---------
>   arch/arm64/boot/dts/renesas/r9a07g043.dtsi    | 70 +++++++++++++++++++
>   .../boot/dts/renesas/r9a07g043u11-smarc.dts   | 11 +++
>   .../dts/renesas/rzg2ul-smarc-pinfunction.dtsi |  6 ++
>   arch/arm64/boot/dts/renesas/rzg2ul-smarc.dtsi | 13 ++++
>   5 files changed, 134 insertions(+), 33 deletions(-)
> 

patches 1,2 and 3 applied

Thanks
Conor Dooley Oct. 9, 2023, 4:17 p.m. UTC | #10
On Mon, Oct 09, 2023 at 04:18:13PM +0200, Daniel Lezcano wrote:
> On 09/10/2023 15:59, Conor Dooley wrote:
> > On Mon, Oct 09, 2023 at 11:52:13AM +0200, Daniel Lezcano wrote:
> > > On 09/10/2023 08:54, Biju Das wrote:
> > > > Hi all,
> > > > 
> > > > Gentle ping. This patch is in the patch work for a while.
> > > > It is acked/reviewed by Conor Dooley and Geert Uytterhoeven.
> > > > 
> > > > Can we apply to mainline if you are happy? Or do you want me
> > > > to RESEND the patches? Please let me know.
> > > 
> > > Krzysztof ?
> > 
> > Daniel ?
> > 
> > (Or for the non-telepathic, what is "Krzysztof ?" supposed to mean?)
> 
> Sorry I missed you were in the DT bindings maintainer list. I was expecting
> Krzysztof tag.

Heh, I figured that that is what was going on, but my telepathy skills
were a bit lacking and I was not 100% sure... NW!