Tsbm master #20
Closed
Tsbm master #20
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
MauritsFassaert
commented
Aug 20, 2024
MauritsFassaert
commented
…-chip#29511) * Start removing enums and structs from top level definitions in the parser * Remove top level structs from matter idl auto-generation * Fix generator type logic * Update test inputs to move global structs * Fix unit tests * Restyle * Remove unused import --------- Co-authored-by: Andrei Litvin <[email protected]>
…oject-chip#29372) * [TC-DRLK-2.4] Handle DUT's that do not support USR & PIN features Fixes CHIP-Specifications/chip-test-plans#3461 * Zap regen * Restyled by prettier-yaml --------- Co-authored-by: Restyled.io <[email protected]>
* Fix script issue: 649, 650, 651, 652, 653, 654, Modified tests: TC_ACL_2_10 TC_ACL_2_7 TC_ACL_2_8 TC_DGWIFI_3_2_Simulated TC_FLW_2_1 TC_LUNIT_3_1 TC_RH_2_1 TC_SC_4_1 Automated test: TC_RVCCLEANM_3_3 Semi-automated tests: TC_REFALM_2_1 Fixes issue: 29331 * Auto generated files * The following YAMLs are removed since there are Python script for the testcases. TC-DT-1.1 TC-IDM-10.1 TC-IDM-11.1 TC-SM-1.1 TC-SM-1.2 * Fixes issue: 656 & 658 Modified tests: TC-GRPKEY-2.1 Removed test: TC-LVL-2.3_Simulated * Modified testcase TC-SC-5.1 * Fixes issue: 655, 660 Modified tests: TC-ACL-2.4 TC-ACL-2.5 TC-ACL-2.6 TC-ACL-2.7 TC-SC-5.2 Revert back the DGGEN-2.1 testcase * Modified test FLW-2.1 * Fixes YAML script issue: 664, 668, 671, 672, 673 TC-DGGEN-2.1 TC-TSTAT-1.1 TC-TSUIC-1.1 TC-TSTAT-2.1 TC-WNCV-3.4 TC-WNCV-3.5 Manual tests: TC-REFALM-2.2 TC-REFALM-2.3 * Restyled by whitespace * Fixes issue 984 TC-TCTL-1.1 * Reverted back changes for tests: TC-DGGEN-2.1 TC-REFALM-2.1 TC-RVCCLEANM-3.3 * Modify ciTest.json file * Fix type --------- Co-authored-by: Restyled.io <[email protected]>
…ect-chip#29522) Bumps [third_party/libwebsockets/repo](https://github.com/warmcat/libwebsockets) from `7ef2065` to `4144c1e`. - [Commits](warmcat/libwebsockets@7ef2065...4144c1e) --- updated-dependencies: - dependency-name: third_party/libwebsockets/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…-chip#29524) Bumps [third_party/openthread/repo](https://github.com/openthread/openthread) from `f0530af` to `eaa6261`. - [Release notes](https://github.com/openthread/openthread/releases) - [Commits](openthread/openthread@f0530af...eaa6261) --- updated-dependencies: - dependency-name: third_party/openthread/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ip#29527) Bumps [third_party/mbedtls/repo](https://github.com/ARMmbed/mbedtls) from `b53e630` to `da635ab`. - [Release notes](https://github.com/ARMmbed/mbedtls/releases) - [Commits](Mbed-TLS/mbedtls@b53e630...da635ab) --- updated-dependencies: - dependency-name: third_party/mbedtls/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…t-chip#29492) * Start removing split defines and setups for InitDataModelHandler * Restyle * Add EmptyDataModelHandler to Matter.framework. --------- Co-authored-by: Andrei Litvin <[email protected]> Co-authored-by: Boris Zbarsky <[email protected]>
…p#29530) This should save about 2.5GB of space. Ideally we should not download these in the first place, however I would have to figure out how to set that up. For now tested locally that compile seems to pass because we use the NDK, however CI will fully validate this.
…chip#29531) * [Telink] Use common Zephyr heap * [Telink] Added default RF TX power selection * [Telink] Update Telink image to 16 --------- Co-authored-by: Andrii Bilynskyi <[email protected]> Co-authored-by: Misha Tkachenko <[email protected]>
…#29532) Bumps [third_party/imgui/repo](https://github.com/ocornut/imgui) from `f4790f6` to `a614387`. - [Release notes](https://github.com/ocornut/imgui/releases) - [Commits](ocornut/imgui@f4790f6...a614387) --- updated-dependencies: - dependency-name: third_party/imgui/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ject-chip#29458) * [Darwin] MTRDeviceTests test017 - fix resubscription expectation * Update src/app/InteractionModelEngine.h Co-authored-by: Boris Zbarsky <[email protected]> --------- Co-authored-by: Boris Zbarsky <[email protected]>
…hip#29484) * [crypto] Extract X509 functions for mbedTLS and PSA PSA crypto backend uses mbedTLS API to implement functions for certificate generation and parsing, so the functions are duplicated between PSA and mbedTLS backends. Extract the functions to reduce the code duplication. * Restyled by whitespace * Restyled by gn * Code review --------- Co-authored-by: Restyled.io <[email protected]>
Bumps [docker/setup-buildx-action](https://github.com/docker/setup-buildx-action) from 2 to 3. - [Release notes](https://github.com/docker/setup-buildx-action/releases) - [Commits](docker/setup-buildx-action@v2...v3) --- updated-dependencies: - dependency-name: docker/setup-buildx-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…t-chip#29519) * Add Chef RVC device type * Update rootnode_roboticvacuumcleaner_1807ff0c49.matter * Restyled by clang-format * Fix nit in chef-rvc-mode-delegate.* * Rollback previous changes regarding to RvcCleanMode * Move ifdef ZCL_USING_RVC_RUN_MODE_CLUSTER_SERVER to latest line since the RVC_CLEAN_MODE will use global variable in RVC_RUN_MODE --------- Co-authored-by: Restyled.io <[email protected]>
* CastingDiscovery: fix snprintf warning snprintf appends a null terminator, as does strcat, so both buffers need to be upsized by 1 to accommodate that. * ID buffer size was also incorrect * Restyled by clang-format --------- Co-authored-by: Restyled.io <[email protected]>
* [NXP] Introducing Matter support for K32W1 Additional information: * SDK: 2.12.5 K32W148-EVK, downloadable from https://mcuxpresso.nxp.com * MCU overview: https://www.nxp.com/products/wireless/multiprotocol-mcus/secure-and-ultra-low-power-mcu-for-matter-over-thread-and-bluetooth-le-5-3:K32W148 Signed-off-by: Doru Gucea <[email protected]> Signed-off-by: Marius Tache <[email protected]> * [NXP] Add CI/CV support for K32W1 Signed-off-by: Marius Tache <[email protected]> * [NXP] Bump ot-nxp submodule Signed-off-by: Marius Tache <[email protected]> * [K32W] Add OPENTHREAD_PLATFORM_CORE_CONFIG_FILE define Signed-off-by: Marius Tache <[email protected]> * [K32W1] Fix misspell Signed-off-by: Marius Tache <[email protected]> * Restyled by whitespace * Restyled by clang-format * Restyled by gn * Restyled by prettier-markdown * Restyled by autopep8 * Restyled by isort * [K32W1] Fix lint errors Signed-off-by: Marius Tache <[email protected]> * [K32W1] Removed deprecated class Signed-off-by: Marius Tache <[email protected]> * Restyled by clang-format * [test] Update testdata/all_targets_linux_x64 with K32W1 targets Signed-off-by: Marius Tache <[email protected]> * [K32W] Fix target names in workflow Signed-off-by: Marius Tache <[email protected]> * [K32W1] Remove deprecated commented sections Signed-off-by: Marius Tache <[email protected]> * [K32W1] Add missing # to reference targets in README Signed-off-by: Marius Tache <[email protected]> * [K32W1] Fix spelling errors Signed-off-by: Marius Tache <[email protected]> * [K32W1] Update CSR crypto flag usage Signed-off-by: Marius Tache <[email protected]> * [K32W1] Remove ResetWatermarks empty implementation Signed-off-by: Marius Tache <[email protected]> * Fix typo in device.gni after fixing conflict Signed-off-by: marius-alex-tache <[email protected]> * [K32W1] Remove anchors and backtick some words Signed-off-by: marius-alex-tache <[email protected]> * [K32W] Fix confusing naming in K32W builder Signed-off-by: marius-alex-tache <[email protected]> * [K32W] Remove commented option Signed-off-by: marius-alex-tache <[email protected]> * [K32W] Fix binary extension format Signed-off-by: marius-alex-tache <[email protected]> * Restyled by autopep8 * [K32W] Fix binary name when copying Signed-off-by: marius-alex-tache <[email protected]> * Restyled by clang-format * [K32W1] Fix ErrorStr header inclusion Signed-off-by: marius-alex-tache <[email protected]> --------- Signed-off-by: Doru Gucea <[email protected]> Signed-off-by: Marius Tache <[email protected]> Signed-off-by: marius-alex-tache <[email protected]> Co-authored-by: Doru Gucea <[email protected]> Co-authored-by: Restyled.io <[email protected]>
* [Chef] Add generic switch device type * Remove MultiPressMax since we don't support MSM We just support LatchingSwitch(LS) bit but not MomentarySwitchMultiPress (MSM) bit. So the MultiPressMax event shouldn't be enabled.
…p#29541) Decouples them from MTRBaseClusters.
…ip#29554) * Implement MTRBaseCluster invokes on top of MTRBaseDevice. This gets rid of command bridge code and shares code a lot better. * Address review comment. * Remove some more code.
* Save some flash on events/commands decode * Regenerate ZAP --------- Co-authored-by: [email protected] <[email protected]>
Fix file paths and update header to be less confusing for newbies
As in project-chip#28890 this zap had a client side command on the server that was causing a duplicate in the generated command list.
project-chip#29234) * Added a way of activating the report on active mode from icd manager with build flags * Restyled by shfmt * Moved the ICD_REPORT_ON_ENTER_ACTIVE_MODE in the report scheduler implementation * Moved the report on active define to the app BUILD.gn and removed un-necessary #ifdef * Removed preset * Added include AppBuildConfig --------- Co-authored-by: Restyled.io <[email protected]>
…p#29561) * Updates to cluster objects: use direct context tag comparisons * Also optimize code size usage for structure decoding * Completely split struct iterations and use std::variant based returns on an iterator * Another space save: using a common err for all decodes and single macro invocation seems to make code even smaller * One more cleanup return and proper use of first/last to remove unused code on empty structs (mainly command inputs) * Added a comment and a missed auto keyword * Missed some first/last iterations
…p#29814) * Fix streamer_esp32.cpp for current esp-idf main branch * Restyled by clang-format --------- Co-authored-by: Restyled.io <[email protected]>
* [NXP] Adding Matter support for RW61x platform * [NXP] Updating ot-nxp submodule to support RW61x * [RW61x] Fix misspell errors * [RW61x] Fix code-lints errors and updating zap generation : expose the EndpointList attribute in PowerSource cluster * [RW61x] Fix build-and-publish errors * [RW61x] Reworking all-clusters README.md to fix build-and-publish errors * [NXP] Fix application build : Adding device_layer_target_define for NXP device platform * [NXP] Addressing review comment : Removing unnecessary factory reset at ConfigurationManagerImpl init * [NXP] Updating zap files * [NXP] Fix all-clusters app build : update path of ErrorStr.h * [NXP] Run clang-format on RW61x/COMMON sources * Restyled by whitespace * Restyled by clang-format * Restyled by gn * Restyled by prettier-markdown --------- Co-authored-by: Restyled.io <[email protected]>
* Move xml to say zapxml since that is the format * Start defining a data_model_xml parser (no functionality for now) * Start adding some basic support for data model xml parsing * make the DM parser executable * Start having the ability to parse clusters * More updates, we seem to have parsing for features * Hard-code global attributes * Remove some comments * Add enumeration handling * Add bitmap handling * Restyle * Parse structs * Re-organize parsing a bit * Make a linter happy * Another linter fix * Handling of events * More handling and logic on events * Add support for access privilege parsing * XMLs have maybe invalid enum entries. Handle them gracefully * More attribute handling updates * restyle * Support deprecate constraint * Support constraint decoding, apply on attributes for now * Restyle * Restyle * Field handling * Field handling * Some bug fixing * Start adding command handling * Restyle * Name normalization and more parsing updates * Name normalization and more parsing updates * Better messaging and fix constraint types * Restyle * Start creating a IDL codegen so we can self-test parsed output * Start with listing clusters * Enum listing * A lot more things supported * Attribute rendering * Support for string and octet string sizes * Timed command support * Restyle * Add descriptions to clusters * Attempt to fix up alignment of things * Alignment looks slightly better * Better command separation * Align comments * Align and output descriptions including clusters * More work regarding loop structures * Apply hex formatting to bitmaps. output now seems identical except one whitespace change * Identical output for now * Support API maturity. Notice that doccomments are lost on maturity :( * Fix doxygen parsing for api maturity at the cluster level * Restyle * Support endpoints, although that is not 1:1 as hex encoding and ordering for events is lost * Restyle * Add todo note that default value does not string escaping * Default rendering and add to files * More updates on file dependencies * Unit test IDL generator * Add the IDL unit test as a standard unit test * Update for python compatibility * Fix unit testing of builds when GSDK root is defined * Added a readme file * Restyle * Make xml parser use the idl codegen * Restyle * look to fix misspell warnings * Undo repo update * Fix linter errors * Codegen as idl for data_model_xml_parser.py * Parsing closer to matter idl content * more normalization and type processing * More mandatory conformance logic * Fix mandatory conditionals * Make unit test pass * Fix tests a bit more and make parsers better * Restyle * Ignore min/max values while parsing xmls, even though raw data internally contains them * Restyle * Fix space after click annotations * Compare support for human reviews * Restyle * Fix slash * Undo submodule change * fix xml to zapxml naming changes * Restyle * Update dates from 2022 to 2023 * Add note about the complex test input * Remove unused imports * Restyle * Add some commends based on code review * Add heuristic for setting enum and bitmap sizes, to make output from XML much more readable * Add support for timed and fabric scoped commands * Add missing import --------- Co-authored-by: Andrei Litvin <[email protected]>
* [Ameba] added implementation for manual operation of several clusters (dishwasher alarm, refrigerator alarm, operational state, rvc operational state, rvc runmode, and rvc cleanmode), test event trigger for smoke alarm, shutdown function of CHIPDeviceManager, and init laundrywashercontrolserver * [Ameba] Run restyle-diff.sh, and modified cmake configuration to add additional implementation * [Ameba] Added CONFIG_ENABLE_AMEBA_TEST_EVENT_TRIGGER macro * [Ameba] added CONFIG_ENABLE_AMEBA_TEST_EVENT_TRIGGER define flag at cmake * [Ameba] Updated switch case at ManualOperationalStateCommand.h and updated raise and lower alarm function at ManualDishWasherAlarmCommand.h * [Ameba] Updated RvcOperationalState::ErrorStateEnum::kUnknownEnumValue to OperationalState::ErrorStateEnum::kUnknownEnumValue because it was removed, and added minor comments.
…9882) Signed-off-by: Chin-Ran Lo <[email protected]>
* Remove PSA Crypto exclusion * fix PSA test failure * Add more comments * Fix comments * Add isKeySet flag * address comments
This change adds shutdown to AndroidDeviceControllerWrapper, which wraps CHIPDeviceController on the Android platform, so that DeviceControllerFactory and CHIPDeviceController can be shutdown. AndroidDeviceControllerWrapper initializes DeviceControllerFactory and CHIPDeviceController, but since it supports only CHIPDeviceController shutdown, DeviceControllerFactory shutdown is required. If you reuse AndroidDeviceControllerWrapper with the call below without shutting down DeviceControllerFactory, sometimes the DeviceControllerSystemState of DeviceControllerFactory becomes incorrect, causing AndroidDeviceControllerWrapper to operate incorrectly, and the problem cannot be solved unless the process is restarted. - AndroidDeviceControllerWrapper::AllocateNew() - AndroidDeviceControllerWrapper::Controller()->Shutdown() Signed-off-by: Youngho Yoon <[email protected]> Signed-off-by: Charles Kim <[email protected]> Signed-off-by: Hunsup Jung <[email protected]> Signed-off-by: sanghyukko <[email protected]> Signed-off-by: Jaehoon You <[email protected]> Signed-off-by: Kyu-Wook Lim <[email protected]>
…ip#29901) Bumps [third_party/pigweed/repo](https://github.com/google/pigweed) from `9504c03` to `fbb318a`. - [Commits](google/pigweed@9504c03...fbb318a) --- updated-dependencies: - dependency-name: third_party/pigweed/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…-chip#29900) Bumps [third_party/openthread/repo](https://github.com/openthread/openthread) from `66eaeec` to `837c7fd`. - [Release notes](https://github.com/openthread/openthread/releases) - [Commits](openthread/openthread@66eaeec...837c7fd) --- updated-dependencies: - dependency-name: third_party/openthread/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…n board 917 NCP (project-chip#29884) * Added changes for the power cycle issue * Added changes for only 917 NCP and firmware version print * fix for build errors * Removed the changes for dissconnect event * Restyled by whitespace * Restyled by clang-format --------- Co-authored-by: yap <[email protected]> Co-authored-by: Restyled.io <[email protected]>
) * Update tag test to include root and omit content app * Restyled by isort * Fix test of the test One line got missed in the commit. --------- Co-authored-by: Restyled.io <[email protected]>
* [Telink] Use -p for activate to install requirements.telink.txt * Update builds to docker version 22 * [Telink] try to use new variable for path --------- Co-authored-by: Andrei Litvin <[email protected]>
TC-CADMIN-1.4 TC-CADMIN-1.6 TC-CADMIN-1.10 TC-CADMIN-1.16 TC-CADMIN-1.21 TC-CADMIN-1.22 TC-CADMIN-1.23 TC-CADMIN-1.24
…ect-chip#29906) Bumps [third_party/libwebsockets/repo](https://github.com/warmcat/libwebsockets) from `18fdb0d` to `a1cbc02`. - [Commits](warmcat/libwebsockets@18fdb0d...a1cbc02) --- updated-dependencies: - dependency-name: third_party/libwebsockets/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…t-chip#29904) Bumps [third_party/ot-br-posix/repo](https://github.com/openthread/ot-br-posix) from `636e53d` to `69ce2a5`. - [Release notes](https://github.com/openthread/ot-br-posix/releases) - [Commits](openthread/ot-br-posix@636e53d...69ce2a5) --- updated-dependencies: - dependency-name: third_party/ot-br-posix/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…p#29902) Bumps [third_party/nanopb/repo](https://github.com/nanopb/nanopb) from `a4bbd6a` to `9766c45`. - [Commits](nanopb/nanopb@a4bbd6a...9766c45) --- updated-dependencies: - dependency-name: third_party/nanopb/repo dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
This triggers the spell checker when I generate the test plans.
|
Superseded by PR21 |
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.