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

Matter 1.4 upmerge #512

Open
wants to merge 2,167 commits into
base: master
Choose a base branch
from
Open

Matter 1.4 upmerge #512

wants to merge 2,167 commits into from

Conversation

adigie
Copy link
Member

@adigie adigie commented Nov 15, 2024

gmarcosb and others added 30 commits October 1, 2024 13:57
…re if the exit code is not 0 AND if the process was responding (we ignore cases where the process is stuck) (#35817)
* Make Subprocess wrapper class even more generic

* Add ECOINFO_2_1 to CI

* Add ECOINFO_2_2 to CI

* Rewrite TC_MCORE_FS CI arguments to YAML

* [Testing] Convenience property for checking PICS_SDK_CI_ONLY

* Guard CI prerequisites setup with PICS SDK CI

* Fix typo
* General updates needed for camera spec

* Generated via ./scripts/tools/zap_regen_all.py

* Apply review suggestions from project-chip/connectedhomeip#35775

* Re-generated using ./scripts/tools/zap_regen_all.py after pigweed update
* Updated PICS Generator to match 1.4 PICS

* Add script to validate PICS and cluster macthing

* Updated based on review feedback

* Apply suggestions from code review

Co-authored-by: C Freeman <cecille@google.com>
Co-authored-by: Andrei Litvin <andy314@gmail.com>

* Add XMLPICSValidator to wordlist

* Restyled by prettier-markdown

* Restyled by isort

---------

Co-authored-by: C Freeman <cecille@google.com>
Co-authored-by: Andrei Litvin <andy314@gmail.com>
Co-authored-by: Restyled.io <commits@restyled.io>
* Adding endpoint information to test steps methods and hooks along with logging.

Also, the TC_ACE_1_3 and TC_BOOLCFG_2_1 tests were updated to demonstrate how to
used the updated hooks to inform the endpoint associated with every step.

* Rolling back the test stop unnecessary exception parameter

* Updating TestDecoratorHooks with endpoint as well.

* Minor grammar fix

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

---------

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
* Fix Android resolve exception issue

* Restyled by google-java-format

---------

Co-authored-by: Restyled.io <commits@restyled.io>
* fan rocksetting; thermostat dry mode

* revert change
* Update rootnode_waterleakdetector_0b067acfa3.zap

Add Power Source cluster to EP1 to simulate a battery

* Update rootnode_waterleakdetector_0b067acfa3.matter

Add Power Source cluster to EP1 to Matter file

* Update rootnode_waterleakdetector_0b067acfa3.matter

* Update rootnode_waterleakdetector_0b067acfa3.zap

* Update rootnode_waterleakdetector_0b067acfa3.matter

Remove Scenes Management Cluster

* Manual correction of device type order due to ZAP tool bug

* Update rootnode_waterleakdetector_0b067acfa3.zap

* Update rootnode_waterleakdetector_0b067acfa3.zap

* Update rootnode_waterleakdetector_0b067acfa3.zap

* Update rootnode_waterleakdetector_0b067acfa3.matter

* Update rootnode_waterleakdetector_0b067acfa3.zap

* Update rootnode_waterleakdetector_0b067acfa3.matter
* [feat] Add littlefs to access flash storage

* fix restyle

* update ci build job and remove regex in BLConfig_littlefs.cpp

* fix restyle

* fix compile error

* fix restyle

* fix compile error after restyle

* fix restyle with include order
* Remove event list attribute support

* Restyle

* Restyled by clang-format

* Remove matter event list from defaults and from cmake builds

---------

Co-authored-by: Restyled.io <commits@restyled.io>
…t is runned with atexit into its own autorelease pool (#35878)
…OL instead of DISPATCH_QUEUE_SERIAL (#35876)
* getter for air quality

* Added Air Quality Sample app for silabs

* Restyled by whitespace

* Restyled by clang-format

* Restyled by prettier-markdown

* added build in CI

* minor fix

* Revert "added build in CI"

This reverts commit 5a1fcda96096a172f1e7fb36fdbe8d269fce741d.

* Revert "minor fix"

This reverts commit 8fa8df289dcbfe6840e425d2330d3ee740577e6b.

* added build for app

* added app to target

* resolved comments

* Restyled by whitespace

* Restyled by clang-format

* using schedulework

* using schedulework instead of chiplock

* Restyled by clang-format

* removing unneccessary loggs, resolved memory leak

---------

Co-authored-by: Restyled.io <commits@restyled.io>
…rsion checks (#35863)

* Update repoting Engine to be able to use DataModel::Provider for version checks.

* Restyle

* Fix an include

* Fix an include

* Fix an include

* Restyled by clang-format

* Update src/app/reporting/Read-Ember.cpp

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

---------

Co-authored-by: Andrei Litvin <andreilitvin@google.com>
Co-authored-by: Restyled.io <commits@restyled.io>
Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
…heckEventSupportStatus from ember-compatibility) (#35873)

* Update event support checks

* Fix an include

* Restyled by clang-format

* Fix typo

* Restyled by clang-format

* Added a EndpointExists call to the datamodel provider along with a default implementation

* Remove missing config use after merge with master

* Restyled by clang-format

---------

Co-authored-by: Restyled.io <commits@restyled.io>
* Reuse Subprocess in run_python_test.py script

* Add --app-ready-pattern option to run_python_test.py

* Replace script-start-delay with app-ready-pattern

* Drop support for script-start-delay

* Use rmtree() instead of explicit "rm -rf" calls

* Update missed python.md section

* Silence output from tests

* Fix removing files

* Restyled by prettier-markdown

* Fix documentation builder warning

* Remove unused import

* Fix Metadata unit test

---------

Co-authored-by: Restyled.io <commits@restyled.io>
* [Chef] Update Refrigerator to TemperatureNumber

* Restyled by clang-format

* Restyled by gn

* Fix comformance issues

checked by

python ./src/python_testing/TC_DeviceBasicComposition.py  --storage-path admin_storage.json --manual-code 34970112332 --bool-arg ignore_in_progress:True allow_provisional:True --PICS src/app/tests/suites/certification/ci-pics-values

python ./src/python_testing/TC_DeviceConformance.py  --storage-path admin_storage.json --manual-code 34970112332 --bool-arg ignore_in_progress:True allow_provisional:True --PICS src/app/tests/suites/certification/ci-pics-values

* Add taglist to Endpoints to fix conformance issue

Caused by  test_TC_DESC_2_2:
 For device types with more than one endpoint listed, ensure each of the listed endpoints has a tag attribute and the tag attributes are not the same

* Restyled by clang-format

* Use MATTER_DM_PLUGIN_REFRIGERATOR_ALARM_SERVER

Use MATTER_DM_PLUGIN_REFRIGERATOR_ALARM_SERVER to enable TagList for
refrigerator since
MATTER_DM_PLUGIN_REFRIGERATOR_AND_TEMPERATURE_CONTROLLED_CABINET_MODE_SERVER
could be used by oven

* Use unique_ptr instead of new

* Restyled by clang-format

---------

Co-authored-by: Restyled.io <commits@restyled.io>
We don't want the LoggerForModule() implementation to call retain/autorelease
on the logger, because its unnecessary, and because LoggerForModule() can be
called from a C++ context without an auto-release pool in place.

To avoid this, we can have LoggerForModule() return a raw pointer, and handle
the bridge cast to os_log_t on the caller side. This also avoids unnecessary
code on the caller side that would otherwise attempt to rescue the
presumed-autoreleased logger object from the ARP.
ArekBalysNordic and others added 17 commits November 15, 2024 10:41
It is better to call the ScheduleFactoryReset method from
the Server, because it also removes all fabrics and emits the Leave
event whereas calling InitiateFactoryReset() removes only the
persistent storage entries.

In this way, the Matter Shell command works in the same way as
other factory reset sources.

We cannot upmerge it in an easy way because many vendors do not
use chip server and there are multiple building issues.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
(cherry picked from commit 973259c)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
This commit updates the IPv6 multicast subscription mechanism by
explicitly using the MLDv2 join procedure. This change ensures proper
registration for multicast addresses, improving reliability in multicast
communication.

Signed-off-by: Łukasz Duda <lukasz.duda@nordicsemi.no>
(cherry picked from commit 08acaf4)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
This commit makes in clear which crypto driver is enabled by default.
Additionally, stack overflow has been fixed for CC3XX variant, and
mbedTLS Heap is disabled for TF-M variant.

Signed-off-by: Łukasz Duda <lukasz.duda@nordicsemi.no>
(cherry picked from commit bdec5fd)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
The upstream WPA supplicant uses k_heap instead of libc_heap, so,
move the heap to k_heap.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
(cherry picked from commit 8c09c31)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
By default nRF70 driver allocates a higher heap to handle most cases,
but as Matter doesn't use much heap, enable the ignore minimum warning
configuration.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
(cherry picked from commit 5c19b19)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
When using nRF54L15 the stack size required for crypto operations
by OpenThread is bigger and must be increased.

Signed-off-by: Kamil Kasperczyk <kamil.kasperczyk@nordicsemi.no>
(cherry picked from commit 3567e53)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
Added a possibility to use ZMS fs backend in Zephyr.
NVS fs backend imply is now controlled by the nrfconnect platform
configuration directly.
All other platforms use NVS fs backend by default.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
(cherry picked from commit 0c49058)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>

[nrf toup] Fix defines for ZMS

It can be squashed with the 0c49058 commit.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
(cherry picked from commit 5148b72)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
Disabled NVS and enabled ZMS fs backend for all devices that
uses RRAM.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
(cherry picked from commit 9a4ea8e)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
When a Matter application provides the VERSION file, the
Matter software version string is set to 4-number string,
such as "2.7.99+0" and the EXTRAVERSION field, which is
typically used to denote the branch or tag name, such as
"dev" or "rc1", is ignored.

If the EXTRAVERSION field is non-empty, set the Matter
software version string to a string such as "2.7.99-dev+0".

(cherry picked from commit 731aac5)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
Added support for multi-image dfu target while using SUIT.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
(cherry picked from commit 7486fd3)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
Targets using cracen crypto backend requires bigger CHIP
stack size.

Signed-off-by: Kamil Kasperczyk <kamil.kasperczyk@nordicsemi.no>
(cherry picked from commit 1fb9792)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
On nRF54L devices we can use KMU to store the DAC private key,
and allow using it directly by CRACEN.
By default the DAC private key is encrypted so it needs
4 KMU slots.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
(cherry picked from commit 5580536)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
Enabled DAC migration for all targets using Thread protocol,
as Wi-Fi does not support PSA yet.

Signed-off-by: Kamil Kasperczyk <kamil.kasperczyk@nordicsemi.no>
(cherry picked from commit 700233b)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
- Fixed wrong dependency in Kconfigs
- Fixed wrong bracket placement that causes overwriting DAC private
key.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
(cherry picked from commit 668f104)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
Until we do not have a proper solution for merging the factory data
hex file with the firmware hex file, we need to flash the factory
data manually. This commit allows doing that on nRF54H20 DK.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
(cherry picked from commit 95a8c0f)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
Enabled storing the DAC private key in KMU if it is available.
It enables storing the DAC private key in KMU for nRF54L15 device.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
(cherry picked from commit aa968a6)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
`FPROTECT` should be disabled for nRF54L15 in app to allow
correctly protecting whole region of mcuboot

Signed-off-by: Maciej Baczmanski <maciej.baczmanski@nordicsemi.no>
(cherry picked from commit 81eb1e9)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
* Disable `Backwards compatibility check`
* Disable `Check for changes to data_model directory without a sha
update`

Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
@adigie adigie requested a review from a team November 15, 2024 12:41
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.