tests: drivers: mbox: mbox_data: add support for s32z

[congnguyenhuu]

Added support for S32Z, ping-pong loopback data on a MBOX channel within one core.
The expected received data must match the sent data when running the sample.

The result for tests/drivers/mbox/mbox_data on s32z2xxdc2/s32z270/rtu0:

Running TESTSUITE mbox_data_tests
===================================================================
START - test_ping_pong_1
 PASS - test_ping_pong_1 in 0.001 seconds
===================================================================
START - test_ping_pong_2
 PASS - test_ping_pong_2 in 0.001 seconds
===================================================================
START - test_ping_pong_3
 PASS - test_ping_pong_3 in 0.001 seconds
===================================================================
START - test_ping_pong_4
 PASS - test_ping_pong_4 in 0.001 seconds
===================================================================
TESTSUITE mbox_data_tests succeeded

------ TESTSUITE SUMMARY START ------

SUITE PASS - 100.00% [mbox_data_tests]: pass = 4, fail = 0, skip = 0, total = 4 duration = 0.004 seconds
 - PASS - [mbox_data_tests.test_ping_pong_1] duration = 0.001 seconds
 - PASS - [mbox_data_tests.test_ping_pong_2] duration = 0.001 seconds
 - PASS - [mbox_data_tests.test_ping_pong_3] duration = 0.001 seconds
 - PASS - [mbox_data_tests.test_ping_pong_4] duration = 0.001 seconds

------ TESTSUITE SUMMARY END ------

===================================================================
PROJECT EXECUTION SUCCESSFUL

[congnguyenhuu]

Hello @kartben, Could you please take a look at my PR? Thanks!

[manuargue]

The hardware MRU on SoC S32Z supports ping-pong loopback data on a MBOX channel within one core.

While it's technically possible to send a message from a CPU to itself, that's not a typical or very meaningful use case -especially in the context of a sample. Mailbox peripherals like the MRU are generally intended for inter-CPU communication. This implementation feels more like an internal test case rather than functionality that should be demonstrated in a generic Zephyr sample.

[congnguyenhuu]

While it's technically possible to send a message from a CPU to itself, that's not a typical or very meaningful use case -especially in the context of a sample. Mailbox peripherals like the MRU are generally intended for inter-CPU communication. This implementation feels more like an internal test case rather than functionality that should be demonstrated in a generic Zephyr sample.

Yes, this implementation is internal and specific to the NXP S32 SoC. Therefore, I created an internal test for the NXP S32 SoC instead of doing in the generic Zephyr test.

[manuargue]

Yes, this implementation is internal and specific to the NXP S32 SoC. Therefore, I created an internal test for the NXP S32 SoC instead of doing in the generic Zephyr test.

By internal I really meant, internal, as not on this Github repo :) I fail to see the benefit of having an NXP sample for the mailbox driver where a CPU send msgs to itself. We could have maybe two Zephyr apps exchanging msgs through the mailbox, it would make more sense.

If the mbox maintainers think this is useful, please keep it in the generic sample as you originally did. I can only advise from the NXP S32 pov.

[kartben]

What @manuargue said, basically. It's really not clear why this new sample would be needed.

[congnguyenhuu]

Hello @kartben, @manuargue, I have updated the change to remove the sample I created. I retained support for tests/drivers/mbox/mbox_data and this test is modified to support loopback test.

[manuargue]

there's no such thing as loopback for the mailboxes. I'd rather call this TEST_SINGLE_CPU or the like, and explain the CPU sends a msg to itself

[congnguyenhuu]

I updated to change the name of kconfig

[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

[congnguyenhuu]

Hello @kartben , could you please revisit this PR? thanks.

addressed