-
Notifications
You must be signed in to change notification settings - Fork 10
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
Sync csa branch with main #316
Merged
Merged
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
* Initial implemenation of camera-controller app * Fix build errors * Remove ICD stuff since it is not needed for camera
…ing page (#37872)
…butes for wifi commissioning. (#37881)
* Only use kMaxIfNameLength instead of IFNAMSIZ I encountered IFNAMSIZ is not defined when doing some porting. I think it's better to use the unified definition for network interface name in this project. * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Andrei Litvin <andy314@gmail.com>
* [app] Parametrize app-common zap-generated path Parametrize app-common zap-generated path to allow building Matter using custom generated zap files stored in different path. When user adds custom clusters, full zap regeneration is needed. Storing these files in fixed place requires regeneration every time user wants to build different project (with different clusters). This commit allows to store zap-generated files separately and provide their path by seting `CHIP_APP_ZAP_DIR` cmake and `chip_app_zap_dir` GN flags. Signed-off-by: Maciej Baczmanski <maciej.baczmanski@nordicsemi.no> * [nrfconnect] Allow switching `CHIP_APP_ZAP_DIR` by cmake. Signed-off-by: Maciej Baczmanski <maciej.baczmanski@nordicsemi.no> --------- Signed-off-by: Maciej Baczmanski <maciej.baczmanski@nordicsemi.no>
* Extract shorten name and add unit tests * Fix ids * Restyled by autopep8 * Add glob support for symbols, to quickly figure out sizes * Update title --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com> Co-authored-by: Restyled.io <commits@restyled.io>
* Disable ABS feature in windowcovering * Add ZAP Generated * Add files for window covering chef delegate * Fix compilation * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * Review comments
* Add on-off command handlers for fan control * Disallow step when fan is Off * Add On/Off cluster to fan device * Fix compilation bugs * Fix compilation bugs * Fix compilation bugs * Handle onOff event in MatterPostAttributeChangeCallback since emberAFCommandHandler callbacks are already defined * Fix compilation * Fix compilation * Fix compilation * Fix unreachable code. * Remove hard checks on on-off to prevent tests from breaking * On command should set Fan to HIGH * Set default OnOff state to Off to match default fanMode
* Add Alchemy generated files for Dishwasher Alarm * Revert change in zap_cluster_list
* Prevent Buffer Overflows with External Attribute Reads * Prevent buffer overflows by blocking creation of dynamic endpoints with attributes that would overflow the static attribute buffer * Added additional sanity checks to block reads of attributes that would overflow the attribute reads; return RESOURCE_EXHAUSTED instead * Fix style + minor compile issues * Fix include order * Address PR Comments
The check for |
* ESP32: Add menuconfig to use PSRAM for Matter memory allocation * Restyled by clang-format * review changes * split to 3 separate files * doc changes * add comment on MemoryInternalCheckPointer --------- Co-authored-by: Restyled.io <commits@restyled.io>
* [ICD] Expose SetModeDurations via ICDManager * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>
mkardous-silabs
approved these changes
Mar 6, 2025
* Darwin style update: remove extra namespace prefixes * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>
* Generate BooleanState with Alchemy * Rearrange ACTIONS_CLUSTER in zap_cluster_list
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
* [NXP] Update submodules: nxp_matter_support and ot-nxp Signed-off-by: Martin Girardot <martin.girardot@nxp.com> * [NXP] Thermostat + all clusters modifications to support SDK next GEN with GN * [NXP][k32w1] Update reference apps Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * [NXP][mcxw71] Update reference apps Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * [NXP] Replace a BLE manager common flag with a CONFIG_ prefix Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * [NXP][k32w1][mcxw71] Disabling nxp_use_smu2_static and nxp_use_smu2_dynamic Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com> * [NXP] Add ResetWatermarks after new freertos API added into last SDK version Signed-off-by: Martin Girardot <martin.girardot@nxp.com> * [NXP][mcxw71_k32w1] Matter build port for platform file rework to support cmake build Signed-off-by: marius-alex-tache <marius.tache@nxp.com> * [NXP] update docker image version to support new SDK Signed-off-by: Martin Girardot <martin.girardot@nxp.com> * Restyled by clang-format * Restyled by gn * [NXP][examples] Update laundry washer example to be align with new sdk, fix zephyr app CERT issues Signed-off-by: Martin Girardot <martin.girardot@nxp.com> * Restyled by gn * [NXP] generate .matter file Signed-off-by: Martin Girardot <martin.girardot@nxp.com> --------- Signed-off-by: Martin Girardot <martin.girardot@nxp.com> Signed-off-by: marius-alex-tache <marius.tache@nxp.com> Signed-off-by: Gatien Chapon <gatien.chapon@nxp.com> Co-authored-by: Jaafar BEN YOUNES <jaafar.benyounes@nxp.com> Co-authored-by: marius-alex-tache <marius.tache@nxp.com> Co-authored-by: Gatien Chapon <gatien.chapon@nxp.com> Co-authored-by: Restyled.io <commits@restyled.io>
* Server cluster interface * ServerClusterInterfaceRegistry adding * Update src/data-model-providers/codegen/ServerClusterInterfaceRegistry.h Co-authored-by: Terence Hampson <thampson@google.com> * Code review: forward declare * Review comment: switch order of members * Updated logic: the condition is not relevant anymore * Code review: added comment * Restyle * Comment update * Updates to make callers use memory allocation * Self review: fix an include * Fix constexpr logic * Restyle * minor comment update so github is not confused about head SHA * Added useless unit test to get coverage * Restyle * Update src/app/server-cluster/ServerClusterInterface.h Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * Update src/data-model-providers/codegen/ServerClusterInterfaceRegistry.cpp Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * Update src/data-model-providers/codegen/ServerClusterInterfaceRegistry.cpp Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * Update src/data-model-providers/codegen/ServerClusterInterfaceRegistry.h Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * Update src/data-model-providers/codegen/ServerClusterInterfaceRegistry.h Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * Disallow null ptr registrations * Restyled by clang-format * Reworded based on code review --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com> Co-authored-by: Terence Hampson <thampson@google.com> Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> Co-authored-by: Restyled.io <commits@restyled.io>
…_8 (#37776) * fix(testing): Move CloseSession call after factory reset in TC_CGEN_2_8 The current implementation of commissioner.CloseSession only marks the session as defunct rather than fully closing it. This state can be reactivated if messages are exchanged on the session. By moving the CloseSession call to after the factory reset step, we ensure that the device session is already destroyed before marking it as defunct, preventing any possibility of message exchange that could reactivate the session. This allows the commissioner to successfully create a new session in subsequent steps. A follow-up change will be needed to properly fix the CloseSession function to use MarkForEviction instead of MarkAsDefunct, and potentially rename the current function to better reflect its behavior.
selissia
approved these changes
Mar 7, 2025
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.
This PR syncs the csa branch with the main branch.
PR MUST BE MERGED WITH MERGE COMMIT - ADMIN MUST ENABLE THE OPTION