mbox series

[6.1.y-cip,v2,00/12] serial: sh-sci: Add support for RZ/G3E serial driver

Message ID 20250319101629.2407727-1-tommaso.merciai.xr@bp.renesas.com (mailing list archive)
Headers show
Series serial: sh-sci: Add support for RZ/G3E serial driver | expand

Message

Tommaso Merciai March 19, 2025, 10:16 a.m. UTC
Dear All,

This series add support for RZ/G3E SCIF interface to linux-6.1.y-cip kernel.
SCIF interface in Renesas RZ/G3E is similar to the one available in RZ/G2L.
This series add also some fixes found during SoC bring up stage.

All patches are cherry-picked from mainline kernel.

base commit: 8da62141b20b (tag: v6.1.129-cip38, linux-cip/linux-6.1.y-cip)

Thanks & Regards,
Tommaso

Biju Das (1):
  dt-bindings: serial: renesas: Document RZ/G3E (r9a09g047) scif

Claudiu Beznea (4):
  serial: sh-sci: Check if TX data was written to device in .tx_empty()
  serial: sh-sci: Move runtime PM enable to sci_probe_single()
  serial: sh-sci: Clean sci_ports[0] after at earlycon exit
  serial: sh-sci: Increment the runtime usage counter for the earlycon
    device

Geert Uytterhoeven (1):
  serial: sh-sci: Use plain struct copy in early_console_setup()

Lad Prabhakar (5):
  dt-bindings: serial: renesas,scif: Move ref for serial.yaml at the end
  dt-bindings: serial: renesas,scif: Validate 'interrupts' and
    'interrupt-names'
  dt-bindings: serial: renesas,scif: Make 'interrupt-names' property as
    required
  dt-bindings: serial: Add documentation for Renesas RZ/V2H(P)
    (R9A09G057) SCIF support
  serial: sh-sci: Add support for RZ/V2H(P) SoC

Wolfram Sang (1):
  serial: sh-sci: describe locking requirements for invalidating RXDMA

 .../bindings/serial/renesas,scif.yaml         | 141 +++++++++++----
 drivers/tty/serial/sh-sci.c                   | 160 +++++++++++++++---
 include/linux/serial_sci.h                    |   1 +
 3 files changed, 242 insertions(+), 60 deletions(-)

Comments

Pavel Machek March 19, 2025, 10:25 a.m. UTC | #1
Hi!

> This series add support for RZ/G3E SCIF interface to linux-6.1.y-cip kernel.
> SCIF interface in Renesas RZ/G3E is similar to the one available in RZ/G2L.
> This series add also some fixes found during SoC bring up stage.
> 
> All patches are cherry-picked from mainline kernel.

Okay, so I just reviewed v1 of this, and was ready to say that I'll
apply it provided there are no other comments and it passes testing.

Now you'll also have to tell me what is the difference between v1 and
v2 :-).

Best regards,
								Pavel
Pavel Machek March 19, 2025, 10:28 a.m. UTC | #2
Hi!

> > This series add support for RZ/G3E SCIF interface to linux-6.1.y-cip kernel.
> > SCIF interface in Renesas RZ/G3E is similar to the one available in RZ/G2L.
> > This series add also some fixes found during SoC bring up stage.
> > 
> > All patches are cherry-picked from mainline kernel.
> 
> Okay, so I just reviewed v1 of this, and was ready to say that I'll
> apply it provided there are no other comments and it passes testing.
> 
> Now you'll also have to tell me what is the difference between v1 and
> v2 :-).

Aha, missing SoB. I'd expect that information in 0/ of the series, but
you wrote it elsewhere. Still good.

Best regards,
								Pavel
Tommaso Merciai March 19, 2025, 10:32 a.m. UTC | #3
Hi Pavel,
Thanks for your comment.

On Wed, Mar 19, 2025 at 11:28:20AM +0100, Pavel Machek wrote:
> Hi!
> 
> > > This series add support for RZ/G3E SCIF interface to linux-6.1.y-cip kernel.
> > > SCIF interface in Renesas RZ/G3E is similar to the one available in RZ/G2L.
> > > This series add also some fixes found during SoC bring up stage.
> > > 
> > > All patches are cherry-picked from mainline kernel.
> > 
> > Okay, so I just reviewed v1 of this, and was ready to say that I'll
> > apply it provided there are no other comments and it passes testing.
> > 
> > Now you'll also have to tell me what is the difference between v1 and
> > v2 :-).
> 
> Aha, missing SoB. I'd expect that information in 0/ of the series, but
> you wrote it elsewhere. Still good.

Yup sorry, I forgot to update the cover letter :'(
Basically on v1 I missed my SOB.

> 
> Best regards,
> 								Pavel
> -- 
> DENX Software Engineering GmbH,        Managing Director: Erika Unter
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

Thanks & Regards,
Tommaso
Pavel Machek March 20, 2025, 10:02 a.m. UTC | #4
Hi!

> Yup sorry, I forgot to update the cover letter :'(
> Basically on v1 I missed my SOB.

No problem.

For some reason, your patches don't seem to hit the cip-dev mailing
list (?).

It passed the testing so I applied the series.

Best regards,
								Pavel
Tommaso Merciai March 20, 2025, 10:12 a.m. UTC | #5
Hi Pavel,
Thanks for your work.

On Thu, Mar 20, 2025 at 11:02:27AM +0100, Pavel Machek wrote:
> Hi!
> 
> > Yup sorry, I forgot to update the cover letter :'(
> > Basically on v1 I missed my SOB.
> 
> No problem.
> 
> For some reason, your patches don't seem to hit the cip-dev mailing
> list (?).

Thanks for let me know.
Mmmm I'm seeing that on [1].

> 
> It passed the testing so I applied the series.

Thanks for this.
I'm going to prepare next series with a minimal support for RZ/G3E then.

> 
> Best regards,
> 								Pavel
> -- 
> DENX Software Engineering GmbH,        Managing Director: Erika Unter
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

[1] https://patchwork.kernel.org/project/cip-dev/list/?series=945527

Thanks & Regards,
Tommaso
Chris Paterson March 20, 2025, 10:26 a.m. UTC | #6
Hi Pavel,

> From: cip-dev@lists.cip-project.org <cip-dev@lists.cip-project.org> On
> Behalf Of Pavel Machek via lists.cip-project.org
> Sent: 20 March 2025 10:02
> 
> Hi!
> 
> > Yup sorry, I forgot to update the cover letter :'(
> > Basically on v1 I missed my SOB.
> 
> No problem.
> 
> For some reason, your patches don't seem to hit the cip-dev mailing
> list (?).

If it helps, I got the patches emailed via the list and they are in the online archives:
https://lore.kernel.org/cip-dev/

Kind regards, Chris

> 
> It passed the testing so I applied the series.
> 
> Best regards,
> 								Pavel
> --
> DENX Software Engineering GmbH,        Managing Director: Erika Unter
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Pavel Machek March 20, 2025, 11:50 a.m. UTC | #7
Hi!

> Thanks for this.
> I'm going to prepare next series with a minimal support for RZ/G3E then.

Ok, thanks. Please note we'll need 6.12 version at this point, too.

Best regards,
								Pavel
Tommaso Merciai March 20, 2025, 1:14 p.m. UTC | #8
Hi Pavel,

On Thu, Mar 20, 2025 at 12:50:02PM +0100, Pavel Machek wrote:
> Hi!
> 
> > Thanks for this.
> > I'm going to prepare next series with a minimal support for RZ/G3E then.
> 
> Ok, thanks. Please note we'll need 6.12 version at this point, too.

Perfect, thanks for the reminder.

> 
> Best regards,
> 								Pavel
> -- 
> DENX Software Engineering GmbH,        Managing Director: Erika Unter
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

Regards,
Tommaso