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

manifest: update sdk-connectedhomeip to bring memory profiling #12114

Closed
wants to merge 1 commit into from

Conversation

LuDuda
Copy link
Contributor

@LuDuda LuDuda commented Aug 23, 2023

This PR would be used to evaluate the RAM memory usage.

@github-actions github-actions bot added changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. manifest labels Aug 23, 2023
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 23, 2023

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
matter nrfconnect/sdk-connectedhomeip@478ba85 nrfconnect/sdk-connectedhomeip#316 nrfconnect/sdk-connectedhomeip#316/files

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

@@ -12,6 +12,7 @@ tests:
platform_allow: nrf52840dk_nrf52840 nrf5340dk_nrf5340_cpuapp nrf7002dk_nrf5340_cpuapp
sample.matter.light_switch.debug:
build_only: true
extra_args: CONFIG_CHIP_MEMORY_PROFILING=y
Copy link
Contributor

Choose a reason for hiding this comment

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

PLease add this config to
sample.matter.light_switch.persistent_subscriptions:
configuration too

This commit introduces a new KConfig called CHIP_MEMORY_PROFILING
which can be used to profile the memory usage.

Signed-off-by: Łukasz Duda <lukasz.duda@nordicsemi.no>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. DNM manifest manifest-matter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants