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

uboot: add Olimex A64 OLinuXino #156034

Merged
merged 1 commit into from
Jan 22, 2022
Merged

Conversation

yurkobb
Copy link
Contributor

@yurkobb yurkobb commented Jan 21, 2022

Motivation for this change
  • The desire to boot NixOS on A64-OLinuXino. Was able to boot successfully thanks to advice on the matrix room from @samueldr.
Things done
  • Built on platform(s)
    • x86_64-linux (built for aarch64)
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 22.05 Release Notes (or backporting 21.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@ofborg ofborg bot added the 8.has: package (new) This PR adds a new package label Jan 21, 2022
@ofborg ofborg bot requested review from lopsided98, samueldr and dezgeg January 21, 2022 09:23
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux labels Jan 21, 2022
Copy link
Member

@samueldr samueldr left a comment

Choose a reason for hiding this comment

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

I'm giving the chance for others to chime in about this question:

Does it matter that this builds only the emmc defconfig?

I assume that even without eMMC, it would work just fine for end-users. The only difference would be the availability of a non-working boot entry... Which would be just as annoying as an SD card boot entry when you don't have an SD card inserted. Which means: not annoying.

@yurkobb
Copy link
Contributor Author

yurkobb commented Jan 22, 2022 via email

@samueldr
Copy link
Member

Then my assumptions hold. Thanks for confirming.

The boot entries don't have a menu to pick from, in U-Boot. It's a semi-baked-in list of targets it tries to boot from.

@samueldr samueldr merged commit 7749ab4 into NixOS:master Jan 22, 2022
@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/arm-olimex-a64-olinuxino/17205/3

@skyrgobbler
Copy link

When trying to build uboot I get a failure due to scp.bin not existing...

error: builder for '/nix/store/fv5n8y277mxgh7sn895njvxs3hwyz49n-uboot-a64-olinuxino-emmc_defconfig-aarch64-unknown-linux-gnu-2023.01.drv' failed with exit code 2;
       last 10 log lines:
       >   DTC     arch/arm/dts/sun50i-a64-teres-i.dtb
       >   SHIPPED dts/dt.dtb
       >   CAT     u-boot-dtb.bin
       >   MKIMAGE u-boot.img
       >   COPY    u-boot.dtb
       >   MKIMAGE u-boot-dtb.img
       >   COPY    u-boot.bin
       >   BINMAN  .binman_stamp
       > binman: Filename 'scp.bin' not found in input path (.,.,./board/sunxi,arch/arm/dts) (cwd='/build/u-boot-2023.01')
       > make: *** [Makefile:1114: .binman_stamp] Error 1
       For full logs, run 'nix log /nix/store/fv5n8y277mxgh7sn895njvxs3hwyz49n-uboot-a64-olinuxino-emmc_defconfig-aarch64-unknown-linux-gnu-2023.01.drv'.

(full log at log.log

This happens for emulated compilation or cross compilation. Any ideas?

@samueldr
Copy link
Member

samueldr commented Aug 4, 2023

Hi, first off, please open a new issue and ping relevant people for new problems not related to a given PR or issue.


Hm right, the U-Boot update in #227947 #218816 was merged without building all variants.

I suspect all allwinner boards will have this issue. add SCP = "/dev/null"; to the attributes given to that build, e.g. right under BL31. (Or provide the proper SCP [crust] build output.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.has: package (new) This PR adds a new package 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants