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.

Sign up for GitHub

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

Jump to bottom

Allow Darwin frawework to commission onto a password-less WiFi network. #21363

Merged
merged 1 commit into from
Jul 29, 2022
Merged

Allow Darwin frawework to commission onto a password-less WiFi network. #21363

merged 1 commit into from
Jul 29, 2022

Conversation

bzbarsky-apple
Copy link
Contributor

We were requiring a WiFi password in order to provide the WiFi SSID to
the underlying SDK. But not all networks have a password.

Problem

Can't commission a Matter device onto an open Wifi network.

Change overview

Fix things so it can be commissioned.

Testing

Needs to be tested by someone with an open network...

@bzbarsky-apple
Allow Darwin frawework to commission onto a password-less WiFi network.
f1c0c42 
We were requiring a WiFi password in order to provide the WiFi SSID to
the underlying SDK.  But not all networks have a password.
@github-actions
Copy link

github-actions bot commented Jul 28, 2022 •
edited
Loading

PR #21363: Size comparison from 00b5886 to f1c0c42

Decreases (4 builds for cyw30739, mbed, telink)
platform target config section 00b5886 f1c0c42 change % change
cyw30739 light cyw930739m2evb_01 (read/write) 583246 583238 -8 -0.0
.app_xip_area 460064 460056 -8 -0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 588970 588962 -8 -0.0
.app_xip_area 466604 466596 -8 -0.0
mbed lock-app CY8CPROTO_062_4343W+release (read/write) 2450672 2450608 -64 -0.0
.text 1413316 1413252 -64 -0.0
telink light-switch-app tlsr9518adk80d text 567248 567246 -2 -0.0
Full report (43 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section 00b5886 f1c0c42 change % change
bl602 lighting-app bl602 (read/write) 1381378 1381378 0 0.0
.bss 117618 117618 0 0.0
.data 4480 4480 0 0.0
.text 1051676 1051676 0 0.0
bl602+rpc (read/write) 1426794 1426794 0 0.0
.bss 125058 125058 0 0.0
.data 4600 4600 0 0.0
.text 1083340 1083340 0 0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 668363 668363 0 0.0
(read/write) 182980 182980 0 0.0
.bss 74236 74236 0 0.0
.data 3372 3372 0 0.0
.rodata 88195 88195 0 0.0
.text 579852 579852 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 633939 633939 0 0.0
(read/write) 157820 157820 0 0.0
.bss 73532 73532 0 0.0
.data 3372 3372 0 0.0
.rodata 77411 77411 0 0.0
.text 556204 556204 0 0.0
lock-ftd LP_CC2652R7 (read only) 671495 671495 0 0.0
(read/write) 170040 170040 0 0.0
.bss 71316 71316 0 0.0
.data 3296 3296 0 0.0
.rodata 76263 76263 0 0.0
.text 594752 594752 0 0.0
lock-mtd LP_CC2652R7 (read only) 653595 653595 0 0.0
(read/write) 183628 183628 0 0.0
.bss 67004 67004 0 0.0
.data 3296 3296 0 0.0
.rodata 100875 100875 0 0.0
.text 552240 552240 0 0.0
pump-app LP_CC2652R7 (read only) 680799 680799 0 0.0
(read/write) 161568 161568 0 0.0
.bss 71380 71380 0 0.0
.data 3296 3296 0 0.0
.rodata 88863 88863 0 0.0
.text 591452 591452 0 0.0
pump-controller-app LP_CC2652R7 (read only) 666527 666527 0 0.0
(read/write) 175976 175976 0 0.0
.bss 71516 71516 0 0.0
.data 3292 3292 0 0.0
.rodata 84679 84679 0 0.0
.text 581368 581368 0 0.0
shell LP_CC2652R7 (read only) 661046 661046 0 0.0
(read/write) 185816 185816 0 0.0
.bss 76556 76556 0 0.0
.data 3376 3376 0 0.0
.rodata 85166 85166 0 0.0
.text 575564 575564 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 583246 583238 -8 -0.0
.app_xip_area 460064 460056 -8 -0.0
.bss 65624 65624 0 0.0
.data 744 744 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 589158 589158 0 0.0
.app_xip_area 461248 461248 0 0.0
.bss 70352 70352 0 0.0
.data 748 748 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 588970 588962 -8 -0.0
.app_xip_area 466604 466596 -8 -0.0
.bss 64864 64864 0 0.0
.data 688 688 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read/write) 1087840 1087840 0 0.0
.bss 133260 133260 0 0.0
.data 2064 2064 0 0.0
.text 952496 952496 0 0.0
BRD4161A+rpc (read/write) 1142148 1142148 0 0.0
.bss 149940 149940 0 0.0
.data 2276 2276 0 0.0
.text 989912 989912 0 0.0
BRD4161A+rs911x (read/write) 973720 973720 0 0.0
.bss 161736 161736 0 0.0
.data 2048 2048 0 0.0
.text 809916 809916 0 0.0
lock-app BRD4161A+wf200 (read/write) 1128392 1128392 0 0.0
.bss 144368 144368 0 0.0
.data 2056 2056 0 0.0
.text 981948 981948 0 0.0
window-app BRD4161A (read/write) 1081316 1081316 0 0.0
.bss 134732 134732 0 0.0
.data 2092 2092 0 0.0
.text 944468 944468 0 0.0
esp32 all-clusters-app c3devkit (read only) 1022520 1022520 0 0.0
(read/write) 1486274 1486274 0 0.0
.dram0.bss 70296 70296 0 0.0
.dram0.data 14600 14600 0 0.0
.flash.rodata 215936 215936 0 0.0
.flash.text 1022520 1022520 0 0.0
.iram0.text 62902 62902 0 0.0
m5stack (read only) 1076151 1076151 0 0.0
(read/write) 488312 488312 0 0.0
.dram0.bss 75808 75808 0 0.0
.dram0.data 34144 34144 0 0.0
.flash.rodata 246364 246364 0 0.0
.flash.text 1070767 1070767 0 0.0
.iram0.text 123267 123267 0 0.0
k32w light k32w0+release (read/write) 642640 642640 0 0.0
.bss 69712 69712 0 0.0
.data 2044 2044 0 0.0
.text 568156 568156 0 0.0
lock k32w0+release (read/write) 699920 699920 0 0.0
.bss 70152 70152 0 0.0
.data 2052 2052 0 0.0
.text 624988 624988 0 0.0
linux all-clusters-app debug (read only) 2990169 2990169 0 0.0
(read/write) 155584 155584 0 0.0
.bss 61888 61888 0 0.0
.data 2064 2064 0 0.0
.data.rel.ro 85272 85272 0 0.0
.dynamic 608 608 0 0.0
.got 4568 4568 0 0.0
.init 27 27 0 0.0
.init_array 1144 1144 0 0.0
.rodata 269195 269195 0 0.0
.text 2542402 2542402 0 0.0
all-clusters-minimal-app debug (read only) 2833209 2833209 0 0.0
(read/write) 147288 147288 0 0.0
.bss 61088 61088 0 0.0
.data 2064 2064 0 0.0
.data.rel.ro 77864 77864 0 0.0
.dynamic 608 608 0 0.0
.got 4488 4488 0 0.0
.init 27 27 0 0.0
.init_array 1136 1136 0 0.0
.rodata 269355 269355 0 0.0
.text 2387890 2387890 0 0.0
bridge-app debug+rpc (read only) 2351561 2351561 0 0.0
(read/write) 127160 127160 0 0.0
.bss 50176 50176 0 0.0
.data 3824 3824 0 0.0
.data.rel.ro 67304 67304 0 0.0
.dynamic 608 608 0 0.0
.got 4392 4392 0 0.0
.init 27 27 0 0.0
.init_array 816 816 0 0.0
.rodata 201480 201480 0 0.0
.text 1987314 1987314 0 0.0
chip-tool debug (read only) 10464689 10464689 0 0.0
(read/write) 641464 641464 0 0.0
.bss 24856 24856 0 0.0
.data 3266 3266 0 0.0
.data.rel.ro 606824 606824 0 0.0
.dynamic 608 608 0 0.0
.got 5088 5088 0 0.0
.init 27 27 0 0.0
.init_array 760 760 0 0.0
.rodata 536021 536021 0 0.0
.text 8444260 8444260 0 0.0
chip-tool-ipv6only arm64 (read only) 9880708 9880708 0 0.0
(read/write) 689041 689041 0 0.0
.bss 32897 32897 0 0.0
.data 3272 3272 0 0.0
.data.rel.ro 634336 634336 0 0.0
.dynamic 560 560 0 0.0
.got 13584 13584 0 0.0
.init 24 24 0 0.0
.init_array 192 192 0 0.0
.rodata 468228 468228 0 0.0
.text 7797348 7797348 0 0.0
lighting-app debug+rpc (read only) 2574425 2574425 0 0.0
(read/write) 130096 130096 0 0.0
.bss 49728 49728 0 0.0
.data 2096 2096 0 0.0
.data.rel.ro 72344 72344 0 0.0
.dynamic 608 608 0 0.0
.got 4392 4392 0 0.0
.init 27 27 0 0.0
.init_array 904 904 0 0.0
.rodata 217232 217232 0 0.0
.text 2186338 2186338 0 0.0
lock-app debug (read only) 2539617 2539617 0 0.0
(read/write) 125176 125176 0 0.0
.bss 48160 48160 0 0.0
.data 1712 1712 0 0.0
.data.rel.ro 69352 69352 0 0.0
.dynamic 608 608 0 0.0
.got 4424 4424 0 0.0
.init 27 27 0 0.0
.init_array 880 880 0 0.0
.rodata 232112 232112 0 0.0
.text 2141426 2141426 0 0.0
ota-provider-app debug (read only) 2343961 2343961 0 0.0
(read/write) 118944 118944 0 0.0
.bss 47776 47776 0 0.0
.data 1936 1936 0 0.0
.data.rel.ro 63336 63336 0 0.0
.dynamic 608 608 0 0.0
.got 4488 4488 0 0.0
.init 27 27 0 0.0
.init_array 760 760 0 0.0
.rodata 207704 207704 0 0.0
.text 1972594 1972594 0 0.0
ota-requestor-app debug (read only) 2464873 2464873 0 0.0
(read/write) 126304 126304 0 0.0
.bss 50144 50144 0 0.0
.data 2240 2240 0 0.0
.data.rel.ro 67992 67992 0 0.0
.dynamic 608 608 0 0.0
.got 4480 4480 0 0.0
.init 27 27 0 0.0
.init_array 824 824 0 0.0
.rodata 211264 211264 0 0.0
.text 2081330 2081330 0 0.0
shell debug (read only) 2573665 2573665 0 0.0
(read/write) 141736 141736 0 0.0
.bss 57736 57736 0 0.0
.data 1264 1264 0 0.0
.data.rel.ro 76944 76944 0 0.0
.dynamic 608 608 0 0.0
.got 4136 4136 0 0.0
.init 27 27 0 0.0
.init_array 1016 1016 0 0.0
.rodata 231250 231250 0 0.0
.text 2184354 2184354 0 0.0
thermostat-no-ble arm64 (read only) 2343516 2343516 0 0.0
(read/write) 141633 141633 0 0.0
.bss 55313 55313 0 0.0
.data 1672 1672 0 0.0
.data.rel.ro 75880 75880 0 0.0
.dynamic 560 560 0 0.0
.got 4992 4992 0 0.0
.init 24 24 0 0.0
.init_array 408 408 0 0.0
.rodata 138964 138964 0 0.0
.text 1967312 1967312 0 0.0
tv-app debug (read only) 3141857 3141857 0 0.0
(read/write) 257608 257608 0 0.0
.bss 167320 167320 0 0.0
.data 4736 4736 0 0.0
.data.rel.ro 79000 79000 0 0.0
.dynamic 608 608 0 0.0
.got 4848 4848 0 0.0
.init 27 27 0 0.0
.init_array 1064 1064 0 0.0
.rodata 254184 254184 0 0.0
.text 2698338 2698338 0 0.0
tv-casting-app debug (read only) 5379529 5379529 0 0.0
(read/write) 158592 158592 0 0.0
.bss 51352 51352 0 0.0
.data 2432 2432 0 0.0
.data.rel.ro 98432 98432 0 0.0
.dynamic 608 608 0 0.0
.got 4736 4736 0 0.0
.init 27 27 0 0.0
.init_array 1024 1024 0 0.0
.rodata 338641 338641 0 0.0
.text 4773810 4773810 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2450672 2450608 -64 -0.0
.bss 214516 214516 0 0.0
.data 5872 5872 0 0.0
.text 1413316 1413252 -64 -0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1172755 1172755 0 0.0
bss 143120 143120 0 0.0
rodata 141828 141828 0 0.0
text 808856 808856 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1152807 1152807 0 0.0
bss 142356 142356 0 0.0
rodata 133360 133360 0 0.0
text 798164 798164 0 0.0
p6 all-clusters-app default (read only) 881560 881560 0 0.0
(read/write) 1686780 1686780 0 0.0
.bss 149136 149136 0 0.0
.data 2648 2648 0 0.0
.text 1526608 1526608 0 0.0
all-clusters-minimal-app default (read only) 882280 882280 0 0.0
(read/write) 1630876 1630876 0 0.0
.bss 148416 148416 0 0.0
.data 2648 2648 0 0.0
.text 1471424 1471424 0 0.0
light-app default (read only) 890584 890584 0 0.0
(read/write) 1552284 1552284 0 0.0
.bss 140320 140320 0 0.0
.data 2440 2440 0 0.0
.text 1401136 1401136 0 0.0
lock-app default (read only) 886112 886112 0 0.0
(read/write) 1589908 1589908 0 0.0
.bss 144776 144776 0 0.0
.data 2456 2456 0 0.0
.text 1434288 1434288 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 799608 799608 0 0.0
bss 70796 70796 0 0.0
noinit 40416 40416 0 0.0
text 567248 567246 -2 -0.0
lighting-app tlsr9518adk80d (read/write) 819748 819748 0 0.0
bss 71640 71640 0 0.0
noinit 40416 40416 0 0.0
text 583850 583850 0 0.0

@woody-apple woody-apple enabled auto-merge (squash) July 29, 2022 19:48
@woody-apple woody-apple merged commit 78fc5ca into project-chip:master Jul 29, 2022
@bzbarsky-apple bzbarsky-apple deleted the allow-no-wifi-password branch July 29, 2022 21:16
github-actions bot pushed a commit that referenced this pull request Jul 29, 2022
@bzbarsky-apple @web-flow
Allow Darwin frawework to commission onto a password-less WiFi networ…
751c4d1 
…k. (#21363)

We were requiring a WiFi password in order to provide the WiFi SSID to
the underlying SDK.  But not all networks have a password.
woody-apple added a commit that referenced this pull request Jul 30, 2022
@woody-apple @bzbarsky-apple
Allow Darwin frawework to commission onto a password-less WiFi networ…
965c9ee 
…k. (#21363) (#21444)

We were requiring a WiFi password in order to provide the WiFi SSID to
the underlying SDK.  But not all networks have a password.

Co-authored-by: Boris Zbarsky <[email protected]>
isiu-apple pushed a commit to isiu-apple/connectedhomeip that referenced this pull request Sep 16, 2022
@bzbarsky-apple @isiu-apple
Allow Darwin frawework to commission onto a password-less WiFi networ…
21f690f 
…k. (project-chip#21363)

We were requiring a WiFi password in order to provide the WiFi SSID to
the underlying SDK.  But not all networks have a password.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vivien-apple vivien-apple vivien-apple approved these changes

@andy31415 andy31415 Awaiting requested review from andy31415

@anush-apple anush-apple Awaiting requested review from anush-apple

@arkq arkq Awaiting requested review from arkq

@Byungjoo-Lee Byungjoo-Lee Awaiting requested review from Byungjoo-Lee

@carol-apple carol-apple Awaiting requested review from carol-apple

@chrisdecenzo chrisdecenzo Awaiting requested review from chrisdecenzo

@chshu chshu Awaiting requested review from chshu

@chulspro chulspro Awaiting requested review from chulspro

@Damian-Nordic Damian-Nordic Awaiting requested review from Damian-Nordic

@dhrishi dhrishi Awaiting requested review from dhrishi

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

@emargolis emargolis Awaiting requested review from emargolis

@erjiaqing erjiaqing Awaiting requested review from erjiaqing

@franck-apple franck-apple Awaiting requested review from franck-apple

@gjc13 gjc13 Awaiting requested review from gjc13

@harimau-qirex harimau-qirex Awaiting requested review from harimau-qirex

@harsha-rajendran harsha-rajendran Awaiting requested review from harsha-rajendran

@hawk248 hawk248 Awaiting requested review from hawk248

@isiu-apple isiu-apple Awaiting requested review from isiu-apple

@jelderton jelderton Awaiting requested review from jelderton

@jepenven-silabs jepenven-silabs Awaiting requested review from jepenven-silabs

@jmartinez-silabs jmartinez-silabs Awaiting requested review from jmartinez-silabs

@jtung-apple jtung-apple Awaiting requested review from jtung-apple

@kpschoedel kpschoedel Awaiting requested review from kpschoedel

@lazarkov lazarkov Awaiting requested review from lazarkov

@LuDuda LuDuda Awaiting requested review from LuDuda

@msandstedt msandstedt Awaiting requested review from msandstedt

@rgoliver rgoliver Awaiting requested review from rgoliver

@robszewczyk robszewczyk Awaiting requested review from robszewczyk

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@tcarmelveilleux tcarmelveilleux Awaiting requested review from tcarmelveilleux

@tecimovic tecimovic Awaiting requested review from tecimovic

@turon turon Awaiting requested review from turon

@vijs vijs Awaiting requested review from vijs

@wbschiller wbschiller Awaiting requested review from wbschiller

@woody-apple woody-apple Awaiting requested review from woody-apple

@xylophone21 xylophone21 Awaiting requested review from xylophone21

@yunhanw-google yunhanw-google Awaiting requested review from yunhanw-google

Assignees
No one assigned
Labels
darwin fast track review - pending
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bzbarsky-apple @vivien-apple @woody-apple