Skip to content

Add system power state notification support for PD controllers - #770

Merged
asasine merged 9 commits into
OpenDevicePartnership:mainfrom
jibz5:personal/jcutlerywala/addpowerstatereg_new
Apr 1, 2026
Merged

asasine merged 9 commits into
OpenDevicePartnership:mainfrom
jibz5:personal/jcutlerywala/addpowerstatereg_new

Conversation

@jibz5

@jibz5 jibz5 commented Mar 31, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Add SystemPowerState enum to controller module
  • Add set_power_state() method to Controller trait
  • Add SetSystemPowerState command to internal and external APIs
  • Implement set_power_state() for TPS6699x driver
  • Add mock controller implementation

Dependencies

  • Requires tps6699x with Sx App Config register support

@jibz5
jibz5 marked this pull request as ready for review March 31, 2026 21:02
@jibz5
jibz5 requested review from a team as code owners March 31, 2026 21:02
Comment thread type-c-service/src/driver/tps6699x.rs Resolved
Comment thread type-c-service/src/driver/tps6699x.rs Resolved
@RobertZ2011

Copy link
Copy Markdown
Contributor

You'll need to do a cargo update tps6699x in both of the example directory as those are separate binaries from the perspective of cargo. You'll also need to implement the new function in mock_controller.rs.

@RobertZ2011
RobertZ2011 requested a review from asasine March 31, 2026 23:04
RobertZ2011
RobertZ2011 previously approved these changes Mar 31, 2026
@asasine
asasine enabled auto-merge (squash) April 1, 2026 20:40
@asasine
asasine merged commit 7faa53f into OpenDevicePartnership:main Apr 1, 2026
15 checks passed
@jibz5
jibz5 deleted the personal/jcutlerywala/addpowerstatereg_new branch April 1, 2026 20:40
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