Skip to content
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

[nrf fromlist] [nrfconnect] Add kconfig for enabling Read Client #368

Merged

Conversation

ArekBalysNordic
Copy link
Contributor

To save some FLASH and RAM space we can disable Read Client functionality in nrfconnect samples.
This commit adds kconfig to enable or disable Read Client in the Interaction Model.

Copy link
Contributor

@markaj-nordic markaj-nordic left a comment

Choose a reason for hiding this comment

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

Does this affect dynamic endpoints with client clusters as well?

@ArekBalysNordic
Copy link
Contributor Author

ArekBalysNordic commented Dec 18, 2023

Does this affect dynamic endpoints with client clusters as well?

In Matter Bridge we use bindings, so we need to enable it there anyway.

@markaj-nordic
Copy link
Contributor

markaj-nordic commented Dec 18, 2023

Does this affect dynamic endpoints with client clusters as well?

In Matter Bridge we use bindings, so we need to enable it there anyway.

Yes, this is what I wanted to confirm, if this config enables read client functionality also on dynamic clusters. Thx,

To save some FLASH and RAM memory space we can disable Read Client
functionality in nrfconnect samples.
This commit adds kconfig to enable or disable Read Client
in the Interaction Model.
@ArekBalysNordic ArekBalysNordic merged commit d95eb1d into nrfconnect:master Dec 19, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants