Skip to content

Conversation

@anangl
Copy link
Contributor

@anangl anangl commented Nov 24, 2025

Add missing getting/putting of the device when the RX interrupt is enabled/disabled. Also fix enabling and disabling of the TX interrupt so that the device is got/put only if the interrupt wasn't already enabled/disabled and device PM reference counting is done correctly.

This fixes console_getchar() that would hang when used with CONFIG_PM_DEVICE_RUNTIME=y.

Upstream PR #: 99888

…terrupt driven API

Add missing getting/putting of the device when the RX interrupt is
enabled/disabled. Also fix enabling and disabling of the TX interrupt
so that the device is got/put only if the interrupt wasn't already
enabled/disabled and device PM reference counting is done correctly.

This fixes `console_getchar()` that would hang when used with
`CONFIG_PM_DEVICE_RUNTIME=y`.

Upstream PR #: 99888

Signed-off-by: Andrzej Głąbek <[email protected]>
@anangl anangl force-pushed the fix_int_driven_uarte_runtime_pm_ncs branch from 02f6b4f to b5e2796 Compare November 25, 2025 06:41
@anangl
Copy link
Contributor Author

anangl commented Nov 25, 2025

Rebased.

@ppryga-nordic
Copy link
Contributor

Functionality tested with Zephyr console and sdk-nrf event_trigger samples. Both are working now.

@anangl anangl merged commit 3b0588e into nrfconnect:main Nov 25, 2025
14 checks passed
@anangl anangl deleted the fix_int_driven_uarte_runtime_pm_ncs branch November 25, 2025 14:35
anangl pushed a commit to NordicBuilder/sdk-nrf that referenced this pull request Nov 25, 2025
Automatically created by action-manifest-pr GH action from PR:
nrfconnect/sdk-zephyr#3527

Signed-off-by: Nordic Builder <[email protected]>
anangl pushed a commit to nrfconnect/sdk-nrf that referenced this pull request Nov 26, 2025
Automatically created by action-manifest-pr GH action from PR:
nrfconnect/sdk-zephyr#3527

Signed-off-by: Nordic Builder <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants