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

[Telink] Use latest Zephyr (SDK 0.15.2) & migrate to Zephyr BLE implementation & Update every image to 0.6.17 #23827

Merged
merged 12 commits into from
Dec 2, 2022
Merged

[Telink] Use latest Zephyr (SDK 0.15.2) & migrate to Zephyr BLE implementation & Update every image to 0.6.17 #23827

merged 12 commits into from
Dec 2, 2022

Conversation

s07641069
Copy link
Contributor

Change overview

  • Adaptation to use latest Telink Zephyr version (SDK 0.15.2)
  • Migrate to Telink Zephyr BLE implementation
  • Update every image to 0.6.17

Known issue

  • Increased flash and ram usage, after which targets with enabled all features can't be compiled (OTA + Factory data).

Testing

Tested manually with chip-tool.

Steps:

  • Run: $ chip-tool pairing ble-thread <...>
  • Wait till success
  • Run: $ chip-tool onoff on ${NODE_ID }1
  • Wait till success
  • Run: $ chip-tool onoff off ${NODE_ID }1

Wait till success

@Damian-Nordic Damian-Nordic enabled auto-merge (squash) December 2, 2022 09:26
@Damian-Nordic Damian-Nordic merged commit f9d9457 into project-chip:master Dec 2, 2022
@s07641069 s07641069 deleted the telink_ble_zephyr_integration branch December 2, 2022 09:33
@bzbarsky-apple bzbarsky-apple mentioned this pull request Jan 11, 2023
Closed
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this pull request Jan 11, 2023
@bzbarsky-apple
Stop holding a reference into a temporary.
44ff13c 
This was drive-by fixed on tip by
project-chip#23827 but we
should fix on 1.0 too.

Fixes project-chip#24367
@bzbarsky-apple bzbarsky-apple mentioned this pull request Jan 11, 2023
Merged
andy31415 pushed a commit that referenced this pull request Jan 12, 2023
@bzbarsky-apple
Stop holding a reference into a temporary. (#24373)
bd94d06 
This was drive-by fixed on tip by
#23827 but we
should fix on 1.0 too.

Fixes #24367
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Damian-Nordic Damian-Nordic Damian-Nordic approved these changes

@andy31415 andy31415 andy31415 approved these changes

@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

@bzbarsky-apple bzbarsky-apple Awaiting requested review from bzbarsky-apple

@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

@dhrishi dhrishi Awaiting requested review from dhrishi

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

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

@gjc13 gjc13 Awaiting requested review from gjc13

@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

@lazarkov lazarkov Awaiting requested review from lazarkov

@lpbeliveau-silabs lpbeliveau-silabs Awaiting requested review from lpbeliveau-silabs

@LuDuda LuDuda Awaiting requested review from LuDuda

@msandstedt msandstedt Awaiting requested review from msandstedt

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@tecimovic tecimovic Awaiting requested review from tecimovic

@vijs vijs Awaiting requested review from vijs

@vivien-apple vivien-apple Awaiting requested review from vivien-apple

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

@xylophone21 xylophone21 Awaiting requested review from xylophone21

Assignees
No one assigned
Labels
config controller examples github gn integrations platform review - approved telink vscode workflows zephyr
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@s07641069 @andy31415 @CLAassistant @Damian-Nordic @andriy-bilynskyy @restyled-commits