[U-Boot] [PATCH v4 14/18] serial: 16550: allow the driver to support MediaTek serial

[Tom Rini]

On Thu, Nov 15, 2018 at 11:59:34AM -0800, Simon Glass wrote:
> Hi Ryder,
> 
> On 13 November 2018 at 17:32, Ryder Lee <ryder.lee at mediatek.com> wrote:
> > On Tue, 2018-11-13 at 11:53 -0800, Simon Glass wrote:
> >> Hi Ryder,
> >>
> >> On 6 November 2018 at 01:47, Ryder Lee <ryder.lee at mediatek.com> wrote:
> >> > This patch adds an extra operation in ns16550.c to suuport MediaTek
> >> > SoCs as we have a highspeed register which influences the calcualtion
> >> > of the divisor.
> >> >
> >> > Note that we don't support the baudrate greater than 115200 currently.
> >> >
> >> > Signed-off-by: Ryder Lee <ryder.lee at mediatek.com>
> >> > Tested-by: Matthias Brugger <matthias.bgg at gmail.com>
> >> > Reviewed-by: Simon Glass <sjg at chromium.org>
> >> > ---
> >> > Changes since v4: None
> >> >
> >> > @Simon
> >> > We have tried the compatible string, but it made the ns16550 driver more complicated.
> >> > To use the compatible string we have to add a new field in ns16550_platdata, and change
> >> > the flow of ns16550_serial_probe().
> >> >
> >> > Moreover, it's totally useless for debug uart. At present using a macro is the easiest way here.
> >> >
> >> > What do you think?
> >> > ---
> >> >  drivers/serial/ns16550.c | 10 ++++++++++
> >> >  1 file changed, 10 insertions(+)
> >>
> >> Please see my reply to the other thread. We should not have
> >> arch-specific code in generic drivers.
> >>
> >> Sorry for not replying earlier on that.
> >>
> >
> > OK.
> >
> > BTW, can you (or Tom) please take the other patches through your tree?
> 
> I think this is more Tom's domain, but I'm happy to do this if Tom wants me to.

I'll be picking up the whole series once everyone seems happy with it,
thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20181116/bb089cf1/attachment.sig>