Skip to content
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 19 commits into from
Mar 7, 2025
Merged

Sync csa branch with main #316

merged 19 commits into from
Mar 7, 2025

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Mar 6, 2025

This PR syncs the csa branch with the main branch.

PR MUST BE MERGED WITH MERGE COMMIT - ADMIN MUST ENABLE THE OPTION

yufengwangca and others added 10 commits March 5, 2025 02:04
* Initial implemenation of camera-controller app

* Fix build errors

* Remove ICD stuff since it is not needed for camera
* 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
Copy link
Author

github-actions bot commented Mar 6, 2025

The check for sl-require-admin-action label CI failure for this job is normal. An admin must do the merge.

wqx6 and others added 2 commits March 6, 2025 05:32
* 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>
andy31415 and others added 7 commits March 6, 2025 15:44
* 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.
@mkardous-silabs mkardous-silabs merged commit ead2028 into main Mar 7, 2025
75 of 76 checks passed
@mkardous-silabs mkardous-silabs deleted the automation/update_main branch March 7, 2025 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.