mbox series

[6.1.y-cip,00/16] Add RPC-IF support for RZ/G2UL SoC

Message ID 20241206091122.67133-1-prabhakar.mahadev-lad.rj@bp.renesas.com (mailing list archive)
Headers show
Series Add RPC-IF support for RZ/G2UL SoC | expand

Message

Prabhakar Dec. 6, 2024, 9:11 a.m. UTC
Hi All,

This patch series aims to add RPC-IF support for RZ/G2UL SoC and enable it
on SMARC platform, along with enabling tx width to 4 on RZ/G2L SMARC
platforms.

All the patches have been cherry picked from upstream kernel.

Cheers,
Prabhakar

Biju Das (5):
  spi: rpc-if: Add missing MODULE_DEVICE_TABLE
  memory: renesas-rpc-if: Use Hi-Z state as the default setting for IOVF
    pins
  arm64: dts: renesas: rzg2ul-smarc-som: Enable serial NOR flash
  arm64: dts: renesas: rzg2lc-smarc-som: Enable 4-bit tx support
  arm64: dts: renesas: rz{g2l,g2lc}-smarc-som: Update partition table
    for spi-nor flash

Cong Dang (1):
  memory: renesas-rpc-if: Remove redundant division of dummy

Geert Uytterhoeven (2):
  memory: renesas-rpc-if: Always use dev in rpcif_probe()
  memory: renesas-rpc-if: Improve Runtime PM handling

Michael Walle (6):
  mtd: spi-nor: sysfs: hide manufacturer if it is not set
  mtd: spi-nor: remember full JEDEC flash ID
  mtd: spi-nor: move function declaration out of sfdp.h
  mtd: spi-nor: fix select_uniform_erase to skip 0 erase size
  mtd: spi-nor: add generic flash driver
  mtd: spi-nor: sysfs: print JEDEC ID for generic flash driver

Miquel Raynal (1):
  spi: rpc-if: differentiate between unsupported and invalid requests

Wolfram Sang (1):
  memory: renesas-rpc-if: Fix PHYCNT.STRTIM setting

 .../ABI/testing/sysfs-bus-spi-devices-spi-nor |  3 +
 .../boot/dts/renesas/rzg2l-smarc-som.dtsi     | 15 ++--
 .../boot/dts/renesas/rzg2lc-smarc-som.dtsi    | 16 +++--
 .../boot/dts/renesas/rzg2ul-smarc-som.dtsi    | 45 ++++++++++++
 .../boot/dts/renesas/rzfive-smarc-som.dtsi    |  4 ++
 drivers/memory/renesas-rpc-if.c               | 70 ++++++++++++-------
 drivers/mtd/spi-nor/core.c                    | 35 +++++++++-
 drivers/mtd/spi-nor/core.h                    |  3 +
 drivers/mtd/spi-nor/debugfs.c                 |  2 +-
 drivers/mtd/spi-nor/sfdp.c                    | 27 +++++++
 drivers/mtd/spi-nor/sfdp.h                    |  2 -
 drivers/mtd/spi-nor/sysfs.c                   |  8 ++-
 drivers/spi/spi-rpc-if.c                      | 19 +++--
 include/linux/mtd/spi-nor.h                   |  3 +
 14 files changed, 207 insertions(+), 45 deletions(-)

Comments

Pavel Machek Dec. 7, 2024, 5:35 p.m. UTC | #1
Hi!

> This patch series aims to add RPC-IF support for RZ/G2UL SoC and enable it
> on SMARC platform, along with enabling tx width to 4 on RZ/G2L SMARC
> platforms.
> 
> All the patches have been cherry picked from upstream kernel.

It is a bit strange that 5.10 series is smaller than this.

But this looks okay, too, and I can merge it if it passes testing etc.

Best regards,
								Pavel
Prabhakar Dec. 8, 2024, 12:42 p.m. UTC | #2
HI Pavel,

> From: Pavel Machek <pavel@denx.de>
> Sent: 07 December 2024 17:36
> To: Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@bp.renesas.com>
> Cc: cip-dev@lists.cip-project.org; Nobuhiro Iwamatsu
> <nobuhiro1.iwamatsu@toshiba.co.jp>; Pavel Machek <pavel@denx.de>; Biju Das
> <biju.das.jz@bp.renesas.com>
> Subject: Re: [PATCH 6.1.y-cip 00/16] Add RPC-IF support for RZ/G2UL SoC
> 
> Hi!
> 
> > This patch series aims to add RPC-IF support for RZ/G2UL SoC and
> > enable it on SMARC platform, along with enabling tx width to 4 on
> > RZ/G2L SMARC platforms.
> >
> > All the patches have been cherry picked from upstream kernel.
> 
> It is a bit strange that 5.10 series is smaller than this.
>
On the 5.10-cip kernel the generic flash driver lacks lots of features as compared to upstream which cannot be easily backported. So instead choose a simpler approach while backporting to 5.10-cip hence less patches there.

> But this looks okay, too, and I can merge it if it passes testing etc.
> 
Thanks.

Cheers,
Prabhakar
Nobuhiro Iwamatsu Dec. 9, 2024, 6:12 a.m. UTC | #3
Hi all,

> -----Original Message-----
> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> Sent: Friday, December 6, 2024 6:11 PM
> To: cip-dev@lists.cip-project.org; iwamatsu nobuhiro(岩松 信洋 ○DITC□
> DIT○OST) <nobuhiro1.iwamatsu@toshiba.co.jp>; Pavel Machek
> <pavel@denx.de>
> Cc: Biju Das <biju.das.jz@bp.renesas.com>
> Subject: [PATCH 6.1.y-cip 00/16] Add RPC-IF support for RZ/G2UL SoC
> 
> Hi All,
> 
> This patch series aims to add RPC-IF support for RZ/G2UL SoC and enable it
> on SMARC platform, along with enabling tx width to 4 on RZ/G2L SMARC
> platforms.
> 
> All the patches have been cherry picked from upstream kernel.
> 
> Cheers,
> Prabhakar
> 
> Biju Das (5):
>   spi: rpc-if: Add missing MODULE_DEVICE_TABLE
>   memory: renesas-rpc-if: Use Hi-Z state as the default setting for IOVF
>     pins
>   arm64: dts: renesas: rzg2ul-smarc-som: Enable serial NOR flash
>   arm64: dts: renesas: rzg2lc-smarc-som: Enable 4-bit tx support
>   arm64: dts: renesas: rz{g2l,g2lc}-smarc-som: Update partition table
>     for spi-nor flash
> 
> Cong Dang (1):
>   memory: renesas-rpc-if: Remove redundant division of dummy
> 
> Geert Uytterhoeven (2):
>   memory: renesas-rpc-if: Always use dev in rpcif_probe()
>   memory: renesas-rpc-if: Improve Runtime PM handling
> 
> Michael Walle (6):
>   mtd: spi-nor: sysfs: hide manufacturer if it is not set
>   mtd: spi-nor: remember full JEDEC flash ID
>   mtd: spi-nor: move function declaration out of sfdp.h
>   mtd: spi-nor: fix select_uniform_erase to skip 0 erase size
>   mtd: spi-nor: add generic flash driver
>   mtd: spi-nor: sysfs: print JEDEC ID for generic flash driver
> 
> Miquel Raynal (1):
>   spi: rpc-if: differentiate between unsupported and invalid requests
> 
> Wolfram Sang (1):
>   memory: renesas-rpc-if: Fix PHYCNT.STRTIM setting
> 

I reviewed this series. Looks good to me. I can apply this series if there are no other comments.

Reviewed-by: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp>

Best regards,
  Nobuhiro
Nobuhiro Iwamatsu Dec. 10, 2024, 11:57 a.m. UTC | #4
Hi,

> -----Original Message-----
> From: iwamatsu nobuhiro(岩松 信洋 ○DITC□DIT○OST)
> Sent: Monday, December 9, 2024 3:12 PM
> To: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>;
> cip-dev@lists.cip-project.org; Pavel Machek <pavel@denx.de>
> Cc: Biju Das <biju.das.jz@bp.renesas.com>
> Subject: RE: [PATCH 6.1.y-cip 00/16] Add RPC-IF support for RZ/G2UL SoC
> 
> Hi all,
> 
> > -----Original Message-----
> > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> > Sent: Friday, December 6, 2024 6:11 PM
> > To: cip-dev@lists.cip-project.org; iwamatsu nobuhiro(岩松 信洋 ○DITC□
> > DIT○OST) <nobuhiro1.iwamatsu@toshiba.co.jp>; Pavel Machek
> > <pavel@denx.de>
> > Cc: Biju Das <biju.das.jz@bp.renesas.com>
> > Subject: [PATCH 6.1.y-cip 00/16] Add RPC-IF support for RZ/G2UL SoC
> >
> > Hi All,
> >
> > This patch series aims to add RPC-IF support for RZ/G2UL SoC and
> > enable it on SMARC platform, along with enabling tx width to 4 on
> > RZ/G2L SMARC platforms.
> >
> > All the patches have been cherry picked from upstream kernel.
> >
> > Cheers,
> > Prabhakar
> >
> > Biju Das (5):
> >   spi: rpc-if: Add missing MODULE_DEVICE_TABLE
> >   memory: renesas-rpc-if: Use Hi-Z state as the default setting for IOVF
> >     pins
> >   arm64: dts: renesas: rzg2ul-smarc-som: Enable serial NOR flash
> >   arm64: dts: renesas: rzg2lc-smarc-som: Enable 4-bit tx support
> >   arm64: dts: renesas: rz{g2l,g2lc}-smarc-som: Update partition table
> >     for spi-nor flash
> >
> > Cong Dang (1):
> >   memory: renesas-rpc-if: Remove redundant division of dummy
> >
> > Geert Uytterhoeven (2):
> >   memory: renesas-rpc-if: Always use dev in rpcif_probe()
> >   memory: renesas-rpc-if: Improve Runtime PM handling
> >
> > Michael Walle (6):
> >   mtd: spi-nor: sysfs: hide manufacturer if it is not set
> >   mtd: spi-nor: remember full JEDEC flash ID
> >   mtd: spi-nor: move function declaration out of sfdp.h
> >   mtd: spi-nor: fix select_uniform_erase to skip 0 erase size
> >   mtd: spi-nor: add generic flash driver
> >   mtd: spi-nor: sysfs: print JEDEC ID for generic flash driver
> >
> > Miquel Raynal (1):
> >   spi: rpc-if: differentiate between unsupported and invalid requests
> >
> > Wolfram Sang (1):
> >   memory: renesas-rpc-if: Fix PHYCNT.STRTIM setting
> >
> 
> I reviewed this series. Looks good to me. I can apply this series if there are no
> other comments.
> 
> Reviewed-by: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp>

Applied, thanks.

Best regards,
  Nobuhiro