Subject: [bug#77148] [PATCH 1/9] gnu: Add u-boot-qemu-ppce550.

From: Maxim Cournoyer <[email-masked]>
Date: Fri, 21 Mar 2025 23:12:07 +0900

Hi,

Efraim Flashner <[email-masked]> writes:

> * gnu/packages/bootloaders.scm (u-boot-qemu-ppce550): New variable.
>
> Change-Id: I92ec83741a90dc9e0c3bdcbeeab0ff5eb47024bf
> ---
>  gnu/packages/bootloaders.scm | 22 +++++++++++++++++++++-
>  1 file changed, 21 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/bootloaders.scm b/gnu/packages/bootloaders.scm
> index fcdd3abbba1..8678e1c480a 100644
> --- a/gnu/packages/bootloaders.scm
> +++ b/gnu/packages/bootloaders.scm
> @@ -6,7 +6,7 @@
>  ;;; Copyright © 2016-2018, 2021-2023 Marius Bakke <[email-masked]>
>  ;;; Copyright © 2016, 2017 Danny Milosavljevic <[email-masked]>
>  ;;; Copyright © 2016, 2017 David Craven <[email-masked]>
> -;;; Copyright © 2017, 2018, 2020-2024 Efraim Flashner <[email-masked]>
> +;;; Copyright © 2017, 2018, 2020-2025 Efraim Flashner <[email-masked]>
>  ;;; Copyright © 2018–2022 Tobias Geerinckx-Rice <[email-masked]>
>  ;;; Copyright © 2019 nee <[email-masked]>
>  ;;; Copyright © 2019 Mathieu Othacehe <[email-masked]>
> @@ -1330,6 +1330,26 @@ (define-public u-boot-qemu-arm64
>                         ;; details.
>                         #:configs '("# CONFIG_FIT_SIGNATURE is not set")))
>  
> +(define-public u-boot-qemu-ppce550
> +  (let ((base
> +          (make-u-boot-package "qemu-ppce500" "powerpc-linux-gnu"
> +                               ;; Disable features that require OpenSSL due
> +                               ;; to GPL/Openssl license incompatibilities.
> +                               ;; See https://bugs.gnu.org/34717 for
> +                               ;; details.
> +                               #:configs '("# CONFIG_FIT_SIGNATURE is not set"))))

I thought the GPL incompatibility had been resolved in recent years (see
https://openssl-library.org/post/2017-03-20-license/) ?

> +    (package
> +      (inherit base)
> +      (arguments
> +       (substitute-keyword-arguments (package-arguments base)
> +         ((#:phases phases #~'%standard-phases)
> +          #~(modify-phases #$phases
> +              (replace 'install
> +                (lambda _
> +                  (mkdir-p (string-append #$output "/share/qemu"))
> +                  (copy-file "u-boot" (string-append
> +                                        #$output "/share/qemu/u-boot.e500")))))))))))

I'm not sure why we need a qemu variant; just having u-boot-ppce550 with
a libvirt/QEMU firmware descriptor should be enough, unless I'm missing
something.

-- 
Thanks,
Maxim