-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
fix: firefly-itx-3588j fails to boot, and audio output #6849
Conversation
...boot/legacy/u-boot-radxa-rk3588/board_firefly-itx-3588j/uboot-firefly-itx-3588j-config.patch
Outdated
Show resolved
Hide resolved
Try this : #6851 |
Thanks!
|
You're welcome! Thanks for testing mainline. I think with the patch in the u-boot patch fodler you will actually change something for the Rock 5A as well, which is not intended. I know it's in its own boot folder, but last time I checked, the boot folder is just cosmetic (to know which patches are from which boards so they can be moved if the board moves) and the patcher will take all patches from all subfolders for all boards. Please try the current board config with |
|
it didn't happened, so I assume keeping the patch won't affect rock-5a? |
Here comes
|
Nice! So it works with |
yeah, works without the patch. But I don't want delete it. |
Why? I don't think I understand 😄 |
It's uncomfort to see a wrong name at every boot, and the patch does not affect the rock-5x boards, so I prefer keeping it. |
I can understand :) In this case, the patch is better suited in your |
But... why? Is because of
I build tested, it did not apply the patch to |
No, not because of that, my previous assumption was not correct, I apologize.
Btw, "Rockchip RK3588 Evaluation Board" is also not the name of the board, it is another board, the rk3588 evb If you like, you can create a customized defconfig for your board though :) |
the vendor u-boot name was RK3588 Evb as we mentioned in #6849 (comment) Apologize for misunderstanding, but I confirmed the board's original u-boot name was rk3588 Evaluation Board, and the board layout was nearly same as the |
Ah now I understand what you mean, you mean the uboot it ships with. They are also just using the rk3588_defconfig :) Try removing this in the rk3588_defconfig: CONFIG_OPTEE_CLIENT=y |
I only removed
|
Hangon I got a mysterious git problem |
Sounds great if that works! Armbian doesn't use Optee in any way so it doesn't make a difference, but if only removing the one line works, great 👍
We've alll been there at some point, don't worry 😄 |
Thanks for encouragement and patience helping me with the patch :) |
Sure! I'm glad that everything works now. If you want to go the extra mile, you could rename the patch to something like I'll let you think about it :) If you don't want to walk the extra mile, it's approved like it currently is 👍 |
why not? here you go! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you aren't annoyed by empty patch subject when patching, then let's go 👍
Thanks for your patience and testing different stuff.
oh I forgot that, sry |
It's fine you don't need to change it 😄 |
Done, everything goes on track as expected and wishes |
Description
Change to mainline uboot, fixed not booting issue, added hdmi0 audio interface alias.
How Has This Been Tested?
./compile.sh build BOARD=firefly-itx-3588j BRANCH=vendor BUILD_DESKTOP=yes BUILD_MINIMAL=no DESKTOP_APPGROUPS_SELECTED='browsers internet' DESKTOP_ENVIRONMENT=gnome DESKTOP_ENVIRONMENT_CONFIG_NAME=config_base KERNEL_CONFIGURE=no RELEASE=bookworm
Checklist:
Please delete options that are not relevant.