Skip to content

arch/tricore: fpu: drop iLLD dependency and fix Makefile - #20018

Open
zhangyu-duck wants to merge 2 commits into
apache:masterfrom
zhangyu-duck:fpu
Open

arch/tricore: fpu: drop iLLD dependency and fix Makefile#20018
zhangyu-duck wants to merge 2 commits into
apache:masterfrom
zhangyu-duck:fpu

Conversation

@zhangyu-duck

Copy link
Copy Markdown
Contributor

Summary

Drop iLLD dependency from TriCore FPU code and fix a missing closing paren in the arch/tricore/src Makefile.

  • Replace __mfcr/__mtcr intrinsics and iLD-provided register symbols with tricore_mfcr/tricore_mtcr and hardcoded register values, so FPU code no longer depends on IfxCpu_Intrinsics.h.
  • Add the missing ) in $(call DELFILE, ...) at arch/tricore/src/Makefile:241, which caused unterminated call to function "call" build error.

Impact

  • Files: arch/tricore/include/irq.h, arch/tricore/src/common/tricore_fpu.c, arch/tricore/src/Makefile.
  • No behavioral change to FPU initialization or context compare logic.
  • Fixes a Makefile syntax error that blocks the build.

Testing

  • Local nxstyle check (tools/checkpatch.sh -f) passes on both source files.
  • Build verified locally on TC4xx board config.

1. replace __mtcr/mfcr with arch's tricore_mtcr/tricore_mfcr, donot dependon illd
2. Simplify header files
3. define registers address self

Signed-off-by: zhangyu117 <zhangyu117@xiaomi.com>
error: Makefile:241: *** unterminated call to function "call": missing ")". Stop.

Signed-off-by: zhangyu117 <zhangyu117@xiaomi.com>
@github-actions github-actions Bot added Arch: tricore Issues related to the TriCore architecture from Infineon Size: S The size of the change in this PR is small labels Aug 31, 2026
@zhangyu-duck
zhangyu-duck marked this pull request as ready for review August 31, 2026 06:22
@github-actions

Copy link
Copy Markdown

MemBrowse Memory Report

No memory changes detected for:

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 Size: S The size of the change in this PR is small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants