Skip to content

[Realtek] Adjust files struct #39100

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

Open
wants to merge 61 commits into
base: master
Choose a base branch
from
Open

Conversation

pankore
Copy link
Contributor

@pankore pankore commented May 21, 2025

Adjust files struct, modify realtek_bee to realtek.

Testing

We conducted thorough testing on local RTL8777G SOC boards. This included building the applications, performing commissioning processes, and verifying the basic functionality of the all-clusters,ota-requestor,thermostat applications.
There is no test added at present. We will temporarily ensure that there are no problems with the version compilation before uploading it.

Conformance tests -

python3 src/python_testing/TC_ACE_1_2.py --commissioning-method ble-thread --discriminator 3840 --passcode 20202021 --thread-dataset-hex
python3 src/python_testing/TC_DeviceConformance.py --manual-code 34970112332
python3 src/python_testing/TC_DeviceBasicComposition.py --manual-code 34970112332

@mergify mergify bot added the conflict label May 21, 2025
@mergify mergify bot added conflict and removed conflict labels May 22, 2025
Copy link

PR #39100: Size comparison from 5dce7ce to bcdb660

Full report (14 builds for cc13x4_26x4, cc32xx, nrfconnect, qpg, stm32, tizen)
platform target config section 5dce7ce bcdb660 change % change
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 FLASH 822092 822092 0 0.0
RAM 120296 120296 0 0.0
lock-ftd LP_EM_CC1354P10_6 FLASH 832112 832112 0 0.0
RAM 125440 125440 0 0.0
pump-app LP_EM_CC1354P10_6 FLASH 779224 779224 0 0.0
RAM 113844 113844 0 0.0
pump-controller-app LP_EM_CC1354P10_6 FLASH 763532 763532 0 0.0
RAM 114060 114060 0 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 549798 549798 0 0.0
RAM 205264 205264 0 0.0
lock CC3235SF_LAUNCHXL FLASH 581630 581630 0 0.0
RAM 205448 205448 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 FLASH 921592 921592 0 0.0
RAM 167400 167400 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 FLASH 858884 858884 0 0.0
RAM 141163 141163 0 0.0
all-clusters-app nrf7002dk_nrf5340_cpuapp FLASH 912872 912872 0 0.0
RAM 145644 145644 0 0.0
qpg lighting-app qpg6105+debug FLASH 668756 668756 0 0.0
RAM 105244 105244 0 0.0
lock-app qpg6105+debug FLASH 627184 627184 0 0.0
RAM 99880 99880 0 0.0
stm32 light STM32WB5MM-DK FLASH 466476 466476 0 0.0
RAM 141496 141496 0 0.0
tizen all-clusters-app arm unknown 5284 5284 0 0.0
FLASH 1817724 1817724 0 0.0
RAM 95764 95764 0 0.0
chip-tool-ubsan arm unknown 20624 20624 0 0.0
FLASH 20863822 20863822 0 0.0
RAM 9079380 9079380 0 0.0

@mergify mergify bot removed the conflict label May 22, 2025
Copy link

PR #39100: Size comparison from a7cf007 to 7d1168e

Full report (3 builds for cc32xx, stm32)
platform target config section a7cf007 7d1168e change % change
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 548254 548254 0 0.0
RAM 205256 205256 0 0.0
lock CC3235SF_LAUNCHXL FLASH 581694 581694 0 0.0
RAM 205448 205448 0 0.0
stm32 light STM32WB5MM-DK FLASH 464964 464964 0 0.0
RAM 141488 141488 0 0.0

Copy link

PR #39100: Size comparison from a7cf007 to 4bc8531

Full report (3 builds for cc32xx, stm32)
platform target config section a7cf007 4bc8531 change % change
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 548254 548254 0 0.0
RAM 205256 205256 0 0.0
lock CC3235SF_LAUNCHXL FLASH 581694 581694 0 0.0
RAM 205448 205448 0 0.0
stm32 light STM32WB5MM-DK FLASH 464964 464964 0 0.0
RAM 141488 141488 0 0.0

@mergify mergify bot added the conflict label May 23, 2025
dmaranges-gd and others added 15 commits May 26, 2025 09:15
* TC-CC-8.1: Automate

* TC-CC-8.1 - Modify spected value for steps 2d and 2e

* TC-CC-8.1 - modify expected result to handle wrap-around

* TC-CC-8.1 - fix expected maxValue

* TC-CC-8.1 - fix expected values and modify wating time

* Fix tolerances

* Update tolerance

* Updated tolerance

---------

Co-authored-by: Diego Martin Maranges <[email protected]>
Co-authored-by: Diego Maranges <[email protected]>
Co-authored-by: Ramiro Recchia <[email protected]>
* Add PSA Crypto initialization for non-TinyCrypt builds in WifiInterfaceImpl

* Update condition for device attestation key signing to use TinyCrypt

* Apply suggestions from code review

Co-authored-by: Sergio Soares <[email protected]>

* Fix condition for PSA Crypto initialization in WifiInterfaceImpl

---------

Co-authored-by: Sergio Soares <[email protected]>
* autocommissioner: Refactor PAI/DAC certificate alloc/de-alloc

This avoids code duplication and the new certicate API may be used by JF.

Signed-off-by: Doru Gucea <[email protected]>

* [E2E][JF] jf-control-app: Add --execute-jcm option

If this option is set during the pairing command, then JCM flow is triggered.

Signed-off-by: Doru Gucea <[email protected]>

* [E2E][JF] core-sdk/jf-control-app: allow enabling/disabling JF support in core-SDK

Enable JF suppport by default when jf-control-app is being compiled.

Signed-off-by: Doru Gucea <[email protected]>

* [E2E][JF] core-sdk/autocommissioner: add skeleton for executing steps 6-7 of JCM

Signed-off-by: Doru Gucea <[email protected]>

* Fix CI/CV issues/comments

* check that the commissioned Admin has an Admin CAT inside its NOC.

Signed-off-by: Doru Gucea <[email protected]>

* [E2E][JF] Fix issues signaled during the review process

* fix the error path when iterating through the attribute cache;
* use a ScopedMemoryBUfferWithSize for DAC, PAI and JF certs.

Signed-off-by: Doru Gucea <[email protected]>

* [E2E][JF] Initialize AdministratorFabricIndex

Set AdministratorFabricIndex on jf-admin-app immediately after
its onboarding by jf-control-app.

Signed-off-by: Doru Gucea <[email protected]>

* [E2E][JF] Refactor pairing command parameters location

Signed-off-by: Doru Gucea <[email protected]>

* [E2E][JF] Fix review comments

* main change is related to creation of AllocateMemoryAndCopySpan
* small cosmetic fixes

Co-authored-by: Boris Zbarsky <[email protected]>
Signed-off-by: Doru Gucea <[email protected]>

---------

Signed-off-by: Doru Gucea <[email protected]>
Co-authored-by: Restyled.io <[email protected]>
Co-authored-by: Boris Zbarsky <[email protected]>
…t-chip#39132)

Alchemy is now assuming that conformance counts toward optionality, but we have
not yet taken the ZAP update that makes that work.
From https://github.com/project-chip/connectedhomeip/actions/runs/15194288746/job/42737814688:

```
2025-05-22T20:37:05.0327720Z WARNING: ThreadSanitizer: data race (pid=29252)
2025-05-22T20:37:05.0327900Z   Read of size 8 at 0x7b5000047358 by thread T42:
2025-05-22T20:37:05.0328160Z     #0 -[MTRDevice_Concrete state] <null> (Matter:x86_64+0x1a86673)
2025-05-22T20:37:05.0328690Z     #1 -[MTRDevice_Concrete _performScheduledTimeUpdate] <null> (Matter:x86_64+0x1a32c83)
...
2025-05-22T20:37:05.0330160Z   Previous write of size 8 at 0x7b5000047358 by thread T44:
2025-05-22T20:37:05.0330470Z     #0 -[MTRDevice_Concrete _changeState:] <null> (Matter:x86_64+0x1a3a64c)
2025-05-22T20:37:05.0330990Z     #1 -[MTRDevice_Concrete _handleUnsolicitedMessageFromPublisher] <null> (Matter:x86_64+0x1a41bee)
```

Co-authored-by: Andrei Litvin <[email protected]>
* Generate ElectricalEnergyMeasurement with Alchemy

* Regenerate files with latest ZAP

---------

Co-authored-by: Andrei Litvin <[email protected]>
* TC-IDM-10.1: Upgrade ValueDecodeFailures to errors

* Remove product appearance from microwave

This was returning an error because it wasn't actually implemented.

Also clean up the other conformance so microwave passes conformance
tests.
* Automate python test WEBRTCR_2_3 in CI

* Add camera-controller as a build target

* Solve merge conflict

* Address review comments

* Mute verbose reponse log

* fix f-string without any placeholders

* Terminate program gracefully

* Update src/python_testing/TC_WEBRTCR_2_3.py

Co-authored-by: Arkadiusz Bokowy <[email protected]>

* Add missing module

* Remove unused module

* Fix CI bug in TC_AVSUM_2_7.py

* Add defautl signal handler

* Move sig handler to InteractiveCommand

* Seperate interactive server command from interactive start command

* Revert "Camera tls spec updates (project-chip#39089)"

This reverts commit 881af5c.

* Use customize port to void conflict

---------

Co-authored-by: Andrei Litvin <[email protected]>
Co-authored-by: Arkadiusz Bokowy <[email protected]>
* initial runner decouple

* lint code

* expose test class to module globals

* restyled

* move assignment inside __main__

* fix issue with CADMIN test

* restyled

* restyled

* fix anothe CADMIN test

* restyled

* add docstrings

* restyled

* resolve merge issues

* revert formatting changes to TC_CADMIN_1_3_4

* revert one more formatting change

* revert formatting changes from matter_testing.py

* revert more formatiing changes from matter_testing.py

* remaining formatting fixes

* revert formatting changes from runner.py

* revert comment format change

* add docstrings as requested

* restyled

* fix test failure by proper import

* fix remaining import issue

* lint

* update imports in TC_FAN_2_3

* Restyled by isort

* fix imports

* Restyled by isort

* fix imports in TC_FAN_2_4 and restyle, finish conflicts resolution

* fix imports in test

---------

Co-authored-by: Andrey Khodyrev <[email protected]>
Co-authored-by: Restyled.io <[email protected]>
Add `LINK_TARGETS` argument to `matter_build` macro to allow linking
additional libraries together with `MATTER_LIBRARIES` as a group.

This may be needed in some cases to prevent undefined reference errors
during linking the application.

Signed-off-by: Adrian Gielniewski <[email protected]>
Co-authored-by: Andrei Litvin <[email protected]>
…oject-chip#39171)

* Report sampled position attribute (EXT) change to subscribers

* Report channel change

* Make channel delegate endpoint aware.
@mergify mergify bot removed the conflict label May 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.