[U-Boot] [PATCH v2 2/2] x86: edison: Add the rest of UARTs present on board

Andy Shevchenko andriy.shevchenko at linux.intel.com
Thu Feb 28 14:45:59 UTC 2019


On Thu, Feb 28, 2019 at 01:29:57PM +0200, Andy Shevchenko wrote:
> On Thu, Feb 28, 2019 at 04:21:48PM +0800, Bin Meng wrote:
> > On Thu, Feb 28, 2019 at 4:12 PM Andy Shevchenko
> > <andriy.shevchenko at linux.intel.com> wrote:
> > >
> > > Intel Edison has three UART ports, i.e.
> > >  port 0 - Bluetooth
> > >  port 1 - auxiliary, available for general purpose use
> > >  port 2 - debugging, usually console output is here
> > >
> > > Enable all of them for future use.
> > >
> > > Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> > > ---
> > >  arch/x86/dts/edison.dts | 18 ++++++++++++++++++
> > >  1 file changed, 18 insertions(+)
> > >
> > 
> > Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
> 
> Thanks, but please defer this series. It made a regression.

I found the root cause.

It's a regression in the

commit a61cbad78e67963944e7d719f0aee27b8aef6c02
Author: Simon Glass <sjg at chromium.org>
Date:   Fri Dec 28 14:23:10 2018 -0700

    dm: serial: Adjust serial_getinfo() to use proper API

I will send a fix soon.

This series is okay as long as future fix will be applied.

Should I make a fix as patch 1 in the series and resend as v3?

-- 
With Best Regards,
Andy Shevchenko




More information about the U-Boot mailing list