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

LINUX-STM32MP: fix 'cpuidle-stm32' for CONFIG_CC_OPTIMIZE_FOR_SIZE' #80

Open
wants to merge 1 commit into
base: kirkstone
Choose a base branch
from

Conversation

AdrianDC
Copy link

WARNING: modpost: vmlinux.o(.exit.text+0x1b24): Section mismatch in reference from the function stm32_cpuidle_driver_exit() to the (unknown reference) .init.data:(unknown) | The function __exit stm32_cpuidle_driver_exit() references | a (unknown reference) __initdata (unknown).
| This is often seen when error handling in the exit function | uses functionality in the init path.
| The fix is often to remove the __initdata annotation of | (unknown) so it may be used outside an init section. |
| ERROR: modpost: Section mismatches detected.
---

Thanks @BernardPuel and @patrickdelaunay for fixing the upstream ST kernel 6.1 through my original PR #54

However I'd be interested to know if fixing the kirkstone faulty .patch from both these PR is possible,
because one year later and being updated to all latest Yocto 4.0.20 layers + STM32MP kirkstone,
and ST kernel current version 5.15.145, this build fix still needs to be kept and maintained locally.

I understand the newest version versus maintenance branches priorities, and isolation between internal Gerrit and PRs,
but the kirkstone branch being LTS and required by some STM32MP1 SOM providers or projects aligned on this LTS,
it should allow minor build fixes as these that affect only the branch (and avoid others to hit & fix these locally again).

Thanks !

WARNING: modpost: vmlinux.o(.exit.text+0x1b24): Section mismatch in reference from the function stm32_cpuidle_driver_exit() to the (unknown reference) .init.data:(unknown)
| The function __exit stm32_cpuidle_driver_exit() references
| a (unknown reference) __initdata (unknown).
| This is often seen when error handling in the exit function
| uses functionality in the init path.
| The fix is often to remove the __initdata annotation of
| (unknown) so it may be used outside an init section.
|
| ERROR: modpost: Section mismatches detected.
---

Signed-off-by: Adrian DC <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant