merge commit #2

* Split out CHIPErrror into a separate source set. CHIPError.cpp uses ErrorStr.h, so ErrorStr gets moved into the same source set. ErrorStr.cpp did not actually use CodeUtils.h for anything except including ErrorStr.h. The random whitespace changes are because we changed our clang-format version but then did not restyle the tree. * Address review comment.

This is a follow-up to #29447 and should really have been done as part of that PR.

1) Add missing isForCommandPayload=true to declarations of the old command payload names. Otherwise we were ending up with SetUtcTime being marked provisional, which is not backwards-compatible. 2) We need to declare fields in the old-name command payloads with the right availability, since the availability in the super-class is "too new". The implementation then needs to use @dynamic. This is similar to how structs work. 3) SetUTCTime cannot be provisional (and hence unavailable) if SetUtcTime is supposed to be available and inherits from it. This requires exposing SetUTCTime and its fields in the first release where we will update Matter.framework in the Xcode SDK to the setup where we started doing "provisionally available". Items 1 and 2 are handled by using "completeDecl" for both the old and new name.

* Darwin: Refactor MTRAsyncWorkQueue to avoid retain cycles - Pass a completion to readyHandler instead of capturing the work item - Hold the context weakly within the queue and handle context loss - Simplify thread-safety by having clear ownership rules for work items * Darwin: Consolidate MTRAsyncWorkQueue headers The internal header is no longer needed because MTRAsyncWorkQueue is entirely private to the project now. Also tidy up duplicate / batching logic a little. * Darwin: invalidate MTRDevice work queue * Address review comments * Add os_unfair_lock_assert_owner and comments * Introduce MTRAsyncWorkItemRetryCountBase * More tweaks from review

* Fix constexpr complaints on test vectors * Attempt #2 - separate PAAs used in non-test * Things work better when we add all the files * Restyled by whitespace * Restyled by clang-format * add tokenizer * add the gni * add pigweed import * Change filenames * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>

* Replace glib char with GAutoPtr<char> in BLEManagerImpl.cpp * Replace glib char with GAutoPtr<char> in DnssdImpl.cpp * Replace glib char with GAutoPtr<char> in WiFiManager.cpp * Add GSource to GLibTypeDeleter.h * Replace glib GSource with GAutoPtr<GSource> in ChipDeviceScanner.cpp * Replace glib GSource with GAutoPtr<GSource> in DnssdImpl.cpp * Replace glib GSource with GAutoPtr<GSource> in PlatformManagerImpl.cpp * Review update

…ests (#29456) * Remove include_yamltests arg, and install always for python tests * CI fixes * CI fixes

Co-authored-by: weicheng <weichengxu@asrmicro.com>

…ncelled and nil-ed sooner for OnError case (#29500)

* Enabled the RvcClean StartUpMode attribute in the zap and mettar files. * Updated the PICS files to note that this device supports testing the StartUpMode attribute for the RVC Clean cluster. * Refactored the run_all_yaml script. * Updated the testing section of the readme to include the test case related to the RvcClean StartUpMode. * Restyled by prettier-markdown * Restyled by shfmt * Fixed bug that was causing the charging and docked states to remain true once set. * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>

Co-authored-by: Andrei Litvin <andreilitvin@google.com>

This test has a python implementation and the yaml is no longer required. It does nothing anyway.

* [Telink] Disable not used shell modules * [Telink] restore default shell stack size after tests with light-switch

* Start removing enums and structs from top level definitions in the parser * Remove top level structs from matter idl auto-generation * Fix generator type logic * Update test inputs to move global structs * Fix unit tests * Restyle * Remove unused import --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com>

…9372) * [TC-DRLK-2.4] Handle DUT's that do not support USR & PIN features Fixes CHIP-Specifications/chip-test-plans#3461 * Zap regen * Restyled by prettier-yaml --------- Co-authored-by: Restyled.io <commits@restyled.io>

* Fix script issue: 649, 650, 651, 652, 653, 654, Modified tests: TC_ACL_2_10 TC_ACL_2_7 TC_ACL_2_8 TC_DGWIFI_3_2_Simulated TC_FLW_2_1 TC_LUNIT_3_1 TC_RH_2_1 TC_SC_4_1 Automated test: TC_RVCCLEANM_3_3 Semi-automated tests: TC_REFALM_2_1 Fixes issue: 29331 * Auto generated files * The following YAMLs are removed since there are Python script for the testcases. TC-DT-1.1 TC-IDM-10.1 TC-IDM-11.1 TC-SM-1.1 TC-SM-1.2 * Fixes issue: 656 & 658 Modified tests: TC-GRPKEY-2.1 Removed test: TC-LVL-2.3_Simulated * Modified testcase TC-SC-5.1 * Fixes issue: 655, 660 Modified tests: TC-ACL-2.4 TC-ACL-2.5 TC-ACL-2.6 TC-ACL-2.7 TC-SC-5.2 Revert back the DGGEN-2.1 testcase * Modified test FLW-2.1 * Fixes YAML script issue: 664, 668, 671, 672, 673 TC-DGGEN-2.1 TC-TSTAT-1.1 TC-TSUIC-1.1 TC-TSTAT-2.1 TC-WNCV-3.4 TC-WNCV-3.5 Manual tests: TC-REFALM-2.2 TC-REFALM-2.3 * Restyled by whitespace * Fixes issue 984 TC-TCTL-1.1 * Reverted back changes for tests: TC-DGGEN-2.1 TC-REFALM-2.1 TC-RVCCLEANM-3.3 * Modify ciTest.json file * Fix type --------- Co-authored-by: Restyled.io <commits@restyled.io>

Bumps [third_party/libwebsockets/repo](https://github.com/warmcat/libwebsockets) from `7ef2065` to `4144c1e`. - [Commits](warmcat/libwebsockets@7ef2065...4144c1e) --- updated-dependencies: - dependency-name: third_party/libwebsockets/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>

Bumps [third_party/openthread/repo](https://github.com/openthread/openthread) from `f0530af` to `eaa6261`. - [Release notes](https://github.com/openthread/openthread/releases) - [Commits](openthread/openthread@f0530af...eaa6261) --- 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>

Bumps [third_party/mbedtls/repo](https://github.com/ARMmbed/mbedtls) from `b53e630` to `da635ab`. - [Release notes](https://github.com/ARMmbed/mbedtls/releases) - [Commits](Mbed-TLS/mbedtls@b53e630...da635ab) --- updated-dependencies: - dependency-name: third_party/mbedtls/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>

* Start removing split defines and setups for InitDataModelHandler * Restyle * Add EmptyDataModelHandler to Matter.framework. --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com> Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

This should save about 2.5GB of space. Ideally we should not download these in the first place, however I would have to figure out how to set that up. For now tested locally that compile seems to pass because we use the NDK, however CI will fully validate this.

* [Telink] Use common Zephyr heap * [Telink] Added default RF TX power selection * [Telink] Update Telink image to 16 --------- Co-authored-by: Andrii Bilynskyi <andrii.bilynskyi@telink-semi.com> Co-authored-by: Misha Tkachenko <misha.tkachenko@telink-semi.com>

Bumps [third_party/imgui/repo](https://github.com/ocornut/imgui) from `f4790f6` to `a614387`. - [Release notes](https://github.com/ocornut/imgui/releases) - [Commits](ocornut/imgui@f4790f6...a614387) --- updated-dependencies: - dependency-name: third_party/imgui/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>

) * [Darwin] MTRDeviceTests test017 - fix resubscription expectation * Update src/app/InteractionModelEngine.h Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> --------- Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

* [crypto] Extract X509 functions for mbedTLS and PSA PSA crypto backend uses mbedTLS API to implement functions for certificate generation and parsing, so the functions are duplicated between PSA and mbedTLS backends. Extract the functions to reduce the code duplication. * Restyled by whitespace * Restyled by gn * Code review --------- Co-authored-by: Restyled.io <commits@restyled.io>

Bumps [docker/setup-buildx-action](https://github.com/docker/setup-buildx-action) from 2 to 3. - [Release notes](https://github.com/docker/setup-buildx-action/releases) - [Commits](docker/setup-buildx-action@v2...v3) --- updated-dependencies: - dependency-name: docker/setup-buildx-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add Chef RVC device type * Update rootnode_roboticvacuumcleaner_1807ff0c49.matter * Restyled by clang-format * Fix nit in chef-rvc-mode-delegate.* * Rollback previous changes regarding to RvcCleanMode * Move ifdef ZCL_USING_RVC_RUN_MODE_CLUSTER_SERVER to latest line since the RVC_CLEAN_MODE will use global variable in RVC_RUN_MODE --------- Co-authored-by: Restyled.io <commits@restyled.io>

* CastingDiscovery: fix snprintf warning snprintf appends a null terminator, as does strcat, so both buffers need to be upsized by 1 to accommodate that. * ID buffer size was also incorrect * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>

* [NXP] Introducing Matter support for K32W1 Additional information: * SDK: 2.12.5 K32W148-EVK, downloadable from https://mcuxpresso.nxp.com * MCU overview: https://www.nxp.com/products/wireless/multiprotocol-mcus/secure-and-ultra-low-power-mcu-for-matter-over-thread-and-bluetooth-le-5-3:K32W148 Signed-off-by: Doru Gucea <doru-cristian.gucea@nxp.com> Signed-off-by: Marius Tache <marius.tache@nxp.com> * [NXP] Add CI/CV support for K32W1 Signed-off-by: Marius Tache <marius.tache@nxp.com> * [NXP] Bump ot-nxp submodule Signed-off-by: Marius Tache <marius.tache@nxp.com> * [K32W] Add OPENTHREAD_PLATFORM_CORE_CONFIG_FILE define Signed-off-by: Marius Tache <marius.tache@nxp.com> * [K32W1] Fix misspell Signed-off-by: Marius Tache <marius.tache@nxp.com> * Restyled by whitespace * Restyled by clang-format * Restyled by gn * Restyled by prettier-markdown * Restyled by autopep8 * Restyled by isort * [K32W1] Fix lint errors Signed-off-by: Marius Tache <marius.tache@nxp.com> * [K32W1] Removed deprecated class Signed-off-by: Marius Tache <marius.tache@nxp.com> * Restyled by clang-format * [test] Update testdata/all_targets_linux_x64 with K32W1 targets Signed-off-by: Marius Tache <marius.tache@nxp.com> * [K32W] Fix target names in workflow Signed-off-by: Marius Tache <marius.tache@nxp.com> * [K32W1] Remove deprecated commented sections Signed-off-by: Marius Tache <marius.tache@nxp.com> * [K32W1] Add missing # to reference targets in README Signed-off-by: Marius Tache <marius.tache@nxp.com> * [K32W1] Fix spelling errors Signed-off-by: Marius Tache <marius.tache@nxp.com> * [K32W1] Update CSR crypto flag usage Signed-off-by: Marius Tache <marius.tache@nxp.com> * [K32W1] Remove ResetWatermarks empty implementation Signed-off-by: Marius Tache <marius.tache@nxp.com> * Fix typo in device.gni after fixing conflict Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * [K32W1] Remove anchors and backtick some words Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * [K32W] Fix confusing naming in K32W builder Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * [K32W] Remove commented option Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * [K32W] Fix binary extension format Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * Restyled by autopep8 * [K32W] Fix binary name when copying Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * Restyled by clang-format * [K32W1] Fix ErrorStr header inclusion Signed-off-by: marius-alex-tache <marius.tache@nxp.com> --------- Signed-off-by: Doru Gucea <doru-cristian.gucea@nxp.com> Signed-off-by: Marius Tache <marius.tache@nxp.com> Signed-off-by: marius-alex-tache <marius.tache@nxp.com> Co-authored-by: Doru Gucea <doru-cristian.gucea@nxp.com> Co-authored-by: Restyled.io <commits@restyled.io>

* [Chef] Add generic switch device type * Remove MultiPressMax since we don't support MSM We just support LatchingSwitch(LS) bit but not MomentarySwitchMultiPress (MSM) bit. So the MultiPressMax event shouldn't be enabled.

Decouples them from MTRBaseClusters.

* Implement MTRBaseCluster invokes on top of MTRBaseDevice. This gets rid of command bridge code and shares code a lot better. * Address review comment. * Remove some more code.

* Save some flash on events/commands decode * Regenerate ZAP --------- Co-authored-by: tennessee.carmelveilleux@gmail.com <tennessee@google.com>

Fix file paths and update header to be less confusing for newbies

As in #28890 this zap had a client side command on the server that was causing a duplicate in the generated command list.

#29234) * Added a way of activating the report on active mode from icd manager with build flags * Restyled by shfmt * Moved the ICD_REPORT_ON_ENTER_ACTIVE_MODE in the report scheduler implementation * Moved the report on active define to the app BUILD.gn and removed un-necessary #ifdef * Removed preset * Added include AppBuildConfig --------- Co-authored-by: Restyled.io <commits@restyled.io>

* Updates to cluster objects: use direct context tag comparisons * Also optimize code size usage for structure decoding * Completely split struct iterations and use std::variant based returns on an iterator * Another space save: using a common err for all decodes and single macro invocation seems to make code even smaller * One more cleanup return and proper use of first/last to remove unused code on empty structs (mainly command inputs) * Added a comment and a missed auto keyword * Missed some first/last iterations

* Add delay_ms to reboot commands, fix wpg * Fix efr32 build * Restyle * Fix type for boufallolab * Ensure 0 initialization for member variable used in checks * Restyle

…val (#29448) Users can decide now what should happen after removing the last fabric: - Do nothing. - Erase the saved network credentials (Thread/WI-FI). - Erase the saved network credentials and start Bluetooth LE advertising. - Erase the saved network credentials and reboot the device.

…rom code review (#29571)

* Implement the tracing macros using a backend for esp32. - This PR is aimed at addressing the review comments on PR #29318. - This is a draft implementation of the backend. * Move esp32 tracing configs around: better location, integrate in build examples, carry over configurations * More options removals * Remove fixme text * Restrict tracing to light app, fix include paths in main, add ignore for authkey since that should not be checked in * Move dependencies around even more - esp32 now seems to compile with tracing enabled * Ensure tracing is actually enabled when insights is on * Restyle * Made esp32_trace backend in sync with existing version * Made the common include on the tracing config for all esp32 examples * Fixed the CI failure --------- Co-authored-by: Andrei Litvin <andy314@gmail.com>

* Enforced Explicit support in scene server and added default values to scene cluster where enabled * Fixed overwrite in attributes

The CancellableReader provides a clean teardown of an RpcClient reader thread at exit. Switching from using readers and RpcClients directly to context statements makes sure the teardown is done. This change is adapts to a recent change in Pigweed's RPC Python library: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/172051

…in unneeded CIPD package (#29548) * Initial test to confirm issue * Remove rust CIPD package * Changed android workflow to no longer delete rust folder * Changed android smoketest workflow to no longer delete rust folder * Fix new clang tidy error * Address PR comment * Update .github/workflows/full-android.yaml Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> --------- Co-authored-by: Andrei Litvin <andy314@gmail.com> Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

…#29584) * Prevent dirtying the buildenv, make sure things are cleaned * Virtualenv seems to be installed in the buildenv ... use that so we do not need to globally install * Force bash shell because source requires it * Stop generating link map files for the python builds..they just take up space and this saves 2GB

* Increment used endpoint count during incoming connection. Due to this issue, subsequent incoming connections were failing to connect as the value was getting decremented from zero resulting in a check failing. Also, remove duplicate code for freeing an active connection and move it into a separate function. * Update src/transport/raw/TCP.h Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> --------- Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

…nctions that were never used (#29596)

* Update summarize_fail.py * Update recent_fail_summary.yaml * Update summarize_fail.py * Update summarize_fail.py * Update summarize_fail.py * Update summarize_fail.py * Update summarize_fail.py * Update recent_fail_summary.yaml * Update recent_fail_summary.yaml * Create build_fail_defs.yaml * Update build_fail_defs.yaml * Update summarize_fail.py * Update summarize_fail.py * Update summarize_fail.py * Update recent_fail_summary.yaml * Update summarize_fail.py * Update summarize_fail.py * Update recent_fail_summary.yaml * Update summarize_fail.py * Update summarize_fail.py * Update summarize_fail.py * Update summarize_fail.py * Update summarize_fail.py * Update summarize_fail.py * Update recent_fail_summary.yaml * Update recent_fail_summary.yaml * Update summarize_fail.py * Update recent_fail_summary.yaml * Update summarize_fail.py * Update summarize_fail.py * Update recent_fail_summary.yaml * Restyled by prettier-yaml * Restyled by autopep8 * Restyled by isort * Update summarize_fail.py --------- Co-authored-by: Restyled.io <commits@restyled.io>

* Enable -Wundef by default on some platforms. * Fixes #29216 * Fix a bunch more errors. * Restrict -Wundef to where it actually passes CI.

* update cluster * Update Generated files * update tests

* Initial implemenation of Kotlin Matter Controller * Rename to KotlinMatterController * Rename Interactions to InteractionClient * Address review comments * Restyled by gn --------- Co-authored-by: Restyled.io <commits@restyled.io>

* Remove unused defines * Do not leave undefined defines * Add missing include * Remove Tizen from exception

* [Telink] add draft air quality app * [Telink] add air-quality example * [Telink] Clear code * [Telink] add air-quality example to github-CI * [Telink] Adopt to latest master chnages * [Telink] update AppTask of air-quality-sensor-app * [Telink] use new APIs * Restyled by whitespace * Restyled by autopep8 * [Telink] Remove read example due to spelling errors * [Telink] fix spelling errors * [Telink] Fix typo --------- Co-authored-by: UR6LAL <ur6lal@gmail.com> Co-authored-by: Serhii Salamakha <serhii.salamakha@gmail.com> Co-authored-by: Restyled.io <commits@restyled.io>

) * fix(QPG): Enable -Wundef and fix any errors for qpg based builds (Addresses #29591) * Restyled by gn --------- Co-authored-by: Restyled.io <commits@restyled.io>

Fixes #29576

…29625)

* Save flash on generated struct encoding logic Problem: - Structural repetition in the generated code caused needless jumps to be inlined in very large numbers in `Encode` methods, in ways that could be factored-out. This PR: - Changes the generated code to use a proxy encoder class that reduces expanded inlined code, reduces 64 bit arguments usage and maintains correct early return behavior, while generating significantly smaller method bodies. Testing done: - Unit tests pass - Integration tests pass * Restyled by clang-format * Address review comments * Update src/app/data-model/WrappedStructEncoder.h Co-authored-by: Damian Królik <66667989+Damian-Nordic@users.noreply.github.com> * Regen --------- Co-authored-by: tennessee.carmelveilleux@gmail.com <tennessee@google.com> Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Damian Królik <66667989+Damian-Nordic@users.noreply.github.com>

…tionChallenge on Darwin. (#29565) This got broken because we had no tests. Also makes attestationChallenge work when invoking via the MTRDevice/MTRBaseDevice interface, not just MTRCluster/MTRBaseCluster.

* Switching all zap files to enabled only content JIRA: ZAPP-1113 * Updating to the latest zap build JIRA: ZAPP-1113 * Minor cleanup with updating the .zap file JIRA: ZAPP-1113 * Minor cleanup JIRA: ZAPP-1113 * Committing regen to fix regen diff failure now that the vacuum cleaner is actually generating the right code JIRA: ZAPP-1113

* Fix handling of a response that comes after MRP resends end. After #29173 we can get into the following situation: 1. A message is sent. 2. Before we get an ack or response, all MRP retries happen, MRP gives up, but the exchange response timer has not been hit yet. 3. We get an actual response. 4. Because our exchange is marked as having an un-acked message, but the incoming message is not treated as an ack (because the MRP state that would do that has been torn down), we do not clear our "have un-acked message" state and end up discarding the incoming message. The fix is as follows: * Rename things to make it clear that what we really have is "waiting for an ack" state, which in fact _does_ get cleared when we run out of MRP retries, not an "un-acked message" state. * Have a separate state bit for tracking that we ran out of MRP retries on a message we sent. * Address review comment.

* [Telink] Enable -Wundef by default on Zephyr platform * [Telink] Fix builds * [Telink] Fix builds * Restyled by clang-format * [Telink] Fix builds * [Telink] Fix builds * [Telink] add defines * [Telink] add defines * [Telink] add defines * [Telink] clean code * [Telink] clean code * [Telink] clean code * [Telink] fix builds * [Telink] clean code * [Telink] fix builds * [Telink] fix builds * [Telink} Fix builds * [Telink] Fix builds * [Telink] fix builds * Clean nrfconnect config --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Damian Krolik <damian.krolik@nordicsemi.no>

* Add pw_unit_test support to GN * Rename old chip_test_suite template to chip_test_suite_using_nltest It will be removed in the future, when it's no longer needed. * Make new chip_test_suite template, using pw_test instead of chip_test * Rename a few GN targets to be consistent with pigweed Make them end with `.lib` or `.run` instead of `_lib` or `_run`. * Reordered declarations in .gn for readability * Update .gn

… root (#29627) The configure script, intended for external build systems such as openwrt, so far only allowed projects contained within the connectedhomeip source tree. This is the case for CHIP examples, but is NOT viable for actual applications, where connectedhomeip is usually a submodule of the main project. This PR changes `scripts/configure` in a backwards compatible way as follows: - absolute paths specified with `--project` are now supported - relative paths specified with `--project` are still relative to CHIP_ROOT, but are also allowed to point outside (using ../../... type path). Tested with actual build of p44mbrd (https://github.com/plan44/p44mbrd) as openwrt package (https://github.com/plan44/plan44-feed/tree/main/p44mbrd)

…se cluster enums. (#29654)

…ist". (#29666) The typical failure there looks like this: ==29620==ERROR: LeakSanitizer: detected memory leaks Direct leak of 88 byte(s) in 1 object(s) allocated from: #0 0x106396e12 in calloc+0xa2 (libclang_rt.asan_osx_dynamic.dylib:x86_64+0x51e12) #1 0x7ff800dc9789 in map_images_nolock+0x24b (libobjc.A.dylib:x86_64h+0x1789) #2 0x7ff800dc94db in map_images+0x42 (libobjc.A.dylib:x86_64h+0x14db) #3 0x113d721fa in invocation function for block in dyld4::RuntimeState::setObjCNotifiers(void (*)(unsigned int, char const* const*, mach_header const* const*), void (*)(char const*, mach_header const*), void (*)(char const*, mach_header const*))+0x112 (dyld:x86_64+0xf1fa) #4 0x113d6d6c8 in dyld4::RuntimeState::withLoadersReadLock(void () block_pointer)+0x28 (dyld:x86_64+0xa6c8) #5 0x113d720e1 in dyld4::RuntimeState::setObjCNotifiers(void (*)(unsigned int, char const* const*, mach_header const* const*), void (*)(char const*, mach_header const*), void (*)(char const*, mach_header const*))+0x51 (dyld:x86_64+0xf0e1) #6 0x113d85d44 in dyld4::APIs::_dyld_objc_notify_register(void (*)(unsigned int, char const* const*, mach_header const* const*), void (*)(char const*, mach_header const*), void (*)(char const*, mach_header const*))+0x4e (dyld:x86_64+0x22d44) #7 0x7ff800dc9343 in _objc_init+0x4fe (libobjc.A.dylib:x86_64h+0x1343) #8 0x7ff800d83992 in _os_object_init+0xc (libdispatch.dylib:x86_64+0x2992) #9 0x7ff800d911b7 in libdispatch_init+0x136 (libdispatch.dylib:x86_64+0x101b7) #10 0x7ff80bd34894 in libSystem_initializer+0xed (libSystem.B.dylib:x86_64+0x1894) #11 0x113d77e4e in invocation function for block in dyld4::Loader::findAndRunAllInitializers(dyld4::RuntimeState&) const+0xb5 (dyld:x86_64+0x14e4e) #12 0x113d9eaac in invocation function for block in dyld3::MachOAnalyzer::forEachInitializer(Diagnostics&, dyld3::MachOAnalyzer::VMAddrConverter const&, void (unsigned int) block_pointer, void const*) const+0xf1 (dyld:x86_64+0x3baac) #13 0x113d95e25 in invocation function for block in dyld3::MachOFile::forEachSection(void (dyld3::MachOFile::SectionInfo const&, bool, bool&) block_pointer) const+0x22c (dyld:x86_64+0x32e25) #14 0x113d64db2 in dyld3::MachOFile::forEachLoadCommand(Diagnostics&, void (load_command const*, bool&) block_pointer) const+0x80 (dyld:x86_64+0x1db2) #15 0x113d95bb6 in dyld3::MachOFile::forEachSection(void (dyld3::MachOFile::SectionInfo const&, bool, bool&) block_pointer) const+0xb2 (dyld:x86_64+0x32bb6) #16 0x113d9e603 in dyld3::MachOAnalyzer::forEachInitializer(Diagnostics&, dyld3::MachOAnalyzer::VMAddrConverter const&, void (unsigned int) block_pointer, void const*) const+0x1d1 (dyld:x86_64+0x3b603) #17 0x113d77d81 in dyld4::Loader::findAndRunAllInitializers(dyld4::RuntimeState&) const+0x8f (dyld:x86_64+0x14d81) #18 0x113d7e659 in dyld4::PrebuiltLoader::runInitializers(dyld4::RuntimeState&) const+0x1d (dyld:x86_64+0x1b659) #19 0x113d8b76d in dyld4::APIs::runAllInitializersForMain()+0x25 (dyld:x86_64+0x2876d) #20 0x113d6938c in dyld4::prepare(dyld4::APIs&, dyld3::MachOAnalyzer const*)+0xd72 (dyld:x86_64+0x638c) #21 0x113d684e3 in start+0x183 (dyld:x86_64+0x54e3)

Text-only logging does not depend on Span, but binary logging (LogByteSpan) does. And Span.h depends on text-only logging. We were manually breaking the cycle caused by having both logging types in CHIPLogging.h by not including Span.h from CHIPLogging.h and forward-declaring Span instead. This fixes things so that we have a TextOnlyLogging.h (which is used by Span.h, via CodeUtils.h, and does not use Span at all), and a BinaryLogging.h which uses Span and includes Span.h.

Helps to figure out how many are happening in settings that do not capture detail logs.

We currently peform validation against PAA checks in encoding. In decoding, we check for cryptographic validity, but do not perform the same checks againts the data. This commit brings the encoding and decoding into parity.

Bumps [third_party/imgui/repo](https://github.com/ocornut/imgui) from `a614387` to `001f102`. - [Release notes](https://github.com/ocornut/imgui/releases) - [Commits](ocornut/imgui@a614387...001f102) --- updated-dependencies: - dependency-name: third_party/imgui/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>

) * Replace ICDEventManager and its event queue logic by the ICDNotifier (Subject/observer pattern). Tweak some ICDManager logic to accound for SIT with active interval and active threshold of 0 ms * Use a pointer array instead of vector for the notifier subscribers * Restyled by clang-format * Restyled by gn * Gate the check for chiplock with CHIP_STACK_LOCK_TRACKING_ENABLED * Rename a class,functions of the ICDNotifier for clarity. Remove taking the chip stack lock. We will assert if we don't have it (caller responsability). Add briefs and comments. move ICDNotifier implementation to a cpp file * Restyled by whitespace * Fix returned error for ICDNotifier subscribe * Rename ICDSubscriber class to ICDListener, Update comment with Boris's suggestion --------- Co-authored-by: Restyled.io <commits@restyled.io>

It's just a server for logs. Fixes #29660

…9642)

This allows targeting group messages at a group id that's a config define. Fixes #29637

Bumps [third_party/nanopb/repo](https://github.com/nanopb/nanopb) from `f5f65a9` to `a4bbd6a`. - [Commits](nanopb/nanopb@f5f65a9...a4bbd6a) --- updated-dependencies: - dependency-name: third_party/nanopb/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>

It looks like #29509 did not update all ZAP files (maybe some got added after it merged?). This just runs scripts/tools/zap_convert_all.py.

Bumps [third_party/ot-br-posix/repo](https://github.com/openthread/ot-br-posix) from `1576ade` to `2dfff2a`. - [Release notes](https://github.com/openthread/ot-br-posix/releases) - [Commits](openthread/ot-br-posix@1576ade...2dfff2a) --- updated-dependencies: - dependency-name: third_party/ot-br-posix/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>

* Added fix for the BLE power cycle issue * Added restyle changes * Added restyle changes * Added changes for the ble config

Bumps [third_party/openthread/repo](https://github.com/openthread/openthread) from `eaa6261` to `786bd7f`. - [Release notes](https://github.com/openthread/openthread/releases) - [Commits](openthread/openthread@eaa6261...786bd7f) --- 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>

* Align Ballast Configuration XML to spec. Fixes #29669 * Auto-update ZAP files that involve Ballast Configuration. * Regenerate generated code.

* Update bridge app to init network commissioning This change modifies the linux bridge app to initialize the Network Commissioning cluster, so that the FeatureMap of the cluster is set to ethernet. In other Linux example apps, the FeatureMap of the Network Commissioning cluster is set to ethernet, but in the bridge app, the cluster is not initialized, so the FeatureMap is set to thread. The default value of the FeatureMap is thread in zap file of example apps. Signed-off-by: Youngho Yoon <34558998+yhoyoon@users.noreply.github.com> * Move the setup code into ApplicationInit Signed-off-by: Youngho Yoon <34558998+yhoyoon@users.noreply.github.com> --------- Signed-off-by: Youngho Yoon <34558998+yhoyoon@users.noreply.github.com>

* Remove an unused partial Problem: - `src/app/zap-templates/partials/im_command_handler_cluster_commands.zapt` had legacy-preserving code that is now 100% obsolete. The remaining part of it that was used can now be inlined in the primary command handling template. This PR: - Removed the dead partial and updated templates Testing done: - Regenerated ZAP and found no changes to code. * Restyled by prettier-json * Fix build --------- Co-authored-by: tennessee.carmelveilleux@gmail.com <tennessee@google.com> Co-authored-by: Restyled.io <commits@restyled.io>

* Activating scenes yaml tests on CI now that the level control cluster handler is implemented * Restyled by whitespace * Restyled by prettier-yaml * Used G1 instead of raw value where possible, modified TC_S_1_1 to be more spec compliant * Restyled by prettier-yaml * Added comments related to issues and remove un-necessary check * Update src/app/tests/suites/certification/Test_TC_S_1_1.yaml Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * Added access control in yaml 2_3, fixed group key format and values and added bitwise check in 1_1 * Restyled by whitespace * Restyled by prettier-yaml * Modified TC_S_2_2 to pass the Store Scene test now that level Control is enabled * Restyled by prettier-yaml * Fixed .matter for all clusters app --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

…29605) * Add maturity markers to clusters and add unit tests * Add maturity to cluster elements. For now tests for enum only * Add tests for bitmap maturity * Add unit tests for bitmap, event and struct api maturity * Add more tests and support maturity for fields * Add unit tests for event field maturity info * Add api maturity for constants and unit tests for bitmaps in particular * Add unit tests for maturity of enum constants * Update readme about supported syntax * Restyle * Add deprecated maturity support * Undo submodule update * Add stable as a supported keyword and update a unit tst --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com>

…#29568) * Darwin: Improve MTRAsyncWorkQueue logging by tracking unique item ids Also return more information from the batching handler so we can log both partial and full batching from the queue itself. * Avoid unused variable warning / error * Include work item id in various log messages as per review * Tweaks from review * restyle

Also makes it possible to use a variable for nodeId.

* Align ICD nommenclature in the app/icd/ file implementation * Restyled by clang-format * rename TestIcdMonitoringTable.cpp --------- Co-authored-by: Restyled.io <commits@restyled.io>

* Add SMOKECO.M.ManuallyControlledTest PICS * Add top-level PICS * Change top-level PICS --------- Co-authored-by: Hare <renilr0@outlook.com>

…#29704) The .matter changes are expected, with this ZAP update.

…pec. (#29707) No codegen changes (codegen does some case-canonicalization already, and is not affected by this bit), so easy fix.

…9676)

…, InterfaceTypeEnum (#29694) * First update for radio fault enum * Fix compile (I hope) * Fix compilation again * Fix BLE constant as well * Undo submodule update * Remove InterfaceTypeEnum from the list of weak enums * Replace all scoped constants with new names * Also convert all InterfaceTypeEnum calls * Fix compilation * Fix boufallolab build * Better namespace names for all other platforms (fix compile) --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com>

* Align OTA cluster XML with the specification. This fixes the enum names to match. The difference in the ProviderLocation name is being addressed in the spec in CHIP-Specifications/connectedhomeip-spec#7580 * Auto-update ZAP files. * Regenerate generated code. * Address review comment.

The on off cluster on EP2 exposes several attributes that require LT feature, but the feature was not marked.

CHIPMem used things from CodeUtils.h which used logging, but logging might need to allocate memory via Memory::Alloc to actually work. This breaks the cycle (both conceptual and real: we could try to Memory::Alloc to log the fact that Memory::Init had not been called!), by making CHIPMem not use any macros that involve logging, at the cost of not having nice error messages when CHIPMem invariants are violated.

* Add tracing dependencies to pycontroller * Restyle --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com>

* Initial implementaton for kotlin-matter-controller * Address review comments * Use lock.wait(remainingTime) to allow the thread to wait for the remaining time or until it's notified

* Name support is not enforced anymore * Restyled by clang-format * Added value initialisation and check on failure of get featuremap before applying a value for namesupport * Apply suggestions from code review Co-authored-by: mkardous-silabs <84793247+mkardous-silabs@users.noreply.github.com> --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: mkardous-silabs <84793247+mkardous-silabs@users.noreply.github.com>

…dpoint cluster. (#29686) * Add support for parsing "handle command" entries in matter files * Make codegen show commands that are handled for each endpoints --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com>

* Update KeyStore for Check-in keys * fix comments * fix comments

…29557) This significantly reduces the size of the framework and speeds up compilation of MTRBaseClusters.

* Replace glib GError with GAutoPtr<GError> in AdapterIterator * Replace glib GError with GAutoPtr<GError> in ChipDeviceScanner * Replace glib GError with GAutoPtr<GError> in Bluez helper * Replace glib GError with GAutoPtr<GError> in ConnectivityMangaerImpl * Replace glib GVariant with GAutoPtr<GVariant> in ConnectivityManagerImpl * Replace glib GVariant with GAutoPtr<GVariant> in Bluez helper * Replace glib and char with GAutoPtr<char> in Bluez helper * Replace glib and GVariantDict with GAutoPtr<GVariantDict> in Bluez helper - review * Fix badly used MakeUniquePointerReceiver * Review changes * Replace g_variant_dict_lookup_value to g_variant_lookup_value in src/platform/Linux/bluez/Helper.cpp * Add template <> struct GAutoPtrDeleter<const char *> * Convert keyMgmts to GAutoPtr in src/platform/Linux/ConnectivityManagerImpl.cpp * Revert in ConnectionDataBundle GVariant convert to GAutoPtr * Review update - remove GVariantDict * Add errorhandling for keyMgmtsHendle

* changes for wifi sdk 3_1_0 * updating the matter support to point to main * added missed commas * Restyled by clang-format * Restyled by gn --------- Co-authored-by: Restyled.io <commits@restyled.io>

* IDT init * Restyled by whitespace * Restyled by prettier-markdown * Restyled by shellharden * Restyled by shfmt * Restyled by isort * Remove f-strings with no placeholders * Remove unknown Pygments lexer * Spelling * License headers * Restyled by prettier-markdown * First response to PR comments * Restyled by prettier-markdown * Respond Dockerfile and README comments * ble nit, venv script * Make TXT logging better * Respond comments * Clarify single host installation * Respond comments * Respond nits * Cleanup BashRunner * Cleanup BashRunner usage * Respond comments, renames, opts * Nits * Restyled by prettier-markdown * Restyled by shellharden * Restyled by shfmt * Restyled by isort * Spelling * Dynamic loading, comments * README nit * Make README generic, add async, any interface * Restyled by prettier-markdown * Restyled by shellharden * Restyled by shfmt * Restyled by isort * Nits, workaround potential issue with shell input * Restyled by isort * Docs CI --------- Co-authored-by: Restyled.io <commits@restyled.io>

… methods (#29695) * [Darwin] MTRClusters commands have incorrect nullability for no-param methods * Also added _Nullable to interval and source file

Specific changes: * Make MTRDevice get its MTRBaseDevice in a way that is XPC-friendly. * Implement readAttributePaths (which is what ends up getting called by MTRDevice's readAttributeWithEndpointID) over XPC, as long as there is just a single attribute path. * Implement _invokeCommandWithEndpointID (which is what ends up getting called by MTRDevice's _invokeKnownCommandWithEndpointID) over XPC, as long as serverSideProcessingTimeout is nil. * Writes already called an MTRBaseDevice function that was implemented over XPC. * Tests for the new setup; these were checked to fail without the other changes. This also makes MTRClusters work over XPC.

* Updating wiseconnect to 2.8.2 * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>

* [Telink] Set default value of COMMON_LIBC_MALLOC_ARENA_SIZE * [Telink] InitBasicOTARequestor after kDnssdInitialized * [Telink] Move OTAUtil into lib * [Telink] restyled

* add different flash sizes * add different flash sizes in exampe's CMake * add offset argument to mfg_tool * generate factory data for different flash sizes * generate factory data for different flash sizes * generate factory data for different flash sizes * generate factory data for different flash sizes * [Telink] Update CI * [Telink] change path * Restyled by prettier-markdown * [Telink] set correct app name --------- Co-authored-by: Dmytro Kashkarov <ur6lal@gmail.com> Co-authored-by: Restyled.io <commits@restyled.io>

…DKs (#29646) * Adds chages to docker image and build scripts * Adds fix for alignment * Adds bugfix for change directory * Removes changes from other PR * Adds book keeping version checks * Reverts changes to be added after docker update * Adds updated SDK as per release * Updates wiseconnect-wifi-bt-sdk to 2.8.1 * Adds updated docker images * Adds update of WiSeConnect 2 SDK 2.8.2 * Adds requirements file for pip freeze --------- Co-authored-by: Rohan Sahay <rohan.sahay@silabs.com>

* Get additional workflow information about diskspace * Get even more diskspace usage info * Initial attempt at collecting info on disk space within checkout submodule * Fix CI issue * Another attempt to get CI running workflow grabbing diskspace usage * Couple other fixes * Another attempt to get CI running * Try this new approach * quick fix * Dump more info * Few more touch ups * quick fix * Restyle * Restyle * quick fix * quick fix

NsdManagerServiceBrowser.java:51: Error: The WIFI_SERVICE must be looked up on the Application context or memory will leak on devices < Android N. Try changing context to context.getApplicationContext() [WifiManagerPotentialLeak] Apply the recommendation from this check.

* Added changes for build failure * Added changes for the wpa3 changes * Added chnages for macro * restyling changes

* Replace bitmaps and enums * Replace all the rest * Zap regen * Replace Percent with percent in yaml tests * Regenerate zap * More type updates

…load (#29741) * delete file * typechange: integrations/docker/images/stage-2/chip-build-efr32/requirements.txt * new file: requirements.silabs_docker.txt * Add --single-branch flag for leaner clone * Update version

When we set a low value to the minimum subscription interval for subscription for the PartList attribute of the Description cluster, and then remove a dynamic endpoint the related bitmask may change after reporting it to a controller. The controller receives the old number of endpoints and does not update it within the following subscriptions until the next change of the PartList attribute occurs. To resolve the issue the endpoint bitmask should be cleared before reporting the PartList attribute of the Descriptor cluster to avoid race conditions.

This requires that those dependencies be separate things that are not the entire "support" library, because some things need Span but can't depend on all of "support". Fixes #29490

…ering (#29750) Co-authored-by: Andrei Litvin <andreilitvin@google.com>

…ted erro… (#29740) * Clarify a warning trace for brd4186a/87c and cleanup unsupported error message * Apply suggestions from code review Co-authored-by: mkardous-silabs <84793247+mkardous-silabs@users.noreply.github.com> --------- Co-authored-by: mkardous-silabs <84793247+mkardous-silabs@users.noreply.github.com>

This should have happened as part of #29704 but didn't.

* Some cleanup and strips on the chip-build image * Restyle --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com>

* Sync the improvements from kotlin-matter-controller * Increase the timeout value to 3s

…for DataModel::Decode (#29739) Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

* Activated the automated yamls for scenes on CI * Added Scenes test to the darwin framework tool * Disabled test 2_2 for Repl as it used unsupported commands for this test and fixed the yaml for Darwin-framework-tool * Restyled by whitespace * Applied fix for Darwin build * Regenerated Zap * Fixed name support typing for Darwin tests --------- Co-authored-by: Restyled.io <commits@restyled.io>

* lwip: Add patch to fix RA LLADDR processing * Fix weird spacing issue in patch. * Restyled by prettier-markdown * Add lladdr to wordlist --------- Co-authored-by: Restyled.io <commits@restyled.io>

#29764) * Update BasicInformation cluster revision from 1 to 2 in zap configuration * Regen all * google-style kotlin files

…erXPC. (#29767) Because MTRDevice will batch reads, we need to handle multiple paths here.

…ull (#29770)

* Move kotlin lib onboardingpayload to package matter * Restyled by google-java-format --------- Co-authored-by: Restyled.io <commits@restyled.io>

They are not really needed; we should really stop generating them.

* fix build error on bl602 for undef enabled * build pass if undef enabled for bl702 and bl702l * correct factory data return and update pt table number * fix cflags operation * Fix restyle * Remove CONFIG_ENABLE_CHIP_SHELL from CHIPDevicePlatformConfig.h

… combinations (#29414) * Adds DISPLAY_ENABLED macro for SPI multiplexed functions * Adds CONFIG_USE_EXTERNAL_FLASH macro for SPI multiplex functions * Added proper MACROs for the features that require multiplexing * temp commit with test file * Adds fix for crash on rs91x * Adds updated build commands * adds DEBUG logs * Adds bugfixes to MACROs * Adds additional logs * Adds fix to UART multiplexing * Remove comments * Adds ported changes * Adds use of SL_CATALOG_UARTDRV_USART_PRESENT, SL_CATALOG_UARTDRV_EUSART_PRESENT * Removes code since WF200 requires SL_CATALOG_UARTDRV_USART_PRESENT only * Restyler changes * Adds additional changes to wrap multiplexing with MACROs * Remove log commit * Adds dummy buffer length check * Restyler changes * Removes use of unnecessary MACRO * adds thread check * Adds comment * updated build script * Adds refactored includes and define macros * Adds spell fix * Adds feature macro for header include * Remove temp code * Adds refactored deassert * Comment reformatting --------- Co-authored-by: Rohan S <3526930+brosahay@users.noreply.github.com>

* Fixed LCD animations for window-app * Modified type conversion from (uint8_t *) to Uint8::from_char as mentioned in comments * As per comments, using BaseApplication to get provision state and updated UpdateLEDs() API. * Added sIsProvisioned as member variable in BaseApplication class. * Restyled by whitespace * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>

* all-clusters: Fix conformance on door-lock Door lock currently supports features 3, 4, 5, and 6, all of which require the DPS feature. As all the supported pieces for this feature are already on, I turned on the feature (bit 5) Door lock currently support feature 0x33, which requires the COTA feature (bit 7), so I turned this feature on. Door lock currently has on the YDSCH and HDSCH features in the feature map. This looks deliberate, but it requires the commands 0x10, 0x11, 0x12 (both ways) and 0x13, so I turned these commands on. * Turn off YDSCH and HDSCH features and cmds / attrs

Attribute 2 requires the MSM feature, which in turn requires two other features that are not on. Instead, turn off this attribute.

* Remove unnecessary globalRef for android im functions localReferences created by sdk's local functions has not yet been removed and are still valid, so there is no need to create globalRef when it is returned from local functions. * Restyled by clang-format * add JniGlobalReference class with move constructor * Restyled by whitespace * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>

Brings in Swift availability annotations on Darwin and fixes accessor generation for global attributes tagged as AttributeAccessInterface-only.

Bumps [third_party/ot-br-posix/repo](https://github.com/openthread/ot-br-posix) from `2dfff2a` to `636e53d`. - [Release notes](https://github.com/openthread/ot-br-posix/releases) - [Commits](openthread/ot-br-posix@2dfff2a...636e53d) --- updated-dependencies: - dependency-name: third_party/ot-br-posix/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>

Bumps [third_party/libwebsockets/repo](https://github.com/warmcat/libwebsockets) from `4144c1e` to `18fdb0d`. - [Commits](warmcat/libwebsockets@4144c1e...18fdb0d) --- updated-dependencies: - dependency-name: third_party/libwebsockets/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>

* Relocate json-related method to JsonUtilities file * address comment * address comment * Restyled by gn * removed unused and broken MakeJsonSingleLine --------- Co-authored-by: Restyled.io <commits@restyled.io>

* Use latest docker version everywhere * Revert java tests since java will only be available in docker image 21 * Fix type checks according to linter * Fix type checks according to linter * Fix type checks according to linter * Undo some odd awk changes due to script automation * Undo some odd awk changes due to script automation * Undo version setting in dockerfile as that seems wrong * One more isinstance change to make linter happy

* [ASR] Fix -Wundef compile issue * update wifi scan implementation --------- Co-authored-by: weicheng <weichengxu@asrmicro.com>

* lwIP: reference upstream patches for ND6 LLADDR * fix link * add to wordlist

* Install openjdk8 in chip-build-java * Make sure vscode also has java installed

Bumps [third_party/openthread/repo](https://github.com/openthread/openthread) from `786bd7f` to `66eaeec`. - [Release notes](https://github.com/openthread/openthread/releases) - [Commits](openthread/openthread@786bd7f...66eaeec) --- 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>

… cluster (#29708) * virtual-device-app: Add powersource usecase Signed-off-by: Jaehoon You <jaehoon.you@samsung.com> Signed-off-by: Charles Kim <chulspro.kim@samsung.com> * virtual-device-app: Add doorlock usecase Signed-off-by: Jaehoon You <jaehoon.you@samsung.com> Signed-off-by: Charles Kim <chulspro.kim@samsung.com> --------- Signed-off-by: Jaehoon You <jaehoon.you@samsung.com> Signed-off-by: Charles Kim <chulspro.kim@samsung.com>

…9793) * Fixed LED0 issue by doing led init after lcd init. * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>

* Remove media clusters from all-clusters-app Both QA and the CI are using the TV app for all testing of the media clusters. The all-clusters versions are not well tested and have conformance issues. Removing these in all-clusters in favour of the TV app. * Remove media servers from esp32

* Added changes for 917 NCP sleepy build failure and formatting ble config file * restyle of the ble config * Restyling changes * Restyle changes * Added changed for rsi_ble_config * Removed the ble config changes

…use RAM as cache (#29755)

* Move android nameindex implementation * Restyled by whitespace * Restyled by clang-format * fix build error in other platform * Fix darwin build error * Update src/inet/InetInterfaceImpl.h Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

* [Telink] Fix DFU target build conflict * [Telink] Fix app path in CI --------- Co-authored-by: Andrei Litvin <andy314@gmail.com>

* Update Manufacturers DB (from September 2023) Signed-off-by: Markus Becker <markus.becker@tridonic.com> * some fixes and 2 additional VIDs Signed-off-by: Markus Becker <markus.becker@tridonic.com> --------- Signed-off-by: Markus Becker <markus.becker@tridonic.com>

Bumps [third_party/pigweed/repo](https://github.com/google/pigweed) from `e765c4d` to `9504c03`. - [Commits](google/pigweed@e765c4d...9504c03) --- updated-dependencies: - dependency-name: third_party/pigweed/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>

Instead of just saying what tests would run, log the actual commands (more or less) that would run, so that it's easier to figure out how to run tests manually.

* Add build flag for changing firmware build time Adding a default fallback firmware build time. This makes the firmware build more consistent, so you get the same binary unless you specifically ask for the build time to be updated. * Get fallback from from file so it can be updated This lets us update the fallback time via a script using a cron job if we want to. * Restyled by isort * Change filename --------- Co-authored-by: Restyled.io <commits@restyled.io>

* Remove static_assert to match test changes * remove verifyordie for now * comment out variable that is not used * Update src/app/icd/ICDManager.cpp Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * Restyled by clang-format --------- Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> Co-authored-by: Restyled.io <commits@restyled.io>

…29825) * Added fix for echo system * Added fic for echo system * Added restyle changes * Added changes for the restyle

* Update Thermostat cluster version to 6. It was updated in all-clusters-app, but not in the XML or other apps. * Regenerate generated files.

Co-authored-by: Andrei Litvin <andreilitvin@google.com>

…29827)

…d/ PlatformManagerImpl.cpp (#29831)

* Update cluster data with api maturity * Pull in latest zap that supports api maturity * mark scenes provisional, mark unit testing and fault injection as internal * Mark more clusters as provisional * Undo domain move * Undo submodule change * make the scenes domain identical with how it was * Restyle --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com>

…ryFailed (#29836)

The factory data documentation contains the wrong example of the factory data script invocation and it did not allow users to use it.

* Remove unnecessary include in linux * Add nodeps build to linux, to ensure the nodeps target never fails to compil again

* Make IDL templates use lowercase for bitmap and enum types * Zap regen * Undo submodule update --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com>

* Added the TableSize bit to the Scenes feature map and ensured the ScenesTableImpl would get the value from zap * Updated Fabric capacity rule to apply less than half of SceneTableSize and adapted the tests * Generated Zap * Adapted the tests to handle new Capacity in spec and regenerated Darwin tests * Attempt to fix signedness error on Darwin CI * Restyled by prettier-yaml --------- Co-authored-by: Restyled.io <commits@restyled.io>

Since this can delay command invocation, accounts for that in the timed invoke case.

* Fixes manual script issues 676, 682, 684, 685, 686, 690, 691, 692 * Restyled by whitespace * Modified scripts: TC_CNET_4_1 TC_CNET_4_10 TC_CNET_4_11 TC_CNET_4_12 TC_CNET_4_13 TC_CNET_4_14 TC_CNET_4_15 TC_CNET_4_16 TC_CNET_4_2 TC_CNET_4_20 TC_CNET_4_21 TC_CNET_4_22 TC_CNET_4_3 TC_CNET_4_4 TC_CNET_4_5 TC_CNET_4_6 TC_CNET_4_9 TC_DA_1_4 TC_OPSTATE_2_2 TC_WNCV_5_1_Simulated TC_WNCV_6_1_Simulated TC_WNCV_7_1_Simulated * Restyled by whitespace * Fixing CNET manual YAML's * Modified test TC-CNET-4.15 --------- Co-authored-by: Restyled.io <commits@restyled.io>

* Fixes issue: 677, 678, 679, 680, 689 DGWIFI-1.1 BRBINFO-2.1 FAN-1.1 MOD-1.1 ALOGIN-12.1 * Auto generated files * Restyled by whitespace * Fixes issue: 987, 675 TC-AUDIOOUTPUT-1.8 TC-BINFO-2.2 * Restyled by whitespace * Modified tests TC-DRLK-2.1 TC-DRLK-2.10 --------- Co-authored-by: Restyled.io <commits@restyled.io>

…ded. (#29855) If we know the command being invoked needs a timed invoke, and the caller did not provide a timed invoke timeout, go ahead and use a default one.

…ports, we should up this (#29852) * Updating intervals * Update unit tests to still use a 2-second MaxIntervalCeiling. * Lowering this for now * Restyled by clang-format --------- Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> Co-authored-by: Restyled.io <commits@restyled.io>

In some cases, a last fabric removal action should be delayed to avoid race conditions and allow the device to finish all currently running tasks. To control the delay use the CHIP_LAST_FABRIC_REMOVED_ACTION_DELAY kconfig. By default it has been set to 500 ms.

* Add UAT and LIT feature map * generated files

… a value lower than spec minimum (#29849)

…pe (#29859) * Added the RVC operational state enums to the EnumsNotUsedAsTypeInXML. * Generated the zap code following changes to the XMLs.

Python lighting-app was broken (see #29508) and got fixed in #29772. Let's make sure that it does not break again by compiling it in CI. Signed-off-by: Markus Becker <markus.becker@tridonic.com>

* Fix FD leak in case of g_variant_lookup failure * Fix GSource deleter

…9858) * [Bouffalolab] Use Openthread stack under Bouffalo Lab SDK by default * Correct spell * only set openthread_root for thread enabled * Update test data * Fix restyle

* Add data model XMLs and script Data model XMLs are scraped from the spec using the data model tools scraper and the provided script. The SHA of the spec used as the source is stored in the spec_sha file. Data types are not yet included in this script or added in the repo. * Stop parsing non-cluster files and remove * Restyled by isort * Remove unused import * Use the 1.2 tag for initial upload --------- Co-authored-by: Restyled.io <commits@restyled.io>

…nee works after a reboot (#29252)

#29169)

* restructuring low level output api * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>

The current implementation of the door-lock server supports emitting the LockOperationEvent only if the credential has been provided remotely. In case we use a custom handling lock credential in the application code the event is emitted only on the correct credential, but we also need a mechanism that allows the door lock server to handle a lock operation error when the wrong credential has been provided. Added handling local lock operation error to emit LockOperationEvent and call HandleWrongCodeEntry.

…when needed." (#29869)

…ive (#29549) * Add intelligence to the exchange context * Restyled by whitespace * Restyled by clang-format * fix name * Apply suggestions from code review Co-authored-by: Tennessee Carmel-Veilleux <tennessee.carmelveilleux@gmail.com> * address review comments * Apply suggestions from code review Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * address review comments * fix rebase * Refactor changes to simplify and remove req on exchange context consummer * fix comment * Apply suggestions from code review Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Tennessee Carmel-Veilleux <tennessee.carmelveilleux@gmail.com> Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

* Do not use BLE_CONNECTION_OBJECT by internal implementation * Wrap BluezConnection in a class * Store const ref to connection in ConnectionDataBundle * Simplify get connection MTU method * Add proxy method for getting device address * Allocate ConnectionDataBundle on stack * Add destructor to IOChannel * Remove Bluez from public method names * Move IO channels creation to BluezConnection * Fix notify callback setup for additional advertising * Fix FD leak in case of g_variant_lookup failure * Move IO channel handlers to BluezConnection class * Manage IOChannel members with GAutoPtr * Use on-stack array for reading characteristic * Document FD closing behavior by setup methods * Fix GSource deleter

…for the RVC device type (#29862) * Removed the StartUpMode attribute from the RVC ModeBase clusters in chef's RVC device. * Reverted the changes made in PR 29248 following the spec change in PR 7576. * Restyled by prettier-markdown --------- Co-authored-by: Restyled.io <commits@restyled.io>

…ded. (#29855) (#29871) If we know the command being invoked needs a timed invoke, and the caller did not provide a timed invoke timeout, go ahead and use a default one.

* Start creating a IDL codegen so we can self-test parsed output * Start with listing clusters * Enum listing * A lot more things supported * Attribute rendering * Support for string and octet string sizes * Timed command support * Restyle * Add descriptions to clusters * Attempt to fix up alignment of things * Alignment looks slightly better * Better command separation * Align comments * Align and output descriptions including clusters * More work regarding loop structures * Apply hex formatting to bitmaps. output now seems identical except one whitespace change * Identical output for now * Support API maturity. Notice that doccomments are lost on maturity :( * Fix doxygen parsing for api maturity at the cluster level * Restyle * Support endpoints, although that is not 1:1 as hex encoding and ordering for events is lost * Restyle * Add todo note that default value does not string escaping * Default rendering and add to files * More updates on file dependencies * Unit test IDL generator * Add the IDL unit test as a standard unit test * Update for python compatibility * Fix unit testing of builds when GSDK root is defined * Added a readme file * Restyle * Make xml parser use the idl codegen * Restyle * look to fix misspell warnings * Undo repo update * Fix linter errors --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com>

* Replaced the use of kGenericFaliure with the more correct kInvalidInMode status code in the RVC example app. * Added missing periods to strings

* [Telink] Cleanup Docker image * [Telink] Update docker version

* Adjust some implementation according to sdk upgrade * Update to get mac address for ethernet * fix compile error * Fix restyle

- Updated by '[Scenes] Enabling TableSize feature (#29762)' Signed-off-by: Jaehoon You <jaehoon.you@samsung.com> Signed-off-by: Charles Kim <chulspro.kim@samsung.com>

* Change java generated files using jinja * Restyled by autopep8 * modify test app list --------- Co-authored-by: Restyled.io <commits@restyled.io>

This commit changes the default value of the UDP listen port used in Android platform's ControllerParams from 5541 to 0. Since the default value of UDP listen port was 5541, if multiple matter applications using default settings are run on one matter device, UDP listen port conflict occurs. If we set the UDP listen port to 0, port collisions can be avoided because the port is bound randomly. Although matter applications can avoid conflicts by setting a UDP listen port, it is better to set this to its default value of 0 so that matter applications that use the default settings can avoid conflicts. Signed-off-by: Youngho Yoon <yhoyoon@samsung.com> Signed-off-by: Charles Kim <chulspro.kim@samsung.com> Signed-off-by: Hunsup Jung <hunsup.jung@samsung.com> Signed-off-by: sanghyukko <sanghyuk.ko@samsung.com> Signed-off-by: Jaehoon You <jaehoon.you@samsung.com> Signed-off-by: Kyu-Wook Lim <kyuwook.lim@samsung.com>

* Fix streamer_esp32.cpp for current esp-idf main branch * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>

* [NXP] Adding Matter support for RW61x platform * [NXP] Updating ot-nxp submodule to support RW61x * [RW61x] Fix misspell errors * [RW61x] Fix code-lints errors and updating zap generation : expose the EndpointList attribute in PowerSource cluster * [RW61x] Fix build-and-publish errors * [RW61x] Reworking all-clusters README.md to fix build-and-publish errors * [NXP] Fix application build : Adding device_layer_target_define for NXP device platform * [NXP] Addressing review comment : Removing unnecessary factory reset at ConfigurationManagerImpl init * [NXP] Updating zap files * [NXP] Fix all-clusters app build : update path of ErrorStr.h * [NXP] Run clang-format on RW61x/COMMON sources * Restyled by whitespace * Restyled by clang-format * Restyled by gn * Restyled by prettier-markdown --------- Co-authored-by: Restyled.io <commits@restyled.io>

* Move xml to say zapxml since that is the format * Start defining a data_model_xml parser (no functionality for now) * Start adding some basic support for data model xml parsing * make the DM parser executable * Start having the ability to parse clusters * More updates, we seem to have parsing for features * Hard-code global attributes * Remove some comments * Add enumeration handling * Add bitmap handling * Restyle * Parse structs * Re-organize parsing a bit * Make a linter happy * Another linter fix * Handling of events * More handling and logic on events * Add support for access privilege parsing * XMLs have maybe invalid enum entries. Handle them gracefully * More attribute handling updates * restyle * Support deprecate constraint * Support constraint decoding, apply on attributes for now * Restyle * Restyle * Field handling * Field handling * Some bug fixing * Start adding command handling * Restyle * Name normalization and more parsing updates * Name normalization and more parsing updates * Better messaging and fix constraint types * Restyle * Start creating a IDL codegen so we can self-test parsed output * Start with listing clusters * Enum listing * A lot more things supported * Attribute rendering * Support for string and octet string sizes * Timed command support * Restyle * Add descriptions to clusters * Attempt to fix up alignment of things * Alignment looks slightly better * Better command separation * Align comments * Align and output descriptions including clusters * More work regarding loop structures * Apply hex formatting to bitmaps. output now seems identical except one whitespace change * Identical output for now * Support API maturity. Notice that doccomments are lost on maturity :( * Fix doxygen parsing for api maturity at the cluster level * Restyle * Support endpoints, although that is not 1:1 as hex encoding and ordering for events is lost * Restyle * Add todo note that default value does not string escaping * Default rendering and add to files * More updates on file dependencies * Unit test IDL generator * Add the IDL unit test as a standard unit test * Update for python compatibility * Fix unit testing of builds when GSDK root is defined * Added a readme file * Restyle * Make xml parser use the idl codegen * Restyle * look to fix misspell warnings * Undo repo update * Fix linter errors * Codegen as idl for data_model_xml_parser.py * Parsing closer to matter idl content * more normalization and type processing * More mandatory conformance logic * Fix mandatory conditionals * Make unit test pass * Fix tests a bit more and make parsers better * Restyle * Ignore min/max values while parsing xmls, even though raw data internally contains them * Restyle * Fix space after click annotations * Compare support for human reviews * Restyle * Fix slash * Undo submodule change * fix xml to zapxml naming changes * Restyle * Update dates from 2022 to 2023 * Add note about the complex test input * Remove unused imports * Restyle * Add some commends based on code review * Add heuristic for setting enum and bitmap sizes, to make output from XML much more readable * Add support for timed and fabric scoped commands * Add missing import --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com>

* [Ameba] added implementation for manual operation of several clusters (dishwasher alarm, refrigerator alarm, operational state, rvc operational state, rvc runmode, and rvc cleanmode), test event trigger for smoke alarm, shutdown function of CHIPDeviceManager, and init laundrywashercontrolserver * [Ameba] Run restyle-diff.sh, and modified cmake configuration to add additional implementation * [Ameba] Added CONFIG_ENABLE_AMEBA_TEST_EVENT_TRIGGER macro * [Ameba] added CONFIG_ENABLE_AMEBA_TEST_EVENT_TRIGGER define flag at cmake * [Ameba] Updated switch case at ManualOperationalStateCommand.h and updated raise and lower alarm function at ManualDishWasherAlarmCommand.h * [Ameba] Updated RvcOperationalState::ErrorStateEnum::kUnknownEnumValue to OperationalState::ErrorStateEnum::kUnknownEnumValue because it was removed, and added minor comments.

Signed-off-by: Chin-Ran Lo <chin-ran.lo@nxp.com>

* Remove PSA Crypto exclusion * fix PSA test failure * Add more comments * Fix comments * Add isKeySet flag * address comments

This change adds shutdown to AndroidDeviceControllerWrapper, which wraps CHIPDeviceController on the Android platform, so that DeviceControllerFactory and CHIPDeviceController can be shutdown. AndroidDeviceControllerWrapper initializes DeviceControllerFactory and CHIPDeviceController, but since it supports only CHIPDeviceController shutdown, DeviceControllerFactory shutdown is required. If you reuse AndroidDeviceControllerWrapper with the call below without shutting down DeviceControllerFactory, sometimes the DeviceControllerSystemState of DeviceControllerFactory becomes incorrect, causing AndroidDeviceControllerWrapper to operate incorrectly, and the problem cannot be solved unless the process is restarted. - AndroidDeviceControllerWrapper::AllocateNew() - AndroidDeviceControllerWrapper::Controller()->Shutdown() Signed-off-by: Youngho Yoon <yho.yoon@samsung.com> Signed-off-by: Charles Kim <chulspro.kim@samsung.com> Signed-off-by: Hunsup Jung <hunsup.jung@samsung.com> Signed-off-by: sanghyukko <sanghyuk.ko@samsung.com> Signed-off-by: Jaehoon You <jaehoon.you@samsung.com> Signed-off-by: Kyu-Wook Lim <kyuwook.lim@samsung.com>

Commits on Oct 7, 2023

remove minimum reqs (#29620 )

mkardous-silabs authored Oct 7, 2023

Configuration menu

View commit details

Copy full SHA for f34b13c

Browse repository at this point

Copy the full SHA

f34b13c View commit details

Browse the repository at this point in the history

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

merge commit #2

merge commit #2

Commits on Sep 29, 2023

Commits on Sep 30, 2023

Commits on Oct 2, 2023

Commits on Oct 3, 2023

Commits on Oct 4, 2023

Commits on Oct 5, 2023

Commits on Oct 6, 2023

Commits on Oct 7, 2023

Commits on Oct 9, 2023

Commits on Oct 10, 2023

Commits on Oct 11, 2023

Commits on Oct 12, 2023

Commits on Oct 13, 2023

Commits on Oct 14, 2023

Commits on Oct 15, 2023

Commits on Oct 16, 2023

Commits on Oct 17, 2023

Commits on Oct 18, 2023

Commits on Oct 19, 2023

Commits on Oct 20, 2023

Commits on Oct 21, 2023

Commits on Oct 22, 2023