Skip to content

Comments

Fix linking error on FreeRTOS when using portYIELD_FROM_ISR#413

Merged
MichalPrincNXP merged 1 commit intoEmbeddedRPC:mainfrom
Thom747:fix-freertos-port
Feb 20, 2026
Merged

Fix linking error on FreeRTOS when using portYIELD_FROM_ISR#413
MichalPrincNXP merged 1 commit intoEmbeddedRPC:mainfrom
Thom747:fix-freertos-port

Conversation

@Thom747
Copy link
Contributor

@Thom747 Thom747 commented Mar 5, 2024

Pull request

Choose Correct

  • bug
  • feature

Describe the pull request

The portYIELD_FROM_ISR() macro declares a variable called ulPortYieldRequired.
FreeRTOS assumes ulPortYieldRequired is in the global namespace.
To prevent linking errors, this PR introduces a function in the global namespace called ErpcPortYieldFromISR that then calls portYIELD_FROM_ISR().

To Reproduce

Build eRPC with FreeRTOS.

Expected behavior

eRPC builds as normal.

Desktop (please complete the following information):

  • OS: Ubuntu 20.04
  • eRPC Version: v1.12.0 and earlier

Steps you didn't forgot to do

  • I checked if other PR isn't solving this issue.
  • I read Contribution details and did appropriate actions.
  • PR code is tested.
  • PR code is formatted.
  • Allow edits from maintainers pull request option is set (recommended).

@Thom747 Thom747 force-pushed the fix-freertos-port branch from f35233b to 8fc7a47 Compare March 5, 2024 12:00
@Thom747
Copy link
Contributor Author

Thom747 commented Mar 27, 2025

@Hadatko or @MichalPrincNXP This PR has not been handled for over a year now. Could one of you pick this up?

@Thom747
Copy link
Contributor Author

Thom747 commented Jan 16, 2026

@Hadatko or @MichalPrincNXP I am once again requesting that you look at this PR. It has not been handled for almost 2 years now and is still relevant.

@MichalPrincNXP MichalPrincNXP changed the base branch from develop to main February 20, 2026 15:31
@MichalPrincNXP MichalPrincNXP self-requested a review February 20, 2026 15:31
@MichalPrincNXP MichalPrincNXP self-assigned this Feb 20, 2026
@MichalPrincNXP MichalPrincNXP merged commit 9967e4a into EmbeddedRPC:main Feb 20, 2026
8 checks passed
@MichalPrincNXP
Copy link
Member

thank you, I am sorry for the delay

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

Labels

Development

Successfully merging this pull request may close these issues.

2 participants