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

s905-autoscript question #139

Open
graugans opened this issue Jun 7, 2022 · 5 comments
Open

s905-autoscript question #139

graugans opened this issue Jun 7, 2022 · 5 comments
Labels

Comments

@graugans
Copy link
Contributor

graugans commented Jun 7, 2022

From my understanding the packages s905-autoscript and s905-autoscript-multiboot are intended to support the vendor U-Boot boot process. Like described in the README

This layer aims to support all mainline supported boards, but some of them have a vendor U-Boot loaded on the eMMC and can only be booted by pressing the "update" button.

But when I do flash the image amlogic-image-headless-sd the file aml_autoscript is missing on the boot partition. This is added by amlogic-s9xxx.inc which is only included in amlogic-s9xxx.conf which looks like a base for boards which do run a mainline U-Boot and not a vendor one.

Or did I understood it wrong?

@superna9999
Copy link
Owner

Those are installed in

IMAGE_BOOT_FILES ?= "${KERNEL_IMAGETYPE} \

This boot scheme is selected in <board>-sdboot.conf machine names, I didn't finish adding the -sdboot.conf variants for all board.

@graugans
Copy link
Contributor Author

graugans commented Jun 7, 2022

Yes, you are right. I was only search half way. I am not sure if this is may related to my set-up but when checking the amlogic-image-headless-sd.env file in the image folder the variable IMAGE_BOOT_FILES only contains the device tree entry.

I will investigate this further.

@graugans
Copy link
Contributor Author

graugans commented Jun 7, 2022

Looks like the order is important.

require conf/machine/include/amlogic-legacy-boot.inc
require conf/machine/include/amlogic-s905x2.inc
require conf/machine/include/amediatech-x96-max-dtb.inc

works for me.

@superna9999
Copy link
Owner

Interesting, it doesn't make sense, could you provide a fix in a PR ?

@graugans
Copy link
Contributor Author

Sorry I was a few days offline, I will do more investigation. But this week will be loaded with other stuff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants