-
Notifications
You must be signed in to change notification settings - Fork 123
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
feat: add support for ODroid HC4 #643
base: main
Are you sure you want to change the base?
Conversation
7d7655b
to
708d78e
Compare
6906bc6
to
1521732
Compare
1521732
to
f855b0c
Compare
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.
cleaned up some minor nits
/ok-to-test |
seems the build is failing. I guess the amlogic script needs something extra |
I believe the amlogic script is calling an amd64 binary, given that it is assumed that the build environment for Any chance the arm64 builder could be updated to support running amd64 binaries? It'll be slow when it's used, but for this particular use case it shouldn't really matter. |
we use dedicated arm64 builder for arm |
anyways what binary it's calling, there could be arm64 variant of it |
I would also check if the scrip is just creating a final binary blog with BL31 appended, then should be easy to change it to use dd |
another option is to build the u-boot for odroid in the amd64 section only https://github.com/siderolabs/pkgs/blob/main/u-boot/pkg.yaml#L20 in the |
Yes, I know. I'm wondering if that builder can be updated to include support for
It's calling aml_encrypt_g12a, which is vendored into the repo.
The script is signing the binaries in addition to other things. This isn't something that can be easily replaced with |
you can build the whole u-boot for odroid in the amd64 section, and use |
Right, but how do we handle this further down the chain? The It seems like it would be a lot cleaner to just enable |
|
Would this also make talos work on an Odroid C4 and Odroid N2+? |
@mirceanton The C4 and HC4 are identical with the exception of the SATA ports, so yes. I'm not as sure about the N2+, but I believe the answer is also yes. All of those boards have support in the upstream kernel, so it's mostly a question of packaging Talos and getting the bootloader situation sorted. |
Adds the kernel module required to ensure proper functioning of the ODroid HC4's GPU so that HDMI output works as expected. Adds the kernel module required to ensure proper functioning of the SATA ODroid HC4's SATA ports. Adds a u-boot variant for the ODroid HC4. This is not used directly, but rather flashed into the SPI of the unit so that it can boot Talos. Documentation on this process can be found in the Talos repo. Signed-off-by: Brandon McNama <[email protected]> Signed-off-by: Noel Georgi <[email protected]>
9f8de0a
to
8ac8e01
Compare
@DWSR this can be now moved to using an overlay. probably a new repo, since amlogic seems to be different from allwinner. See this: https://github.com/siderolabs/overlays/ |
Adds the kernel module required to ensure proper functioning of the
ODroid HC4's GPU so that HDMI output works as expected.
Adds the kernel module required to ensure proper functioning of the
ODroid HC4's SATA ports.
Adds a u-boot variant for the ODroid HC4. This is not used directly, but
rather flashed into the SPI of the unit so that it can boot Talos.
Documentation on this process can be found in the Talos repo.