Subject: Re: [bug#68530] [PATCH 2/2] gnu: Add u-boot-amlogic.

From: Vagrant Cascadian <[email-masked]>
Date: Wed, 17 Jan 2024 11:13:19 -0800

On 2024-01-17, Herman Rimm wrote:
> diff --git a/gnu/packages/bootloaders.scm b/gnu/packages/bootloaders.scm
> index 151b42210c..b354af06c8 100644
> --- a/gnu/packages/bootloaders.scm
> +++ b/gnu/packages/bootloaders.scm
> @@ -894,6 +894,8 @@ (define-public u-boot-tools
>             (lambda* (#:key make-flags test-target #:allow-other-keys)
>               (invoke "test/image/test-imagetools.sh")))
>           ;; Only run full test suite on x86_64 systems, as many tests
> +
> +
>           ;; assume x86_64.
>           ,@(if (string-match "^x86_64-linux"
>                               (or (%current-target-system)

Is this whitespace change needed for some reason?

> @@ -1134,6 +1136,14 @@ (define*-public (make-u-boot-sunxi64-package board triplet scp-firmware
>         (modify-inputs (package-inputs base)
>           (append arm-trusted-firmware-sun50i-a64))))))
>  
> +(define %u-boot-amlogic-description-64-bit
> +  "This is a common 64-bit build of U-Boot for all 64-bit capable Amlogic s905x
> +variants.")
> +
> +(define-public u-boot-amlogic
> +  (make-u-boot-package "p212" "aarch64-linux-gnu"
> +                       #:append-description %u-boot-amlogic-description-64-bit))
> +

This should be called u-boot-p212. There are numerous other potential
amlogic platforms...

Although, an even bigger issues is that amlogic platforms all to my
knowledge require some non-free components in order to sign the binaries
before installing them, see the documentation in u-boot sources:

  doc/board/amlogic/p212.rst

live well,
  vagrant