Skip to content

Comments

DCD Simulator Controller Driver: Add uninitialize API#247

Merged
fdesbiens merged 2 commits intoeclipse-threadx:devfrom
ayedm1:uninit_dcd_sim
Feb 24, 2026
Merged

DCD Simulator Controller Driver: Add uninitialize API#247
fdesbiens merged 2 commits intoeclipse-threadx:devfrom
ayedm1:uninit_dcd_sim

Conversation

@ayedm1
Copy link
Contributor

@ayedm1 ayedm1 commented Jan 27, 2026

  • Adds _ux_dcd_sim_slave_uninitialize() to free the simulated slave DCD controller instance, clear DCD bindings, and return the DCD to UX_UNUSED (idempotent if already unused).
  • Registers the new uninitialize source in CMakeLists.txt.
  • Updates simulator headers to export ux_dcd_sim_slave_uninitialize and does minor whitespace/style cleanup in the simulator header blocks.

Merging changes ahead of hotfix release 202504a
@ayedm1 ayedm1 moved this to In review in ThreadX Roadmap Jan 27, 2026
@ayedm1 ayedm1 requested review from ABOUSTM and rahmanih January 28, 2026 01:13
@fdesbiens fdesbiens changed the base branch from master to dev February 5, 2026 21:39
@rahmanih rahmanih changed the title DCD Simulator Controller Driver: Add uninitializes API DCD Simulator Controller Driver: Add uninitialize API Feb 6, 2026
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this file is part of the HCD simulator.
please move to separate commit.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

update done

- Adds _ux_dcd_sim_slave_uninitialize() to free the simulated slave DCD controller instance, clear DCD bindings, and return the DCD to UX_UNUSED (idempotent if already unused).
- Registers the new uninitialize source in CMakeLists.txt.
- Updates simulator headers to export ux_dcd_sim_slave_uninitialize (and ux_hcd_sim_host_uninitialize) and does minor whitespace/style cleanup in the simulator header blocks.
@ayedm1 ayedm1 requested a review from rahmanih February 9, 2026 21:30
Copy link
Contributor

@rahmanih rahmanih left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 💯

@fdesbiens fdesbiens merged commit 117ac4f into eclipse-threadx:dev Feb 24, 2026
1 check passed
@github-project-automation github-project-automation bot moved this from In review to Done in ThreadX Roadmap Feb 24, 2026
@ayedm1 ayedm1 deleted the uninit_dcd_sim branch February 24, 2026 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants