Skip to content

Minor refactor of early init #547

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.

Sign up for GitHub

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

Jump to bottom
Open
crawfxrd wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Minor refactor of early init #547

crawfxrd wants to merge 2 commits into from

Conversation

@crawfxrd
Copy link
Member

@crawfxrd crawfxrd commented Dec 9, 2025 •
edited
Loading

Move early init logic out of app code and into SoC code.

  • Architecture (8051)
    • It initializes Timer 0 as the systick; other SoCs may use a different configuration
  • GCTRL,ECPM
    • These are ITE system block; other SoCs may not have something called "GCTRL"/"ECPM"

@crawfxrd crawfxrd requested review from a team December 9, 2025 18:57
@crawfxrd crawfxrd force-pushed the dev/refactor-early-init branch 2 times, most recently from 383bed6 to af11194 Compare December 15, 2025 16:16
@crawfxrd crawfxrd changed the title Refactor early init Minor refactor of early init Dec 16, 2025
@crawfxrd crawfxrd force-pushed the dev/refactor-early-init branch from af11194 to 1ddbfac Compare December 16, 2025 20:49
@crawfxrd crawfxrd force-pushed the dev/refactor-early-init branch from 1ddbfac to 9ac7132 Compare January 7, 2026 20:14
@crawfxrd
Move arch init from common to SoC
c693a46 
Since the architecture is very much SoC-specific, have it perform any
needed configurations.

Signed-off-by: Tim Crawford <tcrawford@system76.com>
@crawfxrd
Move GCTRL,ECPM init from common to SoC code
90afad8 
GCTRL and ECPM are ITE system blocks. Move them out of common code.

Signed-off-by: Tim Crawford <tcrawford@system76.com>
@crawfxrd crawfxrd force-pushed the dev/refactor-early-init branch from 9ac7132 to 90afad8 Compare January 16, 2026 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jackpot51 jackpot51 jackpot51 approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@crawfxrd @jackpot51