[U-Boot] [PATCH v1 3/4] serial: add an of-platdata driver for "snps, dw-apb-uart"

[Simon Glass]

Hi Simon,

On Fri, 11 Jan 2019 at 03:02, Simon Goldschmidt
<simon.k.r.goldschmidt at gmail.com> wrote:
>
> On Fri, Jan 11, 2019 at 10:22 AM Andy Shevchenko
> <andy.shevchenko at gmail.com> wrote:
> >
> > On Wed, Jan 9, 2019 at 10:36 AM Alexey Brodkin
> > <alexey.brodkin at synopsys.com> wrote:
> >
> >
> > > Might be a bit naïve question but why depend on SPL_OF_PLATDATA only?
> > > What about CONFIG_OF_EMBED?
> >
> > No driver should depend on OF_EMBED IIUC. New U-Boot spils a warning
> > when production boards are using it.
>
> By now, actually I even dislike the idea of depending on OF_PLATDATA myself.
> Maybe we should fix the OF_PLATDATA to driver matching code to better handle
> drivers with multiple compatible strings instead of adding dedicated drivers.

Sounds good.

One option I rejected with the initial impl was to add tags to the DT
to tell dtoc what to do. I still hope to avoid that, but it is an
option, if helpful.

Regards,
Simon