From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stephen Warren <swarren@wwwdotorg.org>
Date: Fri, 26 Oct 2012 10:23:32 -0600
Subject: [U-Boot] [PATCH] serial: remove calls to serial_assign()
In-Reply-To: <CAPnjgZ2H7m_VTWeSjwSyWMMiDhy8N0teYeWKoqU9Muz6cm8WGw@mail.gmail.com>
References: <1351207814-26591-1-git-send-email-amartin@nvidia.com>
	<CANr=Z=abvz--JZ9=qq8ODqDiRs-EMYhvzdgnEmPyXYaq1pJhDw@mail.gmail.com>
	<201210260910.44625.marex@denx.de> <508AA3E6.5060406@ti.com>
	<CAPnjgZ2H7m_VTWeSjwSyWMMiDhy8N0teYeWKoqU9Muz6cm8WGw@mail.gmail.com>
Message-ID: <508AB904.1000801@wwwdotorg.org>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 10/26/2012 09:50 AM, Simon Glass wrote:
> Hi,
> 
> On Fri, Oct 26, 2012 at 7:53 AM, Tom Rini <trini@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 
>>>>> <amartin@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 <amartin@nvidia.com> ---
>>>>> 
>>>>> Acked-by: Joe Hershberger <joe.hershberger@ni.com>
>>>> 
>>>> Acked-by: Marek Vasut <marex@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

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.

>> Tested on seaboard.
> 
>> Acked-by: Simon Glass <sjg@chromium.org> Tested-by: Simon Glass
>> <sjg@chromium.org>

This patch,
Tested-by: Stephen Warren <swarren@nvidia.com>