On Wed, 4 May 2011 16:30:00 -0700 Simon Glass <s...@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. -Scott _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot