-
Notifications
You must be signed in to change notification settings - Fork 239
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
[controller] add Thread controller interface for unified APIs #2304
[controller] add Thread controller interface for unified APIs #2304
Conversation
1bb443b
to
d3b7f3b
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2304 +/- ##
==========================================
- Coverage 55.77% 55.71% -0.07%
==========================================
Files 87 5 -82
Lines 6890 70 -6820
Branches 0 2 +2
==========================================
- Hits 3843 39 -3804
+ Misses 3047 29 -3018
- Partials 0 2 +2 ☔ View full report in Codecov by Sentry. |
c43481e
to
990f94b
Compare
990f94b
to
280b1f6
Compare
280b1f6
to
2b4418c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
2b4418c
to
8d63db7
Compare
8d63db7
to
390da55
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
This PR defines the Thread Controller Interface which is intended to
provide a set of unified, async thread control APIs for both NCP and
RCP cases.
This PR is a child PR of #2283. In this PR, only
GetDeviceRole
isadded.
HostRcp
implemented the interface and the dbus handler isupdated to directly call this unified API.