[U-Boot] [PATCH 1/2] NS16550: buffer reads

Simon Glass sjg at chromium.org
Wed Oct 12 23:23:37 CEST 2011


Hi Scott,

On Fri, May 6, 2011 at 1:28 PM, Scott Wood <scottwood at freescale.com> wrote:
> On Wed, 4 May 2011 16:30:00 -0700
> Simon Glass <sjg at chromium.org> wrote:
>
>> Hi Scott,
>>
>> This is a very useful patch and it works well. I have taken the liberty of
>> modifying it slightly, because I think you should subtract 1 from the port
>> number that you pass to NS16550. For some reason the 'COM' ports are
>> numbered from 1 instead of 0.
>
> Thanks for spotting!
>
>>Please see below, and sorry if the patch doesn't come through cleanly.
>
> The whitespace got mangled.
>
> There's another thing that needs to be fixed -- the signatures
> of the 16550 functions should not change when the feature is disabled (see
> the discussion on http://patchwork.ozlabs.org/patch/90066/), or it'll
> break boards like linkstation.
>
> I'll include both fixes in my respin.

Are you planning a respin of this? If not I will take this up as it is
pretty important for Tegra.

Regards,
Simon


>
> -Scott
>
>


More information about the U-Boot mailing list