[U-Boot] [PATCH 00/12] tegra: nyan-big: Support chainloading and add documentation

[Tom Warren]

I'm on vacation until Monday, 17 July. I can look at it then.

--
nvpublic

> -----Original Message-----
> From: sjg at google.com [mailto:sjg at google.com] On Behalf Of Simon Glass
> Sent: Saturday, July 08, 2017 9:09 AM
> To: Thomas Hoff <thomashoffltd at gmail.com>
> Cc: Stephen Warren <swarren at wwwdotorg.org>; Marek Vasut
> <marex at denx.de>; Stephen Warren <swarren at nvidia.com>; Stefan Agner
> <stefan.agner at toradex.com>; Jagan Teki <jagan at openedev.com>; U-Boot
> Mailing List <u-boot at lists.denx.de>; Tom Warren <TWarren at nvidia.com>;
> Riley Baird <riley at 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 at gmail.com> wrote:
> > Are you asking me or Tom Warren?
> >
> > On Thu, Jul 6, 2017 at 8:58 PM, Simon Glass <sjg at chromium.org> wrote:
> >>
> >> Hi Tom,
> >>
> >> On 27 June 2017 at 14:09, Thomas Hoff <thomashoffltd at 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 at chromium.org> wrote:
> >> >>
> >> >> Hi Stephen,
> >> >>
> >> >> On 7 June 2017 at 09:41, Stephen Warren <swarren at 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 at gmail.com>
> >> >> >>> wrote:
> >> >> >>>>
> >> >> >>>>
> >> >> >>>> On Wed, May 31, 2017 at 2:28 PM, Matthew Gorski
> >> >> >>>> <matt.gorski at gmail.com> wrote:
> >> >> >>>>>
> >> >> >>>>> On Mon, May 22, 2017 at 7:17 AM, Simon Glass
> >> >> >>>>> <sjg at 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:
> >> >> >
> >> >> > 1) 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.
> >> >>
> >> >> >
> >> >> > 2) I only test on a few HW platforms (beaver, dalmore, Jetson TK1,
> >> >> > p2371-0000, Jetson TX1)
> >> >>
> >> >> That's plenty :-)
> >> >>
> >> >> >
> >> >> > 3) 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