Hi Stephen, On Fri, Oct 26, 2012 at 9:23 AM, Stephen Warren <swar...@wwwdotorg.org> wrote: > On 10/26/2012 09:50 AM, Simon Glass wrote: >> Hi, >> >> On Fri, Oct 26, 2012 at 7:53 AM, Tom Rini <tr...@ti.com> wrote: On >> 10/26/12 00:10, Marek Vasut wrote: >>>>> Dear Joe Hershberger, >>>>> >>>>>> Hi Allen, >>>>>> >>>>>> On Thu, Oct 25, 2012 at 6:30 PM, Allen Martin >>>>>> <amar...@nvidia.com> wrote: >>>>>>> Remove calls to serial_assign() that are failing now that >>>>>>> it returns a proper error code. This calls were not >>>>>>> actually doing anything because they passed the name of a >>>>>>> stdio_dev when a serial_device name is exptectd. >>>>>>> >>>>>>> Signed-off-by: Allen Martin <amar...@nvidia.com> --- >>>>>> >>>>>> Acked-by: Joe Hershberger <joe.hershber...@ni.com> >>>>> >>>>> Acked-by: Marek Vasut <ma...@denx.de> >> >> And this indeed fixes Tegra, right? >> >>> It fixes the serial hang we discussed. I think Tegra still has >>> the SPL problem that Stephen reported (the need for >>> serial_initialize()), or at least I haven't seen a patch for >>> that. > > Allen posted 3 patches to rework Tegra's SPL: > > tegra20: initialize variable to avoid compiler warning > tegra: move to common SPL framework > SPL: make jump_to_image_no_args a weak symbol
OK thank you, I might have missed these, perhaps wasn't copied. I just tried them out and it indeed fixes the SPL serial problem. > > I also posted 5 patches to solve an oversized-SPL issue on Seaboard > and Ventana: > > ARM: tegra: don't request GPIO from Seaboard's SPL > ARM: tegra: select between Seaboard/Ventana at compile time > ARM: tegra: derive CONFIG_SPL_MAX_SIZE instead of hard-coding it > ARM: enhance u-boot.lds to detect over-sized SPL > ARM: fix u-boot.lds for -ffunction-sections/-fdata-sections > > I tested this current patch with all of those applied. Yes I got your ones ok. I tested with this patch with and without them. > >>> Tested on seaboard. >> >>> Acked-by: Simon Glass <s...@chromium.org> Tested-by: Simon Glass >>> <s...@chromium.org> > > This patch, > Tested-by: Stephen Warren <swar...@nvidia.com> Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
