Skip to content
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

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

Closed
jite opened this issue Nov 18, 2022 · 3 comments
Closed

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

jite opened this issue Nov 18, 2022 · 3 comments
Assignees
Labels
bug Something isn't working hardware New hardware or architecture support request

Comments

@jite
Copy link

jite commented Nov 18, 2022

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.

@jite jite added bug Something isn't working hardware New hardware or architecture support request labels Nov 18, 2022
@goldscott goldscott self-assigned this Nov 18, 2022
@goldscott
Copy link
Contributor

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

@jite
Copy link
Author

jite commented Nov 21, 2022

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
Copy link
Contributor

This has been fixed 37f6d0b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working hardware New hardware or architecture support request
Projects
None yet
Development

No branches or pull requests

2 participants