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

rockchip-rk3588: rock-5b current, edge: u-boot: bump to mainline 2024.10-rc3 #7135

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

alexl83
Copy link
Contributor

@alexl83 alexl83 commented Aug 24, 2024

Description

Radxa Rock 5B

  • Bump current and edge to mainline 2024.10-rc3
  • ensure ARMV8_CRYPTO is disabled (currently broken)
  • enable creation of /chosen/kaslr-seed node (to enable Kernel address space layout randomization)

How Has This Been Tested?

  • Built, installed, and succesfully booted trixie current and edge

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings

….10-rc3

 - ensure ARMV8_CRYPTO is disabled (currently broken)
 - enable creation of /chosen/kaslr-seed node (to enable Kernel address space layout randomization
@github-actions github-actions bot added size/small PR with less then 50 lines Hardware Hardware related like kernel, U-Boot, ... labels Aug 24, 2024
@alexl83 alexl83 requested a review from EvilOlaf August 25, 2024 12:01
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.

Alright. However cannot approve since out of scope. Better let Efe or Fate have a look.

@igorpecovnik igorpecovnik added 11 Milestone: Fourth quarter release Needs review Seeking for review labels Sep 15, 2024
@@ -27,16 +27,39 @@ function post_family_tweaks__rock5b_naming_audios() {
return 0
}

# Mainline u-boot or Kwiboo's tree
# Mainline u-boot tree
Copy link
Member

Choose a reason for hiding this comment

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

I think it's better idea to add function without branch notation and use if-return in order to avoid code duplication

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
11 Milestone: Fourth quarter release Hardware Hardware related like kernel, U-Boot, ... Needs review Seeking for review size/small PR with less then 50 lines
Development

Successfully merging this pull request may close these issues.

4 participants