Command handler interface mixin minimal change#207
Draft
ratgr wants to merge 136 commits into
Draft
Conversation
Co-authored-by: Andrei Litvin <andy314@gmail.com>
…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 <dina.benamarelmaaroufi@nxp.com> [rw61x] use wifi osa to register idle function in MoT as well Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com> * [NXP][platform][rt1170] Small reworks to support building Matter-over-Thread with cmake as part of the "chip" module Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> [platform][rt1170] Switching spinel GN options to nxp prefix Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com> [rt1170] Enabling session resumption which is now supported Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com> [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 <dina.benamarelmaaroufi@nxp.com> [platform][rt1060] Switching spinel GN options to nxp prefix Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com> [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 <gatien.chapon@nxp.com> [platform][common] Updating legacy factory data file to support cmake build Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> [platform][common] refactor icd configs Signed-off-by: Doru Gucea <doru-cristian.gucea@nxp.com> [platform][common][ble] Remove BLE Host Task functions/queues and use SDK ones Signed-off-by: Andrei Menzopol <andrei.menzopol@nxp.com> [platform][common] Update include in src/platform/nxp/common/factory_data/legacy/FactoryDataProvider.cpp to use public header Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> [platform][common] Adding missing CHIP configs to be set with Kconfig values Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> [platform][common] Make sure to check if CHIP_DEVICE_CONFIG_ENABLE_SED is defined Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> [platform][common] Use "ifdef" around Kconfig that may not be defined for all platforms Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> * [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 <dina.benamarelmaaroufi@nxp.com> [platform][mcxw71_k32w1] Adding compile options to compile MoT with cmake Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> [platform][mcxw71_k32w1] Moving factory data source files to be added within the platform target Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> [platform][mcxw71] refactor icd configs Signed-off-by: Doru Gucea <doru-cristian.gucea@nxp.com> [platform][mcxw71_k32w1] Remove BLE Host Task function declarations * Functions from SDK will be used instead. Signed-off-by: Andrei Menzopol <andrei.menzopol@nxp.com> [platform][mcxw71_k32w1] Adding dependency on "src/credentials:credentials_header" which is required to build factory data files Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> [platform][mcxw71_k32w1] Moving some CHIP configs under CHIPDevicePlatformConfig.h to replace CHIPProjectConfig.h with Kconfig generation Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> [platform][mcxw71_k32w1] Set CHIP_CONFIG_MAX_FABRICS to 5 Signed-off-by: Andrei Menzopol <andrei.menzopol@nxp.com> [platform][mcxw71_k32w1] Add ifndef around CHIP_DEVICE_CONFIG_KVS_WEAR_STATS to make sure it's not redefined Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> [platform][mcxw71_k32w1] Add ifndef condition to avoid redefinition of CHIP macros Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> * [NXP][platform][k32w0] refactor icd configs Signed-off-by: Doru Gucea <doru-cristian.gucea@nxp.com> * [NXP][examples][common] Updating NXP example code to support cmake build Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com> [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 <dina.benamarelmaaroufi@nxp.com> [examples][common] Moving BleZephyrManagerApp.cpp from thermostat folder to be common under "examples/platform/nxp/common" Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> * [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 <dina.benamarelmaaroufi@nxp.com> * [NXP][examples][rt1170] Update path to BleZephyrManagerApp.cpp in GN file Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> * [NXP][examples][rw61x] Update path to BleZephyrManagerApp.cpp in GN file Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> * [NXP][examples][mcxw71_k32w1] Moving factory data source files to be added within the platform target Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> [examples][mcxw71_k32w1] Updating platform AppFactoryDataExample.cpp to support building with cmake Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> * [NXP][examples][zephyr] Update thermostat CMakeLists.txt with the new path of BleZephyrManagerApp.cpp Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> * [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 <dina.benamarelmaaroufi@nxp.com> * [NXP][platform][rw61x] Remove include fsl_adapter_rng.h which is not necessary Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> * [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 <dina.benamarelmaaroufi@nxp.com> * [NXP] Update nxp_matter_support submodule Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> * [NXP] Update docker image version to use for building NXP examples Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> * [NXP] Update platform BUILD.gn to align with latest updates Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> * Restyled by whitespace * Restyled by gn --------- Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com> Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com> Signed-off-by: Doru Gucea <doru-cristian.gucea@nxp.com> Co-authored-by: Doru Gucea <doru-cristian.gucea@nxp.com> Co-authored-by: Gatien Chapon <gatien.chapon@nxp.com> Co-authored-by: Jaafar BEN YOUNES <jaafar.benyounes@nxp.com> Co-authored-by: Restyled.io <commits@restyled.io>
…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 marius.preda@nxp.com * Restyled by clang-format --------- Signed-off-by: Marius Preda marius.preda@nxp.com Co-authored-by: Restyled.io <commits@restyled.io>
Co-authored-by: Andrei Litvin <andreilitvin@google.com>
…p#38461) Co-authored-by: Andrei Litvin <andreilitvin@google.com>
# 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 <commits@restyled.io>
* 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 <commits@restyled.io>
…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 <commits@restyled.io>
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 <satyanaag.bollimpalli@silabs.com>
* 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 <commits@restyled.io>
…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 <satyanaag.bollimpalli@silabs.com> Co-authored-by: Restyled.io <commits@restyled.io>
…#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 <commits@restyled.io>
…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
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
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