Skip to content

arch/tricore: add support for tc4xx series chips and boards#18391

Merged
jerpelea merged 7 commits intoapache:masterfrom
zhangyu-duck:tricore_tc4_board
Mar 10, 2026
Merged

arch/tricore: add support for tc4xx series chips and boards#18391
jerpelea merged 7 commits intoapache:masterfrom
zhangyu-duck:tricore_tc4_board

Conversation

@zhangyu-duck
Copy link

@zhangyu-duck zhangyu-duck commented Feb 13, 2026

Note: Please adhere to Contributing Guidelines.

Summary

Two main updates:

  1. Uses the official Infineon ILLD repository instead of a private repository.
  2. Adds support for tc4xx series chips and boards

Impact

support tc4xx chip

Testing

  1. Both cmake and makefile methods can compile tc3xx/tc4xx.
  2. pass ostest, basic functions are normal

nsh> hello
Hello, World!! tc4 ostest
nsh>

i give tc3 and tc4 ostest.log in attach files
tc4_ostest.log
tc3_ostest.log

@github-actions github-actions bot added Arch: tricore Issues related to the TriCore architecture from Infineon Board: tricore Size: XL The size of the change in this PR is very large. Consider breaking down the PR into smaller pieces. labels Feb 13, 2026
@zhangyu-duck zhangyu-duck force-pushed the tricore_tc4_board branch 2 times, most recently from 745ed4d to 4edecbe Compare February 13, 2026 08:49
@jerpelea jerpelea changed the title Tricore tc4 board arch/tricore: add support for tc4xx series chips and boards Feb 13, 2026
Copy link
Contributor

@jerpelea jerpelea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please document all non Apache license file to
https://github.com/apache/nuttx/blob/master/LICENSE

Copy link
Contributor

@jerpelea jerpelea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add SPDX license identifiers to all files not only Apache licensed ones

@zhangyu-duck
Copy link
Author

please add SPDX license identifiers to all files not only Apache licensed ones

ok,thanks, i have updata code

@zhangyu-duck
Copy link
Author

please add SPDX license identifiers to all files not only Apache licensed ones

get it, thanks

Copy link
Contributor

@acassis acassis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@zhangyu-duck please update the Documentation https://nuttx.apache.org/docs/latest/platforms/tricore/index.html to include these new chips.

Unfortunately our Documentation is very non-standardized, some Documentation arch are very good, others are very primitive (let say not helpful). So, I suggest you to use https://nuttx.apache.org/docs/latest/platforms/arm/imxrt/index.html as a good reference.

Copy link
Contributor

@linguini1 linguini1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess since this modifies the TC3xx logic as well, could you please include an OSTest log from a tc3 board? The tc4 log looks good.

@linguini1
Copy link
Contributor

Also I won't block the PR about this since I have pretty much no familiarity with the tricore architecture, but is it necessary to use the SDK? It seems like this PR needs to include a lot of workarounds for quirks with the SDK (like the libc stubs and patch files). Is it at all possible to avoid using the SDK directly? I know a few other archs in NuttX use this approach (hence why I don't think it's a dealbreaker or anything), but I also know the preferred method is to avoid using vendor SDKs/HALs entirely if possible.

@wangchdo
Copy link
Contributor

Hi zhangyu-duck,

Great job!

My suggestion is to leave the iLLD unchanged and only configure it using the IFX_XXX macros.

For your reference, I have already ported NuttX to TC4xN based on iLLD in our internal project, and this port does not modify the iLLD itself.

@zhangyu-duck
Copy link
Author

zhangyu-duck commented Feb 26, 2026

Hi zhangyu-duck,

Great job!

My suggestion is to leave the iLLD unchanged and only configure it using the IFX_XXX macros.

For your reference, I have already ported NuttX to TC4xN based on iLLD in our internal project, and this port does not modify the iLLD itself.

Since infineon have open-source plans, I recommend using the ILD library they provide on GitHub.
and what we change for illd are almost bugs, so i think it may ok

@linguini1
Copy link
Contributor

Can you please perform a test on a TV3 device as well since both are modified?

@zhangyu-duck
Copy link
Author

Can you please perform a test on a TV3 device as well since both are modified?

ok, i give tc3

Can you please perform a test on a TV3 device as well since both are modified?

i privoid

Can you please perform a test on a TV3 device as well since both are modified?

ofcourse, i will supply it

add new chip named tc4xx and board tc4evb

Signed-off-by: zhangyu117 <zhangyu117@xiaomi.com>
@linguini1 linguini1 dismissed their stale review March 3, 2026 14:59

TC3 test provided.

Copy link
Contributor

@anchao anchao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jerpelea jerpelea merged commit 72be575 into apache:master Mar 10, 2026
41 checks passed
@jerpelea
Copy link
Contributor

@linguini1 sorry I did not see your comment

@linguini1
Copy link
Contributor

@linguini1 sorry I did not see your comment

It's okay, my comment was resolved!


config HAVE_SECURITY_CORE
bool
--help--
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
--help--
---help---

This is causing NuttX Builds to fail, lemme submit the patch.
https://github.com/lupyuen/nuttx-riscv64/actions/runs/22929878852/job/66548808341#step:5:161

arch/tricore/Kconfig:124: syntax error
arch/tricore/Kconfig:123: unknown option "--help--"
arch/tricore/Kconfig:124:warning: ignoring unsupported character ','
arch/tricore/Kconfig:124: unknown option "In"

@lupyuen
Copy link
Member

lupyuen commented Mar 11, 2026

Patch for Kconfig is here:

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

Labels

Arch: tricore Issues related to the TriCore architecture from Infineon Area: Documentation Improvements or additions to documentation Board: tricore Size: XL The size of the change in this PR is very large. Consider breaking down the PR into smaller pieces.

Projects

None yet

Development

Successfully merging this pull request may close these issues.