Skip to content

CFU Service Refactor#702

Merged
tullom merged 1 commit intoOpenDevicePartnership:v0.2.0from
tullom:cfu-service-refactor
Feb 10, 2026
Merged

CFU Service Refactor#702
tullom merged 1 commit intoOpenDevicePartnership:v0.2.0from
tullom:cfu-service-refactor

Conversation

@tullom
Copy link
Contributor

@tullom tullom commented Feb 4, 2026

Refactor CFU service and move embedded_services::cfu module to cfu_service

  • Updated task function in cfu-service to accept a CfuClient reference directly.
  • Adjusted buffer and splitter tasks to utilize the new CfuClient for device registration and request handling.
  • Refactored examples

As a consequence of moving CFU data structures out of embedded-service, type-c service needs to take a dependency on cfu-service. Chatted with @RobertZ2011 offline and verified that the eventual goal is to move CFU out of type-c service by abstracting out a firmware update trait that the type-c service will use and the CFU service will impl.

@tullom tullom self-assigned this Feb 4, 2026
@tullom tullom requested review from a team as code owners February 4, 2026 21:15
@tullom tullom added the enhancement New feature or request label Feb 4, 2026
@tullom tullom moved this to In review in Embedded Controller Feb 4, 2026
@tullom tullom force-pushed the cfu-service-refactor branch from 6015bd5 to 128333b Compare February 4, 2026 21:29
@tullom tullom requested a review from RobertZ2011 February 6, 2026 16:46
@tullom tullom force-pushed the cfu-service-refactor branch from 128333b to 09b07aa Compare February 10, 2026 16:41
@tullom tullom enabled auto-merge (squash) February 10, 2026 16:41
@tullom tullom merged commit 930a97c into OpenDevicePartnership:v0.2.0 Feb 10, 2026
15 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in Embedded Controller Feb 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants