From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ryder Lee <ryder.lee@mediatek.com>
Date: Mon, 5 Nov 2018 10:20:21 +0800
Subject: [U-Boot] [PATCH v3 14/18] serial: 16550: allow the driver to
 support MediaTek serial
In-Reply-To: <CAPnjgZ1i+TiGT9WxwnS-ENsMj-Sjr4kJG_WnkkkUyZxqUXNRdA@mail.gmail.com>
References: <cover.1541171496.git.ryder.lee@mediatek.com>
 <9031af7d284f63d8d3e7c5f1d6883ba160b72c46.1541171496.git.ryder.lee@mediatek.com>
 <CAPnjgZ1i+TiGT9WxwnS-ENsMj-Sjr4kJG_WnkkkUyZxqUXNRdA@mail.gmail.com>
Message-ID: <1541384421.27315.6.camel@mtkswgap22>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Sat, 2018-11-03 at 00:09 -0600, Simon Glass wrote:
> Hi Ryder,
> 
> On 2 November 2018 at 09:15, Ryder Lee <ryder.lee@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@mediatek.com>
> > Tested-by: Matthias Brugger <matthias.bgg@gmail.com>
> > Reviewed-by: Simon Glass <sjg@chromium.org>
> > ---
> >  drivers/serial/ns16550.c | 10 ++++++++++
> >  1 file changed, 10 insertions(+)
> 
> Actually it seems to me that we should have a compatible string for
> this and deal with this at run-time?
> 
> Is that easy to do here?
> 

How about this:

uart0: serial at 11002000 { 
	compatible = "ns16550a"; 
	....
	mediatek,highspeed = <0>; 
	....

Ryder