Updating SDK pointer to CSA Master #4
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…387) Bumps [cryptography](https://github.com/pyca/cryptography) from 41.0.4 to 43.0.1. - [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst) - [Commits](pyca/cryptography@41.0.4...43.0.1) --- updated-dependencies: - dependency-name: cryptography dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrei Litvin <andy314@gmail.com>
…. (#35344) * add commissionable device type config * Restyled by autopep8 * update the function logic --------- Co-authored-by: Restyled.io <commits@restyled.io>
* Clean up of logs and added checks * Apply suggestions from code review Co-authored-by: Junior Martinez <67972863+jmartinez-silabs@users.noreply.github.com> * Added changes from code review * Replace with chip::platform memory functions * Reduce logs * chore: Fix logging statements and improve error handling in BaseApplication.cpp * Sanitize provisioning WiFi network credentials * Sanitize SSID and SSID length * use CodeUtils min functions * refactor: add void parameter to functions * refactor: fix logging format in wfx_rsi_join_fail_cb and wfx_rsi_init functions * Simplify SSID processing and improve memory handling --------- Co-authored-by: brosahay <3526930+brosahay@users.noreply.github.com> Co-authored-by: Junior Martinez <67972863+jmartinez-silabs@users.noreply.github.com>
* Add matter_enable_recommended meta-setting
This setting defaults to true which leaves current behavior unchanged.
However it can be set to false to achieve a more conservative / minimal set of
defaults, without having to manually disable an ever-increasing set of
features. Consider using this setting as the default value for other settings
that increase code size or add debugging / tracing or similar features that are
not always desired. Especially settings that default to true on Linux and Mac
("because you're probably building for development") should likely use this
option for their default, e.g.
matter_foo = matter_enable_recommended && (current_os == "linux" || current_os == "mac")
* Add more documentation and turn off the setting for minimal CI builds
* Move matter_enable_recommended into /config/recommended.gni
This allows it to be uesd from args.gni (i.e. in the context of a default_args
scope, where variables like current_os are not defined) as well as within a
build / toolchain context.
* Increase the maximum KVS entries our implementation can handle * add gn argument to configure the nvm3 max object size
…nd all the devices. (#35515)
People keep forgetting ATTRIBUTE_MASK_WRITABLE here, then being confused why writes do not work.
* Add DeviceTypeIdToText and it's usage * Update Zap-generated files * Minor ZAP fixes * Update Zap-generated files * Revert empty line * Update missing Zap-generated files
…on matter queue (#35498)
If a test hangs, we end up running until job timeout after 6 hours. But that skips later steps, including log uploads, which makes it harder to figure out why the hang happened. The fix is to use a timeout on the test job that is somewhat shorter than the 6-hour job timeout, so we allow time for the log upload too.
If a resume happened before the async work from a suspend finished running on the Matter queue, we could deadlock: the Matter queue would be holding the device lock and try to acquire the lock to read "suspended" from the controller, while the controller would be holding the "suspended" lock and trying to acquire the device lock as part of synchronous resume work. The fix is to: 1) Stop trying to read the suspended state of the controller in MTRDevice_Concrete except during MTRDevice_Concrete initialization. Instead, store the suspended state directly in MTRDevice_Concrete at init time and update it when the controller notifies about suspend/resume. 2) Fix controller suspend/resume to guarantee that we create the device list snapshot and set our suspended state in one atomic operation, so that devices are either created before that op (and then end up in the list) or after it (and then pick up the new controller suspended state. 3) Remove the locking around controller suspend/resume, because it's proving hard to reason about. This will be replaced by something better once we figure out how threadsafety should work in controller API in general. In the meantime, suspend/resume on a given controller should only be done on a single serial queue. 4) Use an atomic for the suspended state, so reading it can be done from any thread/queue even without locking.
…444) * Updated the service area documentation to encourage the use of CopyCharSpanToMutableCharSpanWithTruncation for memory safety. Updated the rvc-app example to use this method. * Restyled by whitespace --------- Co-authored-by: Restyled.io <commits@restyled.io>
* [nxp][platform][common] Add BLE related headers in common area Signed-off-by: marius-alex-tache <marius.tache@nxp.com> (cherry picked from commit f67f7a16e42127badff01b8af84543810b81b3d7) * [nxp][examples][common] Add mcxw71/k32w1 references in common readme Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * [nxp][examples][mcxw71] Add lighting-app and contact-sensor-app Signed-off-by: marius-alex-tache <marius.tache@nxp.com> [nxp noup][examples][mcxw71] Disable status led from BUILD.gn * CHIP_DEVICE_CONFIG_ENABLE_OTA_REQUESTOR is not propagated in examples folder causing LED_MANAGER_ENABLE_STATUS_LED to always be se to 1. This makes the OTA fail due to status led and ext flash CS pin being wired together. Signed-off-by: Andrei Menzopol <andrei.menzopol@nxp.com> (cherry picked from commit 188301ab68c348c6ff2c0207b19fc0a7291a1b17) * [nxp][scripts] Add mcxw71 build_examples support Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * [nxp][docs][common] Update manufacturing flow * Update factory data write address to APP section * Add hw_params option info * Add reference command for k32w1 factory data generation * Update k32w1 related paths after removing parent folder k32w Signed-off-by: Andrei Menzopol <andrei.menzopol@nxp.com> (cherry picked from commit a2927b74419ed090595993447c40991befd7d6d6) * Add mcxw71 in workflow Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * [nxp][examples][mcxw71_k32w1] Rename app linker script Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * [nxp][examples][k32w1] Update reference to linker script Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * [nxp][examples][mcxw71] Remove k32w1 references Update instructions related to NBU flash. Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * Restyled by gn * Restyled by prettier-markdown * [nxp] Fix readme misspell errors Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * [nxp] Fix lighting app build gn import order Signed-off-by: marius-alex-tache <marius.tache@nxp.com> --------- Signed-off-by: marius-alex-tache <marius.tache@nxp.com> Co-authored-by: Restyled.io <commits@restyled.io>
* [nxp][platform][common] Simplify factory reset procedure when KeyStorage is used Instead of resetting the KetStorage which, in turn, removes the key files one by one, just shut it down to trigger a FileCache flush and then simply format the file system partition. Signed-off-by: Marian Chereji <marian.chereji@nxp.com> (cherry picked from commit 9a2624656feff58849d89272e73559da19078849) * [nxp][platform][common] Update "NXPConfig.h" to allow building multiple key storage solutions The "NXPConfig.h" header was developed to support 2 key storage solutions so far, which were mostly selected using the CHIP_PLAT_NVM_SUPPORT build symbol. In order to be able to support additional key storage solutions (such as Zephyr NVS), which are no longer selected via the above mentioned build symbol, the header Signed-off-by: Marian Chereji <marian.chereji@nxp.com> (cherry picked from commit 7bace6914508ac98124836a1be3973818993dd58) * [nxp][common] Fix KeyStorage use of illegal buffer size when checking the existence of a key The "NXPConfig::ConfigValueExists" member function is using a non-zero buffer capacity combined with a NULL buffer pointer when calling the KeyStorage KS_GetKeyInt() function. This combination is illegal. When using a NULL buffer pointer, a capacity of zero is required to be provided. Signed-off-by: Marian Chereji <marian.chereji@nxp.com> (cherry picked from commit f8f03ef0041bc8b7729fe7959cf2244b6d65133e) * [nxp][platform][common] Add check for read_bytes_size read_bytes_size can be nullptr according to the KVS API. Add a nullptr check to account for use cases where the parameter is not used (implicitly set to nullptr). Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * [nxp][platform][common] Add support for NVS key storage wear statistics Added the support to initialize and update key storage wear statistics. Signed-off-by: Marian Chereji <marian.chereji@nxp.com> Reviewed-by: Doru-Cristian Gucea <doru-cristian.gucea@nxp.com> Reviewed-by: Martin Girardot <martin.girardot@nxp.com> (cherry picked from commit d05884ddf8baac0da0a4d9d2a693280d2bdae86a) * [nxp][platform][common] Update platform names for w0/w1 Signed-off-by: marius-alex-tache <marius.tache@nxp.com> --------- Signed-off-by: marius-alex-tache <marius.tache@nxp.com> Co-authored-by: Marian Chereji <marian.chereji@nxp.com>
Now that we have a concept of suspended devices, just use that to avoid wasting time on read-throughs which would fail anyway.
* fixing the switch ladder and removing warning flags * Restyled by clang-format * adding unused parameter cflag due to wifi sdk * Restyled by gn * adding the correct warning options * updating the warning flag and only deleting fabrics for wifi * adding the remove services to get rid of the matter multicast service --------- Co-authored-by: Restyled.io <commits@restyled.io>
…BINFO cluster (#35510) --------- Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
* Remove unused file * Fix the documentation * Restyled by prettier-markdown --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com> Co-authored-by: Restyled.io <commits@restyled.io>
…35537) * Moving these to async, as the completions already work when needed * Restyled by clang-format * Adding injection for events/attributes * Missing brace * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>
* Switch python builds to latest CI image * Switch logic for pip updates and wheel installation: do not hardcode paths * More fixes for new python, where pip cannot just be used
* Switch infineon builds to latest CI image * Set up an environment for modus toolbox because image does not have it
* [Darwin] Follow-up changes for PR#35475 * restyled * Additional fixes and unit test fix, and rebased to current master
Bumps [third_party/openthread/repo](https://github.com/openthread/openthread) from `aed9cd1` to `2aeb8b8`. - [Release notes](https://github.com/openthread/openthread/releases) - [Commits](openthread/openthread@aed9cd1...2aeb8b8) --- updated-dependencies: - dependency-name: third_party/openthread/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrei Litvin <andy314@gmail.com>
* getting transitive libraries for static linking * Fixing commit * integrating comments
* Updates labels and step numbers * Restyled by prettier-yaml --------- Co-authored-by: Restyled.io <commits@restyled.io>
github-actions
bot
added
documentation
mykrupp
approved these changes
Sep 16, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.