)]}'
{
  "commit": "c4c04b3fecc279b804bb19ec3c92bd2484a91f3e",
  "tree": "791b5e4cd4e71a7079d222bb3420c982fd740cb9",
  "parents": [
    "f38bd61f556cf8d0e86bc64cc2a637d42f6aebab"
  ],
  "author": {
    "name": "PeterC1965",
    "email": "101805108+PeterC1965@users.noreply.github.com",
    "time": "Fri Aug 30 23:22:19 2024 +0100"
  },
  "committer": {
    "name": "GitHub",
    "email": "noreply@github.com",
    "time": "Fri Aug 30 18:22:19 2024 -0400"
  },
  "message": "WaterHeater Management follow up - Adds Boost events, test script updates (#34944)\n\n* Start addressing comments from issue #34565\r\n\r\n* Add WHM event test support\r\n\r\n* Adapt test 2.3 to check for events\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Restyled by autopep8\r\n\r\n* Fix failing CI build\r\n\r\n* Restyled by clang-format\r\n\r\n* Fix some CI build issues\r\n\r\n* Update src/app/clusters/water-heater-management-server/water-heater-management-server.cpp\r\n\r\n* Add event support and fix tank percentage steps\r\n\r\n* Get tank percentage calculation and expected result correct\r\n\r\n* Add a test for emergencyBoost\u003dTrue being specified in the BoostStarted command\r\n\r\n* Add and use a Energy_mWh type\r\n\r\n* [NXP][docker] Update k32w1 SDK (#34927)\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n* [ESP32] Delta OTA Feature (#29011)\r\n\r\n* [ESP32] Delta OTA Feature\r\n\r\n* Fix some issues with header\r\n\r\n* Update readme\r\n\r\n* Some refactoring and cleanups\r\n\r\n* Address review comments\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Shubham Patil \u003cshubham.patil@espressif.com\u003e\r\n\r\n---------\r\n\r\nCo-authored-by: PSONALl \u003csonali.patil@espressif.com\u003e\r\nCo-authored-by: Rohit Jadhav \u003crohit.jadhav@espressif.com\u003e\r\nCo-authored-by: Rohit Jadhav \u003c69809379+jadhavrohit924@users.noreply.github.com\u003e\r\nCo-authored-by: Shubham Patil \u003cshubham.patil@espressif.com\u003e\r\n\r\n* Fix typo in NXP dockerfile: missing backslash (#34931)\r\n\r\n* Fixed test 1.6 and updated the scripts to work in CI. (#34918)\r\n\r\n* Fixed test 1.6 and updated the scripts to work in CI.\r\n\r\n* Restyled by autopep8\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* [NXP][k32w1] Fix docker image (#34935)\r\n\r\nCreate a separate folder for each west workspace.\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n* Checking third party deps doesn\u0027t pass after someone fixes it (#34872)\r\n\r\n* Fixing this to stop breaking PRs\r\n\r\n* Qualifying as expression\r\n\r\n* Making this more readable now that it\u0027s an expression\r\n\r\n* Implement the AndroidChipLogging (#34899)\r\n\r\n* Refine android log priority based on category\r\n\r\nThis change adds ANDROID_LOG_INFO priority for kLogCategory_Progress.\r\nRefine android log priorities to enhance debugging efficiency and\r\nprepare an environment where lower-priority logs can be controlled and\r\nexcluded using the log filter function.\r\n\r\nSigned-off-by: Youngho Yoon \u003c34558998+yhoyoon@users.noreply.github.com\u003e\r\n\r\n* Implement the AndroidChipLogging\r\n\r\nImplemented the AndroidChipLogging.setLogFilter to enhance the logging\r\nmechanism. This method allows for the filtering out of unnecessary or\r\nredundant log messages, significantly reducing noise in the logs.\r\nTo use the log filter, call AndroidChipLogging.setLogFilter with the\r\nappropriate filter criteria. The arg is log level in android.util.Log.\r\nExample: AndroidChipLogging.setLogFilter(android.util.Log.ERROR)\r\n\r\nSigned-off-by: Youngho Yoon \u003c34558998+yhoyoon@users.noreply.github.com\u003e\r\n\r\n* Restyled by google-java-format\r\n\r\n---------\r\n\r\nSigned-off-by: Youngho Yoon \u003c34558998+yhoyoon@users.noreply.github.com\u003e\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Syslog and logging tweaks (#34941)\r\n\r\n* Make syslog ident and facility overridable and tweak options\r\n\r\n* Linux: Be a little less verbose when writing settings\r\n\r\n* Add out-of-band msgs to add and remove Supported Areas and Maps (#34930)\r\n\r\n* update the status field of progress elements at the end of a clean.\r\n\r\n* Added some style suggestions from the review of PR 34887.\r\n\r\n* update the readme.\r\n\r\n* Added a simple message to add a map.\r\n\r\n* Added an out-of-band message to add a supported area.\r\n\r\n* Added out-of-band messages to remove supported maps and supported areas.\r\n\r\n* Updated the rvc-app PICS values. Updated TC-SEAR-1.2 to include the necessary out-of-band messages to run the test in CI.\r\n\r\n* refactored to improve readability.\r\n\r\n* Restyled by clang-format\r\n\r\n* Restyled by prettier-markdown\r\n\r\n* Removed oddly duplicated method.\r\n\r\n* Fixed the json value conversion types.\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Restructured folders in chip-energy-management-app - no code change (#34879)\r\n\r\n* Refactored file location for chip-energy-management-app. Updated esp32 and linux build files.\r\n\r\n* Updated some all-clusters-app build and CMakeLists (linux, esp32, psoc6)\r\n\r\n* Updated remaining build and CMakefiles with new folders.\r\n\r\n* Fixed unit tests BUILD.gn\r\n\r\n* Restyled by gn\r\n\r\n* Missed Ameba .cmake\r\n\r\n* Missed all-clusters-minimal-app/esp32/main/CMakeLists.txt\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* [NXP][config] Adding CMake support to build CHIP libs for NXP FreeRTOS platforms (#34901)\r\n\r\n* [NXP][config][freertos] Adding support to build Matter lib and port files with cmake on NXP freertos platforms\r\n\r\nSigned-off-by: Dina Benamar \u003cdina.benamarelmaaroufi@nxp.com\u003e\r\n\r\n* [NXP][config][common] Create common .cmake file to include common configs between freertos \u0026 zephyr\r\n\r\nSigned-off-by: Dina Benamar \u003cdina.benamarelmaaroufi@nxp.com\u003e\r\n\r\n* [NXP][config][zephyr] Include common.cmake which adds common options to zephyr \u0026 freertos\r\n\r\nSigned-off-by: Dina Benamar \u003cdina.benamarelmaaroufi@nxp.com\u003e\r\n\r\n* [NXP][config][freertos] Updates to align with new flag introduction \"chip_enable_secure_whole_factory_data\"\r\n\r\nSigned-off-by: Dina Benamar \u003cdina.benamarelmaaroufi@nxp.com\u003e\r\n\r\n* [NXP][config][freertos] Set build_matter_standalone_lib to true to support building CHIP libs with CMake in matter-over-thread configuration\r\n\r\nSigned-off-by: Dina Benamar \u003cdina.benamarelmaaroufi@nxp.com\u003e\r\n\r\n---------\r\n\r\nSigned-off-by: Dina Benamar \u003cdina.benamarelmaaroufi@nxp.com\u003e\r\n\r\n* [chip-tool] Adjust the maximum value of \u0027discriminator\u0027 to align with spec (#34940)\r\n\r\n* [examples][lock-common] Remove iostream include (#34952)\r\n\r\n* iostream adds a lot of unnecessary functions when lock-app\r\n   is compiled for non-linux platforms, increasing ram and\r\n   flash consumption.\r\n * linux lock-app compiles without it\r\n\r\nSigned-off-by: Andrei Menzopol \u003candrei.menzopol@nxp.com\u003e\r\n\r\n* Fixed devcontainer creation for huge user ids (#34948)\r\n\r\n* Enabling this for macOS Catalyst apps (#34956)\r\n\r\n* Add new common NXP dockerfile (#34947)\r\n\r\n* [NXP][docker image] Add new common NXP docker image for next SDK version\r\n\r\nSigned-off-by: Martin Girardot \u003cmartin.girardot@nxp.com\u003e\r\n\r\n* [NXP][docker image] remove k32w1 SDK clone as it will use common NXP SDK\r\n\r\nSigned-off-by: Martin Girardot \u003cmartin.girardot@nxp.com\u003e\r\n\r\n---------\r\n\r\nSigned-off-by: Martin Girardot \u003cmartin.girardot@nxp.com\u003e\r\n\r\n* Use timedInteraction for RevokeCommissioning in CCTRL_2_2 (#34957)\r\n\r\nCo-authored-by: Andrei Litvin \u003candreilitvin@google.com\u003e\r\n\r\n* [Fabric-Admin] Fix \u0027fabricsync sync-device\u0027 command always fails (#34938)\r\n\r\n* [Fabric-Admin] Fix \u0027fabricsync sync-device\u0027 command always fails\r\n\r\n* Use default random salt\r\n\r\n* Update examples/fabric-admin/commands/pairing/OpenCommissioningWindowCommand.h\r\n\r\nCo-authored-by: Abdul Samad \u003cabdul.samad@smartthings.com\u003e\r\n\r\n* Address review comments\r\n\r\n---------\r\n\r\nCo-authored-by: Abdul Samad \u003cabdul.samad@smartthings.com\u003e\r\n\r\n* Fix user_params usage (#34964)\r\n\r\n* Fixed the discovery mode in esp32 factory script to a bitmap value instead of enum (#34900)\r\n\r\n* Fix typo in wait_for_user_input (#34965)\r\n\r\n* Remove wrong dependency to attribute id 82 (#34975)\r\n\r\nThis was a left-over attribute id before a spec change where attributes got removed. After that the attribute id 82dec is SetpointHoldExpiryTimestamp which has nothing to do with Presets\r\n\r\n* replacing \"stringcase\" python package with internal implementation + generated java and kotlin code (#34949)\r\n\r\n* changing matter_idl functions\r\n\r\n* changing yaml runner functions\r\n\r\n* removing stringcase from config and requirement files\r\n\r\n* restyle\r\n\r\n* improve string handling\r\n\r\n* fixing generated java and kotlin code\r\n\r\n* regenerate idl golden image for test_generators.py\r\n\r\n* adding unit test\r\n\r\n* restyle\r\n\r\n* adding unit test to build.GN\r\n\r\n* [Tizen] Update Tizen SDK to 8.0 (#34171)\r\n\r\n* Update tizen to 8.0\r\n\r\n* Install platform certificate\r\n\r\n* Reverse restyle diff\r\n\r\n* Review fix - version update\r\n\r\n* Add information about used 7.0 tizen package\r\n\r\n---------\r\n\r\nCo-authored-by: Arkadiusz Bokowy \u003ca.bokowy@samsung.com\u003e\r\n\r\n* updated list of AcceptedCommands list for Thermostat cluster with Preset feature enabled (#34981)\r\n\r\n* Update CCTRL 2.2 to not potentially use uninitialized variables in the class on teardown (#34955)\r\n\r\n* Do not use uninitialized bits in teardown\r\n\r\n* Extra guard for file remove\r\n\r\n* Fix a few more instances of app_process\r\n\r\n---------\r\n\r\nCo-authored-by: Andrei Litvin \u003candreilitvin@google.com\u003e\r\n\r\n* [Python] Auto-import all clusters into the chip.clusters namespace (#34928)\r\n\r\n* Auto-import all clusters into the chip.clusters namespace\r\n\r\n* Fix __all__ lists\r\n\r\n* ZAP-generate __all__ with all clusters\r\n\r\n* fix unused variable issue on non-macOS testing (#34970)\r\n\r\n* Add constraint enforcement on responseTimeoutSeconds in commissioner control cluster (#34982)\r\n\r\n* Add constraint enforcement on responseTimeoutSeconds\r\n\r\nThis adds constraints in the implementation and also fixes\r\nup error codes in CCTRL_2_2 test.\r\n\r\n* Fix comment\r\n\r\n* Fix conversion error (#34989)\r\n\r\n* Update ZAP to newest nightly to get fix for global types (#34996)\r\n\r\n* Update zap.version\r\n\r\n* Update zap.json\r\n\r\n* Bump min ZAP version\r\n\r\n* Regen ZAP files after update\r\n\r\n* Restore extra newline\r\n\r\n* Create new ICD Manager in fabric-admin to service KeepActive Command (#34894)\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: yunhanw \u003cyunhanw@google.com\u003e\r\n\r\n* [Silabs]Provision storage sourceset (#34929)\r\n\r\n* Create a separte source_set for the provision storage implementation\r\n\r\n* Fix efr32 build using provision flash storage\r\n\r\n* fix argument duplication which is not needed.\r\n\r\n* apply suggested renames\r\n\r\n* [Telink] Update Docker image (Zephyr update) (#34980)\r\n\r\n* [Telink] Update Docker image (Zephyr update)\r\n\r\n* [Telink] Update Docker image (Zephyr update)\r\n\r\n---------\r\n\r\nCo-authored-by: Andrei Litvin \u003candy314@gmail.com\u003e\r\n\r\n* [DRLK] Bugfix: return INVALID_COMMAND when attempting to add/modify (#34120)\r\n\r\n* [DRLK] Bugfix: return INVALID_COMMAND when attempting to add/modify\r\ncredential from a different fabric than the User/Credential\u0027s creator fabric\r\n\r\nAdd YAML test steps to verify correct behavior\r\n\r\nFixes #34119\r\n\r\n* Restyled by prettier-yaml\r\n\r\n* Update src/app/tests/suites/DL_UsersAndCredentials.yaml\r\n\r\nCo-authored-by: Andrei Litvin \u003candy314@gmail.com\u003e\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: Andrei Litvin \u003candy314@gmail.com\u003e\r\n\r\n* [NXP][k32w1] Rework reference apps (#34926)\r\n\r\n* [NXP][platform][common] Update pw RPC flag in streamer_nxp\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n(cherry picked from commit 4ae033a1bddd5d914e862791c111eebca41a62e8)\r\n\r\n* [NXP][examples][common] Fix binding issue due to generated Zap flag renaming\r\n\r\nSigned-off-by: Martin Girardot \u003cmartin.girardot@nxp.com\u003e\r\n(cherry picked from commit b50b382478138ad02103e55194b0145b1e61e841)\r\n\r\n* [NXP][examples][common] make-pretty update\r\n\r\n(cherry picked from commit efc51cc65471f4f5f667947ef18a4656e1d29578)\r\n\r\n* [NXP][examples][common] Fix contact-sensor-app always returning empty diagnostic logs\r\n\r\nThe LogProvider class method GetSizeForIntent() was always returning\r\nzero as log size due to an incorrect evaluation of the result of the\r\nSyncGetValue() call.\r\n\r\nGetSizeForIntent() is evaluating the size of the diagnostic log by\r\ntrying to read the key using a zero length buffer. This action will\r\nreturn the size of the key in storage, but it will also yield a\r\nCHIP_ERROR_BUFFER_TOO_SMALL error code. GetSizeForIntent() however\r\nwas not expecting this error code and it was returning zero in all\r\nsituations when the status returned by SyncGetValue() was anything\r\nelse but CHIP_NO_ERROR.\r\n\r\nThe function was updated to take into account that, under these\r\ncircumstances, the SyncGetValue() function always returns\r\nCHIP_ERROR_BUFFER_TOO_SMALL.\r\n\r\nSigned-off-by: Marian Chereji \u003cmarian.chereji@nxp.com\u003e\r\nReviewed-by: Doru-Cristian Gucea \u003cdoru-cristian.gucea@nxp.com\u003e\r\n(cherry picked from commit dd3b3b784318f73cf6f0390f79d1fdf0160169d3)\r\n\r\n* [NXP][examples][common] Call init using the base class reference\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n(cherry picked from commit b7a0a0a4d88bd5d03b70f36b34c2475b614bd296)\r\n\r\n* [NXP][examples][common] Update app task common code\r\n\r\nNew task base API:\r\n- PostEventsProcessedAction - this will be called in the main loop,\r\nafter all events have been processed. An example of concrete action\r\nis to animate some LEDs.\r\n- PrintOnboardingInfo - the default implementation remains the same.\r\nIt can be overwritten by the concrete application task class.\r\n- AppMatter_DisallowDeviceToSleep/AppMatter_AllowDeviceToSleep -\r\nAn application that uses low power mode should overwrite the API to\r\nimplement the allow/disallow entering low power mechanism.\r\nBefore taking the OT mutex, the device is disallowed to enter low power.\r\nThe device is allowed to enter low power after the OT mutex is released.\r\n\r\nOther changes:\r\n- The app event queue handle should be static such that the concrete\r\napplication task can initialize it during Start() call.\r\n- The ticksToWait passed to xQueueReceived is used to specify\r\nthe maximum amount of time the task should block waiting for an event.\r\nThis can be modified according to the application needs, e.g. is needed\r\nby an application that has LED animation.\r\n- PostEvent should take into account ISR context when populating the event queue.\r\n- PostEvent becomes a static method.\r\n- Thread device type is specified by the following flag: CONFIG_THREAD_DEVICE_TYPE\r\nThis is set by default to kThreadDeviceType_Router.\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n(cherry picked from commit d88e78ea6c973eb736915110bb227ad3515eec98)\r\n\r\n* [NXP][examples][common] Add additional common components\r\n\r\n- Add led widget component\r\n- Add diagnostic logs component - chip::NXP::App::DiagnosticLogsDemo::DisplayUsage\r\nshould be implemeneted to showcase how the diagnostic logs provider should be used.\r\n- Add low power component - chip::NXP::App::LowPower::Init should be implemented\r\naccording to the application needs. A default example is implemented, which registers\r\nan empty enter/exit low power callback.\r\n- Add operational keystore component - the API contains two functions:\r\n\t- GetInstance - should return the address of a concrete operational keystore class\r\n\t- Init - can be used to register a persistent storage delegate in case the concrete\r\n\t  class is using persistent storage (which will be our case)\r\n- Add user interface feedback component - this component defines an abstract interface\r\nthat can be implemented by applications in order to provide some kind of feedback to the user\r\nin certain conditions: logging, LED animations, etc. Add example of lighting manager, which\r\nuses LED animation for user feedback.\r\n- Add clusters component - add an API for identify related callbacks. Add implementation for\r\ncommon MatterPostAttributeChangeCallback, which delegates the work to the device manager component.\r\n- Add RPC component - this is a simple Init API that can be used to initialize PW RPC related\r\ncomponents, such as UART.\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n(cherry picked from commit 46a1d3d673b9cae14d33c749e3741727449e6ff9)\r\n\r\n* [NXP][examples][common] Add missing header in Identify\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n(cherry picked from commit 3abfee2c6eea327ae931a150733ac83639d157d5)\r\n\r\n* [NXP][examples][common] Add multi-image OTA requestor initiator\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n(cherry picked from commit 5848e4d4e2d384c4bfe41714bd41e25c4b1ba1e8)\r\n\r\n* [NXP][examples][common] Update pw RPC implementation\r\n\r\nUpdate AppRpc API by adding two new methods used by the common\r\nRPC implementation:\r\n- ButtonHandler\r\n- Reboot\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n(cherry picked from commit 5e40934145825c61444e43e479ed7ac5c26605a4)\r\n\r\n* [NXP][examples][common] Small updates to common app code\r\n\r\n- Add call to feedback manager init.\r\n- Remove PostEventsProcessedAction. Call FeedbackMgr().DisplayInLoop()\r\ndirectly in the main loop.\r\n- Move PostEvent in AppTaskBase interface and make it virtual.\r\n- Add demo of diagnostic logs usage.\r\n- Update PrintOnboardingInfo to take into account user action requried\r\ncommissioning flow.\r\n- Remove usage of deprecated sFunction from Lighting manager.\r\n- Add LowPower::Init call.\r\n- If FSL_OSA_TASK_ENABLE is used, then call AppTaskMain inside Start, instead\r\nof creating the app task.\r\n- Add PrintCurrentVersion API to print the software version string and software\r\nversion.\r\n- Enclose DeviceInfoProviderImpl usage based on CONFIG_DEVICE_INFO_PROVIDER_IMPL\r\n- Add diagnostic logs cluster init under the corresponding flag.\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n(cherry picked from commit 1619c92380275dc0466f9cf9dc9e818671406b06)\r\n\r\n* [NXP][examples][common] Make Animate pure virtual\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n(cherry picked from commit eaf3789fc2d1e562fe030acc758e5f322b5f02d3)\r\n\r\n* [NXP][examples][common] Add pw RPC init\r\n\r\nRPC_TASK_STACK_SIZE and RPC_TASK_PRIORITY can now be defined\r\nin the build system to override the default values.\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n(cherry picked from commit 7962d612b14502dc3f514c5c3d39891b086cfd63)\r\n\r\n* [NXP][examples][common] Allow platforms to redefine the path to the FactoryDataProvider.h that needs to be included\r\n\r\nSigned-off-by: Dina Benamar \u003cdina.benamarelmaaroufi@nxp.com\u003e\r\n(cherry picked from commit 097cd999c4767572f8cc55fac18f93ac3cb5d50c)\r\n\r\n* [NXP][examples][common] Add TestEventTriggerDelegate\r\n\r\n * TestEventTriggerDelegate is now used at Server Initialization\r\n   to add the ICDManager instance as Handler\r\n\r\nSigned-off-by: Andrei Menzopol \u003candrei.menzopol@nxp.com\u003e\r\n(cherry picked from commit edce1dc967130589e74cdd9c8289719171141178)\r\n\r\n* [NXP][examples][common] Minor updates for some components\r\n\r\nThe KVS wear stats are stored in the user diagnostic log key hence only\r\ninitialize this key if the KVS wear stats are not enabled.\r\n\r\nSigned-off-by: Marian Chereji \u003cmarian.chereji@nxp.com\u003e\r\n\r\n[nxp noup][examples][common] Remove UpdateState API\r\n\r\nThe interface can be simplified by calling RestoreState\r\ninstead of UpdateState when needed. This implies that the\r\ncaller must update the cluster attribute by itself.\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n[nxp noup][examples][common] Rename LightingManager to LedManager\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n* [NXP][examples][common] Add initial version of lighting app\r\n\r\n- Copy lighting app sources to common folder\r\n- Add device callbacks\r\n- Use common OTA support\r\n- Use common app factory data\r\n- Use led widgets for lighting app\r\n- Use common app task and button management\r\n- Decouple lighting manager from app task\r\n- Update LED status function\r\n- Update lighting manager related flags\r\n\tFlags:\r\n\t- LIGHTING_MANAGER_ENABLE_DIMMABLE_LED to enable dimmable LED feature\r\n\t- LIGHTING_MANAGER_ENABLE_STATUS_LED to enable status LED usage\r\n\tThese flags can be overwritten in AppConfig.h.\r\n- Make lighting manager more configurable\r\n- Create button manager\r\n\tThe ButtonManager class should define the buttons callbacks and\r\n\tall callback delegates.\r\n\r\n\tMoved AppMatterButton_registerButtons implementation inside the\r\n\tbutton manager source file.\r\n- Update lighting manager interface\r\n- Replace lighting manager calls with feedback manager\r\n\tLightingManager now implements the UserFeedbackInterface abstract class and the\r\n\tdimmable feature was moved to a new class: LightingManagerDimmable.\r\n- Move identify code in a separate file\r\n- Remove deprecated #include statements from all files.\r\n- Add commissioning complete state\r\n\tA boolean static member is added to AppTask. It is set when the\r\n\tkCommissioningComplete event is received. It should be used by\r\n\tother components if they need a way to check the state of the device.\r\n- Add button action to switch to ICD active mode\r\n- Add support for BooleanState in device callbacks\r\n- Add diagnostic logs usage demo\r\n- Enclose feedback manager calls in CONFIG_ENABLE_FEEDBACK flag\r\n\tThis flag should be defined whenever the application must provide\r\n\tsome kind of user interface feedback, such as LED animation.\r\n\tIn case of low power, this flag will not be defined.\r\n- Remove unused features (e.g. diagnostic logs)\r\n- Overwrite event queue and ticks to wait\r\n- Call PlatformMgrImpl API for PreInitMatterStack actions\r\n- Replace PW_RPC_ENABLED with CONFIG_ENABLE_PW_RPC\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n(cherry picked from commit bed6655227d44900279cbd0ef547771da9143895)\r\n\r\n* [NXP][examples][common] Add initial version of contact sensor\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n(cherry picked from commit 637580aa6c63048ae1a956807c284165c2af17ed)\r\n\r\n* [NXP][openthread][k32w1] Update mbedtls target\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n(cherry picked from commit 3c84b1fd3987958bc0dcf49a283c281e48b400f8)\r\n\r\n* [NXP][platform][k32w] OTA/FactoryData changes\r\n\r\n * Add FactoryDataDriver interface which handles factory data update\r\n   functionality and factory data ram backup operations;\r\n   must be implemented for each platform\r\n * Make FactoryDataProvider global instances\r\n * Move Restore mechanism and plain-text SignWithDacKey in\r\n   common FactoryDataProvider\r\n * Adapt Validate and SearchForId to use factory data config\r\n   which stores size, start, payload\r\n * Move OTAFactoryDataProcessor in common folder\r\n * Adapt to be generic and use FactoryDataDriver functions instead\r\n\r\nSigned-off-by: Andrei Menzopol \u003candrei.menzopol@nxp.com\u003e\r\n(cherry picked from commit 3a81c2367013c2a2ed08ecbbeac89ce15943998e)\r\n\r\n* [NXP][platform][k32w1] SDK 2.15 platform updates\r\n\r\n * remove RNG_SetPseudoRandomNoSeed call to accomodate new RNG API changes.\r\n * add SDK root, SDK release, and SDK 2.15 gn arguments. They will be used\r\n   to differentiate between SDK versions (2.12 vs 2.15) and release\r\n   status (repo vs packages).\r\n\r\nSigned-off-by: Andrei Menzopol \u003candrei.menzopol@nxp.com\u003e\r\n\r\n[nxp noup][platform][k32w] Small fixes for k32w platform code\r\n\r\n * correct typos\r\n * remove executable access permissions\r\n * remove unused files\r\n * change openthread_root to ot-nxp/openthread-k32w1\r\n * use nxp_mbedtls gn target\r\n\r\nSigned-off-by: Andrei Menzopol \u003candrei.menzopol@nxp.com\u003e\r\n(cherry picked from commit 1905c1cea645506f8a88e514c2eacecea81ac385)\r\n\r\n* [NXP][platform][k32w1] Add \"chip_key_storage\" solution selector and littlefs support\r\n\r\nUpdated the K32W1 platform build files by adding the \"chip_key_storage\"\r\nselector which will help developers select an implementation option\r\nfor the key storage between \"littlefs\" and \"fwk_nvm\".\r\n\r\nSigned-off-by: Marian Chereji \u003cmarian.chereji@nxp.com\u003e\r\n\r\n[nxp noup][k32w1] Temporarily disable OpenThread settings save-on-idle when using LittleFS key storage\r\n\r\nWhen using LittleFS key storage, the OpenThread settings save-on-idle\r\nfeature requires additional support in the application idle task,\r\nwhich is not yet available. In order for the OT settings saving to\r\nwork at this stage and in this configuration, the save-on-idle\r\nfeature must be disabled until the required idle task support will be\r\nimplemented.\r\n\r\nSigned-off-by: Marian Chereji \u003cmarian.chereji@nxp.com\u003e\r\n\r\n[nxp noup][platform][k32w1] Fix minimum data chunk size required when programming the flash memory\r\n\r\nAccording to the specifications, the internal flash memory of the\r\nK32W1 platform supports programming chunks of memory which are 128\r\nbytes long (instead of 256).\r\n\r\nSigned-off-by: Marian Chereji \u003cmarian.chereji@nxp.com\u003e\r\n\r\n[nxp noup][platform][k32w1] Increase LittleFS cache size to 1K to suppress frequent data synchronizations\r\n\r\nA small cache size forces LittleFS to perform frequent data\r\nsynchronizations with the storage (flash memory) and this consumes a\r\nnumber of data blocks which are not available in the K32W1 Matter\r\napplications. To reduce the fast consumption of flash memory data\r\nblocks, the LittleFS cache size was increased to 1KB.\r\n\r\nSigned-off-by: Marian Chereji \u003cmarian.chereji@nxp.com\u003e\r\n(cherry picked from commit 753f7d7390661bd5dbd29cc9cfa7d3a4e4728ba2)\r\n\r\n* [NXP][platform][k32w1] Add Zephyr NVS based key storage build option\r\n\r\nAdded a new build option (chip_key_storage\u003d\"nvs\") which allows the\r\ndevelopers to use the Zephyr NVS file system as a key storage\r\nsolution.\r\n\r\nSigned-off-by: Marian Chereji \u003cmarian.chereji@nxp.com\u003e\r\n(cherry picked from commit 1d42e48d8f50f6e9999b49025ea3d71545e89a3d)\r\n\r\n* [NXP][examples][k32w1] App updates\r\n\r\n * the sdk root was moved to platform args.\r\n * update readme files\r\n * Add CONNECTIVITY_MANAGER_THREAD_DEVICE_TYPE define.\r\n * Set define accordingly based on FTD/MTD configuration.\r\n * Solve issue when lighting-app is MTD but device type is set to\r\n   Router because it was hardcoded.\r\n * Remove deprecated files\r\n * Move some headers in examples/platform for common usage\r\n\r\nSigned-off-by: Andrei Menzopol \u003candrei.menzopol@nxp.com\u003e\r\n(cherry picked from commit c2b626df132e058ce674df0ee20965b3029c462d)\r\n\r\n* [NXP][examples][k32w1] Update ZclCallbacks after transition to common code\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n* [NXP][k32w1] Remove k32w parent folder\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n* [NXP][platform][k32w1] Add FactoryDataDriver implementation\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n* [NXP][platform][k32w] Remove k32w/common\r\n\r\nThis folder was moved to nxp/common/legacy area.\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n(cherry picked from commit 755081c71bde287e2d065e5fd9b1a0922c3b341a)\r\n\r\n* [NXP][scripts] Minor updates for NXP related scripts\r\n\r\nAdd HWParams APP FactoryData support\r\n\r\n * Add proper format for HWParameters app factory data section\r\n * Put changes under new argument\r\n\r\nSigned-off-by: Andrei Menzopol \u003candrei.menzopol@nxp.com\u003e\r\n\r\n[nxp noup][scripts][common] Fix Crc16 attribute when using latest version\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n[nxp noup][scripts] Update structure k32w1 path\r\n\r\nAll k32w1 folders were moved under nxp, removing the k32w parent.\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n[nxp noup][scripts][ota] Update paths after k32w transition\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n(cherry picked from commit 4963ce98725b74e94f498f959d46edf4933fa56c)\r\n\r\n* [NXP][platform][common] Add nxp_factory_data, nxp_ota gn targets\r\n\r\nThe application can use these targets to integrate the default, platform-specific\r\nNXP factory data provider and OTA implementation. Each platform should define an\r\n\"nxp_factory_data\" and \"nxp_ota\" source sets.\r\n\r\nUsage:\r\ndeps +\u003d [ \"${chip_root}/src/platform/nxp:nxp_factory_data\" ]\r\ndeps +\u003d [ \"${chip_root}/src/platform/nxp:nxp_ota\" ]\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n(cherry picked from commit 4869c4a97134692fe87a59b9b69d33f836d9b2e9)\r\n\r\n* [NXP][platform][common] Create NXPConfigNVS.cpp file, remove NXPConfig BLE dependencies\r\n\r\nSigned-off-by: Martin Girardot \u003cmartin.girardot@nxp.com\u003e\r\n(cherry picked from commit 50e14e4a1b9f8561126310e19ebbaf72ca7bbdbe)\r\n\r\n* [NXP] revert mcxw71 headers\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n* [NXP][k32w1] Add default logging target\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n* [NXP][k32w1] Update paths to reference nxp_matter_support\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n* [NXP] Bump nxp_matter_support\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Restyled by gn\r\n\r\n* Restyled by prettier-markdown\r\n\r\n* Restyled by autopep8\r\n\r\n* Restyled by isort\r\n\r\n* [NXP][k32w1] Fix gn import order\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n* [NXP][k32w1] Fix wrong README reference\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n* [NXP][k32w1] Update workflow docker image\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n* [NXP][rw61x] Adding nxp_ota and nxp_factory_data as they are now mandatory\r\n\r\nSigned-off-by: Gatien Chapon \u003cgatien.chapon@nxp.com\u003e\r\n\r\n* [NXP][platform][common] Fix build warning\r\n\r\n(cherry picked from commit 72b1cbaaf36fe6cf2af57a8c7ad3cc457b45c713)\r\n\r\n* Restyled by gn\r\n\r\n* [NXP][platform] Remove deprecated files\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n* [NXP][workflow] Create separate jobs for k32w0/k32w1\r\n\r\nk32w0 builds use chip-build-k32w Docker image, but k32w1 builds\r\nuse chip-build-nxp Docker image.\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n* [NXP][k32w0] Adding nxp_ota and nxp_factory_data as they are now mandatory\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n* [NXP][scripts] Fix script issue when there is no --args\r\n\r\nSigned-off-by: Martin Girardot \u003cmartin.girardot@nxp.com\u003e\r\n\r\n* [NXP][k32w1] Disable SMU2\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\n\r\n---------\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\nSigned-off-by: Gatien Chapon \u003cgatien.chapon@nxp.com\u003e\r\nSigned-off-by: Martin Girardot \u003cmartin.girardot@nxp.com\u003e\r\nCo-authored-by: Martin Girardot \u003cmartin.girardot@nxp.com\u003e\r\nCo-authored-by: Dina Benamar \u003cdina.benamarelmaaroufi@nxp.com\u003e\r\nCo-authored-by: Marian Chereji \u003cmarian.chereji@nxp.com\u003e\r\nCo-authored-by: Andrei Menzopol \u003candrei.menzopol@nxp.com\u003e\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: Gatien Chapon \u003cgatien.chapon@nxp.com\u003e\r\nCo-authored-by: Yassine El Aissaoui \u003cyassine.elaissaoui@nxp.com\u003e\r\n\r\n* [Fabric-Sync] Simplify and update the fabric sync guide (#34993)\r\n\r\n* [Fabric-Sync] Symplify and update the fabric sync guide\r\n\r\n* Restyled by prettier-markdown\r\n\r\n* Update docs/guides/fabric_synchronization_guide.md\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\n\r\n* Update docs/guides/fabric_synchronization_guide.md\r\n\r\nCo-authored-by: saurabhst \u003cs.kumar9@samsung.com\u003e\r\n\r\n* Address review comments\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\nCo-authored-by: saurabhst \u003cs.kumar9@samsung.com\u003e\r\n\r\n* [Darwin] Duplicate / begin raising `MTRDeviceController` (#34832)\r\n\r\n* Keep MTRDeviceController.mm\r\n\r\n* Copy MTRDeviceController.mm into MTRDeviceController_Concrete.mm\r\n\r\n* Set back MTRDeviceController.mm file\r\n\r\n* Keep MTRDeviceController.h\r\n\r\n* Copy MTRDeviceController.h into MTRDeviceController_Concrete.h\r\n\r\n* Set back MTRDeviceController.h file\r\n\r\n* add `MTRDeviceController_Concrete` files\r\n\r\n* WIP:  bring up `MTRDeviceController_Concrete`\r\n\r\n* temporary init implementation\r\n\r\n* only vend `MTRDeviceController` pointers\r\n\r\nnot concrete class (`instancetype`) pointers\r\n\r\n* add `storedFabricIndex` (currently broken)\r\n\r\n* reduce access of internal properties\r\n\r\n* move C++-style fabric index and keypairs to protected ivars\r\n\r\n* return base class pointer from constructors\r\n\r\nrather than `instancetype`\r\n\r\n* clean up property synthesis/ivars in concrete\r\n\r\n* update merged code\r\n\r\n* WIP:  get us building again, plus plan comments\r\n\r\n* remove superfluous comments\r\n\r\n* Update src/darwin/Framework/CHIP/MTRDeviceControllerFactory.h\r\n\r\n* Restyled by clang-format\r\n\r\n* move MTRDeviceController protected ivars\r\n\r\nto standard `MTRDeviceController.h` header\r\n\r\n* `MTRDeviceController_Concrete` isn\u0027t public API\r\n\r\nthus no `MTR_AVAILABLE`\r\n\r\n* remove commented code\r\n\r\n* move ivar extension back to internal header\r\n\r\n* Fixing TAPI\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n---------\r\n\r\nCo-authored-by: Justin Wood \u003cwoody@apple.com\u003e\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Update silabs docker image (#35011)\r\n\r\nCo-authored-by: Andrei Litvin \u003candy314@gmail.com\u003e\r\n\r\n* Make BRBINFO_4_1 require commissioning (#35017)\r\n\r\n* Make BRVINFO_4_1 require commissioning\r\n\r\n* Also add icd registration\r\n\r\n---------\r\n\r\nCo-authored-by: Andrei Litvin \u003candreilitvin@google.com\u003e\r\n\r\n* Disable TI builds from CI (#35024)\r\n\r\nCo-authored-by: Andrei Litvin \u003candreilitvin@google.com\u003e\r\n\r\n* Fix header usage in rvc (#35025)\r\n\r\nCo-authored-by: Andrei Litvin \u003candreilitvin@google.com\u003e\r\n\r\n* [Fabric-Admin] Add API to commission local bridge within its own fabric  (#35020)\r\n\r\n* Add local bridge handle API\r\n\r\n* Update examples/fabric-admin/commands/fabric-sync/FabricSyncCommand.cpp\r\n\r\nCo-authored-by: saurabhst \u003cs.kumar9@samsung.com\u003e\r\n\r\n* Update examples/fabric-admin/commands/fabric-sync/FabricSyncCommand.cpp\r\n\r\nCo-authored-by: saurabhst \u003cs.kumar9@samsung.com\u003e\r\n\r\n* Use VerifyOrDie to check pairingCommand\r\n\r\n---------\r\n\r\nCo-authored-by: saurabhst \u003cs.kumar9@samsung.com\u003e\r\n\r\n* [Android] Fix mDNS discover ANR issue (#34968)\r\n\r\n* Fix mDNS discover ANR issue\r\n\r\n* Restyled by google-java-format\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* [Android] Implement Establish PASE Connection for SetupCode (#34971)\r\n\r\n* Implement Establish PASE Connection for SetupCode\r\n\r\n* Restyled by google-java-format\r\n\r\n* Restyled by clang-format\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Adding XPC protocols for darwin (#35029)\r\n\r\n* Initial checkin\r\n\r\n* Cleaning up protocol\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Adding nodeID and controllerID for simple routing\r\n\r\n* Moving to implied Non-null\r\n\r\n* Removing these\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Set FabricFiltered to True when reading Fabrics. (#35023)\r\n\r\nOtherwise we get a fabric view on the list of fabrics which does\r\nnot change over time.\r\n\r\nCo-authored-by: Andrei Litvin \u003candreilitvin@google.com\u003e\r\n\r\n* Fix BRBINFO_4_1 to run further (#35026)\r\n\r\n* Fix BRBINFO_4_1 to run further\r\n\r\n* Restyled by autopep8\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* fabric sync guide: fix device-remote-port (#35035)\r\n\r\nSigned-off-by: Doru Gucea \u003cdoru-cristian.gucea@nxp.com\u003e\r\n\r\n* [TI] Update TI BLE Manager Task priority (#34852)\r\n\r\n* Update TI BLE Manager task priority for initialization sequence\r\n\r\n* Restyled by clang-format\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Use RootEndponId for reading SupportsConcurrentConnection in CNET_1_4 (#34972)\r\n\r\n* [Telink] Disable Wi-Fi LPM during the OTA (#35006)\r\n\r\n* [Telink] Add OTAImageProcessorImplWiFi.h\r\n\r\n* [Telink] remove IPv4 temporarry debug configs\r\n\r\n* [Telink] w91 remove not connected PWM channels (#35005)\r\n\r\n* telink: w91 remowe PWM channels\r\n\r\nSigned-off-by: Dmytro Kashkarov \u003cur6lal@gmail.com\u003e\r\n\r\n* Restyled by whitespace\r\n\r\n---------\r\n\r\nSigned-off-by: Dmytro Kashkarov \u003cur6lal@gmail.com\u003e\r\nCo-authored-by: Dmytro Kashkarov \u003cur6lal@gmail.com\u003e\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Optimize RAM utilization for WriteHandler (#34992)\r\n\r\n* Pull RAM usage reduction from write impl\r\n\r\n* remove some extra added includes\r\n\r\n* Apply review comment from previous PR\r\n\r\n* [Tizen] Implement GetThreadVersion() for Thread driver (#34976)\r\n\r\n* [Tizen] Implement GetThreadVersion in thread stack manager\r\n\r\n* Move otThreadGetVersion() usage to Thread stack manager class\r\n\r\n* Update C++ style\r\n\r\n* Make GetThreadVersion public interface\r\n\r\n* Implement mock _GetThreadVersion() for Linux, NuttX and WebOS\r\n\r\n* Copy over data model flags (#34995)\r\n\r\n* TC-IDM-10.5: Plumb through allow_provisional flag (#35037)\r\n\r\n* Fixes the Service Area server\u0027s `SkipArea` handling (#35019)\r\n\r\n* Updated the rvc-example PICS XML files.\r\n\r\n* update the handling of the SkipArea command to match the latest spec.\r\n\r\n* Updated the name of SkipCurrentArea to SkipArea in the Instance and Delegate classes.\r\n\r\n* Refactored SkipCurrentArea names to SkipArea in the RVC example app.\r\n\r\n* Restyled by clang-format\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Minimal TC_SWTCH fixes for 1.4-TE2 (#34986)\r\n\r\n- Fixes only issues raised on script that block success\r\n  - https://github.com/project-chip/matter-test-scripts/issues/337\r\n  - https://github.com/project-chip/matter-test-scripts/issues/338\r\n\r\n- Fixes done:\r\n  - Prompt text improvement/fixes\r\n  - Fixed crash in 2.5/2.6\r\n\r\n* [MCORE.FS 1.3] make \"app not found\" easier to debug (#35038)\r\n\r\n* Remove unused method, make missing app error easier to find\r\n\r\n* Remove more todo for confirm we can run apps: we confirmed we can\r\n\r\n---------\r\n\r\nCo-authored-by: Andrei Litvin \u003candreilitvin@google.com\u003e\r\n\r\n* [FS Example] Update the FS Example apps to support fabric sync setup process part II (#34990)\r\n\r\n* examples/fabric-bridge-app/linux/CommissionerControl.cpp\r\n\r\n* Implement reverse commissioning\r\n\r\n* Update function names to align with spec\r\n\r\n* Update compatible builds to docker version 74 (#35009)\r\n\r\n* Check which build can update to latest version\r\n\r\n* temporarry use docker version 72\r\n\r\n* revert image for failed builds\r\n\r\n* downgrade K32W image\r\n\r\n* use K32W docker image 69\r\n\r\n* Update compatible builds to docker version 74\r\n\r\n* use docker version 73 for EFR32\r\n\r\n* Prevent cancellation of task with timeout in asyncio runner.\r\n\r\nI am yet unclear why on new python/os/websocket we do not get\r\na timeout error, however from what I could tell, wait_for would\r\ncancel the websocket.recv and that stops instead of throwing\r\na timeout exception, resulting in a full success result.\r\n\r\nMaking this change seems to make the test TestPurposefulFailureExtraReportingOnToggle\r\npass on my machine (well ... fail as expected instead of passing with a\r\nsuccessful stop)\r\n\r\n---------\r\n\r\nCo-authored-by: Andrei Litvin \u003candreilitvin@google.com\u003e\r\n\r\n* Fix test code build issues when INET_CONFIG_TCP_ENDPOINT is set to false (#35027)\r\n\r\n* Prevent cancellation of task with timeout in asyncio runner. (#35045)\r\n\r\nI am yet unclear why on new python/os/websocket we do not get\r\na timeout error, however from what I could tell, wait_for would\r\ncancel the websocket.recv and that stops instead of throwing\r\na timeout exception, resulting in a full success result.\r\n\r\nMaking this change seems to make the test TestPurposefulFailureExtraReportingOnToggle\r\npass on my machine (well ... fail as expected instead of passing with a\r\nsuccessful stop)\r\n\r\nCo-authored-by: Andrei Litvin \u003candreilitvin@google.com\u003e\r\n\r\n* TC-IDM-10.2: Fix error message (#35044)\r\n\r\n* [Darwin] New XPC Service (#34961)\r\n\r\n* empty xpc service files\r\n\r\n* XPC service needs to be C++ to use MTRLogging\r\n\r\n* add tests; stub method signature\r\n\r\n* start configuring test XPC listener\r\n\r\n* add first test method to XPC service protocol\r\n\r\n* tests: remove peformance test; add headers\r\n\r\n* make MTRXPCService testable\r\n\r\n* more working lines of XPC listener setup\r\n\r\n* add dummy XPC service for testing\r\n\r\n* add replying method to service protocol / implementation\r\n\r\n* remove log - reply contents can be asserted\r\n\r\n* make test interesting\r\n\r\n* linty formatty\r\n\r\n* remove personal development team from pbxproj\r\n\r\n* fix new file headers\r\n\r\n* less whitespace\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* add new XPC device controller\r\n\r\n* sketch out a first XPC method\r\n\r\n* store WIP for rebase\r\n\r\n* fix pbxproj merge mistake\r\n\r\n* make WIP slightly more sensible\r\n\r\n* WIP: remote proxy obj in `MTRDeviceController_XPC`\r\n\r\n* more experiments\r\n\r\n* WIP:  return type issue\r\n\r\nrun `MTRXPCServiceTests testMTRXPCServiceSetup` and see selection of proxy object method in `MTRDeviceController_XPC.m`/`initWithXPCListenerEndpointForTesting` around line 44 to see the problem.\r\n\r\n* add `MTRDevice_XPC`\r\n\r\nnothing exciting yet\r\n\r\n* add device controller over XPC parameters stub\r\n\r\n* add init split for XPC device controllers\r\n\r\nnot yet implemented, but there\r\n\r\n* rename new XPC parameters class\r\n\r\n`OverXPC` is already taken / means something else\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* remove failing exploratory test\r\n\r\n* lintfix\r\n\r\n* Examples\r\n\r\n* quickly attempt to fix test\r\n\r\nunused variable -\u003e warning -\u003e error\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* inevitably the file will need to be obj-C++\r\n\r\n* prepare to use mach service name to create XPC connection\r\n\r\n* allow for other XPC controller connection types for now\r\n\r\nonce mach service is working, i don\u0027t expect to need others, but easier to collapse classes than raise.\r\n\r\n* constant for well-known matter plugin service name\r\n\r\n* note to self and re-ordering of this code now what `XPCParameters` is a subclass\r\n\r\n* XPC parameters should support secure coding\r\n\r\n* don\u0027t keep around object proxy; make a new one when needed.  try calling the `checkIn` method when connecting.  remove old test methods that served their purpose.\r\n\r\n* that doesn\u0027t need to be there.\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* no longer fighting with compiler, have some `instancetype`\r\n\r\n* extremely normal object construction\r\n\r\n* somehow this header got thrown out\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Updating XPC interfaces\r\n\r\n* hide `initWithMachServiceName:options:` constructor\r\n\r\ntvOS/iOS compilation issues need a look\r\n\r\n* make `MTRDeviceController_XPC` an XPC client\r\n\r\nvia `MTRXPCClientProtocol_MTRDeviceController`, which for the moment is empty\r\n\r\n* remove initial tests\r\n\r\nthey served their purpose well but are no longer relevant\r\n\r\n* call mach service constructor when appropriate\r\n\r\nalso logging\r\n\r\n* remove some obsolete test methods\r\n\r\n* more obsoletes fixed\r\n\r\n* more obsolete removal and logging tweaks\r\n\r\n* buildability tweaks\r\n\r\n* Moving along\r\n\r\n* Fixing header\r\n\r\n* Fixing macros and codable\r\n\r\n* Adding invoke?\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Actually hooking up to XPCConnection\r\n\r\n* Actually hooking up to XPCConnection\r\n\r\n* Adding XPC Parameters\r\n\r\n* Adding XPC Parameter hookup\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* true \u003d\u003e YES\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: Justin Wood \u003cwoody@apple.com\u003e\r\n\r\n* [Darwin] more XPC service tweaks (#35048)\r\n\r\n* return `MTRDevice_XPC`s from XPC controller\r\n\r\n* more logging\r\n\r\n* move shadow property declarations to internal header\r\n\r\n* declare `_setupDeviceForNodeID` as common internal device controller method\r\n\r\n* prefetchedClusterData is nullable\r\n\r\n* fix a few properties that needed raising to base class\r\n\r\n* you get a log and you get a log EVERYONE GETS A LOG\r\n\r\n* convert device map lock for use in subclasses\r\n\r\n* check for optional delegate method impl before calling\r\n\r\n* ivar no longer necessary with accessor method\r\n\r\nunderlying lock is the only state needed\r\n\r\n* Restyled by clang-format\r\n\r\n* remove more obsolete lock bits from `MTRDeviceController_XPC`\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* ModeSelect: Add setter API for SupportedModesManager. (#34809)\r\n\r\n* Add setter API for SupportedModesManager\r\n\r\n* Example changes\r\n\r\n* Restyled by clang-format\r\n\r\n* Fix namespace usage\r\n\r\n* Address review comments\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Python testing: Fix reporting on setup_class error (#35016)\r\n\r\n* Python testing: Fix reporting on setup_class error\r\n\r\nAlso add error text to make the error easier to find.\r\n\r\n* Restyled by isort\r\n\r\n* Fix lint\r\n\r\n* Change exemption to not carry chip_error reference\r\n\r\nchip_error is a ctypes struct with a const char* pointer internally.\r\nThis cannot be pickled, so it\u0027s causing problems with the mobly\r\nframework.\r\n\r\n* Fix some tests using removed class member\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Fixing darwin crashes, and XPC invalidation (#35056)\r\n\r\n* Fixing crashes\r\n\r\n* Restyled by clang-format\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* [TI] Switch Application for CC13x4_26x4 (#34916)\r\n\r\n* Switch Application for TI CC13x4_26x4\r\n\r\n* Added light-switch to CI and restyled changes\r\n\r\n* OOB configs\r\n\r\n* CI fix\r\n\r\n* CI fix in all_targets_linux_x64.txt\r\n\r\n* revert CI changes for switch-app\r\n\r\n* updated args.gni comment for logging\r\n\r\n* [Fabric-Admin] Fix the new added device failed to get reported (#35058)\r\n\r\n* [Fabric-Admin] Fix the new added device failed to get reported\r\n\r\n* Restyled by prettier-markdown\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Fix failed to sync device when using dynamic setupPIN (#35061)\r\n\r\n* improve `MTRDeviceController`/`_Concrete` (#35060)\r\n\r\nlogging\r\n\r\n* Updates to BRBINFO_4_1 after issues discovered during TE2 (#35040)\r\n\r\n* Updates to BRBINFO_4_1 after issues discovered during TE2\r\n\r\n* Update test step 3 text\r\n\r\n* Restyled by autopep8\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Add fabric scoping to ECOINFO cluster attributes (#35022)\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: Andrei Litvin \u003candy314@gmail.com\u003e\r\n\r\n* RVC DirectModeChange Feature Update for RVCCLEANM and RVCRUNM (#34994)\r\n\r\n* Python Scripts TC_RVCCLEANM_2_2.py and TC_RVCRUNM_2_2.py updated\r\n* Updated the python script TC_RVCCLEANM_2_2.py with two new steps \u00277a\u0027 For reading FeatureMap Attribute and \u00277b\u0027 for send ChangeToMode Command\r\n\r\n* Updated the python script TC_RVCRUNM_2_2.py with two new steps \u00276a\u0027 For reading FeatureMap Attribute and \u00276b\u0027 for send ChangeToMode Command\r\n\r\n* Updated TC_RVCOPSTATE_2_4.py with updated PICS as per Test Plan\r\n\r\n* Python Script TC_RVCOPSTATE_2_4.py\r\n\r\n* Updated TC_RVCOPSTATE_2_4.py with steps numbers update\r\n\r\n* Restyled by autopep8\r\n\r\n* Python Scripts TC_RVCCLEANM_2_2.py and TC_RVCRUNM_2_2.py updated\r\n* Updated the python script TC_RVCCLEANM_2_2.py with two new steps \u00277a\u0027 For reading FeatureMap Attribute and \u00277b\u0027 for send ChangeToMode Command\r\n\r\n* Updated the python script TC_RVCRUNM_2_2.py with two new steps \u00276a\u0027 For reading FeatureMap Attribute and \u00276b\u0027 for send ChangeToMode Command\r\n\r\n* Updated TC_RVCOPSTATE_2_4.py with updated PICS as per Test Plan\r\n\r\n* Python Script TC_RVCOPSTATE_2_4.py\r\n\r\n* Updated TC_RVCOPSTATE_2_4.py with steps numbers update\r\n\r\n* Restyled by autopep8\r\n\r\n* Python Script TC_RVCRUNM_2_2.py and TC_RVCCLEANM_2_2.py\r\n\r\n* Updated TC_RVCRUNM_2_2.py and TC_RVCCLEANM_2_2.py as per comments from ROB\r\n\r\n* Restyled by autopep8\r\n\r\n* Updated CI PICS Value For TC_RVCOPSTATE_2_4.py\r\n\r\n* Updated PICS.yaml For TC_RVCOPSTATE_2_4.py\r\n\r\n* Updated CI PICS file by removing RVCOPSTATE.S.C128.Rsp\r\n\r\n* Updated RVC PICS value at location examples/rvc-app/rvc-common/pics/rvc-app-pics-values\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* [Fabric-Admin] We should only start the reverse commission process if the local fabric bridge is ready. (#35079)\r\n\r\n* [Fabric-Admin] We should only start the reverse commission process if the local fabric is ready.\r\n\r\n* Fix compile error\r\n\r\n* [ICD] Add ICDM 3.2 Python Certification Script (#34782)\r\n\r\n* Add ICDM 3.2 Python Certification Script\r\n\r\n* fix restyle\r\n\r\n* Restyled by prettier-json\r\n\r\n* Restyled by isort\r\n\r\n* restyle and linter fix\r\n\r\n* Add CI test arguments\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* [FabricAdmin] Refactor FS specific attribute handlings out of general HandleAttributeData function (#35080)\r\n\r\n* [FabricAdmin] Refactor attribute Data handling\r\n\r\n* Use reference to pass chip::TLV::TLVReader\r\n\r\n* Update examples/fabric-admin/device_manager/DeviceManager.h\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\n\r\n* Update function name\r\n\r\n---------\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\n\r\n* Move step (#35070)\r\n\r\n* Use PRIu32 instead of %u to print unsigned integer (#35069)\r\n\r\n* soc: riscv: telink_b9x, telink_w91: Auto calculation of SETTINGS_NVS_SECTOR_COUNT (#34958)\r\n\r\n- Remove sector count of NVS from metter kconfig\r\n\r\n- Sector count of NVS now calculates on Zephyr side as the storage partition size divided by the sector size (4 KB)\r\n\r\nSigned-off-by: Borys Nykytiuk \u003cborys.nykytiuk@telink-semi.com\u003e\r\n\r\n* riscv: telink: tlsr9528a_4m_flash.overlay (#34945)\r\n\r\n- follow telink flash rules .\r\n\r\nSigned-off-by: Haiwen Xia \u003chaiwen.xia@telink-semi.com\u003e\r\nCo-authored-by: Haiwen Xia \u003chaiwen.xia@telink-semi.com\u003e\r\n\r\n* [Silabs] Update Silabs sdks version and related enablement (#35083)\r\n\r\n* Update all sdks to new verions\r\n\r\n* 9116 changes to support sdk update\r\n\r\n* 917 changes to support sdk update\r\n\r\n* Update to provision storage for configurable address\r\n\r\n* bump matter_support submodule\r\n\r\n* Bump docker image version used for silabs ci\r\n\r\n* Restyled by clang-format\r\n\r\n* Fix wifi ncp builds\r\n\r\n---------\r\n\r\nCo-authored-by: Mathieu Kardous \u003cmathieu.kardous@silabs.com\u003e\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* In TC-ICDM-3.4 make sure that a new session will be used after DUT reboots (#35092)\r\n\r\n* after DUT reboots make sure that a new session will be used\r\nfixes #34985\r\n\r\n* updated comment to reflect closing the session than expiring it\r\n\r\n* re-establish session only when not in CI\r\n\r\n* [Darwin] MTRDeviceController_XPC client protocol xpc interface fix (#35087)\r\n\r\n* Fix SkipArea logic and removed the use of memcpy (#35075)\r\n\r\n* Updated the SkipArea serever handle to not error if the CurrentArea is null, according to the spec updates.\r\n\r\n* Replaced the use of memcpy with CopyCharSpanToMutableCharSpan.\r\n\r\n* Restyled by clang-format\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Move delegate management into shared MTRDevice super-class. (#35084)\r\n\r\n* Move delegate management into shared MTRDevice super-class.\r\n\r\nMTRDevice_XPC and MTRDevice_Concrete can then share that code.\r\n\r\n* Address review comments, fix TAPI build.\r\n\r\n* [Darwin] MTRDevice_XPC delegate callbacks need to hold lock before _callDelegatesWithBlock (#35095)\r\n\r\n* [Darwin] MTRDevice_XPC delegate callbacks need to hold lock before _callDelegatesWithBlock\r\n\r\n* Add back _callDelegatesWithBlock to avoid conflict\r\n\r\n* Add plumbing that allows fabric-bridge endpoints to report CADMIN attr (#35076)\r\n\r\n* Make the MTRDevice ivars protected. (#35101)\r\n\r\nIt seems like by default we have the following visibility options here:\r\n\r\n@private - subclasses can\u0027t touch, not workable.\r\n@public - anyone can touch, not desirable.\r\n@package - @public inside Matter.framework, @private ouside it. Does not export\r\n           the symbols, but anyone inside Matter.framework can touch.\r\n@protected - only subclasses can touch, but exports the symbols in case we have\r\n             out-of-framework subclasses who want to touch it.  Since the\r\n             declarations are in a project header that TAPI does not know about\r\n             in release builds, we get complaints about mismatches between\r\n             what\u0027s declared public and what\u0027s exported.\r\n\r\nWhat we would really want here is \"@protected inside Matter.framework, @private\r\nouside it\", but that does not exist.  So this switches to @protected, and uses\r\nlinker arguments to not export the symbols in release builds.  Since the header\r\nitself is not public, this accomplishes the same goal.  In debug builds, we do\r\nexpose project headers to TAPI, hence there we want to keep exporting the\r\nsymbols.\r\n\r\nThe linker arguments just prevent exporting all ivar symbols, since we shouldn\u0027t\r\nbe exporting any of those anyway.\r\n\r\n* IDM-12.1: Also dump to log (#35098)\r\n\r\n* IDM-12.1: Also dump to log\r\n\r\n* simplify print command\r\n\r\n* Address post merge comment (#35103)\r\n\r\n* Address post merge comment\r\n\r\n* Quick fix\r\n\r\n* Update default file header for Matter.framework files. (#35106)\r\n\r\nRemoves blank line at the top, makes copyright year be saner.\r\n\r\n* Add availability annotations to new XPC protocols. (#35107)\r\n\r\n* TC-IDM-10.2: Add check for MACL (#35086)\r\n\r\n* TC-IDM-10.2: Add check for MACL\r\n\r\nTest: unit tests. Note this has not been tested against the example\r\n      app as it is still under development.\r\n\r\n* Update src/python_testing/TC_DeviceConformance.py\r\n\r\nCo-authored-by: Tennessee Carmel-Veilleux \u003ctennessee.carmelveilleux@gmail.com\u003e\r\n\r\n* Update src/python_testing/TestConformanceTest.py\r\n\r\n---------\r\n\r\nCo-authored-by: Tennessee Carmel-Veilleux \u003ctennessee.carmelveilleux@gmail.com\u003e\r\n\r\n* [Darwin] New XPC bits need to initialize properly (#35110)\r\n\r\n* Revert \"Enabling this for macOS Catalyst apps (#34956)\" (#35117)\r\n\r\nThis reverts commit 1d2ddd4131f3b6a10092d93a74ea58f0fe09510b.\r\n\r\n* [Fabric-Sync] Update the FS setup guide to wait for reverse commissioning complete (#35115)\r\n\r\n* [Fabric-Sync] Update the FS setup guide to wait for reverse commissing\r\n\r\n* Restyled by prettier-markdown\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* [Fabric-Sync] Add VID/PID fields to IPC method CommissionNode (#35116)\r\n\r\n* Add VID/PID field to IPC methode CommissionNode\r\n\r\n* Use QRCodeSetupPayloadGenerator to generate pairing code\r\n\r\n* Revert \"TC-IDM-10.2: Add check for MACL (#35086)\" (#35111)\r\n\r\nThis reverts commit 927c81857ae9288ffab4ffbd529979412f8cc5f1.\r\n\r\n* Fix UnregisterAllCommandHandlersForEndpoint to work correctly. (#35100)\r\n\r\nFixes https://github.com/project-chip/connectedhomeip/issues/34953\r\n\r\n* Add missing CFLAGS to instrument things for fuzzers. (#35082)\r\n\r\n-fno-sanitize-coverage\u003dpc-table is added since we are already using\r\ninline-8bit-counters and trace-cmp to provide instrumentation for the\r\nfuzzers. It also helps reduce memory usage and improve fuzzer performance.\r\n\r\n* adding mandatory endpointComposition data to XML for ZAP to consume (#34818)\r\n\r\n* adding endpointComposition data to XML for ZAP to consume\r\n\r\n* explicitly adding constraint as defined by the spec and defining compositions types\r\n\r\n* fix composition type tag\r\n\r\n* changing back to prevent both types being true which is not allowed\r\n\r\n* explciitly setting conformance and contraint to account for all scenarios\r\n\r\n* Colorcontrol xml file sync with specs (#33612)\r\n\r\n* doorlock xml file sync with specs and colorcontrol xml type changes\r\n\r\n* build error resolved\r\n\r\n* Restyled by clang-format\r\n\r\n* build error resolved\r\n\r\n* addressed review comments\r\n\r\n* Restyled by clang-format\r\n\r\n* Addressed review comments\r\n\r\n* Restyled by clang-format\r\n\r\n* build error solved\r\n\r\n* build error solved\r\n\r\n* build error solved\r\n\r\n* build error solved\r\n\r\n* build issue resolved\r\n\r\n* Restyled by clang-format\r\n\r\n* resolved build errors\r\n\r\n* removed doorlock xml changes\r\n\r\n* Restyled by clang-format\r\n\r\n* resolved build errors\r\n\r\n* Restyled by clang-format\r\n\r\n* resolved build errors\r\n\r\n* addressed review comments\r\n\r\n* addressed review comments\r\n\r\n* resolved build errors\r\n\r\n* resolved build errors\r\n\r\n* zap generated changes\r\n\r\n* Restyled by clang-format\r\n\r\n* Revert colorloopdirection type change\r\n\r\n* build error resolved\r\n\r\n* Restyled by clang-format\r\n\r\n* rebased and zap generated\r\n\r\n* color control xml file updated\r\n\r\n* build error solved\r\n\r\n* ameba build error resolved\r\n\r\n* Restyled by clang-format\r\n\r\n* ameba build error resolved\r\n\r\n* doorlock changes removed\r\n\r\n* ameba build error resolved\r\n\r\n* ameba build error resolved\r\n\r\n* addressed review comments\r\n\r\n* addressed review comments\r\n\r\n* Restyled by clang-format\r\n\r\n* Addressed review comments\r\n\r\n* Addressed review comments\r\n\r\n* Restyled by clang-format\r\n\r\n* build error resolved\r\n\r\n* build error resolved\r\n\r\n* addressed reviews comments and build error resolved\r\n\r\n* Restyled by clang-format\r\n\r\n* ZAP issue resolved\r\n\r\n* scripts updated\r\n\r\n* scripts updated\r\n\r\n* added enum values in compatenum.h file\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* added enum values in compatenum.h file\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* added enum values in compatenum.h file\r\n\r\n* added header file for to_underlying\r\n\r\n* build error resolved\r\n\r\n* build error resolved\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Update Darwin availability annotations. (#35108)\r\n\r\n* Make descriptions for MTRDevice clearly say whether it\u0027s the XPC version. (#35112)\r\n\r\nThis requires hoisting _nodeID and _deviceController ivars clearly into the\r\nMTRDevice superclass, so they can be accessed from subclasses.\r\n\r\nThe XPC version does not have a bunch of the state the non-XPC one does, so for\r\nnow it does not try to log that\r\n\r\n* removed invalid symlinks (#35129)\r\n\r\n* Update MCORE_FS_1_2 to latest testplan (#35097)\r\n\r\n* Removed Arithmetics operations relying on Unit test cluster from TC_S_2_2 and TC_S_2_3 (#35130)\r\n\r\n* Add expiry to KeepActive as per spec into fabric-admin example (#35099)\r\n\r\n* Add expiry to KeepActive as per spec into fabric-admin example\r\n\r\n* Restyled by clang-format\r\n\r\n* Self Review fix\r\n\r\n* Restyled by clang-format\r\n\r\n* Name fix\r\n\r\n* Address PR comments\r\n\r\n* Address PR comments\r\n\r\n* Restyled by clang-format\r\n\r\n* Address PR comments\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Move TC_BR_5 to the right directory (#35128)\r\n\r\n* Update the preset handle field in the SetActivePresetRequest c… (#35105)\r\n\r\n* Update the preset handle field in the SetActivePresetRequestRequest command to be nullable\r\n\r\n- Update the deleagte API for getting the active preset handle to return a nullable handle\r\n\r\n* Return error if CopySpanToMutableSpan fails\r\n\r\n* Restyled by clang-format\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* Fix fabric-bridge is accidentally added into sync list (#35132)\r\n\r\n* Yaml scripts update (#35093)\r\n\r\n* Removed scripts Test_TC_SC_4_10.yaml and Test_TC_SC_4_5.yaml as Test plan have removed them\r\n\r\n* step numbers corrected  in Test_TC_CC_7_3.yaml and action id is updated in Test_TC_ACT_3_2.yaml\r\n\r\n* Removed scripts Test_TC_SC_4_10.yaml and Test_TC_SC_4_5.yaml as Test plan have removed them\r\n\r\n* step numbers corrected  in Test_TC_CC_7_3.yaml and action id is updated in Test_TC_ACT_3_2.yaml\r\n\r\n* Reverted Action of Test_TC_ACT_3_2.yaml\r\n\r\n* [Darwin] XPC interface for invoke needs to take into account the reply may contain non-property-list objects (#35131)\r\n\r\n* Fix Darwin to handle renames of Color Control data types. (#35135)\r\n\r\nhttps://github.com/project-chip/connectedhomeip/pull/33612 made the following changes to the Color Control cluster:\r\n\r\n1) Renamed HueDirection to DirectionEnum\r\n2) Renamed the ShortestDistance and LongestDistance values of DirectionEnum to\r\n   Shortest and Longest, respectively.\r\n3) Renamed HueMoveMode to MoveModeEnum.\r\n4) Renamed HueStepMode to StepModeEnum.\r\n5) Removed SaturationMoveMode in favor of MoveModeEnum.\r\n6) Removed SaturationStepMode in favor of StepModeEnum.\r\n7) Renamed ColorMode to ColorModeEnum.\r\n8) Renamed the ColorTemperature value to ColorTemperatureMireds\r\n8) Renamed ColorCapabilities to ColorCapabilitiesBitmap.\r\n9) Renamed various fields of ColorCapabilitiesBitmap.\r\n10) Renamed ColorLoopUpdateFlags to UpdateFlagsBitmap.\r\n11) Renamed ColorLoopAction to ColorLoopActionEnum.\r\n12) Added OptionsBitmap, EnhancedColorModeEnum, DriftCompensationEnum.\r\n13) Renamed ColorLoopDirection to ColorLoopDirectionEnum.\r\n14) Renamed the DecrementHue and IncrementHue values of ColorLoopDirectionEnum\r\n    to Decrement and Increment, respectively.\r\n\r\nThis change adds the right renamed/introduced/deprecated annotations for the\r\nabove changes, and adds manual shims for the enums that got removed.\r\n\r\n* Re-route deprecated android casting APIs to new ones (#35136)\r\n\r\n* Remove subscription establishment bits from the MTRDevice base class. (#35139)\r\n\r\n_setupSubscriptionWithReason was only called from\r\n_reattemptSubscriptionNowIfNeededWithReason.\r\n\r\n_reattemptSubscriptionNowIfNeededWithReason was only called from:\r\n\r\n* _handleUnsolicitedMessageFromPublisher, which is only called in callbacks set\r\n   up by _setupSubscriptionWithReason.\r\n* _triggerResubscribeWithReason, which only calls it when\r\n   _reattemptingSubscription is YES.  But that\u0027s only set to YES in\r\n   _doHandleSubscriptionReset, see next item.\r\n* _doHandleSubscriptionReset, called from:\r\n   * _handleSubscriptionReset: only called from callbacks set up by\r\n     _setupSubscriptionWithReason.\r\n   * _resetSubscriptionWithReasonString: called from unitTestResetSubscription\r\n     and us detecting persisted data corruption in _clusterDataForPath.  But we\r\n     don\u0027t actually have persisted data in the MTRDevice base class anymore, so\r\n     that\u0027s not reachable.\r\n\r\nSo _setupSubscriptionWithReason is not reachable and can be removed.  And\r\nunitTestResetSubscription can be made a no-op on the base MTRDevice.  And we can\r\nremove the _resetSubscriptionWithReasonString call in _clusterDataForPath.\r\n\r\nOnce those are done, the following become unreachable:\r\n\r\n* _handleSubscriptionReset\r\n* _handleSubscriptionError\r\n* Writing a nonzero value to _unitTestAttributesReportedSinceLastCheck\r\n* _markDeviceAsUnreachableIfNeverSubscribed\r\n* _handleResubscriptionNeededWithDelay\r\n* _handleSubscriptionEstablished\r\n* _handleUnsolicitedMessageFromPublisher\r\n* _createDataVersionFilterListFromDictionary\r\n* _setupConnectivityMonitoring\r\n* _resetSubscriptionWithReasonString\r\n\r\nOnce those have been removed, the following become unreachable:\r\n\r\n* HadSubscriptionEstablishedOnce\r\n* _stopConnectivityMonitoring\r\n* _doHandleSubscriptionError\r\n* _doHandleSubscriptionReset\r\n\r\nOnce those have been removed, the following become unreachable:\r\n\r\n* _setLastSubscriptionAttemptWait\r\n* Setting _lastSubscriptionFailureTime to non-nil\r\n* _clearSubscriptionPoolWork\r\n* _scheduleSubscriptionPoolWork\r\n* Use of _initialSubscribeStart\r\n* _storePersistedDeviceData\r\n* Use of _connectivityMonitor\r\n* Use of _lastSubscriptionFailureTimeForDescription\r\n* Any code depending on reattemptingSubscription being YES.\r\n* Use of _lastSubscriptionAttemptWait\r\n* Use of _lastSubscriptionAttemptWaitForDescription\r\n\r\nOnce those have been removed, _reattemptSubscriptionNowIfNeededWithReason is\r\nunreachable and can be removed.\r\n\r\nAt this point _currentReadClient and _currentSubscriptionCallback are always\r\nnil, so all code conditioned on those can be removed.  Once that\u0027s done,\r\n_triggerResubscribeWithReason is a no-op, so it and all calls to it can be\r\nremoved.\r\n\r\nSince _lastSubscriptionFailureTime is always nil, all code guarded on it not\r\nbeing nil can be removed, so _readThroughSkipped becomes a no-op and\r\n_lastSubscriptionFailureTime can be removed.\r\n\r\nAt this point, the following are not reachable:\r\n\r\n* NeedToStartSubscriptionSetup\r\n* _changeInternalState\r\n\r\nOnce those are removed, _internalDeviceStateForDescription is unused, and\r\n_internalDeviceState is always MTRInternalDeviceStateUnsubscribed.  So\r\nHaveSubscriptionEstablishedRightNow() always returns NO and can be removed.\r\n\r\nAt this point SubscriptionCallback is unused and can also be removed.\r\n\r\n* Enhance chip-tool\u0027s formatting for some IDs (#35088)\r\n\r\n* Added logging functions\r\n\r\n* Modify ZAP template file generation\r\n\r\n* Updated files for CommandId support\r\n\r\n* Updated manual tests for the new formatting\r\n\r\n* Fixed variable name shadowing\r\n\r\n* Fixed typo\r\n\r\n* Added logging functions to fabric admin\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Fixed linking\r\n\r\n* Fixed typo\r\n\r\n* Fixed CommandId logging\r\n\r\n* Updated ZAP generated files\r\n\r\n* Restyled by clang-format\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Truncate map and area names (#35127)\r\n\r\n* Updated the setting of the area and map names so that if the name given is greater than the buffer size, it\u0027s truncated rater to set to empty.\r\n\r\n* Restyled by clang-format\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Add new CHIP_CONFIG_ADDRESS_RESOLVE_MIN_LOOKUP_TIME_MS and (#35102)\r\n\r\nCHIP_CONFIG_ADDRESS_RESOLVE_MAX_LOOKUP_TIME_MS macros to CHIPConfig.\r\nUse new lookup time min/max macros inside AddressResolve for default\r\nkMinLookupTimeMsDefault and kMaxLookupTimeMsDefault to allow for applications\r\nto override the time required to wait for an address lookup before timing\r\nout.\r\n\r\n* Fix CHIP REPL tests runner after changes in e407d40 (#34453)\r\n\r\n* Fix CHIP REPL tests runner after changes in e407d40\r\n\r\nThe click framework does not have a support for async functions. The\r\nasync needs to be synchronized before applying click wrappers.\r\n\r\n* Accept 0x, 0b or 0o prefix for int values\r\n\r\n* Fix for non-string numbers\r\n\r\n* Exclude Test_TC_BRBINFO_2_1 from chip-repl engine\r\n\r\n* Log what happened in case of pseudo cluster creation failure\r\n\r\n* Fix typo\r\n\r\n* Fix typo when accessing TestGlobalStruct\r\n\r\n* Fix new line\r\n\r\n* use CHIP_DEVICE_CONFIG_FAILSAFE_EXPIRY_LENGTH_SEC define to arme the failsafe (#35137)\r\n\r\n* Remove unnecessary invokeCommand overrides from MTRDevice_Concrete. (#35151)\r\n\r\nThe one part that is not shared with the XPC implementation is\r\n_invokeCommandWithEndpointID:....  Everything else is just generic argument\r\nmassaging and forwarding that can keep living in the base MTRDevice.\r\n\r\n* Remove readAttributeWithEndpointID implementation from MTRDevice. (#35150)\r\n\r\nThis is implemented (differently) by the different subclasses.\r\n\r\nOnce this implementation is removed, the following become unreachable and can be\r\nremoved:\r\n\r\n* _attributeValueDictionaryForAttributePath\r\n* _subscriptionAbleToReport\r\n* _readThroughSkipped\r\n\r\nAt that point _subscriptionsAllowed becomes unreachable and can be removed.\r\n\r\n* Fix paths for fabric apps compiled with build_examples.py (#35123)\r\n\r\n* Fix paths for fabric apps compiled with build_examples.py\r\n\r\n* Require RPC to be enabled when building fabric-admin and fabric-bridge\r\n\r\n* [Fabric-Sync] Allow to specify custom port in fabric-bridge (#35146)\r\n\r\n* [Darwin] XPC invoke should implement bottom-most method (#35159)\r\n\r\n* [Darwin] XPC invoke should implement bottom-most method\r\n\r\n* Restyled fix\r\n\r\n* Update PositionTag and AreaDesc field names (#35094)\r\n\r\n* Updated the PositionTag and AreaDesc field names in the service area XML.\r\n\r\n* Zap generated after XML update.\r\n\r\n* Updated the PositionTag and AreaDesc field names in the SDK server code and test scripts.\r\n\r\n* Restyled by clang-format\r\n\r\n* Fixed issues caused by merge.\r\n\r\n* reverted changes in generated code.\r\n\r\n* encoding\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* AccountLogin Login/Logout command support (#34162)\r\n\r\n* AccountLogin Login/Logout command support\r\n\r\n[Problem]\r\n* ContentAppPlatform should send AccountLogin::Login command after\r\nsuccessfull commissioning if user was shown setupPIN prompt.\r\n* Handling Login/Logout commands is currently not implemented in the\r\n  Android.\r\n\r\n[Solution]\r\n* Call AccountLoginManager::HandleLogin if commissioning succeeded\r\n  successfully with setupPIN prompt flow.\r\n* Implement HandleLogin (and HandleLogout) for Android\r\n  AccountLoginManager using the ContentAppCommandDelegate.\r\n\r\n[Testing]\r\nWIP\r\n\r\n* HandleLogin in ContentAppPlatform::ManageClientAccess\r\n\r\n* Cache rotating ID in OnUserDirectedCommissioningRequest\r\n\r\n* Restyled by google-java-format\r\n\r\n* Restyled by clang-format\r\n\r\n* Update content app\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Update response\r\n\r\n* Update method name\r\n\r\n* Restyled by google-java-format\r\n\r\n* Update src/app/app-platform/ContentApp.h\r\n\r\nCo-authored-by: chrisdecenzo \u003c61757564+chrisdecenzo@users.noreply.github.com\u003e\r\n\r\n* Update content app platform validation\r\n\r\n* Update validation logic\r\n\r\n* Update logic\r\n\r\n* Update responses\r\n\r\n* Restyled by clang-format\r\n\r\n* Simplify logic\r\n\r\n* Restyled by whitespace\r\n\r\n* clean up\r\n\r\n* Update code\r\n\r\n* Update content app with dynamic pin code\r\n\r\n* Restyled by google-java-format\r\n\r\n* Remove getRotatingIdSpan class methods\r\n\r\n* Restyled by clang-format\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: Lazar Kovacic \u003clkovacic@amazon.com\u003e\r\nCo-authored-by: chrisdecenzo \u003c61757564+chrisdecenzo@users.noreply.github.com\u003e\r\n\r\n* [Fabric-Sync] Symplify the build instructions with build_examples.py (#35162)\r\n\r\n* Fix init of all TC-OCC-* tests (#35001)\r\n\r\n* Fix init of all TC-OCC-* tests\r\n\r\n- During TE2 if was found that all TC-OCC-* tests require\r\n  to pass a `--int-arg endpoint:N` argument which is not the right\r\n  way to specify endpoint. This is fixed here.\r\n- Some pre-steps were done in Step 1 which should have been done after Step 1\r\n  so that errors are not marked as Commissioning Failure when it\u0027s test failure.\r\n\r\nFixes https://github.com/project-chip/matter-test-scripts/issues/340\r\n\r\n* More fixes to OCC tests\r\n\r\n- 3.2 now properly using feature map\r\n- Fixed typos\r\n- Added queue flushing examples\r\n\r\n* Restyle\r\n\r\n* Fix line endings\r\n\r\n* Restyled by autopep8\r\n\r\n* Fix CI\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Remove DuplicatedAreas error (#35126)\r\n\r\n* Removed the DuplicatedAreas error from the XML\r\n\r\n* Generated code after XML update.\r\n\r\n* The service are server ignores any duplicate values before calling the delegate. Example was updated accodingly.\r\n\r\n* Updated test SEAR-1.3 following changes to the duplicated areas error.\r\n\r\n* Restyled by clang-format\r\n\r\n* Made select areas const.\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Mark return types nullable in MTRDevice_XPC when nil can be returned. (#35152)\r\n\r\n* check (#34998)\r\n\r\n* minimal_mdns: Fix filter for Compressed Fabric Id when browsing operational nodes (#35063)\r\n\r\n* minimal_mdns: Fix filter for Compressed Fabric Id when browsing operational nodes\r\n\r\n* Add check for subtype number\r\n\r\n* Docs re-organization (#34765)\r\n\r\n* Docs re-organization\r\n\r\nThe second phase of the handbook involves linking over to the SDK\r\ndocs from the handbook site. Before we do this, the SDK documentation\r\nneeds a re-org and an update. This is a continuation of the re-org,\r\nfollowing the platform updates. Remaining: Zap, tools, examples, IDs.\r\n\r\nDone in this PR:\r\n- remove API - this is a 4 year old design doc for a test dispatch\r\nsystem that was never implemented. This does not make sense to keep\r\nin the docs folder since it gives the incorrect impression that\r\nthis system was implemented at some point. In reality, our testing\r\nis quite different and is decently well documented in the testing section.\r\n\r\n- rename discussion to product consideration. This contains only the\r\nipv6 considerations doc, which isn\u0027t a discussion. Instead, we\u0027re\r\ngoing to use this location to land documents related to product\r\nconsiderations that affect matter, but are provided outside of the SDK,\r\nex. required network stack support (current doc), and factory\r\nconsiderations (coming later) for things like DACs, individually\r\nprovisioned values, QR codes, config options, certification\r\ndeclaration inclusion, DCL.\r\n\r\n- move designing cluster for testing and portability from testing to\r\ncluster design - it just makes more sense there.\r\n\r\n- Creating a tips and troubleshooting section to land small guides,\r\nmove in avahi troubleshooting and guide around discovery from a host computer.\r\n\r\n- remove quick start guide - everything in here is outdated, we\r\nhave a getting started guide and a readme that covers all the\r\ngetting started material. Added references to that in the readme\r\n-\r\n\r\n* couple more refs\r\n\r\n* remove readme\r\n\r\n* [HVAC] Sync atomic write error order with spec (#34936)\r\n\r\n* Add support for Presets attributes and commands to the Thermostat cluster\r\n\r\nClean up the Thermostat cluster and remove the TemperatureSetpointHoldPolicy attribute\r\nand SetTemperatureSetpointHoldPolicy command\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Restyled by gn.\r\n\r\n* Fix build error for Linux configure build of all-clusters-app\r\n\r\n* Fix Darwin CI issues\r\n\r\nEditorial fixes\r\n\r\n* Restyled by clang-format\r\n\r\n* More fixes\r\n\r\n* Restyled by clang-format\r\n\r\n* BUILD.gn fixes for CI\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* Address review comments.\r\n\r\n* Restyled by clang-format\r\n\r\n* Regenerate Thermostat XML from spec\r\n\r\n* Move atomic enum to global-enums.xml, actually\r\n\r\n# Conflicts:\r\n#\tsrc/app/zap-templates/zcl/data-model/chip/global-structs.xml\r\n\r\n* Regenerate XML and convert thermostat-server to atomic writes\r\n\r\n* Pull in ACCapacityFormat typo un-fix\r\n\r\n* Update Test_TC_TSTAT_1_1 to know about AtomicResponse command.\r\n\r\n* Restyled patch\r\n\r\n* Fix weird merge with upstream\r\n\r\n* Fix emberAfIsTypeSigned not understanding temperature type\r\n\r\n* Merge fixes from atomic write branch\r\n\r\n* Relocate thermostat-manager sample code to all-clusters-common\r\n\r\n* Fix g++ build error on linux\r\n\r\n* Fix C formatter for long int, cast whole expression\r\n\r\n* Sync cast fix with master\r\n\r\n* Add thermostat-common dependency to thermostat app under linux\r\n\r\n* Remove MatterPostAttributeChangeCallback from thermostat-manager, as it conflicts with other implementations\r\n\r\n* Convert Atomic enums and structs to global\r\n\r\n* Restyled patch\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* Regen with alchemy 0.6.1\r\n\r\n* Updates based on comments\r\n\r\n* Add TC_MCORE_FS_1_3.py test implementation (#34650)\r\n\r\n* Fix most TC-SWTCH-2.4 remaining issues (#34677)\r\n\r\n- Move 2.4 in a better place in the file\r\n- Add test steps properly\r\n- Allow default button press position override\r\n\r\nIssue #34656\r\n\r\nTesting done:\r\n\r\n- Test still passes on DUT with automation\r\n\r\n* Initial test script for Fabric Sync TC_MCORE_FS_1_2 (#34675)\r\n\r\n* Initial test script for Fabric Sync TC_MCORE_FS_1_2\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: C Freeman \u003ccecille@google.com\u003e\r\n\r\n* Address Review Comments\r\n\r\n* Address review comments\r\n\r\n* Fix default timeout after other timeouts changed\r\n\r\n* Restyled by autopep8\r\n\r\n* Fix linter error\r\n\r\n---------\r\n\r\nCo-authored-by: C Freeman \u003ccecille@google.com\u003e\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Test automation for FabricSync ICD BridgedDeviceBasicInfoCluster (#34628)\r\n\r\n* WIP Bridged ICD, commissioning to both fabrics\r\n\r\n* wip testing sending KeepActive\r\n\r\n* wip most steps implemented\r\n\r\n* using SIGSTOP and SIGCONT to control ICD server pausing\r\n\r\n* Update src/python_testing/TC_BRBINFO_4_1.py\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\n\r\n* comments addressed\r\n\r\n* more comments addressed\r\n\r\n* lint pass\r\n\r\n* Update src/python_testing/TC_BRBINFO_4_1.py\r\n\r\nCo-authored-by: C Freeman \u003ccecille@google.com\u003e\r\n\r\n* comments addressed, incl TH_SERVER configurable\r\n\r\n* added setupQRCode and setupManualCode as options for DUT commissioning\r\n\r\n* Restyled by autopep8\r\n\r\n* Restyled by isort\r\n\r\n* Update src/python_testing/TC_BRBINFO_4_1.py\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\n\r\n* Update src/python_testing/TC_BRBINFO_4_1.py\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\n\r\n* Update src/python_testing/TC_BRBINFO_4_1.py\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\n\r\n* comments addressed\r\n\r\n* Restyled by autopep8\r\n\r\n---------\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\nCo-authored-by: C Freeman \u003ccecille@google.com\u003e\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* ServiceArea test scripts (#34548)\r\n\r\n* initial commit\r\n\r\n* fix bugs\r\n\r\n* fix issues reported by the linter\r\n\r\n* fix bug in checking for unique areaDesc\r\n\r\n* add TC 1.5\r\n\r\n* Update src/python_testing/TC_SEAR_1_2.py\r\n\r\nCo-authored-by: William \u003chicklin@users.noreply.github.com\u003e\r\n\r\n* Update src/python_testing/TC_SEAR_1_2.py\r\n\r\nCo-authored-by: William \u003chicklin@users.noreply.github.com\u003e\r\n\r\n* address code review comments\r\n\r\n* fix issue introduced by the previous commit\r\n\r\n* address code review feedback\r\n\r\n* Update src/python_testing/TC_SEAR_1_2.py\r\n\r\nCo-authored-by: Kiel Oleson \u003ckielo@apple.com\u003e\r\n\r\n* address code review feedback\r\n\r\n* remove PICS checked by the TC_SEAR_1.6\r\n\r\n* more code review updates\r\n\r\n* Restyled by autopep8\r\n\r\n---------\r\n\r\nCo-authored-by: William \u003chicklin@users.noreply.github.com\u003e\r\nCo-authored-by: Kiel Oleson \u003ckielo@apple.com\u003e\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Remove manual tests for Thermostat presets (#34679)\r\n\r\n* Dump details about leaked ExchangeContexts before aborting (#34617)\r\n\r\n* Dump details about leaked ExchangeContexts before aborting\r\n\r\nThis is implemented via a VerifyOrDieWithObject() variant of the existing\r\nVerifyOrDie() macro that calls a DumpToLog() method on the provided object if\r\nit exists (otherwise this is simply a no-op).\r\n\r\nIf CHIP_CONFIG_VERBOSE_VERIFY_OR_DIE is not enabled, VerifyOrDieWithObject()\r\nsimply behaves like a plain VerifyOrDie(). DumpToLog() implementations can use\r\nChipLogFormatRtti to log type information about an object (usually a delegate);\r\nif RTTI is disabled this simply outputs whether the object was null or not.\r\n\r\n* Address review comments\r\n\r\n* Make gcc happy and improve documentation\r\n\r\n* Remove unused include\r\n\r\n* Fix compile error without CHIP_CONFIG_VERBOSE_VERIFY_OR_DIE\r\n\r\n* Avoid unused parameter warning\r\n\r\n* [TI] CC13x4_26x4 build fixes (#34682)\r\n\r\n* lwip pbuf, map file, and hex creation when OTA is disabled\r\n\r\n* added cc13x4 family define around the non OTA hex creation\r\n\r\n* whitespace fix\r\n\r\n* reversed custom factoy data flash with cc13x4 check\r\n\r\n* more whitespace fixes\r\n\r\n* [ICD] Add missing polling function to NoWifi connectivity manager (#34684)\r\n\r\n* Add missing polling function to NoWifi connectivity manager\r\n\r\n* Update GenericConnectivityManagerImpl_NoWiFi.h\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n---------\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* [OPSTATE] Add Q test script for CountdownTime (#34632)\r\n\r\n* Add Q test\r\n\r\n* Added test to test set\r\n\r\n* Remove unused var\r\n\r\n* Restyled by autopep8\r\n\r\n* Restyled by isort\r\n\r\n* Fix name\r\n\r\n* Use pics over other method\r\n\r\n* Removed unused stuff\r\n\r\n* Added pipe commands\r\n\r\n* Fix reset\r\n\r\n* Get example to report appropriate changes.\r\n\r\n* WiP\r\n\r\n* Added some comments\r\n\r\n* Changes to make things work\r\n\r\n* Removed dev msgs\r\n\r\n* Missed some\r\n\r\n* Removed dev msgs\r\n\r\n* Straggler\r\n\r\n* Restyled by clang-format\r\n\r\n* Restyled by autopep8\r\n\r\n* Restyled by isort\r\n\r\n* Commented unused var\r\n\r\n* Update examples/all-clusters-app/linux/AllClustersCommandDelegate.cpp\r\n\r\n* Fix bug\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* YAML update to BRBINFO, ProductId (#34513)\r\n\r\n* Bridged Device Information Cluster, Attribute ProductID test reflects marking as O, not X\r\n\r\n* Update src/app/tests/suites/certification/Test_TC_BRBINFO_2_1.yaml\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\n\r\n* corrected pics\r\n\r\n* corrected pics\r\n\r\n* WIP Bridged ICD, commissioning to both fabrics\r\n\r\n* wip testing sending KeepActive\r\n\r\n* update to bridged-device-basic-information.xml and zap generated files\r\n\r\n* removed unrelated file\r\n\r\n---------\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\nCo-authored-by: Andrei Litvin \u003candy314@gmail.com\u003e\r\n\r\n* Fix simplified Linux tv-casting-app gn build error. (#34692)\r\n\r\n* adding parallel execution to restyle-diff (#34663)\r\n\r\n* adding parallel execution to restyle-diff\r\n\r\n* using xargs to call restyle-paths\r\n\r\n* fixing Copyright year\r\n\r\n* restyle the restyler\r\n\r\n* Add some bits to exercise global structs/enums to Unit Testing cluster. (#34540)\r\n\r\n* Adds things to the Unit Testing cluster XML.\r\n* This requires those things to be enabled in all-clusters-app,\r\n  all-clusters-minimal-app, and one of the chef contact sensors to pass CI.\r\n* That requires an implementation in test-cluster-server\r\n* At which point might as well add a YAML test to exercise it all.\r\n\r\n* [Silabs] Port platform specific Multi-Chip OTA work  (#34440)\r\n\r\n* Pull request #1836: Cherry multi ota\r\n\r\nMerge in WMN_TOOLS/matter from cherry-multi-ota to silabs_slc_1.3\r\n\r\nSquashed commit of the following:\r\n\r\ncommit 4320bb46571658bc44fb82345348265def394991\r\nAuthor: Michael Rupp \u003cmichael.rupp@silabs.com\u003e\r\nDate:   Fri May 10 14:26:07 2024 -0400\r\n\r\n    remove some unwanted diffs in provision files\r\n\r\ncommit be160931dc600de7e7ead378b70d6a43c3945e46\r\nAuthor: Michael Rupp \u003cmichael.rupp@silabs.com\u003e\r\nDate:   Fri May 10 14:24:25 2024 -0400\r\n\r\n    revert changes to generator.project.mak\r\n\r\ncommit 14b6605887166e6d5284a61feb2bf407d850bdcf\r\nAuthor: Michael Rupp \u003cmichael.rupp@silabs.com\u003e\r\nDate:   Fri May 10 13:06:12 2024 -0400\r\n\r\n    revert NVM key changes and script changes\r\n\r\n... and 8 more commits\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Restyled by gn\r\n\r\n* Restyled by autopep8\r\n\r\n* remove unused libs caught by linter\r\n\r\n* update doctree with new readmes\r\n\r\n* rerun CI, cirque failing for unknown reasons\r\n\r\n* fix include guards in provision examples\r\n\r\n* Restyled by clang-format\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Add python tests for Thermostat presets feature (#34693)\r\n\r\n* Add python tests for Thermostat presets feature\r\n\r\n* Restyled by autopep8\r\n\r\n* Restyled by isort\r\n\r\n* Update the PICS code for presets attribute\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* removing unneccessary git fetch (#34698)\r\n\r\n* Restyle patch\r\n\r\n* Regen to fix ordering of global structs\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* Return correct AtomicResponse when committing or rolling back\r\n\r\n* Patch tests for atomic write of presets\r\n\r\n* Fix tests to work with the new setup.\r\n\r\nSpecific changes:\r\n\r\n* Enable SetActivePresetRequest command in all-clusters-app.\r\n* Fix assignment of a PresetStructWithOwnedMembers to another\r\n  PresetStructWithOwnedMembers to actually work correctly.\r\n* Move constraint checks that happen on write from commit to write.\r\n* Fix sending of atomic responses to not have use-stack-after-return.\r\n* Fix PICS for the tests involved.\r\n\r\n* Fix PICS values for atomic requests\r\n\r\n* Remove PresetsSchedulesEditable and QueuedPreset from various places\r\n\r\n* Restyled patch\r\n\r\n* Restyled patch, again\r\n\r\n* Remove PICS value for PresetsSchedulesEditable\r\n\r\n* clang-tidy fixes\r\n\r\n* clang-tidy fixes\r\n\r\n* Clear associated atomic writes when fabric is removed\r\n\r\n* Add tests for fabric removal and lockout of clients outside of atomic write\r\n\r\n* Python linter\r\n\r\n* Restyled patch\r\n\r\n* Clear timer when fabric is removed\r\n\r\n* Check for open atomic write before resetting\r\n\r\n* Revert auto delegate declaration on lines where there\u0027s no collision\r\n\r\n* Allow Thermostat delegate to provide timeout for atomic requests\r\n\r\n* Relocate thermostat example code to thermostat-common\r\n\r\n* Remove thermostat-manager code, replace with thermostat delegate\r\n\r\n* Sync atomic write error order with spec\r\n\r\n* Restyle patch\r\n\r\n* Drop memset of atomic write sessions\r\n\r\n* Add PreCommit stage to allow rollback of multiple attributes when only one fails\r\n\r\n* Separate OnTimerExpired method, vs ResetWrite\r\n\r\n* Method documentation\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Nivi Sarkar \u003c55898241+nivi-apple@users.noreply.github.com\u003e\r\n\r\n* Remove unused InWrite check\r\n\r\n* Drop imcode alias\r\n\r\n* Switch AtomicWriteState to enum class\r\n\r\n* DRY up atomic write manager\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Nivi Sarkar \u003c55898241+nivi-apple@users.noreply.github.com\u003e\r\n\r\n* Drop duplicate doc comments\r\n\r\n* Rename GetAtomicWriteScopedNodeId to GetAtomicWriteOriginatorScopedNodeId\r\n\r\n* Updates based on comments\r\n\r\n* Add MatterReportingAttributeChangeCallback calls for updated attributes\r\n\r\n* Relocate thermostat example code to thermostat-common, and remove thermostat-manager\r\n\r\n* Merge atomic write code back into thermostat-server\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* Fix build after suggestions\r\n\r\n* Actually track attribute IDs associated with atomic write\r\n\r\n* Only commit presets if all attribute precommits were successful\r\n\r\n* Fix scope on err\r\n\r\n* Add documentation to methods\r\n\r\n* Remove duplicate preset check.\r\n\r\n* Move various functions into anonymous namespaces, or Thermostat namespace\r\n\r\n* Drop impossible non-atomic attribute status after rollback\r\n\r\n* Namespace workaround for compilers on other platforms\r\n\r\n* Apply suggestions from code review\r\n\r\n---------\r\n\r\nCo-authored-by: Nivedita Sarkar \u003cnivedita_sarkar@apple.com\u003e\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: Nivi Sarkar \u003c55898241+nivi-apple@users.noreply.github.com\u003e\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\nCo-authored-by: Tennessee Carmel-Veilleux \u003ctennessee.carmelveilleux@gmail.com\u003e\r\nCo-authored-by: Chris Letnick \u003ccletnick@google.com\u003e\r\nCo-authored-by: C Freeman \u003ccecille@google.com\u003e\r\nCo-authored-by: Douglas Rocha Ferraz \u003crochaferraz@google.com\u003e\r\nCo-authored-by: Petru Lauric \u003c81822411+plauric@users.noreply.github.com\u003e\r\nCo-authored-by: William \u003chicklin@users.noreply.github.com\u003e\r\nCo-authored-by: Kiel Oleson \u003ckielo@apple.com\u003e\r\nCo-authored-by: Karsten Sperling \u003c113487422+ksperling-apple@users.noreply.github.com\u003e\r\nCo-authored-by: Anu Biradar \u003c104591549+abiradarti@users.noreply.github.com\u003e\r\nCo-authored-by: mkardous-silabs \u003c84793247+mkardous-silabs@users.noreply.github.com\u003e\r\nCo-authored-by: Rob Bultman \u003crob.Bultman@gmail.com\u003e\r\nCo-authored-by: Andrei Litvin \u003candy314@gmail.com\u003e\r\nCo-authored-by: Shao Ling Tan \u003c161761051+shaoltan-amazon@users.noreply.github.com\u003e\r\nCo-authored-by: Amine Alami \u003c43780877+Alami-Amine@users.noreply.github.com\u003e\r\nCo-authored-by: Michael Rupp \u003c95718139+mykrupp@users.noreply.github.com\u003e\r\n\r\n* esp32: fix compiling when enabling ble commissioner (#35085)\r\n\r\n* esp32: fix compiling when enabling ble commissioner\r\n\r\n* Restyled by clang-format\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Remove writeAttributeWithEndpointID implementation from MTRDevice. (#35170)\r\n\r\nThis is implemented (differently) by the different subclasses.\r\n\r\nOnce this implementation is removed, removeExpectedValueForAttributePath becomes\r\nunused and can be removed.\r\n\r\nAlso removes the unused setExpectedValues declaration in MTRDevice_Internal.h\r\nand the implementations of it.\r\n\r\n* Address followup issues for preset/atomic-write implementation. (#35175)\r\n\r\n* Puts some file-local functions in an anonymous namespace.\r\n* Fixes the \"is this attribute supported?\" check to correctly check for global\r\n  attributes that are not in Ember metadata.\r\n* Moves the comment explaining why it\u0027s OK to skip the spec-required ACL check\r\n  to the place where that check is being skipped.\r\n* Removes a non-spec-compliant \"error if the timeout is 0\" bit.\r\n\r\nFixes https://github.com/project-chip/connectedhomeip/issues/35168\r\n\r\n* [Darwin] Enable taking assertion on MTRDeviceController (#35148)\r\n\r\n* [Darwin] Enable taking assertion on MTRDeviceController\r\n\r\n- Added ability to take assertion on MTRDeviceController to keep it\r\n  running until all assertions are removed\r\n- A request to shutdown will take place only if there are no assertions\r\n  are present\r\n\r\n* Fixed format string\r\n\r\n* Account for existing controller that is asserted in factory when creating a new one\r\n\r\n* Simplified to use lock for assertion tracking\r\n\r\n* Fixed build error\r\n\r\n* Removed unneeded includes\r\n\r\n* Fixed bugs with wrong match logic\r\n\r\n* resytle fixes\r\n\r\n* Restyle fixes\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* Fixed build failure from review suggestions and added comment per review feedback\r\n\r\n---------\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* Cleanup/updates tied with latest sdk update (#35140)\r\n\r\n* [Silabs] Adding the gn build header for the lwip IPv4 and RS9116 (#35142)\r\n\r\n* adding the gn build header for the lwip and rs9116\r\n\r\n* Restyled by clang-format\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* [Silabs] [WiFi] Fixing the retry logic for the WiFi Devices (#34225)\r\n\r\n* Modifying the retry logic for the wifi devices\r\n\r\n* Baseapplication.cpp modification of the delegate fix\r\n\r\n* Restyled by clang-format\r\n\r\n* removing the soc and sleepy condition\r\n\r\n* Restyled by clang-format\r\n\r\n* Restyled by gn\r\n\r\n* moving the variable to efr32_sdk.gni to make it available everywhere\r\n\r\n* correcting the path of the 917 ncp file\r\n\r\n* fixing the build for the 917 ncp\r\n\r\n* addressing the review comments\r\n\r\n* fixing the build and replacing the define\r\n\r\n* removing the unused else case\r\n\r\n* Update third_party/silabs/efr32_sdk.gni\r\n\r\nCo-authored-by: mkardous-silabs \u003c84793247+mkardous-silabs@users.noreply.github.com\u003e\r\n\r\n* addressing the review comments\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* fixing the build\r\n\r\n* fixing the efr32 build\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: mkardous-silabs \u003c84793247+mkardous-silabs@users.noreply.github.com\u003e\r\n\r\n* Fixed missing trailing zero in example documentation (#35149)\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* [Silabs] Fixing the build for the custom boards on NCP (#35055)\r\n\r\n* fixing the build for the custom boards on ncp\r\n\r\n* Restyled by clang-format\r\n\r\n* applying the restyler\r\n\r\n* addressing the review comments and adding sl_custom_board.h file\r\n\r\n* adding the warning message for 917 NCP\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Yufeng Wang \u003cyufengwang@google.com\u003e\r\n\r\n* Restyled by clang-format\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: Junior Martinez \u003c67972863+jmartinez-silabs@users.noreply.github.com\u003e\r\nCo-authored-by: Yufeng Wang \u003cyufengwang@google.com\u003e\r\n\r\n* [Darwin] Adding missing property on MTRDeviceController_XPC (#35188)\r\n\r\n- Added controllerID to match the OverXPC interface\r\n\r\n* [Fabric-Bridge] Add state machine to CommissionerControlDelegate to guard various unexpected commands (#35186)\r\n\r\n* Add state machine to CommissionerControlDelegate to guard various unexpected commands\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Do not reset state when commissionNode fail to allow retry\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Move the constraint check for number of presets not to exceed the num… (#35167)\r\n\r\n* Move the constraint check for number of presets not to exceed the number of presets supported from the atomic write commit command handling to the write request\r\n\r\n* Minor fix\r\n\r\n* Do not check if totalExpectedCount \u003e 0 since its an unsigned integer\r\n\r\n- Also no need to check if numberOfPresetsSupported is 0.\r\n\r\n* Update src/app/clusters/thermostat-server/thermostat-server-presets.cpp\r\n\r\n* Update src/app/clusters/thermostat-server/thermostat-server-presets.cpp\r\n\r\n---------\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* TV App - Add validation for invalid product and vendor id. (#35157)\r\n\r\n* Add validation for pid\u0026vid\r\n\r\n* Restyled by clang-format\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* [Darwin] Find cached controllers when creating existing controller on fabric as well (#35205)\r\n\r\n- Missed another call site in #35148\r\n\r\n* Remove handling of the unitTestInject* bits in base MTRDevice. (#35180)\r\n\r\nThese injections need to be handled by subclasses, because the subclasses do\r\nvery different report processing and have different mechanisms for getting\r\nreports.\r\n\r\nOnce these implementations are removed, the following become unreachable and can\r\nbe removed:\r\n\r\n* _handleEventReport\r\n* _handleReportBegin\r\n* _handleAttributeReport\r\n* _handleReportEnd\r\n\r\nOnce those are removed, the following become unreachable or unused and can be\r\nremoved:\r\n\r\n* _receivingReport (always false)\r\n* _receivingPrimingReport (always false, because _receivingReport is always false)\r\n* _changeState\r\n* _callDelegateDeviceCachePrimed\r\n* unreportedEvents\r\n* _getAttributesToReportWithReportedValues\r\n\r\nOnce those are removed, the following become unreachable or unused and can be\r\nremoved:\r\n\r\n* _setCachedAttributeValue\r\n* _noteDataVersion\r\n* _deviceConfigurationChanged\r\n* estimatedStartTimeFromGeneralDiagnosticsUpTime\r\n* _attributeAffectsDeviceConfiguration\r\n* _pruneStoredDataForPath\r\n\r\nOnce those are removed, the following become unreachable or unused and can be\r\nremoved:\r\n\r\n* _pruneEndpointsIn\r\n* _pruneClustersIn\r\n* _pruneAttributesIn\r\n* _attributePathAffectsDescriptionData\r\n* _addInformationalAttributesToCurrentMetricScope\r\n* AttributeHasChangesOmittedQuality\r\n\r\nOnce those are removed, the following become unreachable or unused and can be\r\nremoved:\r\n\r\n* _removeAttributes\r\n* _removeClusters\r\n\r\nOnce those are removed, the following become unreachable or unused and can be\r\nremoved:\r\n\r\n* _removeCachedAttribute\r\n\r\n* darwin-framework-tool: Add support for paseonly pairing (#35206)\r\n\r\nThis includes code-paseonly and by-index-paseonly commands, as as interacting\r\nwith devices over PASE by preferring an existing MTRBaseDevice connected over\r\nPASE to creating a new CASE connection when looking up a device by node id.\r\n\r\n* esp32: update esp_rcp_update version to 1.2.0 (#35114)\r\n\r\n* esp32: update esp_rcp_update version to 1.2.0\r\n\r\n* change for ESP32ThreadInit\r\n\r\n* fix compile error when lwip ipv6 address number \u003e 8\r\n\r\n* Fix the PICS code for atomic response to be lower case and update the… (#35204)\r\n\r\n* Fix the PICS code for atomic response to be lower case and update the PICS code for the Generated command list for the presets and schedules feature\r\n\r\n* Updated the PICS code for atomic response in PICS.yaml\r\n\r\n* TC-LVL-2.3: Update expected number of reports (#35179)\r\n\r\nplease see\r\nhttps://github.com/CHIP-Specifications/chip-test-plans/pull/4456#discussion_r1715722836\r\n\r\n* [Telink] Add b95 flash overlay (#35133)\r\n\r\n* riscv: telink: add ota event. (#35118)\r\n\r\n- add all the ota event in ota impl.\r\n\r\nSigned-off-by: Haiwen Xia \u003chaiwen.xia@telink-semi.com\u003e\r\nCo-authored-by: Haiwen Xia \u003chaiwen.xia@telink-semi.com\u003e\r\n\r\n* Added changes for ble connection interval (#35198)\r\n\r\n* Add missing occupancy event (#35220)\r\n\r\n* XML: Add OccupancyChanged event\r\n\r\n* Regen zap\r\n\r\n* Managed ACL: Add AccessRestrictionList support (#34932)\r\n\r\n* Add AccessRestrictionList support\r\n\r\n* Update src/access/AccessConfig.h\r\n\r\nCo-authored-by: C Freeman \u003ccecille@google.com\u003e\r\n\r\n* Reworked data manipulators and other cleanup\r\n\r\n* Fixed encode/decode so reading CommissioningARL and Arl attributes work\r\n\r\n* Reworked ARL storage\r\n\r\nPreviously ARL related data was persisted in KVS.  This has been\r\nremoved and now the responsibility for managing/maintaining the\r\nrelated data (CommissioningARL and ARL attributes) is up to the\r\napp to set on AccessRestrictionProvider class.\r\n\r\n* Review fixes\r\n\r\ncleanup ArlEncoder interface.\r\nreturn error to client if arl review request fails\r\nreturn token to client in FabricRestrictionReviewUpdate\r\n\r\n* Fixed GetEntries vector pointer arg\r\n\r\n* Updated core restriction logic/integration\r\n\r\n* Restyled by clang-format\r\n\r\n* fixed include check for renamed AccessRestrictionProvider.h file\r\n\r\n* M-ACL updates\r\n\r\n- refactored AccessControl::Check into CheckACL and CheckARL\r\n- added placeholders for the upcoming CHIP_ERROR_ACCESS_RESTRICTED_BY_ARL\r\n- extracted ARL exception processing to standalone class for better\r\n  testing\r\n\r\n* Add plumbing for subject descriptor IsCommissioning field\r\n\r\n- Make session manager update that state on a message-per-message basis\r\n- Add tests\r\n\r\nMissing test: MRP test against a not-yet-committed fabric over CASE showing\r\nthat IsCommissioning is true.\r\n\r\n* Fix crash\r\n\r\n* Use new IsCommissioning in ARL check\r\n\r\n* Updates for review comments\r\n\r\n* restyled\r\n\r\n* Review updates\r\n\r\n- fixed return type for some command failures\r\n- enhanced unit tests\r\n\r\n* restyled\r\n\r\n* Updated ARL tests per review comments\r\n\r\n* work around nuttx and jsoncpp contention\r\n\r\n* Review comments and nuttx build failure fix attempt\r\n\r\n* review updates\r\n\r\n---------\r\n\r\nCo-authored-by: C Freeman \u003ccecille@google.com\u003e\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: tennessee.carmelveilleux@gmail.com \u003ctennessee.carmelveilleux@gmail.com\u003e\r\n\r\n* Adding TimeoutMs to KeepActive command (#35154)\r\n\r\n* Remove RemovedOn attribute from ECOINFO cluster (#34988)\r\n\r\n* Remove RemovedOn attribute from ECOINFO cluster\r\n\r\n* Restyled by autopep8\r\n\r\n* Remove artifact that were only initially used to debug script locally\r\n\r\n* Move attr IDs after RemovedOn removal\r\n\r\n* Empty-Commit\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* fix #35210: tc-icdm-2.1, 3.1, 3.4 (#35227)\r\n\r\n* fix 35210: tc-icdm-2.1, 3.1, 3.4\r\n\r\n* missing changes.\r\n\r\n* [HVAC] Set thermostat delegate in thermostat app (#35230)\r\n\r\n* [HVAC] Handle null BuiltIn field on Preset write according to spec (#35161)\r\n\r\n* Add support for Presets attributes and commands to the Thermostat cluster\r\n\r\nClean up the Thermostat cluster and remove the TemperatureSetpointHoldPolicy attribute\r\nand SetTemperatureSetpointHoldPolicy command\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Restyled by gn.\r\n\r\n* Fix build error for Linux configure build of all-clusters-app\r\n\r\n* Fix Darwin CI issues\r\n\r\nEditorial fixes\r\n\r\n* Restyled by clang-format\r\n\r\n* More fixes\r\n\r\n* Restyled by clang-format\r\n\r\n* BUILD.gn fixes for CI\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* Address review comments.\r\n\r\n* Restyled by clang-format\r\n\r\n* Regenerate Thermostat XML from spec\r\n\r\n* Move atomic enum to global-enums.xml, actually\r\n\r\n# Conflicts:\r\n#\tsrc/app/zap-templates/zcl/data-model/chip/global-structs.xml\r\n\r\n* Regenerate XML and convert thermostat-server to atomic writes\r\n\r\n* Pull in ACCapacityFormat typo un-fix\r\n\r\n* Update Test_TC_TSTAT_1_1 to know about AtomicResponse command.\r\n\r\n* Restyled patch\r\n\r\n* Fix weird merge with upstream\r\n\r\n* Fix emberAfIsTypeSigned not understanding temperature type\r\n\r\n* Merge fixes from atomic write branch\r\n\r\n* Relocate thermostat-manager sample code to all-clusters-common\r\n\r\n* Fix g++ build error on linux\r\n\r\n* Fix C formatter for long int, cast whole expression\r\n\r\n* Sync cast fix with master\r\n\r\n* Add thermostat-common dependency to thermostat app under linux\r\n\r\n* Remove MatterPostAttributeChangeCallback from thermostat-manager, as it conflicts with other implementations\r\n\r\n* Convert Atomic enums and structs to global\r\n\r\n* Restyled patch\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* Regen with alchemy 0.6.1\r\n\r\n* Updates based on comments\r\n\r\n* Add TC_MCORE_FS_1_3.py test implementation (#34650)\r\n\r\n* Fix most TC-SWTCH-2.4 remaining issues (#34677)\r\n\r\n- Move 2.4 in a better place in the file\r\n- Add test steps properly\r\n- Allow default button press position override\r\n\r\nIssue #34656\r\n\r\nTesting done:\r\n\r\n- Test still passes on DUT with automation\r\n\r\n* Initial test script for Fabric Sync TC_MCORE_FS_1_2 (#34675)\r\n\r\n* Initial test script for Fabric Sync TC_MCORE_FS_1_2\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: C Freeman \u003ccecille@google.com\u003e\r\n\r\n* Address Review Comments\r\n\r\n* Address review comments\r\n\r\n* Fix default timeout after other timeouts changed\r\n\r\n* Restyled by autopep8\r\n\r\n* Fix linter error\r\n\r\n---------\r\n\r\nCo-authored-by: C Freeman \u003ccecille@google.com\u003e\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Test automation for FabricSync ICD BridgedDeviceBasicInfoCluster (#34628)\r\n\r\n* WIP Bridged ICD, commissioning to both fabrics\r\n\r\n* wip testing sending KeepActive\r\n\r\n* wip most steps implemented\r\n\r\n* using SIGSTOP and SIGCONT to control ICD server pausing\r\n\r\n* Update src/python_testing/TC_BRBINFO_4_1.py\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\n\r\n* comments addressed\r\n\r\n* more comments addressed\r\n\r\n* lint pass\r\n\r\n* Update src/python_testing/TC_BRBINFO_4_1.py\r\n\r\nCo-authored-by: C Freeman \u003ccecille@google.com\u003e\r\n\r\n* comments addressed, incl TH_SERVER configurable\r\n\r\n* added setupQRCode and setupManualCode as options for DUT commissioning\r\n\r\n* Restyled by autopep8\r\n\r\n* Restyled by isort\r\n\r\n* Update src/python_testing/TC_BRBINFO_4_1.py\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\n\r\n* Update src/python_testing/TC_BRBINFO_4_1.py\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\n\r\n* Update src/python_testing/TC_BRBINFO_4_1.py\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\n\r\n* comments addressed\r\n\r\n* Restyled by autopep8\r\n\r\n---------\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\nCo-authored-by: C Freeman \u003ccecille@google.com\u003e\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* ServiceArea test scripts (#34548)\r\n\r\n* initial commit\r\n\r\n* fix bugs\r\n\r\n* fix issues reported by the linter\r\n\r\n* fix bug in checking for unique areaDesc\r\n\r\n* add TC 1.5\r\n\r\n* Update src/python_testing/TC_SEAR_1_2.py\r\n\r\nCo-authored-by: William \u003chicklin@users.noreply.github.com\u003e\r\n\r\n* Update src/python_testing/TC_SEAR_1_2.py\r\n\r\nCo-authored-by: William \u003chicklin@users.noreply.github.com\u003e\r\n\r\n* address code review comments\r\n\r\n* fix issue introduced by the previous commit\r\n\r\n* address code review feedback\r\n\r\n* Update src/python_testing/TC_SEAR_1_2.py\r\n\r\nCo-authored-by: Kiel Oleson \u003ckielo@apple.com\u003e\r\n\r\n* address code review feedback\r\n\r\n* remove PICS checked by the TC_SEAR_1.6\r\n\r\n* more code review updates\r\n\r\n* Restyled by autopep8\r\n\r\n---------\r\n\r\nCo-authored-by: William \u003chicklin@users.noreply.github.com\u003e\r\nCo-authored-by: Kiel Oleson \u003ckielo@apple.com\u003e\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Remove manual tests for Thermostat presets (#34679)\r\n\r\n* Dump details about leaked ExchangeContexts before aborting (#34617)\r\n\r\n* Dump details about leaked ExchangeContexts before aborting\r\n\r\nThis is implemented via a VerifyOrDieWithObject() variant of the existing\r\nVerifyOrDie() macro that calls a DumpToLog() method on the provided object if\r\nit exists (otherwise this is simply a no-op).\r\n\r\nIf CHIP_CONFIG_VERBOSE_VERIFY_OR_DIE is not enabled, VerifyOrDieWithObject()\r\nsimply behaves like a plain VerifyOrDie(). DumpToLog() implementations can use\r\nChipLogFormatRtti to log type information about an object (usually a delegate);\r\nif RTTI is disabled this simply outputs whether the object was null or not.\r\n\r\n* Address review comments\r\n\r\n* Make gcc happy and improve documentation\r\n\r\n* Remove unused include\r\n\r\n* Fix compile error without CHIP_CONFIG_VERBOSE_VERIFY_OR_DIE\r\n\r\n* Avoid unused parameter warning\r\n\r\n* [TI] CC13x4_26x4 build fixes (#34682)\r\n\r\n* lwip pbuf, map file, and hex creation when OTA is disabled\r\n\r\n* added cc13x4 family define around the non OTA hex creation\r\n\r\n* whitespace fix\r\n\r\n* reversed custom factoy data flash with cc13x4 check\r\n\r\n* more whitespace fixes\r\n\r\n* [ICD] Add missing polling function to NoWifi connectivity manager (#34684)\r\n\r\n* Add missing polling function to NoWifi connectivity manager\r\n\r\n* Update GenericConnectivityManagerImpl_NoWiFi.h\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n---------\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* [OPSTATE] Add Q test script for CountdownTime (#34632)\r\n\r\n* Add Q test\r\n\r\n* Added test to test set\r\n\r\n* Remove unused var\r\n\r\n* Restyled by autopep8\r\n\r\n* Restyled by isort\r\n\r\n* Fix name\r\n\r\n* Use pics over other method\r\n\r\n* Removed unused stuff\r\n\r\n* Added pipe commands\r\n\r\n* Fix reset\r\n\r\n* Get example to report appropriate changes.\r\n\r\n* WiP\r\n\r\n* Added some comments\r\n\r\n* Changes to make things work\r\n\r\n* Removed dev msgs\r\n\r\n* Missed some\r\n\r\n* Removed dev msgs\r\n\r\n* Straggler\r\n\r\n* Restyled by clang-format\r\n\r\n* Restyled by autopep8\r\n\r\n* Restyled by isort\r\n\r\n* Commented unused var\r\n\r\n* Update examples/all-clusters-app/linux/AllClustersCommandDelegate.cpp\r\n\r\n* Fix bug\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* YAML update to BRBINFO, ProductId (#34513)\r\n\r\n* Bridged Device Information Cluster, Attribute ProductID test reflects marking as O, not X\r\n\r\n* Update src/app/tests/suites/certification/Test_TC_BRBINFO_2_1.yaml\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\n\r\n* corrected pics\r\n\r\n* corrected pics\r\n\r\n* WIP Bridged ICD, commissioning to both fabrics\r\n\r\n* wip testing sending KeepActive\r\n\r\n* update to bridged-device-basic-information.xml and zap generated files\r\n\r\n* removed unrelated file\r\n\r\n---------\r\n\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\nCo-authored-by: Andrei Litvin \u003candy314@gmail.com\u003e\r\n\r\n* Fix simplified Linux tv-casting-app gn build error. (#34692)\r\n\r\n* adding parallel execution to restyle-diff (#34663)\r\n\r\n* adding parallel execution to restyle-diff\r\n\r\n* using xargs to call restyle-paths\r\n\r\n* fixing Copyright year\r\n\r\n* restyle the restyler\r\n\r\n* Add some bits to exercise global structs/enums to Unit Testing cluster. (#34540)\r\n\r\n* Adds things to the Unit Testing cluster XML.\r\n* This requires those things to be enabled in all-clusters-app,\r\n  all-clusters-minimal-app, and one of the chef contact sensors to pass CI.\r\n* That requires an implementation in test-cluster-server\r\n* At which point might as well add a YAML test to exercise it all.\r\n\r\n* [Silabs] Port platform specific Multi-Chip OTA work  (#34440)\r\n\r\n* Pull request #1836: Cherry multi ota\r\n\r\nMerge in WMN_TOOLS/matter from cherry-multi-ota to silabs_slc_1.3\r\n\r\nSquashed commit of the following:\r\n\r\ncommit 4320bb46571658bc44fb82345348265def394991\r\nAuthor: Michael Rupp \u003cmichael.rupp@silabs.com\u003e\r\nDate:   Fri May 10 14:26:07 2024 -0400\r\n\r\n    remove some unwanted diffs in provision files\r\n\r\ncommit be160931dc600de7e7ead378b70d6a43c3945e46\r\nAuthor: Michael Rupp \u003cmichael.rupp@silabs.com\u003e\r\nDate:   Fri May 10 14:24:25 2024 -0400\r\n\r\n    revert changes to generator.project.mak\r\n\r\ncommit 14b6605887166e6d5284a61feb2bf407d850bdcf\r\nAuthor: Michael Rupp \u003cmichael.rupp@silabs.com\u003e\r\nDate:   Fri May 10 13:06:12 2024 -0400\r\n\r\n    revert NVM key changes and script changes\r\n\r\n... and 8 more commits\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Restyled by gn\r\n\r\n* Restyled by autopep8\r\n\r\n* remove unused libs caught by linter\r\n\r\n* update doctree with new readmes\r\n\r\n* rerun CI, cirque failing for unknown reasons\r\n\r\n* fix include guards in provision examples\r\n\r\n* Restyled by clang-format\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Add python tests for Thermostat presets feature (#34693)\r\n\r\n* Add python tests for Thermostat presets feature\r\n\r\n* Restyled by autopep8\r\n\r\n* Restyled by isort\r\n\r\n* Update the PICS code for presets attribute\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* removing unneccessary git fetch (#34698)\r\n\r\n* Restyle patch\r\n\r\n* Regen to fix ordering of global structs\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* Return correct AtomicResponse when committing or rolling back\r\n\r\n* Patch tests for atomic write of presets\r\n\r\n* Fix tests to work with the new setup.\r\n\r\nSpecific changes:\r\n\r\n* Enable SetActivePresetRequest command in all-clusters-app.\r\n* Fix assignment of a PresetStructWithOwnedMembers to another\r\n  PresetStructWithOwnedMembers to actually work correctly.\r\n* Move constraint checks that happen on write from commit to write.\r\n* Fix sending of atomic responses to not have use-stack-after-return.\r\n* Fix PICS for the tests involved.\r\n\r\n* Fix PICS values for atomic requests\r\n\r\n* Remove PresetsSchedulesEditable and QueuedPreset from various places\r\n\r\n* Restyled patch\r\n\r\n* Restyled patch, again\r\n\r\n* Remove PICS value for PresetsSchedulesEditable\r\n\r\n* clang-tidy fixes\r\n\r\n* clang-tidy fixes\r\n\r\n* Clear associated atomic writes when fabric is removed\r\n\r\n* Add tests for fabric removal and lockout of clients outside of atomic write\r\n\r\n* Python linter\r\n\r\n* Restyled patch\r\n\r\n* Clear timer when fabric is removed\r\n\r\n* Check for open atomic write before resetting\r\n\r\n* Revert auto delegate declaration on lines where there\u0027s no collision\r\n\r\n* Allow Thermostat delegate to provide timeout for atomic requests\r\n\r\n* Relocate thermostat example code to thermostat-common\r\n\r\n* Remove thermostat-manager code, replace with thermostat delegate\r\n\r\n* Sync atomic write error order with spec\r\n\r\n* Restyle patch\r\n\r\n* Drop memset of atomic write sessions\r\n\r\n* Add PreCommit stage to allow rollback of multiple attributes when only one fails\r\n\r\n* Separate OnTimerExpired method, vs ResetWrite\r\n\r\n* Method documentation\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Nivi Sarkar \u003c55898241+nivi-apple@users.noreply.github.com\u003e\r\n\r\n* Remove unused InWrite check\r\n\r\n* Drop imcode alias\r\n\r\n* Switch AtomicWriteState to enum class\r\n\r\n* DRY up atomic write manager\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Nivi Sarkar \u003c55898241+nivi-apple@users.noreply.github.com\u003e\r\n\r\n* Drop duplicate doc comments\r\n\r\n* Rename GetAtomicWriteScopedNodeId to GetAtomicWriteOriginatorScopedNodeId\r\n\r\n* Updates based on comments\r\n\r\n* Add MatterReportingAttributeChangeCallback calls for updated attributes\r\n\r\n* Relocate thermostat example code to thermostat-common, and remove thermostat-manager\r\n\r\n* Merge atomic write code back into thermostat-server\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* Fix build after suggestions\r\n\r\n* Actually track attribute IDs associated with atomic write\r\n\r\n* Only commit presets if all attribute precommits were successful\r\n\r\n* Fix scope on err\r\n\r\n* Add documentation to methods\r\n\r\n* Remove duplicate preset check.\r\n\r\n* Move various functions into anonymous namespaces, or Thermostat namespace\r\n\r\n* Drop impossible non-atomic attribute status after rollback\r\n\r\n* Allow null BuiltIn field when saving Presets\r\n\r\n* Namespace workaround for compilers on other platforms\r\n\r\n* Fix bad merge\r\n\r\n* Fix readability issue\r\n\r\n* Force built-in to false on new presets\r\n\r\n---------\r\n\r\nCo-authored-by: Nivedita Sarkar \u003cnivedita_sarkar@apple.com\u003e\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: Nivi Sarkar \u003c55898241+nivi-apple@users.noreply.github.com\u003e\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\nCo-authored-by: Tennessee Carmel-Veilleux \u003ctennessee.carmelveilleux@gmail.com\u003e\r\nCo-authored-by: Chris Letnick \u003ccletnick@google.com\u003e\r\nCo-authored-by: C Freeman \u003ccecille@google.com\u003e\r\nCo-authored-by: Douglas Rocha Ferraz \u003crochaferraz@google.com\u003e\r\nCo-authored-by: Petru Lauric \u003c81822411+plauric@users.noreply.github.com\u003e\r\nCo-authored-by: William \u003chicklin@users.noreply.github.com\u003e\r\nCo-authored-by: Kiel Oleson \u003ckielo@apple.com\u003e\r\nCo-authored-by: Karsten Sperling \u003c113487422+ksperling-apple@users.noreply.github.com\u003e\r\nCo-authored-by: Anu Biradar \u003c104591549+abiradarti@users.noreply.github.com\u003e\r\nCo-authored-by: mkardous-silabs \u003c84793247+mkardous-silabs@users.noreply.github.com\u003e\r\nCo-authored-by: Rob Bultman \u003crob.Bultman@gmail.com\u003e\r\nCo-authored-by: Andrei Litvin \u003candy314@gmail.com\u003e\r\nCo-authored-by: Shao Ling Tan \u003c161761051+shaoltan-amazon@users.noreply.github.com\u003e\r\nCo-authored-by: Amine Alami \u003c43780877+Alami-Amine@users.noreply.github.com\u003e\r\nCo-authored-by: Michael Rupp \u003c95718139+mykrupp@users.noreply.github.com\u003e\r\n\r\n* run a regen (#35233)\r\n\r\n* Update ColorControl bounds based on spec fixes (#35156)\r\n\r\n* Update color defaults in samples\r\n\r\n* Update color control server logic to avoid div-by-zero\r\n\r\n* Update TC-CC-2.1 with new bounds on ColorControl\r\n\r\n* Restyled by clang-format\r\n\r\n* More fixes to CI\r\n\r\n* Set default startup mireds to null\r\n\r\n* Fix some comments\r\n\r\n* Try to fix CI\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Add plumbing in FS example for TimeoutMs parameter in KeepActive (#35174)\r\n\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: Yufeng Wang \u003cyufengwang@google.com\u003e\r\n\r\n* Updated BRBINFO-4-1 test after timeoutMs added (#35160)\r\n\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: saurabhst \u003cs.kumar9@samsung.com\u003e\r\n\r\n* [Chef] BasicVideoPlayer: Fix MediaInput and MediaPlayback not reporting attribute changes due to using AttributeAccessInterface (#35122)\r\n\r\n* Fix media-input accessinterface\r\n\r\n1. SelectInput command won\u0027t call reportData since\r\n   MatterReportingAttributeChangeCallback isn\u0027t called\r\n\r\n2. PW RPC Writing CurrentInput attribute of MediaInput cluster through won\u0027t reportData. The\r\n   value is also mismatch with PW RPC Read. (But this is still under WIP\r\nand not working)\r\n\r\n* Not writing the MediaInput cluster through RPC\r\n\r\n* Remove unused function HandleSetCurrentInput\r\n\r\n* Move all attributes logic to Chef application\r\n\r\n* Fix typo\r\n\r\n* Fix CurrentState(Playback Cluster) not report data\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Set CurrentState \u0026 PlaybackSpeed in MediaPlayback cluster\r\n\r\n* Set some attributes to persist (NVM)\r\n\r\n* Remove mPlaybackSpeed/mCurrentState/mCurrentInput\r\n\r\n* modify playSpeed from 1.0 to 1\r\n\r\n* Restyled by clang-format\r\n\r\n* Fix type in error messages\r\n\r\n* Restyled by clang-format\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\n\r\n* Add water heater to energy management app (#34886)\r\n\r\n* Refactored file location for chip-energy-management-app. Updated esp32 and linux build files.\r\n\r\n* Updated some all-clusters-app build and CMakeLists (linux, esp32, psoc6)\r\n\r\n* Updated remaining build and CMakefiles with new folders.\r\n\r\n* Fixed unit tests BUILD.gn\r\n\r\n* Restyled by gn\r\n\r\n* Missed Ameba .cmake\r\n\r\n* Missed all-clusters-minimal-app/esp32/main/CMakeLists.txt\r\n\r\n* Added basic hooks to ESP32 to allow EVSE or Water Heater app\r\n\r\n* Added initial WaterHeaterMain.cpp - needs rework\r\n\r\n* Configure runner command to use --application \u003cdem|eevse|whm\u003e option\r\n\r\n* Unify the WHM and EEVSE/DEM apps\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Restyled by gn\r\n\r\n* Rename application names\r\n\r\n* Get ESP32 energy management app building with water-heater. Move water heater files from all-clusters-app to examples/energy-management-app/energy-management-common/water-heater\r\n\r\n* Get WHM triggers working\r\n\r\n* Get energy tests passing\r\n\r\n* Add some more build infrastructure to set the DEM feature map\r\n\r\n* Address review comments from James and fix some CI tests\r\n\r\n* Restyled by clang-format\r\n\r\n* Restyled by gn\r\n\r\n* Undo changes to Objects.py\r\n\r\n* Address review from James and fix forecast subscription issues\r\n\r\n* Address review from James\r\n\r\n* Added ENERGY_REPORTING_TRIGGERS option to ESP32 and updated ESP32 README.md\r\n\r\n* Guard against a EVSE and WHM app being simultaneously defined\r\n\r\n* Updated Linux README.md to highlight --application and --featureSet args.\r\n\r\n* Fixed mis-spell\r\n\r\n* Removed duplicate WaterHeaterMode::Shutdown()\r\n\r\n* Fixed missing free of WaterHeaterMode and made a call to both EvseMain shutdown and WaterHeaterMain shutdown regardless of application mode.\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Applied code review comments, attempt to fix clang build failure with type in loop.\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Get DEM and EEM tests passing when running combined app in WHM mode\r\n\r\n* Restyled by clang-format\r\n\r\n* Ensure absMinPower and absMaxPower are set\r\n\r\n* Fix CI build error\r\n\r\n* Restyled by whitespace\r\n\r\n* Restyled by clang-format\r\n\r\n* Fix power levels so they fit within absMinPower and absMaxPower\r\n\r\n* Fix definition of CHIP_DEVICE_CONFIG_ENABLE_ENERGY_REPORTING_TRIGGER\r\n\r\n---------\r\n\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: pcoleman \u003cpcoleman@geotogether.com\u003e\r\nCo-authored-by: PeterC1965 \u003c101805108+PeterC1965@users.noreply.github.com\u003e\r\n\r\n* Darwin: Improve CommissioningType naming (#35209)\r\n\r\n* Darwin: Improve CommissioningType naming\r\n\r\n* Restore WaitForCommissioneeCommand behavior to always use CASE\r\n\r\n* THNETDIR and WIFINM: Align with latest spec and other small fixes (#35191)\r\n\r\n* YAML runner: Add support for EstablishPASESession command\r\n\r\n* Enable python constraints in TC_THNETDIR_*\r\n\r\n* THNETDIR: Align with latest spec\r\n\r\n- Implement ActiveTimestamp check for updates\r\n- Implement CASE session requirement for GetOperationalDataset\r\n- Update / add certification YAML tests accordingly\r\n\r\n* WIFINM: Implement CASE session requirement for NetworkPassphraseRequest\r\n\r\n- Implement CASE session requirement for NetworkPassphraseRequest\r\n- Add a corresponding YAML test\r\n\r\n* Apply test script suggestions from code review\r\n\r\nCo-authored-by: marchemi \u003c56955785+marchemi@users.noreply.github.com\u003e\r\n\r\n* Enable negative timed-invoke tests in TC_THNETDIR_2_2\r\n\r\nInstead, disable the test in CI for darwin-framework-tool.\r\n\r\nFixes matter-test-scripts#359\r\n\r\n---------\r\n\r\nCo-authored-by: marchemi \u003c56955785+marchemi@users.noreply.github.com\u003e\r\n\r\n* Re-run zap_convert_all on the tree. (#35235)\r\n\r\nThere have been various XML modifications that have not made it into the relevant .zap files.\r\n\r\n* increasing the chip task size for the 917soc (#35213)\r\n\r\n* Restyled by clang-format\r\n\r\n* Adding some useful debug prints\r\n\r\n* Update following code review comments on test spec\r\n\r\n* Update following code review comments on test spec\r\n\r\n* Update examples/energy-management-app/energy-management-common/water-heater/include/WhmDelegate.h\r\n\r\nCo-authored-by: jamesharrow \u003c93921463+jamesharrow@users.noreply.github.com\u003e\r\n\r\n* Update following code review comments on test spec\r\n\r\n* Update src/python_testing/TC_EWATERHTR_2_2.py\r\n\r\n* Update following code review comment\r\n\r\n* Update src/app/clusters/water-heater-management-server/water-heater-management-server.cpp\r\n\r\n* Fixes TP #4504 - Cannot run TC_EWATERHTR_2.1 under test harness using PIXIT values - so using featureMap instead.\r\n\r\n* Add some energy specific types\r\n\r\n* Removed unused PIXIT args in test-runner header\r\n\r\n* Fix issue with step 3\r\n\r\n* Removed duplicate steps 10a/10b and aligned to test plan.\r\n\r\n* Update src/python_testing/TC_EWATERHTR_2_3.py\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* Apply suggestions from code review\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* Fix code review comments - Changed temperature types to int16_t (signed). Avoided divide by 0.\r\n\r\n* Restyled by whitespace\r\n\r\n* Update to ESP32 featureMap to avoid static cast per review comment.\r\n\r\n* Fixed potential divide by 0 - Update examples/energy-management-app/energy-management-common/water-heater/src/WhmDelegateImpl.cpp\r\n\r\n* Restyled by clang-format\r\n\r\n* Update examples/energy-management-app/energy-management-common/water-heater/src/WhmDelegateImpl.cpp\r\n\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\n\r\n* Quick fix - needs to be Active not Inactive - Update src/python_testing/TC_EWATERHTR_2_1.py\r\n\r\n* Updated the test steps to align to feedback on Test Plan PR.\r\n\r\n* Another tweak to add \u0027inclusive\u0027 and align to test plan comment.\r\n\r\n---------\r\n\r\nSigned-off-by: marius-alex-tache \u003cmarius.tache@nxp.com\u003e\r\nSigned-off-by: Youngho Yoon \u003c34558998+yhoyoon@users.noreply.github.com\u003e\r\nSigned-off-by: Dina Benamar \u003cdina.benamarelmaaroufi@nxp.com\u003e\r\nSigned-off-by: Andrei Menzopol \u003candrei.menzopol@nxp.com\u003e\r\nSigned-off-by: Martin Girardot \u003cmartin.girardot@nxp.com\u003e\r\nSigned-off-by: Gatien Chapon \u003cgatien.chapon@nxp.com\u003e\r\nSigned-off-by: Doru Gucea \u003cdoru-cristian.gucea@nxp.com\u003e\r\nSigned-off-by: Dmytro Kashkarov \u003cur6lal@gmail.com\u003e\r\nSigned-off-by: Borys Nykytiuk \u003cborys.nykytiuk@telink-semi.com\u003e\r\nSigned-off-by: Haiwen Xia \u003chaiwen.xia@telink-semi.com\u003e\r\nCo-authored-by: Restyled.io \u003ccommits@restyled.io\u003e\r\nCo-authored-by: jamesharrow \u003c93921463+jamesharrow@users.noreply.github.com\u003e\r\nCo-authored-by: Marius Tache \u003c102153746+marius-alex-tache@users.noreply.github.com\u003e\r\nCo-authored-by: PSONALl \u003c77670766+PSONALl@users.noreply.github.com\u003e\r\nCo-authored-by: PSONALl \u003csonali.patil@espressif.com\u003e\r\nCo-authored-by: Rohit Jadhav \u003crohit.jadhav@espressif.com\u003e\r\nCo-authored-by: Rohit Jadhav \u003c69809379+jadhavrohit924@users.noreply.github.com\u003e\r\nCo-authored-by: Shubham Patil \u003cshubham.patil@espressif.com\u003e\r\nCo-authored-by: Andrei Litvin \u003candy314@gmail.com\u003e\r\nCo-authored-by: William \u003chicklin@users.noreply.github.com\u003e\r\nCo-authored-by: Justin Wood \u003cwoody@apple.com\u003e\r\nCo-authored-by: Youngho Yoon \u003c34558998+yhoyoon@users.noreply.github.com\u003e\r\nCo-authored-by: Karsten Sperling \u003c113487422+ksperling-apple@users.noreply.github.com\u003e\r\nCo-authored-by: dinabenamar \u003c108664279+dinabenamar@users.noreply.github.com\u003e\r\nCo-authored-by: Yufeng Wang \u003cyufengwang@google.com\u003e\r\nCo-authored-by: andrei-menzopol \u003c96489227+andrei-menzopol@users.noreply.github.com\u003e\r\nCo-authored-by: Miłosz Tomkiel \u003cxyz.toripuru@gmail.com\u003e\r\nCo-authored-by: Martin Girardot \u003c165289184+Martin-NXP@users.noreply.github.com\u003e\r\nCo-authored-by: Andrei Litvin \u003candreilitvin@google.com\u003e\r\nCo-authored-by: Abdul Samad \u003cabdul.samad@smartthings.com\u003e\r\nCo-authored-by: shripad621git \u003c79364691+shripad621git@users.noreply.github.com\u003e\r\nCo-authored-by: Stefan Bauer-Schwan \u003c62264998+bauerschwan@users.noreply.github.com\u003e\r\nCo-authored-by: Amine Alami \u003c43780877+Alami-Amine@users.noreply.github.com\u003e\r\nCo-authored-by: Jakub Latusek \u003cj.latusek@samsung.com\u003e\r\nCo-authored-by: Arkadiusz Bokowy \u003ca.bokowy@samsung.com\u003e\r\nCo-authored-by: fesseha-eve \u003c88329315+fessehaeve@users.noreply.github.com\u003e\r\nCo-authored-by: Kiel Oleson \u003ckielo@apple.com\u003e\r\nCo-authored-by: Terence Hampson \u003cthampson@google.com\u003e\r\nCo-authored-by: yunhanw \u003cyunhanw@google.com\u003e\r\nCo-authored-by: Junior Martinez \u003c67972863+jmartinez-silabs@users.noreply.github.com\u003e\r\nCo-authored-by: Alex Tsitsiura \u003cs07641069@gmail.com\u003e\r\nCo-authored-by: jrhees-cae \u003c61466710+jrhees-cae@users.noreply.github.com\u003e\r\nCo-authored-by: Martin Girardot \u003cmartin.girardot@nxp.com\u003e\r\nCo-authored-by: Dina Benamar \u003cdina.benamarelmaaroufi@nxp.com\u003e\r\nCo-authored-by: Marian Chereji \u003cmarian.chereji@nxp.com\u003e\r\nCo-authored-by: Andrei Menzopol \u003candrei.menzopol@nxp.com\u003e\r\nCo-authored-by: Gatien Chapon \u003cgatien.chapon@nxp.com\u003e\r\nCo-authored-by: Yassine El Aissaoui \u003cyassine.elaissaoui@nxp.com\u003e\r\nCo-authored-by: saurabhst \u003cs.kumar9@samsung.com\u003e\r\nCo-authored-by: mkardous-silabs \u003c84793247+mkardous-silabs@users.noreply.github.com\u003e\r\nCo-authored-by: joonhaengHeo \u003c85541460+joonhaengHeo@users.noreply.github.com\u003e\r\nCo-authored-by: doru91 \u003cdoru-cristian.gucea@nxp.com\u003e\r\nCo-authored-by: adabreuti \u003c76965454+adabreuti@users.noreply.github.com\u003e\r\nCo-authored-by: cdj \u003c45139296+DejinChen@users.noreply.github.com\u003e\r\nCo-authored-by: Dmytro Kashkarov \u003cur6lal@gmail.com\u003e\r\nCo-authored-by: C Freeman \u003ccecille@google.com\u003e\r\nCo-authored-by: Tennessee Carmel-Veilleux \u003ctennessee.carmelveilleux@gmail.com\u003e\r\nCo-authored-by: Pradip De \u003cpradipd@google.com\u003e\r\nCo-authored-by: Anu Biradar \u003c104591549+abiradarti@users.noreply.github.com\u003e\r\nCo-authored-by: Harshith-GRL \u003c145322529+Harshith-GRL@users.noreply.github.com\u003e\r\nCo-authored-by: Rob Bultman \u003crob.Bultman@gmail.com\u003e\r\nCo-authored-by: Borys Nykytiuk \u003c165832970+BorysNykytiuk@users.noreply.github.com\u003e\r\nCo-authored-by: Haiwen Xia \u003chaiwen.xia@telink-semi.com\u003e\r\nCo-authored-by: Mathieu Kardous \u003cmathieu.kardous@silabs.com\u003e\r\nCo-authored-by: Jeff Tung \u003c100387939+jtung-apple@users.noreply.github.com\u003e\r\nCo-authored-by: Boris Zbarsky \u003cbzbarsky@apple.com\u003e\r\nCo-authored-by: Paul Regan \u003c64710345+paulr34@users.noreply.github.com\u003e\r\nCo-authored-by: Thirupathi S \u003c108743108+Thirsrin@users.noreply.github.com\u003e\r\nCo-authored-by: sarthak shaha \u003c130495524+Sarthak-Shaha@users.noreply.github.com\u003e\r\nCo-authored-by: lpbeliveau-silabs \u003c112982107+lpbeliveau-silabs@users.noreply.github.com\u003e\r\nCo-authored-by: Nivi Sarkar \u003c55898241+nivi-apple@users.noreply.github.com\u003e\r\nCo-authored-by: Sharad Binjola \u003c31142146+sharadb-amazon@users.noreply.github.com\u003e\r\nCo-authored-by: Miłosz Tomkiel \u003cm.tomkiel@samsung.com\u003e\r\nCo-authored-by: Sean \u003c97990350+s-mcclain@users.noreply.github.com\u003e\r\nCo-authored-by: mthiesc \u003cmthiesc@amazon.com\u003e\r\nCo-authored-by: Lazar Kovacic \u003clkovacic@amazon.com\u003e\r\nCo-authored-by: chrisdecenzo \u003c61757564+chrisdecenzo@users.noreply.github.com\u003e\r\nCo-authored-by: Wang Qixiang \u003c43193572+wqx6@users.noreply.github.com\u003e\r\nCo-authored-by: Hasty Granbery \u003chasty@granbery.org\u003e\r\nCo-authored-by: Nivedita Sarkar \u003cnivedita_sarkar@apple.com\u003e\r\nCo-authored-by: Chris Letnick \u003ccletnick@google.com\u003e\r\nCo-authored-by: Douglas Rocha Ferraz \u003crochaferraz@google.com\u003e\r\nCo-authored-by: Petru Lauric \u003c81822411+plauric@users.noreply.github.com\u003e\r\nCo-authored-by: Shao Ling Tan \u003c161761051+shaoltan-amazon@users.noreply.github.com\u003e\r\nCo-authored-by: Michael Rupp \u003c95718139+mykrupp@users.noreply.github.com\u003e\r\nCo-authored-by: Anush Nadathur \u003canush@apple.com\u003e\r\nCo-authored-by: chirag-silabs \u003c100861685+chirag-silabs@users.noreply.github.com\u003e\r\nCo-authored-by: waqar rashid \u003c6962179+wickywaka@users.noreply.github.com\u003e\r\nCo-authored-by: shgutte \u003c102281713+shgutte@users.noreply.github.com\u003e\r\nCo-authored-by: Thomas Lea \u003c35579828+tleacmcsa@users.noreply.github.com\u003e\r\nCo-authored-by: yinyihu-silabs \u003c83772323+yinyihu-silabs@users.noreply.github.com\u003e\r\nCo-authored-by: Erwin Pan \u003cerwinpan1@gmail.com\u003e\r\nCo-authored-by: marchemi \u003c56955785+marchemi@users.noreply.github.com\u003e\r\nCo-authored-by: James Harrow \u003cjames.harrow@gmail.com\u003e",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "27d8cd7cd96ecfba0c728c49b0ba2e6b56a3df59",
      "old_mode": 33188,
      "old_path": "examples/energy-management-app/energy-management-common/water-heater/include/WhmDelegate.h",
      "new_id": "dffa91b5297efc32c3032c8e9cfe4602c14d60f6",
      "new_mode": 33188,
      "new_path": "examples/energy-management-app/energy-management-common/water-heater/include/WhmDelegate.h"
    },
    {
      "type": "modify",
      "old_id": "af09a13a0dab2d8769ea5add99ff87c1f936a3d8",
      "old_mode": 33188,
      "old_path": "examples/energy-management-app/energy-management-common/water-heater/src/WhmDelegateImpl.cpp",
      "new_id": "79a64d5226f486de577cf42c51e35dc99ab79294",
      "new_mode": 33188,
      "new_path": "examples/energy-management-app/energy-management-common/water-heater/src/WhmDelegateImpl.cpp"
    },
    {
      "type": "modify",
      "old_id": "af01f36bc5da69180414e0837e20e68169dc3f3f",
      "old_mode": 33188,
      "old_path": "examples/energy-management-app/energy-management-common/water-heater/src/WhmManufacturer.cpp",
      "new_id": "446d684e41be646e6a601bdd5bfd015144ec3059",
      "new_mode": 33188,
      "new_path": "examples/energy-management-app/energy-management-common/water-heater/src/WhmManufacturer.cpp"
    },
    {
      "type": "modify",
      "old_id": "33623882cc2e7cb7b3928f9765c6912cf1ef8690",
      "old_mode": 33188,
      "old_path": "examples/energy-management-app/esp32/main/main.cpp",
      "new_id": "d235fa4291ac9828d0616aac7008150dff8c31ec",
      "new_mode": 33188,
      "new_path": "examples/energy-management-app/esp32/main/main.cpp"
    },
    {
      "type": "modify",
      "old_id": "3cbe76f60d38f45fbaf392922eb23765cb8ac664",
      "old_mode": 33188,
      "old_path": "src/app/clusters/water-heater-management-server/water-heater-management-server.cpp",
      "new_id": "165fc94fb319a296fd2c9ebb931f962d575b8c3b",
      "new_mode": 33188,
      "new_path": "src/app/clusters/water-heater-management-server/water-heater-management-server.cpp"
    },
    {
      "type": "modify",
      "old_id": "71a52de9b45eef28dc423923e44ccf7516a211ea",
      "old_mode": 33188,
      "old_path": "src/app/clusters/water-heater-management-server/water-heater-management-server.h",
      "new_id": "25ff16265b6df728395fab629fece22eb277ae58",
      "new_mode": 33188,
      "new_path": "src/app/clusters/water-heater-management-server/water-heater-management-server.h"
    },
    {
      "type": "modify",
      "old_id": "97aebe690423c0bbaf88f9d475b536c376defd38",
      "old_mode": 33188,
      "old_path": "src/app/util/basic-types.h",
      "new_id": "449756fe90ef42ec4e11b573ce48d10d9f47ed36",
      "new_mode": 33188,
      "new_path": "src/app/util/basic-types.h"
    },
    {
      "type": "modify",
      "old_id": "055b5923cf6f073e4bbcf6cf9b7860a48f612be8",
      "old_mode": 33188,
      "old_path": "src/python_testing/TC_EWATERHTR_2_1.py",
      "new_id": "df5d2821ece9e4c119629f8ecfadd81178787775",
      "new_mode": 33188,
      "new_path": "src/python_testing/TC_EWATERHTR_2_1.py"
    },
    {
      "type": "modify",
      "old_id": "5fd40e4130a64e9deb54b9203cb7e6b8d7db4ade",
      "old_mode": 33188,
      "old_path": "src/python_testing/TC_EWATERHTR_2_2.py",
      "new_id": "8c96e52d4524c840ba85edab124010b41351c150",
      "new_mode": 33188,
      "new_path": "src/python_testing/TC_EWATERHTR_2_2.py"
    },
    {
      "type": "modify",
      "old_id": "d2bc97d33b511375154cfc1746bcbb852dcf01ad",
      "old_mode": 33188,
      "old_path": "src/python_testing/TC_EWATERHTR_2_3.py",
      "new_id": "e39257134b3db5787571e399730dd4b0a87cb272",
      "new_mode": 33188,
      "new_path": "src/python_testing/TC_EWATERHTR_2_3.py"
    }
  ]
}