Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

samples: drivers: mbox: Disable UART Flow Control on nrf54h20 #75309

Merged

Conversation

nordic-segl
Copy link
Contributor

@nordic-segl nordic-segl commented Jul 2, 2024

Nrf54h20dk has UART Flow Control enabled by default on all cores.
It's a source of problem when mbox sample is run in Twister.
Twister opens serial port only on host core, thus nobody confirms reception of logs from remote core.
As a result, remote core stucks on printing boot banner.

Twister fails sample due to the timeout while waiting for messages from remote to host.

Disable Flow Control on remote core when mbox sample is executed on nrf54h20.

@nordic-segl
Copy link
Contributor Author

FYI @nordic-piks , @anangl , @jaz1-nordic

@nordic-segl nordic-segl force-pushed the NRFX-NONE_Improve-mbox-on-nrf54h20 branch from 2646b3a to 5fe13c1 Compare July 2, 2024 09:18
@nordic-segl nordic-segl force-pushed the NRFX-NONE_Improve-mbox-on-nrf54h20 branch from 5fe13c1 to c5cc868 Compare July 4, 2024 06:07
@nordic-segl
Copy link
Contributor Author

Dear reviewers, please find some time to do the review.

jaz1-nordic
jaz1-nordic previously approved these changes Jul 11, 2024
nordic-segl added a commit to nordic-segl/sdk-zephyr that referenced this pull request Jul 15, 2024
…rf54h20

Nrf54h20dk has UART Flow Control enabled by default on all cores.
It's a source of problem when mbox sample is run in Twister.
Twister opens serial port only on host core,
thus nobody confirms reception of logs from remote core.
As a result, remote core stucks on printing boot banner.

Twister fails sample due to the timeout while waiting for messages
from remote to host.

Disable Flow Control on remote core when mbox sample is executed
on nrf54h20.

Upstream PR: zephyrproject-rtos/zephyr#75309

Signed-off-by: Sebastian Głąb <[email protected]>
rlubos pushed a commit to nrfconnect/sdk-zephyr that referenced this pull request Jul 15, 2024
…rf54h20

Nrf54h20dk has UART Flow Control enabled by default on all cores.
It's a source of problem when mbox sample is run in Twister.
Twister opens serial port only on host core,
thus nobody confirms reception of logs from remote core.
As a result, remote core stucks on printing boot banner.

Twister fails sample due to the timeout while waiting for messages
from remote to host.

Disable Flow Control on remote core when mbox sample is executed
on nrf54h20.

Upstream PR: zephyrproject-rtos/zephyr#75309

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl nordic-segl force-pushed the NRFX-NONE_Improve-mbox-on-nrf54h20 branch from c5cc868 to 867bb00 Compare July 16, 2024 13:27
@nordic-segl nordic-segl force-pushed the NRFX-NONE_Improve-mbox-on-nrf54h20 branch from 867bb00 to 1c82233 Compare July 16, 2024 13:38
nordic-segl added a commit to nordic-segl/sdk-zephyr that referenced this pull request Jul 18, 2024
…rf54h20 and nrf54l15

Nrf54h20dk and nrf54l15 have UART Flow Control enabled
by default on all cores.
It's a source of problem when mbox sample is run in Twister.
Twister opens serial port only on host core,
thus nobody confirms reception of logs from remote core.
As a result, remote core stucks on printing boot banner.

Twister fails sample due to the timeout while waiting for messages
from remote to host.

Disable Flow Control on remote core when mbox sample is executed
on nrf54h20 or nrf54l15.

Upstream PR: zephyrproject-rtos/zephyr#75309

Signed-off-by: Sebastian Głąb <[email protected]>
rlubos pushed a commit to nrfconnect/sdk-zephyr that referenced this pull request Jul 18, 2024
…rf54h20 and nrf54l15

Nrf54h20dk and nrf54l15 have UART Flow Control enabled
by default on all cores.
It's a source of problem when mbox sample is run in Twister.
Twister opens serial port only on host core,
thus nobody confirms reception of logs from remote core.
As a result, remote core stucks on printing boot banner.

Twister fails sample due to the timeout while waiting for messages
from remote to host.

Disable Flow Control on remote core when mbox sample is executed
on nrf54h20 or nrf54l15.

Upstream PR: zephyrproject-rtos/zephyr#75309

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl nordic-segl force-pushed the NRFX-NONE_Improve-mbox-on-nrf54h20 branch from 1c82233 to 920dc23 Compare August 1, 2024 08:56
…54l15

Nrf54h20dk and nrf54l15 have UART Flow Control enabled
by default on all cores.
It's a source of problem when mbox sample is run in Twister.
Twister opens serial port only on host core,
thus nobody confirms reception of logs from remote core.
As a result, remote core stucks on printing boot banner.

Twister fails sample due to the timeout while waiting for messages
from remote to host.

Disable Flow Control on remote core when mbox sample is executed
on nrf54h20 or nrf54l15.

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl nordic-segl force-pushed the NRFX-NONE_Improve-mbox-on-nrf54h20 branch from 920dc23 to 9623a53 Compare August 19, 2024 06:34
@nordic-segl
Copy link
Contributor Author

rebased on main@ad3e941ad3ed9e3e2f9032b0151558a948e99fb9

@nashif nashif merged commit 4a9ca27 into zephyrproject-rtos:main Aug 19, 2024
17 checks passed
@nordic-segl nordic-segl deleted the NRFX-NONE_Improve-mbox-on-nrf54h20 branch August 20, 2024 05:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants