[v5] add I2C DTS support for S32G2/S32G3 SoCs

[PATCH v5 0/3] add I2C DTS support for S32G2/S32G3 SoCs

Posted by Ciprian Costea 11 months, 1 week ago

From: Ciprian Marian Costea <ciprianmarian.costea@oss.nxp.com>

This patchset aims to add two changes to the S32G2/S32G3 dtsi support:
- Adding I2C dts support for S32G SoC based boards
- Centralize the common part of 'S32G-EVB' and 'S32G-RDB' board revisions
into dtsi files. This refactor will serve I2C in this patchset, but in the
future it will also be used for other modules such as : FlexCAN and DSPI.

Changes in V5:
- Updated several commit titles and descriptions.
- Moved 'reg' dtsi node entry after 'compatible'.
- Squashed commit 3/4.
- Changed 'ina231' dtsi node name to 'current-sensor'.

Changes in V4:
- Moved I2C nodes '#address-size' and '#address-cells' entries from board
common level to S32G2/S32G3 SoC level.

Changes in V3:
- Separated patchset into multiple stages: common 'I2C' dts entries, board
'I2C' dts entries and the introduction of common 'S32GXXXA-EVB/RDB' dtsi.
- Added missing changelog for V2 of this patchset

Changes in V2:
- Moved I2C end device '#address-size' and '#address-cells' entries from
board dts to common 's32gxxxa-evb/rdb' common dtsi.

Ciprian Marian Costea (3):
  arm64: dts: s32g: add I2C[0..2] support for s32g2 and s32g3
  arm64: dts: s32g: add common 'S32G-EVB' and 'S32G-RDB' board support
  arm64: dts: s32g399a-rdb3: Add INA231 sensor entry over I2C4

 arch/arm64/boot/dts/freescale/s32g2.dtsi      |  55 +++++++
 .../arm64/boot/dts/freescale/s32g274a-evb.dts |   1 +
 .../boot/dts/freescale/s32g274a-rdb2.dts      |   1 +
 arch/arm64/boot/dts/freescale/s32g3.dtsi      |  60 +++++++
 .../boot/dts/freescale/s32g399a-rdb3.dts      |   9 ++
 .../boot/dts/freescale/s32gxxxa-evb.dtsi      | 150 ++++++++++++++++++
 .../boot/dts/freescale/s32gxxxa-rdb.dtsi      | 122 ++++++++++++++
 7 files changed, 398 insertions(+)
 create mode 100644 arch/arm64/boot/dts/freescale/s32gxxxa-evb.dtsi
 create mode 100644 arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi

-- 
2.45.2

Re: [PATCH v5 0/3] add I2C DTS support for S32G2/S32G3 SoCs

Posted by Shawn Guo 10 months ago

On Mon, Jan 13, 2025 at 01:05:09PM +0200, Ciprian Costea wrote:
> Ciprian Marian Costea (3):
>   arm64: dts: s32g: add I2C[0..2] support for s32g2 and s32g3
>   arm64: dts: s32g: add common 'S32G-EVB' and 'S32G-RDB' board support
>   arm64: dts: s32g399a-rdb3: Add INA231 sensor entry over I2C4

Applied all, thanks!

Re: [PATCH v5 0/3] add I2C DTS support for S32G2/S32G3 SoCs

Posted by Ciprian Marian Costea 10 months, 2 weeks ago

On 1/13/2025 1:05 PM, Ciprian Costea wrote:
> From: Ciprian Marian Costea <ciprianmarian.costea@oss.nxp.com>
> 
> This patchset aims to add two changes to the S32G2/S32G3 dtsi support:
> - Adding I2C dts support for S32G SoC based boards
> - Centralize the common part of 'S32G-EVB' and 'S32G-RDB' board revisions
> into dtsi files. This refactor will serve I2C in this patchset, but in the
> future it will also be used for other modules such as : FlexCAN and DSPI.
> 
> Changes in V5:
> - Updated several commit titles and descriptions.
> - Moved 'reg' dtsi node entry after 'compatible'.
> - Squashed commit 3/4.
> - Changed 'ina231' dtsi node name to 'current-sensor'.
> 
> Changes in V4:
> - Moved I2C nodes '#address-size' and '#address-cells' entries from board
> common level to S32G2/S32G3 SoC level.
> 
> Changes in V3:
> - Separated patchset into multiple stages: common 'I2C' dts entries, board
> 'I2C' dts entries and the introduction of common 'S32GXXXA-EVB/RDB' dtsi.
> - Added missing changelog for V2 of this patchset
> 
> Changes in V2:
> - Moved I2C end device '#address-size' and '#address-cells' entries from
> board dts to common 's32gxxxa-evb/rdb' common dtsi.
> 
> Ciprian Marian Costea (3):
>    arm64: dts: s32g: add I2C[0..2] support for s32g2 and s32g3
>    arm64: dts: s32g: add common 'S32G-EVB' and 'S32G-RDB' board support
>    arm64: dts: s32g399a-rdb3: Add INA231 sensor entry over I2C4
> 
>   arch/arm64/boot/dts/freescale/s32g2.dtsi      |  55 +++++++
>   .../arm64/boot/dts/freescale/s32g274a-evb.dts |   1 +
>   .../boot/dts/freescale/s32g274a-rdb2.dts      |   1 +
>   arch/arm64/boot/dts/freescale/s32g3.dtsi      |  60 +++++++
>   .../boot/dts/freescale/s32g399a-rdb3.dts      |   9 ++
>   .../boot/dts/freescale/s32gxxxa-evb.dtsi      | 150 ++++++++++++++++++
>   .../boot/dts/freescale/s32gxxxa-rdb.dtsi      | 122 ++++++++++++++
>   7 files changed, 398 insertions(+)
>   create mode 100644 arch/arm64/boot/dts/freescale/s32gxxxa-evb.dtsi
>   create mode 100644 arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi
> 

Hello,

Is there any more feedback on this patchset ? I see that it is hanging 
for a while. Just want to check.

Best Regards,
Ciprian

Re: [PATCH v5 0/3] add I2C DTS support for S32G2/S32G3 SoCs

Posted by Frank Li 10 months, 2 weeks ago

On Tue, Feb 04, 2025 at 12:12:34PM +0200, Ciprian Marian Costea wrote:
> On 1/13/2025 1:05 PM, Ciprian Costea wrote:
> > From: Ciprian Marian Costea <ciprianmarian.costea@oss.nxp.com>
> >
> > This patchset aims to add two changes to the S32G2/S32G3 dtsi support:
> > - Adding I2C dts support for S32G SoC based boards
> > - Centralize the common part of 'S32G-EVB' and 'S32G-RDB' board revisions
> > into dtsi files. This refactor will serve I2C in this patchset, but in the
> > future it will also be used for other modules such as : FlexCAN and DSPI.
> >
...
> >
>
> Hello,
>
> Is there any more feedback on this patchset ? I see that it is hanging for a
> while. Just want to check.

After linus create v6.14-rc1 tag, shawn will take care.

Frank

>
> Best Regards,
> Ciprian