From: Arnaud Patard (Rtp) <arnaud.patard@rtp-net.org>
To: "Vincent Stehlé" <vincent.stehle@arm.com>
Cc: u-boot@lists.denx.de, Simon Glass <sjg@chromium.org>,
Philipp Tomsich <philipp.tomsich@vrull.eu>,
Kever Yang <kever.yang@rock-chips.com>,
Lukasz Majewski <lukma@denx.de>
Subject: Re: [BUG] USB boot issue on ROCKPro64 with UEFI
Date: Fri, 03 Sep 2021 19:02:13 +0200 [thread overview]
Message-ID: <87h7f14mp6.fsf@lechat.rtp-net.org> (raw)
In-Reply-To: <20210903152439.5778-1-vincent.stehle@arm.com> ("Vincent \=\?utf-8\?Q\?Stehl\=C3\=A9\=22's\?\= message of "Fri, 3 Sep 2021 17:24:38 +0200")
Vincent Stehlé <vincent.stehle@arm.com> writes:
Hi,
> Hi U-Boot folks,
>
> Hopefully this is the right way to report bugs. If not, please do not hesitate
> to let me know.
>
> I am hitting an issue with U-Boot v2021.07 on the ROCKPro64, when booting Linux
> with UEFI from USB. The kernel EFI stub will hang:
>
> EFI stub: Booting Linux Kernel...
> EFI stub: Using DTB from configuration table
> EFI stub: Exiting boot services and installing virtual address map...
>
> After tracking it down, it appears efi_exit_boot_services() is ultimately
> calling OHCI hc_reset(), which hangs at this line:
>
> 1804 if (ohci_readl(&ohci->regs->control) & OHCI_CTRL_IR) {
>
> This seems to indicate reading the OHCI hardware control register cannot
> complete, which hints at a clocking issue.
>
> Looking in more details at the clocks changes performed on the
> efi_exit_boot_services() path, a workaround is indeed to prevent the USB2PHY
> clocks from being disabled (see the patch following this e-mail).
>
> I don't know enough about the RK3399 clock tree to fully understand what is
> going on here, and what would be a proper fix. Hopefully others will be able to
> continue from there.
I may be wrong but seems like the issue solved by
https://patchwork.ozlabs.org/project/uboot/patch/20210406151059.1187379-1-icenowy@aosc.io/
Unfortunately, I think that there has been no progress since the
submission of this patch.
Arnaud
next prev parent reply other threads:[~2021-09-03 17:02 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-03 15:24 [BUG] USB boot issue on ROCKPro64 with UEFI Vincent Stehlé
2021-09-03 15:24 ` [PATCH WORKAROUND] clk: rk3399: do not disable the USB2PHY clk Vincent Stehlé
2021-09-03 17:02 ` Arnaud Patard [this message]
2021-09-03 17:30 ` [BUG] USB boot issue on ROCKPro64 with UEFI Vincent Stehlé
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=87h7f14mp6.fsf@lechat.rtp-net.org \
--to=arnaud.patard@rtp-net.org \
--cc=kever.yang@rock-chips.com \
--cc=lukma@denx.de \
--cc=philipp.tomsich@vrull.eu \
--cc=sjg@chromium.org \
--cc=u-boot@lists.denx.de \
--cc=vincent.stehle@arm.com \
/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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox