From: Pierre Langlois <pierre.langlois@gmx.com>
To: Vagrant Cascadian <vagrant@debian.org>
Cc: 49550@debbugs.gnu.org
Subject: bug#49550: Enable to boot from emmc on rockpro64
Date: Tue, 27 Jul 2021 20:28:54 +0100 [thread overview]
Message-ID: <87bl6nh7yh.fsf@gmx.com> (raw)
In-Reply-To: <87sg0eqszi.fsf@yucca>
[-- Attachment #1.1: Type: text/plain, Size: 3312 bytes --]
Hi Vagrant,
I just had a break-through with this! See inline.
Vagrant Cascadian writes:
> On 2021-07-13, Pierre Langlois wrote:
>> I'm afraid since commit 3a851d45576e046d696fcf35b34d57b2cd28ea49 [0]
>> I've not been able to boot from eMMC on the rockpro64 board, it freezes
>> before loading the kernel. Re-introducing the work-around fixed the
>> issue for me (see patch attached).
>>
>> Did we mean to remove the work-around? I'm wondering if there's
>> something I'm missing, or if I should instead boot from a USB device.
>
> I definitely tested this on pinebook-pro and probably just assumed it
> would also fix the issue on rockpro64...
>
>
>> I've also updated u-boot to 2021.07 but I'm seeing the same boot issue
>> so it doesn't appear to be fixed upstream :-/
>>
>> Anybody know of any other work around? If not, are you happy with the
>> patch attached?
>
> I'll try to test on rockpro64 and see if I can confirm the issue and
> your fix...
>
>
> live well,
> vagrant
>
>> From f3c647bb447706f465a3fb4b8d6e09cd089dbed9 Mon Sep 17 00:00:00 2001
>> From: Pierre Langlois <pierre.langlois@gmx.com>
>> Date: Sat, 8 May 2021 16:19:23 +0100
>> Subject: [PATCH 1/3] gnu: u-boot-rockpro64-rk3399: Disable USB boot.
>>
>> * gnu/packages/bootloaders.scm (u-boot-rockpro64-rk3399)[arguments]: Add
>> 'patch-rockpro64-config phase.
>> ---
>> gnu/packages/bootloaders.scm | 11 ++++++++---
>> 1 file changed, 8 insertions(+), 3 deletions(-)
>>
>> diff --git a/gnu/packages/bootloaders.scm b/gnu/packages/bootloaders.scm
>> index 742992a119..75705a27c1 100644
>> --- a/gnu/packages/bootloaders.scm
>> +++ b/gnu/packages/bootloaders.scm
>> @@ -12,7 +12,7 @@
>> ;;; Copyright © 2019 Mathieu Othacehe <m.othacehe@gmail.com>
>> ;;; Copyright © 2020 Björn Höfling <bjoern.hoefling@bjoernhoefling.de>
>> ;;; Copyright © 2018, 2019, 2020 Vagrant Cascadian <vagrant@debian.org>
>> -;;; Copyright © 2020 Pierre Langlois <pierre.langlois@gmx.com>
>> +;;; Copyright © 2020, 2021 Pierre Langlois <pierre.langlois@gmx.com>
>> ;;; Copyright © 2021 Vincent Legoll <vincent.legoll@gmail.com>
>> ;;; Copyright © 2021 Brice Waegeneire <brice@waegenei.re>
>> ;;;
>> @@ -931,11 +931,16 @@ to Novena upstream, does not load u-boot.img from the first partition.")
>> (substitute-keyword-arguments (package-arguments base)
>> ((#:phases phases)
>> `(modify-phases ,phases
>> + (add-after 'unpack 'patch-rockpro64-config
>> + ;; Fix regression in 2020.10 causing freezes on boot with USB boot enabled.
>> + ;; See https://gitlab.manjaro.org/manjaro-arm/packages/core/uboot-rockpro64/-/issues/4
>> + (lambda _
>> + (substitute* "configs/rockpro64-rk3399_defconfig"
>> + (("CONFIG_USE_PREBOOT=y") "CONFIG_USE_PREBOOT=n"))))
>> (add-after 'patch-rockpro64-config 'set-environment
Wait a minute, there was a bug here on master! The 'set-environment
phase runs after 'patch-rockpro64-config phase, it should be 'unpack.
This was the reason it was hanging on boot for me, it wasn't to do with
PREBOOT, ah-ha! :-D
That's nice, I was really confused about this issue, thinking I maybe
had a faulty board.
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 519 bytes --]
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-u-boot-rockpro64-rk3399-Enable-set-environment-p.patch --]
[-- Type: text/x-patch, Size: 1251 bytes --]
From 055d1d358c55811cafd4c56b919865b75e59d936 Mon Sep 17 00:00:00 2001
From: Pierre Langlois <pierre.langlois@gmx.com>
Date: Tue, 27 Jul 2021 20:15:55 +0100
Subject: [PATCH] gnu: u-boot-rockpro64-rk3399: Enable 'set-environment phase.
* gnu/packages/bootloaders.scm (u-boot-rockpro64-rk3399)[arguments]: Run
'set-environment phase after 'unpack instead of non-existent
'patch-rockpro64-config phase.
---
gnu/packages/bootloaders.scm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gnu/packages/bootloaders.scm b/gnu/packages/bootloaders.scm
index 742992a119..4f7a8853e8 100644
--- a/gnu/packages/bootloaders.scm
+++ b/gnu/packages/bootloaders.scm
@@ -931,7 +931,7 @@ to Novena upstream, does not load u-boot.img from the first partition.")
(substitute-keyword-arguments (package-arguments base)
((#:phases phases)
`(modify-phases ,phases
- (add-after 'patch-rockpro64-config 'set-environment
+ (add-after 'unpack 'set-environment
(lambda* (#:key inputs #:allow-other-keys)
(setenv "BL31" (string-append (assoc-ref inputs "firmware")
"/bl31.elf"))
--
2.32.0
[-- Attachment #3: Type: text/plain, Size: 591 bytes --]
>> (lambda* (#:key inputs #:allow-other-keys)
>> (setenv "BL31" (string-append (assoc-ref inputs "firmware")
>> - "/bl31.elf"))
>> - #t))
>> + "/bl31.elf"))))
>> ;; Phases do not succeed on the bl31 ELF.
>> (delete 'strip)
>> (delete 'validate-runpath)))))
>> --
>> 2.32.0
>>
>>
>>
>> [0]: https://lists.gnu.org/archive/html/guix-commits/2021-05/msg00085.html
Thanks,
Pierre
next prev parent reply other threads:[~2021-07-27 19:30 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-13 19:36 bug#49550: Enable to boot from emmc on rockpro64 Pierre Langlois
2021-07-16 13:38 ` Vagrant Cascadian
2021-07-27 19:28 ` Pierre Langlois [this message]
2021-07-27 19:38 ` Pierre Langlois
2021-07-31 11:38 ` Pierre Langlois
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87bl6nh7yh.fsf@gmx.com \
--to=pierre.langlois@gmx.com \
--cc=49550@debbugs.gnu.org \
--cc=vagrant@debian.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.