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

[Tizen] Enable ble configuration in chip-tool for tizen platform #23317

Merged

Conversation

jlatusek
Copy link
Contributor

@jlatusek jlatusek commented Oct 24, 2022

Problem

During the compilation of chip-tool for the Tizen platform, the configuration of ble was disabled. Bonding a new device didn't work because of missing setting if the Tizen device is type central or not.

Changes

Add CHIP_DEVICE_LAYER_TARGET_TIZEN to conditional compilation.

Testing

Run chip-lighting-app on one device:

./out/linux-x64-light/chip-lighting-app --discriminator 42 --passcode 1234

and chip-tool on the Tizen device:

./chip-tool pairing ble-wifi 0x01 wifi-name wifi-pass 1234 42

@github-actions
Copy link

PR #23317: Size comparison from b9d32ec to 5708907

Increases (2 builds for k32w, telink)
platform target config section b9d32ec 5708907d change % change
k32w contact k32w0+release (read/write) 661396 661412 16 0.0
.text 563140 563156 16 0.0
telink ota-requestor-app tlsr9518adk80d text 565044 565046 2 0.0
Decreases (2 builds for telink)
platform target config section b9d32ec 5708907d change % change
telink light-switch-app tlsr9518adk80d text 554016 554014 -2 -0.0
lighting-app tlsr9518adk80d text 566176 566174 -2 -0.0
Full report (11 builds for bl702, k32w, mbed, nrfconnect, telink)
platform target config section b9d32ec 5708907d change % change
bl702 lighting-app bl702 0 0 0 0.0
(read only) 3262 3262 0 0.0
(read/write) 1189427 1189427 0 0.0
.bleromro 6296 6296 0 0.0
.bleromrw 124 124 0 0.0
.boot2 688 688 0 0.0
.bss 67118 67118 0 0.0
.bss_psram 29696 29696 0 0.0
.comment 48 48 0 0.0
.data 4272 4272 0 0.0
.debug_abbrev 1508924 1508924 0 0.0
.debug_aranges 132088 132088 0 0.0
.debug_frame 484668 484668 0 0.0
.debug_info 37916953 37916953 0 0.0
.debug_line 5257385 5257385 0 0.0
.debug_loc 3369084 3369084 0 0.0
.debug_ranges 358520 358520 0 0.0
.debug_str 3452369 3452369 0 0.0
.hbn 509 509 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 144 144 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 116744 116744 0 0.0
.rsvd 3188 3188 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 565491 565491 0 0.0
.symtab 171760 171760 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3262 3262 0 0.0
.text 957778 957778 0 0.0
bl702+rpc 0 0 0 0.0
(read only) 3262 3262 0 0.0
(read/write) 1285091 1285091 0 0.0
.bleromro 6296 6296 0 0.0
.bleromrw 124 124 0 0.0
.boot2 688 688 0 0.0
.bss 75166 75166 0 0.0
.bss_psram 29936 29936 0 0.0
.comment 48 48 0 0.0
.data 4800 4800 0 0.0
.debug_abbrev 1646331 1646331 0 0.0
.debug_aranges 139592 139592 0 0.0
.debug_frame 510040 510040 0 0.0
.debug_info 41824009 41824009 0 0.0
.debug_line 5632105 5632105 0 0.0
.debug_loc 3561786 3561786 0 0.0
.debug_ranges 380976 380976 0 0.0
.debug_str 3848339 3848339 0 0.0
.hbn 509 509 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 160 160 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 130136 130136 0 0.0
.rsvd 3188 3188 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 624664 624664 0 0.0
.symtab 189568 189568 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3262 3262 0 0.0
.text 1031224 1031224 0 0.0
k32w contact k32w0+release (read/write) 661396 661412 16 0.0
.bss 77040 77040 0 0.0
.data 2104 2104 0 0.0
.text 563140 563156 16 0.0
light k32w0+release (read/write) 671640 671640 0 0.0
.bss 74840 74840 0 0.0
.data 2060 2060 0 0.0
.text 592012 592012 0 0.0
lock k32w0+release (read/write) 633220 633220 0 0.0
.bss 75600 75600 0 0.0
.data 2080 2080 0 0.0
.text 552812 552812 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2452328 2452328 0 0.0
.bss 215028 215028 0 0.0
.data 5872 5872 0 0.0
.text 1414972 1414972 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1183787 1183787 0 0.0
bss 144457 144457 0 0.0
rodata 144428 144428 0 0.0
text 816016 816016 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1162471 1162471 0 0.0
bss 143684 143684 0 0.0
rodata 136000 136000 0 0.0
text 803912 803912 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 790476 790476 0 0.0
bss 72480 72480 0 0.0
noinit 43520 43520 0 0.0
text 554016 554014 -2 -0.0
lighting-app tlsr9518adk80d (read/write) 805732 805732 0 0.0
bss 73240 73240 0 0.0
noinit 43520 43520 0 0.0
text 566176 566174 -2 -0.0
ota-requestor-app tlsr9518adk80d (read/write) 804616 804616 0 0.0
bss 74052 74052 0 0.0
noinit 43520 43520 0 0.0
text 565044 565046 2 0.0

@bzbarsky-apple bzbarsky-apple merged commit 5d95baf into project-chip:master Oct 24, 2022
adbridge pushed a commit to ARM-software/connectedhomeip that referenced this pull request Nov 18, 2022
adbridge pushed a commit to ARM-software/connectedhomeip that referenced this pull request Nov 18, 2022
@jlatusek jlatusek deleted the tizen-chip-tool-central branch February 17, 2023 10:34
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.

3 participants