forked from project-chip/connectedhomeip
-
Notifications
You must be signed in to change notification settings - Fork 0
Command handler interface mixin minimal change #207
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Draft
ratgr
wants to merge
136
commits into
andy31415:cluster_metadata_generation
Choose a base branch
from
ratgr:command-handler-interface-mixin-minimal-change
base: cluster_metadata_generation
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Command handler interface mixin minimal change #207
ratgr
wants to merge
136
commits into
andy31415:cluster_metadata_generation
from
ratgr:command-handler-interface-mixin-minimal-change
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: Andrei Litvin <[email protected]>
…ct-chip#38431) * [NXP][platform][rw61x] Small reworks to support building Matter-over-Thread with cmake as part of the "chip" module Signed-off-by: Dina Benamar <[email protected]> [rw61x] use wifi osa to register idle function in MoT as well Signed-off-by: Gatien Chapon <[email protected]> * [NXP][platform][rt1170] Small reworks to support building Matter-over-Thread with cmake as part of the "chip" module Signed-off-by: Dina Benamar <[email protected]> [platform][rt1170] Switching spinel GN options to nxp prefix Signed-off-by: Gatien Chapon <[email protected]> [rt1170] Enabling session resumption which is now supported Signed-off-by: Gatien Chapon <[email protected]> [rt1170] use wifi osa to register idle function in MoT as well * [NXP][platform][rt1060] Small reworks to support building Matter-over-Thread with cmake as part of the "chip" module Signed-off-by: Dina Benamar <[email protected]> [platform][rt1060] Switching spinel GN options to nxp prefix Signed-off-by: Gatien Chapon <[email protected]> [rt1060] use wifi osa to register idle function in MoT as well * [NXP][platform][common] Updating NXP platform code to support cmake build Signed-off-by: Gatien Chapon <[email protected]> [platform][common] Updating legacy factory data file to support cmake build Signed-off-by: Dina Benamar <[email protected]> [platform][common] refactor icd configs Signed-off-by: Doru Gucea <[email protected]> [platform][common][ble] Remove BLE Host Task functions/queues and use SDK ones Signed-off-by: Andrei Menzopol <[email protected]> [platform][common] Update include in src/platform/nxp/common/factory_data/legacy/FactoryDataProvider.cpp to use public header Signed-off-by: Dina Benamar <[email protected]> [platform][common] Adding missing CHIP configs to be set with Kconfig values Signed-off-by: Dina Benamar <[email protected]> [platform][common] Make sure to check if CHIP_DEVICE_CONFIG_ENABLE_SED is defined Signed-off-by: Dina Benamar <[email protected]> [platform][common] Use "ifdef" around Kconfig that may not be defined for all platforms Signed-off-by: Dina Benamar <[email protected]> * [NXP][platform][mcxw71_k32w1] Updates and reworks in platform files for CMake build support and improvements [platform][mcxw71_k32w1] Reworks to use CHIP config and app config generation with Kconfig Signed-off-by: Dina Benamar <[email protected]> [platform][mcxw71_k32w1] Adding compile options to compile MoT with cmake Signed-off-by: Dina Benamar <[email protected]> [platform][mcxw71_k32w1] Moving factory data source files to be added within the platform target Signed-off-by: Dina Benamar <[email protected]> [platform][mcxw71] refactor icd configs Signed-off-by: Doru Gucea <[email protected]> [platform][mcxw71_k32w1] Remove BLE Host Task function declarations * Functions from SDK will be used instead. Signed-off-by: Andrei Menzopol <[email protected]> [platform][mcxw71_k32w1] Adding dependency on "src/credentials:credentials_header" which is required to build factory data files Signed-off-by: Dina Benamar <[email protected]> [platform][mcxw71_k32w1] Moving some CHIP configs under CHIPDevicePlatformConfig.h to replace CHIPProjectConfig.h with Kconfig generation Signed-off-by: Dina Benamar <[email protected]> [platform][mcxw71_k32w1] Set CHIP_CONFIG_MAX_FABRICS to 5 Signed-off-by: Andrei Menzopol <[email protected]> [platform][mcxw71_k32w1] Add ifndef around CHIP_DEVICE_CONFIG_KVS_WEAR_STATS to make sure it's not redefined Signed-off-by: Dina Benamar <[email protected]> [platform][mcxw71_k32w1] Add ifndef condition to avoid redefinition of CHIP macros Signed-off-by: Dina Benamar <[email protected]> * [NXP][platform][k32w0] refactor icd configs Signed-off-by: Doru Gucea <[email protected]> * [NXP][examples][common] Updating NXP example code to support cmake build Signed-off-by: Gatien Chapon <[email protected]> [examples][common] Updates in examples files to align with CMake reworks [examples][common] Updating contact-sensor-app and lighting-app applications by adding "vApplicationStackOverflowHook" to the main.cpp Signed-off-by: Dina Benamar <[email protected]> [examples][common] Moving BleZephyrManagerApp.cpp from thermostat folder to be common under "examples/platform/nxp/common" Signed-off-by: Dina Benamar <[email protected]> * [NXP][examples][rt1060] Updates in examples files to align with CMake reworks [examples][rt1060] keep empty implementation for button [examples][rt1060] Update path to BleZephyrManagerApp.cpp in GN file Signed-off-by: Dina Benamar <[email protected]> * [NXP][examples][rt1170] Update path to BleZephyrManagerApp.cpp in GN file Signed-off-by: Dina Benamar <[email protected]> * [NXP][examples][rw61x] Update path to BleZephyrManagerApp.cpp in GN file Signed-off-by: Dina Benamar <[email protected]> * [NXP][examples][mcxw71_k32w1] Moving factory data source files to be added within the platform target Signed-off-by: Dina Benamar <[email protected]> [examples][mcxw71_k32w1] Updating platform AppFactoryDataExample.cpp to support building with cmake Signed-off-by: Dina Benamar <[email protected]> * [NXP][examples][zephyr] Update thermostat CMakeLists.txt with the new path of BleZephyrManagerApp.cpp Signed-off-by: Dina Benamar <[email protected]> * [NXP][examples][rt1060][ota] fix flash size for RT1060-EVKC on gn build * [NXP][platform][rt1060] Remove include fsl_adapter_rng.h which is not necessary Signed-off-by: Dina Benamar <[email protected]> * [NXP][platform][rw61x] Remove include fsl_adapter_rng.h which is not necessary Signed-off-by: Dina Benamar <[email protected]> * [NXP][rt1060][cmake][FileSystem] switch to NVS FileSystem * [NXP][rt1170][cmake][FileSystem] switch to NVS FileSystem * [NXP] Update copyrights year date Signed-off-by: Dina Benamar <[email protected]> * [NXP] Update nxp_matter_support submodule Signed-off-by: Dina Benamar <[email protected]> * [NXP] Update docker image version to use for building NXP examples Signed-off-by: Dina Benamar <[email protected]> * [NXP] Update platform BUILD.gn to align with latest updates Signed-off-by: Dina Benamar <[email protected]> * Restyled by whitespace * Restyled by gn --------- Signed-off-by: Gatien Chapon <[email protected]> Signed-off-by: Dina Benamar <[email protected]> Signed-off-by: Doru Gucea <[email protected]> Co-authored-by: Doru Gucea <[email protected]> Co-authored-by: Gatien Chapon <[email protected]> Co-authored-by: Jaafar BEN YOUNES <[email protected]> Co-authored-by: Restyled.io <[email protected]>
…ssControl Cluster (project-chip#38263) * Modifying how ACL Attribute is written: making it use a non-empty initial ReplaceAll list * removing Test modifications on ACL Extensions * adding reference to issue in TODO comment * integrating comments * integrating comments * Revert "removing Test modifications on ACL Extensions" This reverts commit 055af80. * Clarifying AttributeDataIB Checkpoint * adding Valid CAT values * Integrating comments * Add comments clarifying AttributeDataIB rollback * Integrating comments again * change method order * integrating new comments * revert change to IM Error Code return from AccessControl cluster * changing constants use to reserveBuffer for TLV writing * Integrating more Boris Comments * removing state related to IsWriteRequestChunked * Create TryToStartList that dynamically starts message in case of NO Memory
…38449) New status code added in Spec PR project-chip#11555. Update IM check to return new code when wrong transport used for `L` flagged commands.
…roject-chip#38370) All webrtc commands require a LargePayload session, so it is mandatory for all commands.
* [NXP] Update border router name in the TBRM cluster Update border router name in TBRM cluster to be consistent with the Border Router service name present in MeshCop service. Signed-off-by: Marius Preda [email protected] * Restyled by clang-format --------- Signed-off-by: Marius Preda [email protected] Co-authored-by: Restyled.io <[email protected]>
Co-authored-by: Andrei Litvin <[email protected]>
…p#38461) Co-authored-by: Andrei Litvin <[email protected]>
# Conflicts: # .github/workflows/protocol_compatibility.yaml
…oject-chip#38451) * ESP32: add sdkconfig to build lighting-app for Matter Over Thread - Added the sdkconfig.defaults.esp32c6_thread which can be used to build the lighting-app for thread transport - Also, added steps in the README * Restyled by prettier-markdown * refine the heading --------- Co-authored-by: Restyled.io <[email protected]>
* Cleanup typos in the code * Simplify IDL usage in scripts * Fix matter_idl_generator_files * Keep tests only in matter-idl GN * Restyled by isort * Fix Java CI * Activate env prior to gn gen in gn_silabs_example.sh --------- Co-authored-by: Restyled.io <[email protected]>
…ip#38128) * da-revocation: add test data for dac and pai both revoked case Clubbed the different revocation set into single file and adjusted the doc accordingly. * Added certification test cases for TC-DA-1-9 Added the test cases for testing device attestation revocation feature * fix the yaml indentation * Restyled by whitespace * Restyled by prettier-json * Restyled by prettier-markdown * Restyled by prettier-yaml * Add CRL for revoked PAI CA and adjust the unit tests after clubbing data into single file * remove manual test case for TC-DA-1-9 * update comment in dac provider test vectors * merge revocation sets * add python test for tc-da-1-9 * Restyled by autopep8 * Restyled by isort * remove unnecessary if * skip running tc-da-1-9 in ci * Restyled by prettier-yaml * change test case as per python testing requirements * enable tc-da in ci * add a todo * Restyled by autopep8 * fix the app path * Restyled by autopep8 * fix paths for ci * remove unused commnet * address review comments * add asserts when required arguments are not present and use the default log path for storing app logs, also fix the ci by passing --PICS argument --------- Co-authored-by: Restyled.io <[email protected]>
To address CHIP-Specifications/chip-test-plans#5067 this test case needs to be removed because the Oven Cavity Operational State cluster does not support pause and resume what this step is testing for.
- Simplify call sites by not passing the stringified module name - Avoid duplicate plain and os_log string constants - Actually avoid retain/release of the logger Overall this saves about 50k of code and 100k of strings in Matter.framework
…#38122) * Closure device type updates as per latest specification * Adding App to MTRDeviceTypeClass Enum * Updating closure app with new device types --------- Co-authored-by: sabollim <[email protected]>
* Add handling for non-volatile attributes * Restyled by whitespace * Restyled by clang-format * Add missing clearing of the provided vectors --------- Co-authored-by: Restyled.io <[email protected]>
…on (project-chip#38424) * Update Closure dimension cluster as per latest specification * Adding missing zap files * Updating server implementation as per new spec changes * Removing constrain check for boolean * Restyled by clang-format * Restyled by prettier-json * Add missing Git SHA in XML * Removing old struct files --------- Co-authored-by: sabollim <[email protected]> Co-authored-by: Restyled.io <[email protected]>
…#38445) * Allow generation of no-ICAC cert chains in Python tests - Allow generation of no-ICAC cert chains (valid) in Python tests - Cache steps list to avoid possible recursive calls in upcoming PR in matter_testing_support.py Testing done: - All existing tests still pass * Restyled by clang-format * Fix lint * Backport project-chip#38467 --------- Co-authored-by: Restyled.io <[email protected]>
…8413) The documentation was misleading as to the sematics of the value.
…37930) * Generate ConcentrationMeasurement cluster with Alchemy * Regenerate cluster with new alchemy * Add missing ZAP generated files
…rice' as money, remove pricelevel from components) (project-chip#38474) * Update commodity-price-cluster.xml based on spec PR 11569 * zap regen all * Update EntryToText.cpp - restore newline at EOF * Update EntryToText.h - restore newline at EOF * Deleted unused java files so CI passes
* Generate AccessControl cluster with Alchemy * Add matter files generated by ZAP * Update generated files
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Temporary PR for testing and showing differences
Testing