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

kernel: show rk3588 vendor kernel overlays in overlays section #446

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

Conversation

efectn
Copy link
Member

@efectn efectn commented Feb 17, 2025

Description

Show rk3588 vendor kernel overlays which don't have prefix in overlay selection section.
Issue reference:
Related documentation:

Implementation Details

Provide a detailed description of the implementation. Include the following:

  • Key changes introduced by this PR
  • Justification for the changes
  • Confirmation that no new external dependencies or modules have been introduced

Documentation Summary

  • Metadata Included:
    Did you include the metadata (associative arrays) in the code? Ensure that metadata for modules, jobs, and runtime has been updated appropriately.

  • Document Generated:
    Did you generate the updated documentation using armbian-configng --doc? Confirm if the command was run to update README.md and provide any relevant details.

Testing Procedure

Describe the tests you ran to verify your changes. Provide relevant details about your test configuration.

  • Tested and confirmed working.

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have ensured that my changes do not introduce new warnings or errors
  • No new external dependencies are included
  • Changes have been tested and verified
  • I have included necessary metadata in the code, including associative arrays

@github-actions github-actions bot added the size/small PR with less then 50 lines label Feb 17, 2025
@@ -35,6 +35,23 @@ function manage_dtoverlays () {
else
available_overlays=$(ls -1 ${overlaydir}/${overlay_prefix}*.dtbo | sed 's/^.*\('${overlay_prefix}'.*\)/\1/g' | sed 's/'${overlay_prefix}'-//g' | sed 's/.dtbo//g')
fi

# Check the branch in case it is not available in /etc/armbian-release
if [[ -z "${BRANCH}" ]]; then
Copy link
Member Author

@efectn efectn Feb 17, 2025

Choose a reason for hiding this comment

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

Not sure about here. I've copied these codes from other part because i realized that BRANCH is not exists on /etc/armbian-release

Copy link
Member

@igorpecovnik igorpecovnik Feb 17, 2025

Choose a reason for hiding this comment

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

Yes, we determine BRANCH at boot time script. When kernel is changing via armbian-config, this might not be the case. Needs to be investigated.

Copy link
Member

@igorpecovnik igorpecovnik Feb 18, 2025

Choose a reason for hiding this comment

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

Move this part to tools/modules/functions/set_runtime_variables.sh It might be helpful for other functions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/small PR with less then 50 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants