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

Infineon p6 all clusters update #19245

Merged

Conversation

keithmwheeler
Copy link
Contributor

Problem

Fixed commissioning issue with all clusters; updated network commissioning wifi driver with needed lock/unlock sequences.

Change overview

Updated P6 all clusters and all clusters minimal apptask to support commissioning only on endpoint 0. Added necessary lock/unlock calls to P6 network commissioning wifi driver.

Testing

Tested by commissioning both all clusters and all clusters minimal on P6 hardware.

@CLAassistant
Copy link

CLAassistant commented Jun 6, 2022

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@bzbarsky-apple bzbarsky-apple left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should not be making changes to the EFR32 SDK or the ZAP repo... and probably not to mbedtls.

@boring-cyborg boring-cyborg bot added the lib label Jun 6, 2022
@github-actions
Copy link

github-actions bot commented Jun 8, 2022

PR #19245: Size comparison from 8cc291d to 17ff172

Decreases (1 build for nrfconnect)
platform target config section 8cc291d 17ff172 change % change
nrfconnect all-clusters-app nrf52840dk_nrf52840 text 818812 818808 -4 -0.0
Full report (2 builds for nrfconnect)
platform target config section 8cc291d 17ff172 change % change
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1194327 1194327 0 0.0
bss 141385 141385 0 0.0
rodata 155216 155216 0 0.0
text 818812 818808 -4 -0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1142035 1142035 0 0.0
bss 140602 140602 0 0.0
rodata 131708 131708 0 0.0
text 790864 790864 0 0.0

@andy31415 andy31415 merged commit 5d8599d into project-chip:master Jun 9, 2022
mkardous-silabs pushed a commit to mkardous-silabs/connectedhomeip that referenced this pull request Jun 10, 2022
rochaferraz pushed a commit to rochaferraz/connectedhomeip that referenced this pull request Jun 15, 2022
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.

4 participants