[U-Boot] [PATCH v1 1/2] WIP: serial: Introduce ->getinfo() callback
Andy Shevchenko
andriy.shevchenko at linux.intel.com
Thu Nov 15 16:03:37 UTC 2018
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 at 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
More information about the U-Boot
mailing list