-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
use prep_c for xtensa to align with other architectures #75665
use prep_c for xtensa to align with other architectures #75665
Conversation
nashif
commented
Jul 9, 2024
- xtensa: remove mention of crt0-app.S
- xtensa: introduce prep_c for xtensa
- xtensa: adapt soc code to use prep_c
- xtensa: move arch_kernel_init code into prep_c
f5c33a3
to
63cd77b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SOF CI reports a FW exception in tests with this PR. From what I see, the tests on power flow from D3 are failing.
crt0-app.S does not exist, remove it from comments to avoid confusion. Signed-off-by: Anas Nashif <[email protected]>
xtensa is the only architecutre doing thing differently and introduces inconsistency in the init process and dependencies as we attemp to cleanup init levels and remove misused of SYS_INIT. Introduce prep_c for this architecture and align with other architectures. Signed-off-by: Anas Nashif <[email protected]>
Many xtensa target jump from soc code directly into cstart and depend on architecture code being initialized in arch_kernel_init(). Instead of jumping to cstart, jump to newly introduced prep_c similar to all other architectures, where common platfotm initialization will happen. Signed-off-by: Anas Nashif <[email protected]>
arch_kernel_init() was misused for all architecture initialization code that is done in prep_c and prior to cstart on other architectures. arch_kernel_init() is late in the init process and comes after EARLY init level, making xtensa have a very special boot path. Signed-off-by: Anas Nashif <[email protected]>
Restore order of execution. Code that was run in EARLY init level is now too late. Signed-off-by: Anas Nashif <[email protected]>
dbdcb4e
63cd77b
to
dbdcb4e
Compare
should be fixed now. |
@tmleman please have another look. |