configng icon indicating copy to clipboard operation
configng copied to clipboard

[Bug]: Only two overlays on armbian-config for OrangePi 5 with clean installation

Open Robolightning opened this issue 9 months ago • 12 comments

What happened?

In armbian-config overlays only two positions are displayed: [ ] opp-oc-24ghz [ ] panthor-gpu

The rest of the overlays work if they are added via /boot/armbianEnv.txt directly, in the /boot/ folder all the necessary overlays are there. The problem is in armbian-config. I have already tried several different Armbian Ubuntu 24 images and on the Linux kernel version 6.

It is not caused by my settings or packages. It happens even with a clean installation of the system (Armbian Ubuntu 24 minimal on OrangepPi 5)

Image

How to reproduce?

Clean install system (Armbian Ubuntu 24 minimal) on OrangepPi 5 and go to the armbian-config > System > Kernel > Manage devicetree overlays

On which OS are you observing this problem?

Armbian Ubuntu 24.04 Noble

Code of Conduct

  • [x] I agree to follow this project's Code of Conduct

Robolightning avatar Apr 10 '25 15:04 Robolightning

prefixes are wrong. configng is looking for rockchip-rk3588 and only finds those two since they have this prefix in file name. All generic overlays start with rk3588 only and therefore not visible. Either prefixes should be unified in build or configng adjusted to show these generics too.

EvilOlaf avatar Apr 10 '25 16:04 EvilOlaf

prefixes are wrong. configng is looking for rockchip-rk3588 and only finds those two since they have this prefix in file name. All generic overlays start with rk3588 only and therefore not visible. Either prefixes should be unified in build or configng adjusted to show these generics too.

Is this a problem on Armbian side, Armbian config or OrangePi/Rockchip developers side?

For my local solution, should I rename the prefixes on the overlays or change the prefix in armbianEnv.txt? Or edit the armbian-config code to add support for both types of prefixes for rk-3588?

Robolightning avatar Apr 10 '25 16:04 Robolightning

Might be an issue of both Armbian and armbian-config. Former because inconsistent overlay naming and latter for not detecting such 😁 . Jokes aside, not my decision to determine this.

Anyway, if you don't need armbian-config, simply edit /boot/armbianEnv.txt manually to enable/disable overlays by adjusting overlays= to your needs. If you need an example, simply enable like the panthor overlay and check how /boot/armbianEnv.txt looks afterwards. overlay_prefix may need adjustment, depending on used overlays. As for myself I do the not recommended way and rename the dtb files so they match up 🙈

EvilOlaf avatar Apr 10 '25 16:04 EvilOlaf

Might be an issue of both Armbian and armbian-config. Former because inconsistent overlay naming and latter for not detecting such 😁 . Jokes aside, not my decision to determine this.

Anyway, if you don't need armbian-config, simply edit /boot/armbianEnv.txt manually to enable/disable overlays by adjusting overlays= to your needs. If you need an example, simply enable like the panthor overlay and check how /boot/armbianEnv.txt looks afterwards. overlay_prefix may need adjustment, depending on used overlays. As for myself I do the not recommended way and rename the dtb files so they match up 🙈

Are you sure that Armbian itself generates overlays during assembly, and does not take them automatically from a third party repository (OrangePi/Rockchip), taking the names "as is"?

Robolightning avatar Apr 10 '25 16:04 Robolightning

Depends. In this particular case the vendor kernel source is maintained by Armbian, therefore it is an Armbian issue. https://github.com/armbian/linux-rockchip

EvilOlaf avatar Apr 10 '25 17:04 EvilOlaf

Depends. In this particular case the vendor kernel source is maintained by Armbian, therefore it is an Armbian issue. https://github.com/armbian/linux-rockchip

Ok, then it is reasonable to create an issue there to rename overlays and overlay_prefix and consider that there is no problem on the armbian-config side?

Robolightning avatar Apr 10 '25 17:04 Robolightning

Again, depends. If the consest is to prefix all overlays with rockchip-rk3588 then there is no issue in configng. The other way around (prefix all with rk3588 only) however there is since those are not detected and that is actually the majority atm.

@amazingfate tl;dr: What's your opinion? Should rk3588 overlays file names start with the prefix rockchip-rk3588 or rk3588 only? At the moment we have both causing issues.

@Tearran I see you are aware of the situation. Just want to state the obvious for anyone else: Depending on outcome from above a minor adjustment in configng might be necessary.

EvilOlaf avatar Apr 11 '25 06:04 EvilOlaf

Both topics are duplicates: #505 and #523.

@Robolightning As a reminder from your last issue regarding overlays:

It possible you are looking for the "device tree overlay". for i2c, URT, spi, etc?
Kernel > SY210 - Manage device tree overlays
Image Beyond that this would be more likely a "feature request"
NOTE: Not a bug

What level of responsibility are you willing to assume as an image-space user? Are you prepared to make a proper feature request, considering the favor you are asking volunteers to address, or will you continue searching for someone else to take responsibility?

Tearran avatar Apr 11 '25 15:04 Tearran

Both topics are duplicates: #505 and #523.

@Robolightning As a reminder from your last issue regarding overlays:

It possible you are looking for the "device tree overlay". for i2c, URT, spi, etc? Kernel > SY210 - Manage device tree overlays Image Beyond that this would be more likely a "feature request" NOTE: Not a bug

What level of responsibility are you willing to assume as an image-space user? Are you prepared to make a proper feature request, considering the favor you are asking volunteers to address, or will you continue searching for someone else to take responsibility?

The previous issue was wrong, I just couldn't find the right tab in the new config. At the moment, I solved the problem of the current issue locally, as discussed above, by renaming the overlays and the prefix for the overlays. It just seems to me that the problem should also be solved on the maintainers' side

Robolightning avatar Apr 11 '25 15:04 Robolightning

It just seems to me that the problem should also be solved on the maintainers' side

@Robolightning First Thank you for your time and perspective. That toppic is https://github.com/armbian/configng/issues/523.

As i see it the build developers have more pressing matters then to make sure a image-space convenience is made.

I solved the problem of the current issue locally, as discussed above, by renaming the overlays and the prefix for the overlays Again Thank you for your time and experience . out of respect for the time you have given

here is the line you have issue with. if [[ $BOARDFAMILY == "rockchip-rk3588" ]] && [[ $BRANCH == "vendor" ]]; then my quick guess at start would be if [[ $BOARDFAMILY == "rockchip-rk3588" || $BOARDFAMILY == "rk3588" ]] && [[ $BRANCH == "vendor" ]]; then

AI Suggested

if [[ "$BOARDFAMILY" =~ ^(rockchip-rk3588|rk3588)$ && "$BRANCH" == "vendor" ]]; then
    # Add support for rk3588 vendor kernel overlays
    # Place your logic here
fi

How you want to prosed is up to you! Do you want to continue to burden the developers with things tinkers, hobbyist. and AI can do or help with the prefix issue?

Tearran avatar Apr 11 '25 18:04 Tearran

@Robolightning Sorry timing was off there https://github.com/armbian/configng/pull/528

Tearran avatar Apr 11 '25 19:04 Tearran

@EvilOlaf and @amazingfate My apologies for interfering with your Conversation. I will step away.

Tearran avatar Apr 11 '25 20:04 Tearran