I'm on vacation until Monday, 17 July. I can look at it then.
-- nvpublic
-----Original Message----- From: sjg@google.com [mailto:sjg@google.com] On Behalf Of Simon Glass Sent: Saturday, July 08, 2017 9:09 AM To: Thomas Hoff thomashoffltd@gmail.com Cc: Stephen Warren swarren@wwwdotorg.org; Marek Vasut marex@denx.de; Stephen Warren swarren@nvidia.com; Stefan Agner stefan.agner@toradex.com; Jagan Teki jagan@openedev.com; U-Boot Mailing List u-boot@lists.denx.de; Tom Warren TWarren@nvidia.com; Riley Baird riley@openmailbox.org Subject: Re: [U-Boot] [PATCH 00/12] tegra: nyan-big: Support chainloading and add documentation
Hi,
On 7 July 2017 at 01:27, Thomas Hoff thomashoffltd@gmail.com wrote:
Are you asking me or Tom Warren?
On Thu, Jul 6, 2017 at 8:58 PM, Simon Glass sjg@chromium.org wrote:
Hi Tom,
On 27 June 2017 at 14:09, Thomas Hoff thomashoffltd@gmail.com wrote:
Thanks a bunch Simon (and everyone else who contributed)!
I followed the instructions and was able to get uboot to boot successfully. :D
Now I just got to figure out how to get the kernel to boot in HYP mode.
Cheers,
Thomas
On Fri, Jun 9, 2017 at 5:27 AM, Simon Glass sjg@chromium.org wrote:
Hi Stephen,
On 7 June 2017 at 09:41, Stephen Warren swarren@wwwdotorg.org
wrote:
On 06/07/2017 09:32 AM, Tom Warren wrote: > > Simon, > >> From: Simon Glass
... >> >> On 1 June 2017 at 07:52, Matthew Gorski >> matt.gorski@gmail.com >> wrote: >>> >>> >>> On Wed, May 31, 2017 at 2:28 PM, Matthew Gorski >>> matt.gorski@gmail.com wrote: >>>> >>>> On Mon, May 22, 2017 at 7:17 AM, Simon Glass >>>> sjg@chromium.org> >>>> wrote: >>>>> >>>>> >>>>> Every now and then someone wants to chain-load U-Boot on a >>>>> Chromebook. >>>>> The procedure is not very complicated but there are some >>>>> oddities. >>>>> >>>>> This series updates a few things with allow U-Boot to start >>>>> on nyan-big in this way and adds documentation on how to do it. >>>>> This >>>>> provides a central place where the procedure on different >>>>> devices can be documented.
... >>>> >>>> Thank You for this Simon! Very much appreciated! I am >>>> going to give this a try later tonight and report back my >>>> personal results...
...
>>> Confirmed! Chain-loading working on my nyan_big. I >>> appreciate this very much Simon and all who contributed. >> >> >> I'd like to pull in this series. Are there any comments / reviews? >> If >> affects both >> tegra and rockchip. > > I'm fine with it - I assume you'll pull it in to one of your repos > and > post a PR to get it into mainstream U-Boot? > I don't have the HW to test on (my nyan-big has been shelved), but > trust > in Matthew's testing, and that Stephen's U-Boot test scripts will > catch > any > problems, so LGTM.
Thanks for looking at this.
Yes, my testing should catch most issues with any patches, although there are still some coverage holes:
- I don't test cold boot; U-boot is always loaded over USB RCM which
may hide bugs specific to cold booting.
I tested cold bold on Nyan, but I use USB RCM also so don't normally test cold boot.
- I only test on a few HW platforms (beaver, dalmore, Jetson TK1,
p2371-0000, Jetson TX1)
That's plenty :-)
- Obviously I don't test the new features introduced by this patch;
chain-loading.
It's OK I can rely on others for that so long as the changes are acceptable to the normal boot case.
Is this ready to be applied at some point?
Tom Warren - he is the Tegra maintainer.
Regards, Simon
Regards, Simon