mbox series

[v3,0/3] add FlexCAN support for S32G2/S32G3 SoCs

Message ID 20241119080144.4173712-1-ciprianmarian.costea@oss.nxp.com (mailing list archive)
Headers show
Series add FlexCAN support for S32G2/S32G3 SoCs | expand

Message

Ciprian Marian Costea Nov. 19, 2024, 8:01 a.m. UTC
From: Ciprian Marian Costea <ciprianmarian.costea@oss.nxp.com>

S32G2 and S32G3 SoCs share the FlexCAN module with i.MX SoCs, with some
hardware integration particularities.

Main difference covered by this patchset relates to interrupt management.
On S32G2/S32G3 SoC, there are separate interrupts for state change, bus
errors, MBs 0-7 and MBs 8-127 respectively.

The intent of this patchset is to be upstream'ed on the official Linux
repo [0].

Since S32G2/S32G3 SoCs share the FlexCAN controller with I.MX platforms,
we find value in an allignment on Linux Factory tree [1]. Hence, we are
looking forward to integrate any feedback which you have based on your
expertise on this proposed patchset, before finally submitting upstream
for review.

[0] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/
[1] https://bitbucket.sw.nxp.com/projects/LFAC/repos/linux-nxp/browse

Changes in V3:
- Refactored FlexCan binding documentation changes
- Rephrased/Clarified some commit messages

Changes in V2:
- Fixed several issues in FlexCan binding documentation

Ciprian Marian Costea (3):
  dt-bindings: can: fsl,flexcan: add S32G2/S32G3 SoC support
  can: flexcan: add NXP S32G2/S32G3 SoC support
  can: flexcan: handle S32G2/S32G3 separate interrupt lines

 .../bindings/net/can/fsl,flexcan.yaml         | 25 +++++++++++++--
 drivers/net/can/flexcan/flexcan-core.c        | 31 +++++++++++++++++++
 drivers/net/can/flexcan/flexcan.h             |  3 ++
 3 files changed, 56 insertions(+), 3 deletions(-)

Comments

Ciprian Marian Costea Nov. 19, 2024, 8:07 a.m. UTC | #1
On 11/19/2024 10:01 AM, Ciprian Costea wrote:
> From: Ciprian Marian Costea <ciprianmarian.costea@oss.nxp.com>
> 
> S32G2 and S32G3 SoCs share the FlexCAN module with i.MX SoCs, with some
> hardware integration particularities.
> 
> Main difference covered by this patchset relates to interrupt management.
> On S32G2/S32G3 SoC, there are separate interrupts for state change, bus
> errors, MBs 0-7 and MBs 8-127 respectively.
> 
> The intent of this patchset is to be upstream'ed on the official Linux
> repo [0].
> 
> Since S32G2/S32G3 SoCs share the FlexCAN controller with I.MX platforms,
> we find value in an allignment on Linux Factory tree [1]. Hence, we are
> looking forward to integrate any feedback which you have based on your
> expertise on this proposed patchset, before finally submitting upstream
> for review.
> 
> [0] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/
> [1] https://bitbucket.sw.nxp.com/projects/LFAC/repos/linux-nxp/browse
> 
> Changes in V3:
> - Refactored FlexCan binding documentation changes
> - Rephrased/Clarified some commit messages
> 
> Changes in V2:
> - Fixed several issues in FlexCan binding documentation
> 
> Ciprian Marian Costea (3):
>    dt-bindings: can: fsl,flexcan: add S32G2/S32G3 SoC support
>    can: flexcan: add NXP S32G2/S32G3 SoC support
>    can: flexcan: handle S32G2/S32G3 separate interrupt lines
> 
>   .../bindings/net/can/fsl,flexcan.yaml         | 25 +++++++++++++--
>   drivers/net/can/flexcan/flexcan-core.c        | 31 +++++++++++++++++++
>   drivers/net/can/flexcan/flexcan.h             |  3 ++
>   3 files changed, 56 insertions(+), 3 deletions(-)
> 

Hello,

Please disregard this patch series. I will send a new one with 
appropriate description.

Ciprian