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

Rock 4C+: update and cleanup boot config #6414

Merged
merged 2 commits into from
Mar 25, 2024
Merged

Conversation

gerculanum
Copy link
Contributor

Description

Rock 4C+ image experienced instability during boot due to the bootloader initializing with an incorrect memory clock of 800MHz, instead of the maximum supported 666MHz.

This update incorporates a newer version of U-Boot that includes a default configuration for the 4C+ and specifies binaries for DDR and BL31.

How Has This Been Tested?

  • Compiled and verified default image builds for 4C+ on working hardware
  • Confirmed that bootloader sets memory clock to 666MHz
  • Confirmed stability of boots

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings

@github-actions github-actions bot added size/small PR with less then 50 lines Hardware Hardware related like kernel, U-Boot, ... labels Mar 21, 2024
@EvilOlaf
Copy link
Member

Please review linting

@gerculanum
Copy link
Contributor Author

Please review linting

Done

Use newer uboot that includes defconfig for 4C+.
Fixes unstable boots.
Copy link
Member

@EvilOlaf EvilOlaf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If its tested and works I am fine with

@igorpecovnik igorpecovnik added Ready to merge Reviewed, tested and ready for merge 05 Milestone: Second quarter release labels Mar 21, 2024
@igorpecovnik igorpecovnik merged commit fc98d2e into armbian:main Mar 25, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
05 Milestone: Second quarter release Hardware Hardware related like kernel, U-Boot, ... Ready to merge Reviewed, tested and ready for merge size/small PR with less then 50 lines
Development

Successfully merging this pull request may close these issues.

3 participants