-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[Telink] fix factory data as pr #23385 #23559
[Telink] fix factory data as pr #23385 #23559
Conversation
PR #23559: Size comparison from 1a8cf55 to 36fa49a Increases (3 builds for cc13x2_26x2, telink)
Decreases (9 builds for bl602, bl702, cc13x2_26x2, psoc6, telink)
Full report (44 builds for bl602, bl702, cc13x2_26x2, cyw30739, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess we should think of merging some of the factory data stuff between nRF and Telink given the scripts also work for you :)
I agree, nRF solution pretty good and Telink use Zephyr as well. |
Problem
Telink Factory Data brocken
Need the same fix for Telink platform as #23385
Change overview
Applied same fixes as in #23385
Testing
Tested manually with chip-tool.
Steps:
Build Telink application with flags: -DCONFIG_CHIP_FACTORY_DATA=y -DCONFIG_CHIP_FACTORY_DATA_BUILD=y -DCONFIG_CHIP_FACTORY_DATA_MERGE_WITH_FIRMWARE=y
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