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

Update esp_cpu.h to include esp_attr.h (IDFGH-10113) #11388

Merged
merged 1 commit into from
May 21, 2023

Conversation

byteofsoren
Copy link
Contributor

If esp_attr.h is not included then there are no definitions for the symbol 'FORCE_INLINE_ATTR'.

If esp_attr.h is not included then there are no definitions for the symbol 'FORCE_INLINE_ATTR'.
@espressif-bot espressif-bot added the Status: Opened Issue is new label May 11, 2023
@github-actions github-actions bot changed the title Update esp_cpu.h to include esp_attr.h Update esp_cpu.h to include esp_attr.h (IDFGH-10113) May 11, 2023
@ESP-Marius
Copy link
Collaborator

@byteofsoren Thanks for the fix, LGTM!

@ESP-Marius
Copy link
Collaborator

sha=004d55a5ed97a0c5af6f863bf6098885e8f8a4eb

@ESP-Marius ESP-Marius added the PR-Sync-Merge Pull request sync as merge commit label May 12, 2023
@CLAassistant
Copy link

CLAassistant commented May 12, 2023

CLA assistant check
All committers have signed the CLA.

@byteofsoren
Copy link
Contributor Author

No problem, glad to help!

@espressif-bot espressif-bot added Status: Done Issue is done internally Resolution: NA Issue resolution is unavailable and removed Status: Opened Issue is new labels May 16, 2023
@espressif-bot espressif-bot merged commit 0036e55 into espressif:master May 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR-Sync-Merge Pull request sync as merge commit Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants