-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Create an AAI registry for PwRPC services to allow registration of custom AAI classes #37351
base: master
Are you sure you want to change the base?
Conversation
…ites to look for custom AAIs before using datamodel provider
PR #37351: Size comparison from 95d5de5 to a130e5f Full report (67 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
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.
code in src/data-model-providers should not be touched for example development.
Co-authored-by: Tennessee Carmel-Veilleux <[email protected]>
PR #37351: Size comparison from 95d5de5 to 4393084 Full report (67 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #37351: Size comparison from 95d5de5 to a8bcf87 Full report (67 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #37351: Size comparison from 95d5de5 to 95c1d6b Full report (67 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
Chef delegates can now register attribute access classes in
AttributeAccessorRegistry
to serve PwRPC write requests. It is not ideal for PwRPC writes to take the same path as Matter Attribute Writes, as they have no constraint of being spec compliant. This PR adds the support for PwRPC write requests to directly reach custom handlers without going through Matter Attribute Writes pathway.When attribute access registration is not found, it falls back to the old route.
Testing
Added the following custom handler in
chef-air-quality.cpp
Tested with
rootnode_airqualitysensor_e63187f6c9
Send Write via RPC console -
Read value
Output = 3
Logs -
Testing backwards compatibility
Device:
rootnode_airqualitysensor_e63187f6c9
Set MeasuredValue to 421.5
Read MeasuredValue
Output: 421.5
Logs -