From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Schwierzeck Date: Wed, 22 Jan 2020 22:09:15 +0100 Subject: [PATCH v2 6/7] serial: serial_cortina: add UART DM driver for CAxxxx SoCs In-Reply-To: <1579601912-27737-7-git-send-email-alex.nemirovsky@cortina-access.com> References: <1579601912-27737-1-git-send-email-alex.nemirovsky@cortina-access.com> <1579601912-27737-7-git-send-email-alex.nemirovsky@cortina-access.com> Message-ID: <59daf4ca-d723-e515-eafc-4a048f2bd560@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Am 21.01.20 um 11:19 schrieb Alex Nemirovsky: > From: Jason Li > > Add serial UART driver support for all Cortina Access > CAxxxx family of SoCs. > > Signed-off-by: Jason Li > Signed-off-by: Alex Nemirovsky > > > --- > > Changes in v2: > - Rename driver in DT namespace for consistency between all > CA drivers. > - Remove blank line after SPDX identifier > - Remove authorship comment as it is already recorded within Git > and is redundant > - Merge serial_cortina.h reg defines into serial_cortina.c > - Modify ca_serial_pending() and API to get resource. > > drivers/serial/Kconfig | 7 ++ > drivers/serial/Makefile | 2 +- > drivers/serial/serial_cortina.c | 164 ++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 172 insertions(+), 1 deletion(-) > create mode 100644 drivers/serial/serial_cortina.c Reviewed-by: Daniel Schwierzeck nits below > > diff --git a/drivers/serial/Kconfig b/drivers/serial/Kconfig > index ece7d87..9f76596 100644 > --- a/drivers/serial/Kconfig > +++ b/drivers/serial/Kconfig > @@ -539,6 +539,13 @@ config BCM6345_SERIAL > help > Select this to enable UART on BCM6345 SoCs. > > +config CORTINA_UART > + bool "Cortina UART support" > + depends on DM_SERIAL > + help > + Select this to enable UART support for Cortina-Access UART devices > + found on CAxxxx SoCs. > + > config FSL_LINFLEXUART > bool "Freescale Linflex UART support" > depends on DM_SERIAL > diff --git a/drivers/serial/Makefile b/drivers/serial/Makefile > index 06ee306..c8f2db4 100644 > --- a/drivers/serial/Makefile > +++ b/drivers/serial/Makefile > @@ -28,13 +28,13 @@ obj-$(CONFIG_PL010_SERIAL) += serial_pl01x.o > obj-$(CONFIG_PL011_SERIAL) += serial_pl01x.o > obj-$(CONFIG_SYS_NS16550_SERIAL) += serial_ns16550.o > endif > - > obj-$(CONFIG_ALTERA_UART) += altera_uart.o > obj-$(CONFIG_ALTERA_JTAG_UART) += altera_jtag_uart.o > obj-$(CONFIG_AR933X_UART) += serial_ar933x.o > obj-$(CONFIG_ARM_DCC) += arm_dcc.o > obj-$(CONFIG_ATMEL_USART) += atmel_usart.o > obj-$(CONFIG_BCM6345_SERIAL) += serial_bcm6345.o > +obj-$(CONFIG_CORTINA_UART) += serial_cortina.o > obj-$(CONFIG_EFI_APP) += serial_efi.o > obj-$(CONFIG_LPC32XX_HSUART) += lpc32xx_hsuart.o > obj-$(CONFIG_MCFUART) += mcfuart.o > diff --git a/drivers/serial/serial_cortina.c b/drivers/serial/serial_cortina.c > new file mode 100644 > index 0000000..3af64ba > --- /dev/null > +++ b/drivers/serial/serial_cortina.c > @@ -0,0 +1,164 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * (C) Copyright 2020 > + * Cortina-Access Ltd. I think this should be on one line > + * > + */ > + > +/* Common UART Driver for Cortina Access CAxxxx line of SoCs */ this should be moved to the previous comment block > + > +#include > +#include > +#include > +#include > +#include > +#include > +#include > + > +/* Register definitions */ > +#define UCFG 0x00 /* UART config register */ > +#define UFC 0x04 /* Flow Control */ > +#define URX_SAMPLE 0x08 /* UART RX Sample register */ > +#define URT_TUNE 0x0C /* Fine tune of UART clk */ > +#define UTX_DATA 0x10 /* UART TX Character data */ > +#define URX_DATA 0x14 /* UART RX Character data */ > +#define UINFO 0x18 /* UART Info */ > +#define UINT_EN0 0x1C /* UART Interrupt enable 0 */ > +#define UINT_EN1 0x20 /* UART Interrupt enable 1 */ > +#define UINT0 0x24 /* UART Interrupt 0 setting/clearing */ > +#define UINT1 0x28 /* UART Interrupt 1 setting/clearing */ > +#define UINT_STAT 0x2C /* UART Interrupt Status */ > + > +/* UART Control Register Bit Fields */ > +#define UCFG_BAUD_COUNT BIT(8) > +#define UCFG_EN BIT(7) > +#define UCFG_RX_EN BIT(6) > +#define UCFG_TX_EN BIT(5) > +#define UCFG_PARITY_EN BIT(4) > +#define UCFG_PARITY_SEL BIT(3) > +#define UCFG_2STOP_BIT BIT(2) > +#define UCFG_CNT1 BIT(1) > +#define UCFG_CNT0 BIT(0) > +#define UCFG_CHAR_5 0 > +#define UCFG_CHAR_6 1 > +#define UCFG_CHAR_7 2 > +#define UCFG_CHAR_8 3 > + > +#define UINFO_TX_FIFO_EMPTY BIT(3) > +#define UINFO_TX_FIFO_FULL BIT(2) > +#define UINFO_RX_FIFO_EMPTY BIT(1) > +#define UINFO_RX_FIFO_FULL BIT(0) > + > +#define UINT_RX_NON_EMPTY BIT(6) > +#define UINT_TX_EMPTY BIT(5) > +#define UINT_RX_UNDERRUN BIT(4) > +#define UINT_RX_OVERRUN BIT(3) > +#define UINT_RX_PARITY_ERR BIT(2) > +#define UINT_RX_STOP_ERR BIT(1) > +#define UINT_TX_OVERRUN BIT(0) > +#define UINT_MASK_ALL 0x7F > + > +struct ca_uart_priv { > + void __iomem *base; > +}; > + > +int ca_serial_setbrg(struct udevice *dev, int baudrate) > +{ > + struct ca_uart_priv *priv = dev_get_priv(dev); > + unsigned int uart_ctrl, baud, sample; > + > + baud = CORTINA_UART_CLOCK / baudrate; > + > + uart_ctrl = readl(priv->base + UCFG); > + uart_ctrl |= (baud << 8); > + writel(uart_ctrl, priv->base + UCFG); you should clear the old baud value at first before writing a new one. AFAIK _setbrg() is supposed to be able to be called multiple times. With or'ing only you would possibly end up with a wrong value. clrsetbits_32() should do the job. > + > + sample = baud / 2; > + sample = (sample < 7) ? 7 : sample; > + writel(sample, priv->base + URX_SAMPLE); > + > + return 0; > +} > + > +static int ca_serial_getc(struct udevice *dev) > +{ > + struct ca_uart_priv *priv = dev_get_priv(dev); > + int ch; > + > + ch = readl(priv->base + URX_DATA) & 0xFF; > + > + return (int)ch; > +} > + > +static int ca_serial_putc(struct udevice *dev, const char ch) > +{ > + struct ca_uart_priv *priv = dev_get_priv(dev); > + unsigned int status; > + > + /* Retry if TX FIFO full */ > + status = readl(priv->base + UINFO); > + if (status & UINFO_TX_FIFO_FULL) > + return -EAGAIN; > + > + writel(ch, priv->base + UTX_DATA); > + > + return 0; > +} > + > +static int ca_serial_pending(struct udevice *dev, bool input) > +{ > + struct ca_uart_priv *priv = dev_get_priv(dev); > + unsigned int status; > + > + status = readl(priv->base + UINFO); > + > + if (input) > + return (status & UINFO_RX_FIFO_EMPTY) ? 0 : 1; > + else > + return (status & UINFO_TX_FIFO_FULL) ? 1 : 0; > +} > + > +static int ca_serial_probe(struct udevice *dev) > +{ > + struct ca_uart_priv *priv = dev_get_priv(dev); > + u32 uart_ctrl; > + > + /* Set data, parity and stop bits */ > + uart_ctrl = UCFG_EN | UCFG_TX_EN | UCFG_RX_EN | UCFG_CHAR_8; > + writel(uart_ctrl, priv->base + UCFG); > + > + return 0; > +} > + > +static int ca_serial_ofdata_to_platdata(struct udevice *dev) > +{ > + struct ca_uart_priv *priv = dev_get_priv(dev); > + > + priv->base = dev_remap_addr_index(dev, 0); > + if (!priv->base) > + return -ENOENT; > + > + return 0; > +} > + > +static const struct dm_serial_ops ca_serial_ops = { > + .putc = ca_serial_putc, > + .pending = ca_serial_pending, > + .getc = ca_serial_getc, > + .setbrg = ca_serial_setbrg, > +}; > + > +static const struct udevice_id ca_serial_ids[] = { > + {.compatible = "cortina,ca-uart"}, > + {} > +}; > + > +U_BOOT_DRIVER(serial_cortina) = { > + .name = "serial_cortina", > + .id = UCLASS_SERIAL, > + .of_match = ca_serial_ids, > + .ofdata_to_platdata = ca_serial_ofdata_to_platdata, > + .priv_auto_alloc_size = sizeof(struct ca_uart_priv), > + .probe = ca_serial_probe, > + .ops = &ca_serial_ops > +}; > -- - Daniel