GNU bug report logs - #64068
[PATCH] gnu: Add u-boot-lichee-rv-dock.

Previous Next

Package: guix-patches;

Reported by: Z572 <873216071 <at> qq.com>

Date: Wed, 14 Jun 2023 14:10:01 UTC

Severity: normal

Tags: patch

Full log


Message #11 received at 64068 <at> debbugs.gnu.org (full text, mbox):

From: Z572 <873216071 <at> qq.com>
To: Vagrant Cascadian <vagrant <at> debian.org>
Cc: Ludovic Courtès <ludo <at> gnu.org>,
 Efraim Flashner <efraim <at> flashner.co.il>, guix-patches <at> gnu.org,
 64068 <at> debbugs.gnu.org
Subject: Re: [bug#64068] [PATCH] gnu: Add u-boot-lichee-rv-dock.
Date: Sat, 17 Jun 2023 22:25:48 +0800
[Message part 1 (text/plain, inline)]
Vagrant Cascadian <vagrant <at> debian.org> writes:

> [[PGP Signed Part:Undecided]]
> On 2023-06-14, Z572 wrote:
>> * gnu/packages/bootloaders.scm (u-boot-lichee-rv-dock): New variable.
>> * gnu/packages/patches/u-boot-lichee-rv-dock-disable-openssl.patch: New file.
>> * gnu/local.mk(dist_patch_DATA): register it.
>
> Thanks for the patches!
>
> This also reminds me that u-boot is falling a little behind (there is
> 2023.01, 2023.04 and soon 2023.07)... and while I would not want to block
> adding support for this board, I would be curious if it still works with
> newer u-boot versions...

for now is not upstreamed.

>
>
>> +(define-public u-boot-lichee-rv-dock
> ...
>> +                  (search-patches
>> +                   "u-boot-lichee-rv-dock-disable-openssl.patch")
>> +                  ;;; rebase from %u-boot-allow-disabling-openssl-patch
>> +                  (filter (negate (cut string-contains <> "openssl"))
>> +                          (origin-patches (package-source base)))))
>
> This seems a little tricky and possibly error-prone if another patch
> with openssl in the name is included at a later time, it could break
> this package. I almost wonder if it wouldn't be better to merge the
> functionality of the two patches disabling openssl than applying a
> board-specific patch?

this new patch is modified from the original patch, because original
patch can't apply to smaeul/u-boot.

I attach a new patch.

>
>
> I have not had a chance to test that this package builds; I presume you
> have tested that it actually boots?

yes, u-boot can boot, but cann't boot guix system, i guess because
initrd is too big(even though the board has 1GB of RAM), initrd.cpio is 60M. so i send this patch first,
and try "remove initrd" or "make initrd small" in future .
(I'm not familiar with u-boot)

I attach some photo, scm file and boot log.

https://wiki.sipeed.com/hardware/en/lichee/RV/Dock.html

[guix-riscv.scm (text/plain, attachment)]
[u-boot.png (image/png, attachment)]
[截图 2023-06-18 00-08-55.png (image/png, attachment)]
[955455260.jpg (image/jpeg, attachment)]
[Message part 6 (text/plain, inline)]
>
>
> live well,
>   vagrant
>
> [[End of PGP Signed Part]]


[0001-gnu-Add-u-boot-lichee-rv-dock.patch (text/x-patch, attachment)]
[lichee-rv-dock-boot.log (text/plain, attachment)]
[Message part 9 (text/plain, inline)]
-- 
over

This bug report was last modified 2 years and 39 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.