unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
From: Pierre Langlois <pierre.langlois@gmx.com>
To: 49550@debbugs.gnu.org
Cc: Vagrant Cascadian <vagrant@debian.org>
Subject: bug#49550: Enable to boot from emmc on rockpro64
Date: Tue, 27 Jul 2021 20:38:18 +0100	[thread overview]
Message-ID: <878s1rh7it.fsf@gmx.com> (raw)
In-Reply-To: <87bl6nh7yh.fsf@gmx.com>

[-- Attachment #1: Type: text/plain, Size: 4117 bytes --]

Whoops, I hadn't realised the bug email wasn't included in the email,
sorry!

Pierre Langlois writes:

> 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.
>
>
>
>>>                  (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


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 519 bytes --]

  reply	other threads:[~2021-07-27 19:39 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
2021-07-27 19:38     ` Pierre Langlois [this message]
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

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=878s1rh7it.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 public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).