[U-Boot] [PATCH v1 1/2] WIP: serial: Introduce ->getinfo() callback

[Andy Shevchenko <andriy.shevchenko@linux.intel.com>]

On Tue, Sep 25, 2018 at 10:24:42AM +0800, Bin Meng wrote:
> On Sat, Sep 22, 2018 at 9:05 PM Andy Shevchenko
> <andriy.shevchenko@linux.intel.com> wrote:

I found a bit of time to look at this again.

> > +static int ns16550_serial_getinfo(struct udevice *dev, struct serial_device_info *info)
> > +{
> > +       struct NS16550 *const com_port = dev_get_priv(dev);
> > +       struct ns16550_platdata *plat = com_port->plat;
> > +
> > +       info->addr_space = 0;

> This one is tricky. Looks current 8250 DT binding does not have a
> property to describe the address space to be MMIO or PIO. I am not
> sure what's the best option here.

For now I suggest to leave it hard coded to MMIO.

> > +       info->reg_width = 8;

> For this one, I believe we can use "reg-io-width" property from device tree.

I will try this path.

> > +       info->reg_shift = plat->reg_shift;
> > +       info->reg_offset = plat->reg_offset;
> > +       info->addr = plat->base;
> > +       return 0;
> > +}

> Generally this patch looks OK to me.

Thanks! Still I have few amendments for the real submission of the series.

> BTW: why doesn't the kernel use the 'earlycon' command line?

Not all hardware supports it or supports it in a right way.

-- 
With Best Regards,
Andy Shevchenko