-
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] Zephyr 3.3 & 3.7 support #37582
Merged
mergify
merged 65 commits into
project-chip:master
from
s07641069:telink_add_zephyr_3.7_support
Mar 3, 2025
Merged
[Telink] Zephyr 3.3 & 3.7 support #37582
mergify
merged 65 commits into
project-chip:master
from
s07641069:telink_add_zephyr_3.7_support
Mar 3, 2025
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Adopted changes for Telink B9x board series for upcoming update of Zephyr version. Fixed err output from InitChipStack in mainCommon. Tested advertisement tested manually.
- STREAM_FLASH_ERASE has direct dependencies STREAM_FLASH - GETOPT_LONG multiple definition error
- Apply change for flash overlays <=2Mb - Possible after reducing CONFIG_BOOT_MAX_IMG_SECTORS
reverted GetWifiInterface in InetUtils changes, added syntax fixes to Kconfigs.default
Reduced mcuboot partition by 0x1000 and removed two other partitions getting 0x3000 to distribute 0x4000 between slot0 and slot 1. Kconfig.defaults changes to reduce ROM usage on B92 lighting OTA. To be improved [Telink] reduced MCUboot partition for most targets increase of slot0,1, bootloader config changes [Telink] Fixed B9x platform startup Lack of MBEDTLS_ZEPHYR_ENTROPY config in defaults which led to InitChipStack 172 error. As result entropy failed in src/platform/Zephyr/PlatformManagerImpl.cpp [Telink] fixed B9x commissioning Reverted network buffers configs [Telink] TL3218 commissioning fixes Changes to fix TLx commissioning on new Zephyr [Telink] Zephyr revision update After merge updates. Kconfig change to fix TL3218 commissioning Reduced RAM usage to build TL3218 by COMMON_LIBC_MALLOC_ARENA_SIZE changing [Telink] zephyr version change, cleanup Changed MCUBOOT logs config, same for LZMA
[Telink] ROM usage optimization
Fixes full config target commissioning for W91. Been failing on `SendAttestationRequest` step, finished with `general error 0x01`. Tested manually with chip-tool on RPi on tlsr9118bdk40d
Increased NET_MGMT_EVENT_STACK_SIZE by 122 B and IPC_SERVICE_BACKEND_ICMSG_WQ_STACK_SIZE by 256 B so that net_mgmt and icmsg_workq stack were used <90%. Jenkins reported thread analyzer warnings on that.
Reverted NET_BUF_RX/TX_COUNT
- update the default values for BT_RX_STACK_SIZE and COMMON_LIBC_MALLOC_ARENA_SIZE for TL721X Signed-off-by: Jinmiao Yu <jinmiao.yu@telink-semi.com>
PR #37582: Size comparison from 1b3f616 to 2e0beb7 Full report (75 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, tizen)
|
2e0beb7
to
ef1cedc
Compare
PR #37582: Size comparison from ceb97d1 to ef1cedc Full report (75 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, tizen)
|
- adjust the zephyr hash to 930b6d4 . Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com>
* riscv: telink: Update Zephyr hash - adjust the zephyr hash to 9e39d9f . - add tl3218x_retention target . Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com> * riscv: telink: remove retention build target for buteo Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com> * [Telink] Add comment --------- Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com> Co-authored-by: Alex Tsitsiura <s07641069@gmail.com>
TODO: fix compilation warnings
ef1cedc
to
a417d41
Compare
PR #37582: Size comparison from 6947f9b to a417d41 Full report (76 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, tizen)
|
a417d41
to
4b9e6dd
Compare
PR #37582: Size comparison from 4e7ce33 to 4b9e6dd Increases above 0.2%:
Full report (94 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
4b9e6dd
to
08ea9ac
Compare
PR #37582: Size comparison from 28c1d83 to 08ea9ac Full report (96 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
andy31415
approved these changes
Mar 3, 2025
gmarcosb
pushed a commit
to gmarcosb/connectedhomeip
that referenced
this pull request
Mar 4, 2025
* [Telink] Zephyr v3.7 update changes Adopted changes for Telink B9x board series for upcoming update of Zephyr version. Fixed err output from InitChipStack in mainCommon. Tested advertisement tested manually. * [Telink] Manual Zephyr SDK update to check CI jobs * [Telink] Fix build errors: - STREAM_FLASH_ERASE has direct dependencies STREAM_FLASH - GETOPT_LONG multiple definition error * [Telink] CONFIG_BOOT_MAX_IMG_SECTORS_AUTO used * [Telink] Fix build errors * [Telink] Reduce mcuboot partition size to 60Kb - Apply change for flash overlays <=2Mb - Possible after reducing CONFIG_BOOT_MAX_IMG_SECTORS * [Telink] Reduce NET_BUF_*_COUNT (to be tested) * [Telink] avoid using NEWLIB_LIBC (use PICOLIB) * Restyled by whitespace * Restyled by clang-format * [Telink] Update Zephyr revision & SDK * [Telink] Adopt TL3218 to latest Zephyr version * [Telink] Use 4mb flash for (B92) Light Switch App target build * [Telink] Fix W91 target build * [Telink] Fix B91 Mars target build * [Telink] Use correct if SOC_RISCV_TELINK_TL321X * [Telink] Fix retention target builds * [Telink] fixed W91 WiFi interface init reverted GetWifiInterface in InetUtils changes, added syntax fixes to Kconfigs.default * [Telink] Update to latest zephyr (develop_new_matter_fixes) hash * [Telink] memory layout and default config changes Reduced mcuboot partition by 0x1000 and removed two other partitions getting 0x3000 to distribute 0x4000 between slot0 and slot 1. Kconfig.defaults changes to reduce ROM usage on B92 lighting OTA. To be improved [Telink] reduced MCUboot partition for most targets increase of slot0,1, bootloader config changes [Telink] Fixed B9x platform startup Lack of MBEDTLS_ZEPHYR_ENTROPY config in defaults which led to InitChipStack 172 error. As result entropy failed in src/platform/Zephyr/PlatformManagerImpl.cpp [Telink] fixed B9x commissioning Reverted network buffers configs [Telink] TL3218 commissioning fixes Changes to fix TLx commissioning on new Zephyr [Telink] Zephyr revision update After merge updates. Kconfig change to fix TL3218 commissioning Reduced RAM usage to build TL3218 by COMMON_LIBC_MALLOC_ARENA_SIZE changing [Telink] zephyr version change, cleanup Changed MCUBOOT logs config, same for LZMA * [Telink] W91 OTA, Factory data commissioning fix Fixes full config target commissioning for W91. Been failing on `SendAttestationRequest` step, finished with `general error 0x01`. Tested manually with chip-tool on RPi on tlsr9118bdk40d * [Telink] W91 stacks increase Increased NET_MGMT_EVENT_STACK_SIZE by 122 B and IPC_SERVICE_BACKEND_ICMSG_WQ_STACK_SIZE by 256 B so that net_mgmt and icmsg_workq stack were used <90%. Jenkins reported thread analyzer warnings on that. * [Telink] review changes draft Reverted NET_BUF_RX/TX_COUNT * [Telink] Add tl7218 board. * [Telink] Fix compilation warnings. * [Telink] Fix config option. * [Telink] Cleanup MbedTLS configuration. * [Telink] Update tl721x default Kconfig - update the default values for BT_RX_STACK_SIZE and COMMON_LIBC_MALLOC_ARENA_SIZE for TL721X Signed-off-by: Jinmiao Yu <jinmiao.yu@telink-semi.com> * [Telink] Remove unused files * [Telink] Update Zephyr revision * [Telink] Clean config types * [Telink] Update Zephyr revision * [Telink] Increase BLE_CTRL_THREAD_STACK_SIZE (B92 SoC retention test) * [Telink] Fix compilation warninig * [Telink] TL7218 upgrade to support version A2 * [Telink] Remove unused file after resolve merge conflict * [Telink] Add tl7218_retention target * riscv: telink: seperate ram for tl3218x . - use retention mode to seperate ram. - reserve more ram for ilm. Signed-off-by: Haiwen Xia <haiwen.xia@telink-semi.com> * [Telink] Fix broken GitHub CI after adding tl721x target * [Telink] remove temporary LZMA for tl321x due to bug * [Telink] Fix TELINK_TLX_MATTER_RETENTION_LAYOUT typo for BOARD_TL3218 * [Telink] Add GPIO/PWM required for TL721X telink-semi/hal_telink@62edeab * [Telink] Temporary disable B95 OTA failed build B95 will be removed fully in next commits * [Telink] Add missed clean out build output step * [Telink] adopt WiFi driver * [Telink] remove b95 SoC / TLSR9258a board (replaced by TL721x) * [Telink] Revert InetUtils.cpp Telink dependency * [Telink] zephyr versions compatibility changes (project-chip#382) * [Telink] zephyr versions compatibility changes * [Telink] Change Window Covering App default SoC W91 doesn't support PM * [Telink] Update Zephyr revision * [Telink] sync B92 flash layout * [Telink] Remove temp b92 4mb change * [Telink] Avoid main stack owerflow for w91 * [Telink] Update Zephyr revision * [Telink] manual sync after merge master * [Telink] Increase Main Stack size for W91 Update Zephyr revision * [Telink] Zephyr 3.3 backward compatibility support * riscv: telink: fix insufficient RAM when config LZMA (project-chip#384) - adjust the zephyr hash to 930b6d4 . Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com> * riscv: telink: add retention mode for buteo (project-chip#385) * riscv: telink: Update Zephyr hash - adjust the zephyr hash to 9e39d9f . - add tl3218x_retention target . Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com> * riscv: telink: remove retention build target for buteo Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com> * [Telink] Add comment --------- Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com> Co-authored-by: Alex Tsitsiura <s07641069@gmail.com> * [Telink] Fix commisioning issue due to missed MbedTLS module TODO: fix compilation warnings * telink: tl7218x: fix tercel retention (Update Zephyr hash) * [Telink] Optimize add MbedTLS definitions for Zephyr 3.3 without warnings * [Telink] Use latest hash for Zephyr 3.3 * [Telink] Disable debug part --------- Signed-off-by: Jinmiao Yu <jinmiao.yu@telink-semi.com> Signed-off-by: Haiwen Xia <haiwen.xia@telink-semi.com> Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com> Co-authored-by: interfer <diman1436@gmail.com> Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Dmytro Huz <75682372+interfer@users.noreply.github.com> Co-authored-by: Andrii Bilynskyi <andrii.bilynskyi@telink-semi.com> Co-authored-by: Haiwen Xia <haiwen.xia@telink-semi.com> Co-authored-by: Serhii Salamakha <serhii.salamakha@gmail.com> Co-authored-by: fengtai-telink <fengtai.xie@telink-semi.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Change overview
Testing
Verified by CI