NXP S32 introduce support SENT

[congnguyenhuu]

Introduce support Single Edge Nibble Transmission (SENT) driver using peripheral SENT receiver on S32Z. This driver allows to communication (read data) with SENT device.
The test result: SENT-RECEIVER-TESTING-RESULT.pptx

RFC: #83983

[zephyrbot]

The following west manifest projects have changed revision in this Pull Request:

Name	Old Revision	New Revision	Diff
hal_nxp	zephyrproject-rtos/hal_nxp@d0e49d6	zephyrproject-rtos/hal_nxp#455	zephyrproject-rtos/hal_nxp#455/files

⛔ DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

[manuargue]

@congnguyenhuu I'd suggest you to open an RFC (request for comments) and link it to this pr in order to involve community to review the proposal and reach consensus: https://docs.zephyrproject.org/latest/contribute/proposals_and_rfcs.html

[Copilot]

Pull Request Overview

This pull request introduces support for the SENT driver using the peripheral SENT receiver on S32Z hardware. Key changes include adding new dt-binding macros and YAML files for SENT configuration, developing a dedicated SENT driver API with callback registration, and updating board and MAINTAINERS files to integrate and support the new driver.

Reviewed Changes

Copilot reviewed 21 out of 32 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
include/zephyr/dt-bindings/saej2716/saej2716.h	Added macro definitions for CRC and binding constants
include/zephyr/drivers/saej2716/saej2716.h	Introduced the SENT driver API and callback definitions
dts/bindings/saej2716/saej2716-controller.yaml	Defined YAML configurations for the SENT controller binding
dts/bindings/saej2716/nxp,s32-saej2716.yaml	Provided YAML for SENT receiver controller configuration
boards/nxp/s32z2xxdc2/s32z2xxdc2_s32z270_rtu1_D.yaml	Updated board configuration to include support for saej2716
boards/nxp/s32z2xxdc2/s32z2xxdc2_s32z270_rtu1.yaml	Updated board configuration to include support for saej2716
boards/nxp/s32z2xxdc2/s32z2xxdc2_s32z270_rtu0_D.yaml	Updated board configuration to include support for saej2716
boards/nxp/s32z2xxdc2/s32z2xxdc2_s32z270_rtu0.yaml	Updated board configuration to include support for saej2716
MAINTAINERS.yml	Added maintainer and collaborator details along with relevant labels

Files not reviewed (11)

• doc/hardware/peripherals/index.rst: Language not supported
• doc/hardware/peripherals/saej2716.rst: Language not supported
• drivers/CMakeLists.txt: Language not supported
• drivers/Kconfig: Language not supported
• drivers/saej2716/CMakeLists.txt: Language not supported
• drivers/saej2716/Kconfig: Language not supported
• drivers/saej2716/Kconfig.nxp_s32: Language not supported
• dts/arm/nxp/nxp_s32z27x_r52.dtsi: Language not supported
• samples/drivers/saej2716/CMakeLists.txt: Language not supported
• samples/drivers/saej2716/README.rst: Language not supported
• samples/drivers/saej2716/boards/s32z2xxdc2_s32z270_rtu0.overlay: Language not supported

Comments suppressed due to low confidence (1)

include/zephyr/drivers/saej2716/saej2716.h:78

• [nitpick] Consider renaming the parameter 'num_frame' to 'num_frames' to better reflect that it represents a count.

enum saej2716_status status, uint32_t num_frame,

[congnguyenhuu]

I have updated to the v2 changes:

• combined property crc-status-nibble-include to property fast-crc due to this only apply for fast frame CRC
• Added require true for mandatory properties

[congnguyenhuu]

I have updated to the v3 changes: added the check for the valid value of the binding properties fast-crc and short-serial-crc

[Dat-NguyenDuy]

Looks fine from my side

[rruuaanng]

Why isn't it called SNET?

[manuargue]

I commented on the ambiguity of the previously proposed API names in #80117 (comment) -it was something like sent_rx_- and one of the proposals that was picked is to use the standard name as a prefix, similarly as done with IEEE802154, IEEE80211*. Although it removes the ambiguety, I acknowledge this protocol is more commonly known as SENT.

[manuargue]

hi @carlescufi , @henrikbrixandersen . Can we remove the architecture review label or is there anything else to discuss? Thanks

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud