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

@Thom747 Thom747 deleted the fix-freertos-port branch February 23, 2026 11:03
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