search

eclipse-threadx / threadx

Eclipse ThreadX is an advanced real-time operating system (RTOS) designed specifically for deeply embedded applications.
https://github.com/eclipse-threadx/rtos-docs/blob/main/rtos-docs/threadx/index.md
MIT License
2.87k stars 782 forks source link

Cortex-M55 ac6: Compilation error due to arm_compat.h include #217

Closed jite closed 1 year ago

jite commented 1 year ago

I'm trying to compile latest master of threadx (as static library) for cortex-m55, using armclang version 6+.

I'm running into compilation issues where the include of arm_compat.h (ports/cortex_m55/ac6/inc/tx_port.h) causes conflicts with CMSIS definitions for __enable_irq and __disable_irq.

Specifically in the cortex-m55 port I can not find that the use of arm_compat.h is necessary, removing the include solves the issue for me.

goldscott commented 1 year ago

Thanks for the report. We'll remove arm_compat.h from the include. This will be done in the next ThreadX release.

jite commented 1 year ago

Thanks. I suspect the same might be true for cortex-m85 and cortex-m33 (perhaps others too) as tx_port.h seems to be copy-paste. I have only tested cortex-m55 though so don't take my word on it.

goldscott commented 1 year ago

This has been fixed https://github.com/azure-rtos/threadx/commit/37f6d0b39c970b297c263cbb8388b9060b92e5c9