Skip to content

ignore, just for code compare , to update matter1.6 for tl3238x - #48

Open
haiwentelink wants to merge 54 commits into
dev-tl3238x_prj_v1.5.1_basefrom
dev-tl3238x_prj_v1.5.1
Open

ignore, just for code compare , to update matter1.6 for tl3238x #48
haiwentelink wants to merge 54 commits into
dev-tl3238x_prj_v1.5.1_basefrom
dev-tl3238x_prj_v1.5.1

Conversation

@haiwentelink

Copy link
Copy Markdown
Collaborator

Summary

Related issues

Testing

Readability checklist

The checklist below will help the reviewer finish PR review in time and keep the
code readable:

  • PR title is
    descriptive
  • Apply the
    “When in Rome…”
    rule (coding style)
  • PR size is short
  • Try to avoid "squashing" and "force-update" in commit history
  • CI time didn't increase

See: Pull Request Guidelines

haiwentelink and others added 30 commits November 26, 2025 13:26
- add tl3238x into matter.

Signed-off-by: Haiwen Xia <haiwen.xia@telink-semi.com>
- close bootstrap and bootvalitdate.
- adjust rf settings.

Signed-off-by: Haiwen Xia <haiwen.xia@telink-semi.com>
- add 2m partition for customer.
- setting different discriminator.
- add dual mode logic.
- enable ota.

Signed-off-by: Haiwen Xia <haiwen.xia@telink-semi.com>
- after change the ble controller , the thread need to enlarge in pm mode.

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
* [Telink] set persistent subscriptions by default

Now included to all Telink targets.

Persists Matter subscriptions on the publisher node. This feature allows
a Matter node to faster get back to the previous operation after it went
offline, for example, due to a power outage. That is, the node can use the
persisted subscription information to quickly re-establish the previous
subscriptions instead of waiting for the subscriber node to realize that
the publisher is alive again.

* [Telink] added dependenant configs & Buteo condition

Exclude persistent subscription for Buteo with retention
due to footprint.
Added dependencies for CHIPPlatformConfig.h

* Restyled by whitespace

---------

Co-authored-by: Restyled.io <commits@restyled.io>
- open ota and add lzma.

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- enlarge heap size for tl3238x.

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- enlarge ble thread stack for jenkins alarm to 100%.

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- ajust the rf power to 3dbm.
- add the extra code for tlx optimization.
- add the demo setting for ICD MODE.

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- after adjust the ot queue for ram section changes .

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- clean code for dumplicate partition .

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- update ICD parameters for light-switch .

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- select different dual mode ,if none means matter only mode .

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- add switch setting and code logic .

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
 - add the 2m_lzma partition and origin 4m partition .

Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com>
- update application code for auto-switch mode .
- clean code.

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- bootloader will use const overlay ,so should keep the bootloader and application use the same .

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- in pm mode ,should open optimize mode to save power .

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- in pm mode ,should open optimize mode to save power .

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- need to use 2m back up file to replace 2m partition .

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- open ble ota by default for pm mode and non-pm mode .

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- in pm mode , should apply this patch to modules/lib/openthread .

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- fix the logic for zigbee switch to matter.

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- add optional argument --secure-programming-verification and --chip-id to
generate factory data and independent DAC.
- remove redundant Config SECURE_PROGRAMMING in Kconfig.defaults.
- Add gatt service for read chip-id when adv and conn.
- Update CONFIG_SECURE_PROGRAMMING.
- update logging in mfg_tool.py
- Add aes_decryption_be for TL3238
- Add GetDACData for separate DAC partition
- Add logging to print DAC key pairs and cert
- Add testing factory data blocks
- directly use the flash adr as the pointer for the dac cert part.
- use efuse_get_ieee_addr instead of efuse_get_chip_id
- append 8 bytes of '0x00' at the end of mac addr to generated chip_id
- Need update efuse_get_ieee_addr return from void to drv_api_status_e
- Enhance the Manufacturing Date handling to support extended string formats (e.g.,
 20250825CNLS0413) while maintaining compatibility with the previous numeric format
 (YYYY-MM-DD).
- Some modifications are Matter public implementation changes, use as reference to
implement your own requirements.
- Verification command: ./chip-tool basicinformation read manufacturing-date 1 0
- remove PKI certs
- remove factory data
- update prj.conf files.
- clean code for parse factory data and prj.conf.
- fix the bugs for the impl of manufacutring_string.

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>

---------

Signed-off-by: jinmiao.yu <jinmiao.yu@telink-semi.com>
Signed-off-by: Jinmiao Yu <jinmiao.yu@telink-semi.com>
Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com>
Signed-off-by: pulin1103 <zhenghuan.zhang1103@gmail.com>
Signed-off-by: Damien Ji <yinghao.ji@telink-semi.com>
Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
Signed-off-by: Hao Wu <hao.wu@telink-semi.com>
Co-authored-by: jinmiao.yu <jinmiao.yu@telink-semi.com>
Co-authored-by: hao.wu <hao.wu@telink-semi.com>
Co-authored-by: Fengtai Xie <fengtai.xie@telink-semi.com>
Co-authored-by: pulin1103 <zhenghuan.zhang1103@gmail.com>
Co-authored-by: Damien Ji <yinghao.ji@telink-semi.com>
- clean demo jump code zigbee switch demo code .
- clean code kFailSafeTimerExpired.

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
- fix compilation error when enabling CONFIG_EXPOSE_CHIP_ID_VIA_BLE

Signed-off-by: Damien Ji <yinghao.ji@telink-semi.com>
 - Adjust HandleChipIDRead format .

Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com>
- close CONFIG_BOOT_BANNER  .

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
haiwentelink and others added 19 commits February 7, 2026 14:24
- seperate the commission mode and enhance mode.

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
* [Telink] Add BLE extended advertising

* [Telink] Fix DISCOVERY_TIMEOUT

* [Telink] Don't enable extended advertising by default

* [Telibk] Fix typo
- lowdown the secure partition and add user config partition for user.

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
 - Open CONFIG_ADC in lighting-app/light-switch-app/contact-sensor-app .
 - Fix the compilation will fail when CONFIG_LOG=n .

Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com>
 - Accelerate factory_reset run time .

Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com>
- after change the ble lib , the ble stack need to enlarge 256.

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
* Merge SBoot or ZB FW

- Add commands to merge Secure Boot or Zigbee FW if exits
- Enable BOOT_VALIDATE_SLOT0 but disable BOOT_BANNER to output chip_id

Signed-off-by: Damien Ji <yinghao.ji@telink-semi.com>

* Enable BOOT_VALIDATE_SLOT0_ONCE

to validate slot0 for only once.

Signed-off-by: Damien Ji <yinghao.ji@telink-semi.com>

* update boot signature key file

- CONFIG_BOOT_SIGNATURE_KEY_FILE
- CONFIG_MCUBOOT_SIGNATURE_KEY_FILE

Signed-off-by: Damien Ji <yinghao.ji@telink-semi.com>

* Extend Boot Signature Key File in More Apps

- add CONFIG_MCUBOOT_SIGNATURE_KEY_FILE in contact-sensor-app,
 light-switch-app, lighting-app, and lock-app
- Disable CONFIG_BOOT_SIGNATURE_KEY_FILE and
CONFIG_MCUBOOT_SIGNATURE_KEY_FILE in configuration files

Signed-off-by: Damien Ji <yinghao.ji@telink-semi.com>

---------

Signed-off-by: Damien Ji <yinghao.ji@telink-semi.com>
…roject-chip#438)

 - Add ota event callback .
 - Add deferred storage attributes for provider .
 - Increase maximum group number per fabric .
 - Apply HW Hash and set BOOT_IMG_HASH_DIRECTLY_ON_STORAGE .
 - Increase the ISR_STACK_SIZE to 1024 .
 - Increase the SYSTEM_WORKQUEUE_STACK_SIZE to 2048 .
 - Increase the HEAP_MEM_POOL_SIZE to 1280 .
 - Increase the COMMON_LIBC_MALLOC_ARENA_SIZE to 20716 .
 - Increase the NET_IF_MCAST_IPV6_ADDR_COUNT to 47 .
 - Increase the TL_BLE_CTRL_THREAD_STACK_SIZE to 1536 .
 - Increase the OPENTHREAD_THREAD_STACK_SIZE to 4096 .
 - Increase the OPENTHREAD_RADIO_WORKQUEUE_STACK_SIZE to 2048 .
 - Increase the OPENTHREAD_IP6_MAX_EXT_MCAST_ADDRS to 40 .
 - Increase the NET_RX_STACK_SIZE to 2048 .

Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com>
Co-authored-by: Damien Ji <yinghao.ji@telink-semi.com>
 - Add OTA resume watchdog to avoid the question .

Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com>
- change the logic from the ScheduleFactoryReset() to earse NVS.

Signed-off-by: haiwen.xia <haiwen.xia@telink-semi.com>
* [Telink] Fix TC-OPCREDS-3.6 test

* Restyled by clang-format

* [Telink] Decrease CHIP_MAX_FABRICS for BOARD_TL3218X_RETENTION.

* [Telink] Fix default value for BOARD_TL3218X_RETENTION

---------

Co-authored-by: Restyled.io <commits@restyled.io>
…ice.

- fix incorrect factory reset logic for commissioned device when fabric is 0.

Signed-off-by: Hao Wu <hao.wu@telink-semi.com>
… initialization (observed on NXP Zephyr platform) (project-chip#71368) (project-chip#463)

* [common][platform][openthread] Add OT mutex lock when calling OT api to avoid race condition during initialization

    * The mutex lock is necessary for Zephyr because the openthread thread is initialized by the kernel init, and therefore it runs before the Matter initialization
    * so when we InitThreadStack, we must ensure that the OT api calls are protected with a mutex lock to avoid concurrent access to OT shared resources



* Restyled by whitespace

* [common][platform][openthread] Add comment to explain why the LockThreadStack() is required



---------

Signed-off-by: Dina Benamar <dina.benamarelmaaroufi@nxp.com>
Co-authored-by: dinabenamar <108664279+dinabenamar@users.noreply.github.com>
Co-authored-by: Restyled.io <commits@restyled.io>
…RY (#3)

* Update Telink OpenThread Path Selection

fixes the incorrect OpenThread repository selection logic in Telink platform builds:

   - Explicitly set ZEPHYR_OPENTHREAD_MODULE_DIR to use standard openthread path
     when CONFIG_OPENTHREAD_TELINK_LIBRARY is not enabled
   - Replace the directory-existence check with CONFIG_OPENTHREAD_TELINK_LIBRARY
     based path selection
   - Ensures build logs always print the correct git revision information
     matching the actually used OpenThread repository

Now builds correctly distinguish between:
- CONFIG_OPENTHREAD_TELINK_LIBRARY=y: uses modules/lib/openthread_telink_lib
- CONFIG_OPENTHREAD_TELINK_LIBRARY=n: uses modules/lib/openthread

Signed-off-by: Damien Ji <yinghao.ji@telink-semi.com>

* Restyled by clang-format

- update zephyr revision to
3a3683860df05a6e9d441a66f50097c9c03258bb

Signed-off-by: Damien Ji <yinghao.ji@telink-semi.com>

---------

Signed-off-by: Damien Ji <yinghao.ji@telink-semi.com>
Co-authored-by: Restyled.io <commits@restyled.io>
* [Telink] Check if the thread dataset and fabirc were successfully written.

Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com>
…-chip#471)

 - Improve the commissioning flag process and maintain boundary security .
 - Copy a mcuboot.bin from build_mcuboot/.../zephyr.bin in CMakeLists .

Signed-off-by: Fengtai Xie <fengtai.xie@telink-semi.com>
@coderabbitai

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: a775405c-d613-411b-8a72-76c74dfac02d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@mergify

mergify Bot commented Sep 3, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

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.

9 participants