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

Test #5

Closed
wants to merge 17 commits into from
Closed

Test #5

wants to merge 17 commits into from

Conversation

adigie
Copy link
Owner

@adigie adigie commented Feb 26, 2025

fooo

fdfdfd bar fdfd

NCS PR: nrfconnect/sdk-nrf/pull/

adigie and others added 15 commits February 21, 2025 14:48
NET_SOCKETS_POSIX_NAMES has been removed from Zephyr.

Signed-off-by: Adrian Gielniewski <[email protected]>
Rename CONFIG_MBEDTLS_ZEPHYR_ENTROPY to
CONFIG_MBEDTLS_ENTROPY_POLL_ZEPHYR.

Signed-off-by: Adrian Gielniewski <[email protected]>
Add missing configs.

Signed-off-by: Adrian Gielniewski <[email protected]>
Use correct path when running ctest.

Signed-off-by: Adrian Gielniewski <[email protected]>
* Build Zephyr native tests for 32-bit target.
* Use native_sim as native_posix_64 is deprecated and removed in recent
version of Zephyr.

Upstream PR #: 37686

Signed-off-by: Adrian Gielniewski <[email protected]>
…ride

CHIP_CONFIG_LAMBDA_EVENT_SIZE should not be needed when building for
32-bit target.

Upstream PR #: 37686

Signed-off-by: Adrian Gielniewski <[email protected]>
Update Docker image version to 115.

Signed-off-by: Adrian Gielniewski <[email protected]>
…#37513)

* Remove ICD server sources from chip_data_model.cmake

ICD server sources are built by gn, there is no need to compile them
twice.

Signed-off-by: Adrian Gielniewski <[email protected]>

* Fix ICD dependencies

Don't depend on ICD if it's not enabled.

Signed-off-by: Adrian Gielniewski <[email protected]>

* Fix gn include warnings

Signed-off-by: Adrian Gielniewski <[email protected]>

* iOS TVCasting app: Add missing chip_project_config_include

`chip_project_config_include` has to be set to get correct include order
of configuration files in `CHIPDeviceConfig.h` and prevent macro
redefined warnings coming from overriding defines already defined by
`CHIPConfig.h`.

Signed-off-by: Adrian Gielniewski <[email protected]>

---------

Signed-off-by: Adrian Gielniewski <[email protected]>
(cherry picked from commit abd2e1dad08396e51bc4a6f503f0d3e419507f96)
…ement server cluster is included (#37653)

* Fix ICD management server cluster dependecies

Add ICDConfigurationData to app sources when ICD management server
cluster is included in sample, but ICD support is disabled, e.g.
lock-app on some platforms.

Related to: #32321

Signed-off-by: Adrian Gielniewski <[email protected]>

* Add TODO comments for #32321

Add TODO comments for issue related to ICD management server.

Signed-off-by: Adrian Gielniewski <[email protected]>

---------

Signed-off-by: Adrian Gielniewski <[email protected]>
(cherry picked from commit 00c6f4ee91b79da9905eb827b5ae7e3018e34783)
Remove workaround as issue is resolved.

Signed-off-by: Adrian Gielniewski <[email protected]>
(cherry picked from commit 0086ee2073ed45d591068a7e4a8321d9dab1e96d)
* Move some classes out of af-types.h

Move AttributesChangedListener and MarkAttributeDirty out of af-types.h

Signed-off-by: Adrian Gielniewski <[email protected]>

* Remove dependency to cluster-objects.h

Signed-off-by: Adrian Gielniewski <[email protected]>

* Remove cluster-objects from chip_data_model.cmake

Signed-off-by: Adrian Gielniewski <[email protected]>

* [Chef] Fix includes

Fix missing includes after changes in Accessors.h

Signed-off-by: Adrian Gielniewski <[email protected]>

* [Infineon] Fix includes

Fix missing includes after changes in Accessors.h

Signed-off-by: Adrian Gielniewski <[email protected]>

* [Silabs] Fix includes

Fix missing includes after changes in Accessors.h

Signed-off-by: Adrian Gielniewski <[email protected]>

---------

Signed-off-by: Adrian Gielniewski <[email protected]>
(cherry picked from commit 38cc24bd33f738ef02e2a81d67046325098d2349)
* [crypto] Introduce PSAKeyAllocator

- Moved the PSA key definitions from CHIPCryptoPALPSA.h file to
the newly created PSAKeyAllocator.

- The new PSAKeyAllocator class allows for the allocation of keys
in secure storage. Users can create their own PSAKeyAllocator
implementation and set it to be used by the Matter stack.

- If the custom implementation is not provided the default one is
used and it works as the legacy solution and the mechanism is
about stored keys in the PSA ITS storage.

Signed-off-by: Arkadiusz Balys <[email protected]>

* [crypto] Added a unit test for PSAKeyAllocator

The unit test:
- Verifies the defaultKeyAllocator instance, key allocation and
if attributes are ont changed.
- Creates a new testing key allocator and verifies if it works
as expected and differently than the default ones.
- Switches back to the defaultKeyAllocator instance and checks
if code works properly.

Signed-off-by: Arkadiusz Balys <[email protected]>

cherry-picked from 06edeee7c0d839f9e8249cb5f8274594de8c7576

Signed-off-by: Arkadiusz Balys <[email protected]>
KMUKeyAllocator is used to overwrite PSAKeyAllocator for the
specific use-cases of KMU driver.

This feature is available only on the device which run Cracen.

Signed-off-by: Arkadiusz Balys <[email protected]>
Allow setting session keystore by platform. If any keystore is not
set, assign the default one. This is a workaround that should not
be upstreamed.

Signed-off-by: Arkadiusz Balys <[email protected]>
…uirements

We need to overwrite the PersistICDKey method of
the PSASessionKeystore because we must export the volatile key first
and then import it again into the KMU slot instead of copying it.

Currently copying keys from the RAM location to the KMU slot is not
supported.

This is workaround that should not be upstreamed. It can be removed
once the other solution is found.

Signed-off-by: Arkadiusz Balys <[email protected]>
Signed-off-by: Adrian Gielniewski <[email protected]>
Signed-off-by: Adrian Gielniewski <[email protected]>
@adigie adigie closed this Feb 26, 2025
@adigie adigie deleted the test branch February 26, 2025 17:53
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.

2 participants