[U-Boot] [PATCH 2/4] serial: pxa: integrate optional driver model handling
Marek Vasut
marex at denx.de
Fri Oct 28 22:58:31 CEST 2016
On 10/28/2016 10:50 PM, Marcel Ziswiler wrote:
> Optional driver model handling integration.
>
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler at toradex.com>
> ---
>
> drivers/serial/serial_pxa.c | 181 ++++++++++++++++++++++------------
> include/dm/platform_data/serial_pxa.h | 56 +++++++++++
> 2 files changed, 174 insertions(+), 63 deletions(-)
> create mode 100644 include/dm/platform_data/serial_pxa.h
>
> diff --git a/drivers/serial/serial_pxa.c b/drivers/serial/serial_pxa.c
> index 1eb19ec..b927a3e 100644
> --- a/drivers/serial/serial_pxa.c
> +++ b/drivers/serial/serial_pxa.c
> @@ -14,6 +14,9 @@
> *
> * Copyright (C) 1999 2000 2001 Erik Mouw (J.A.K.Mouw at its.tudelft.nl)
> *
> + * Modified to add driver model (DM) support
> + * (C) Copyright 2016 Marcel Ziswiler <marcel.ziswiler at toradex.com>
> + *
> * SPDX-License-Identifier: GPL-2.0+
> */
>
> @@ -21,75 +24,32 @@
> #include <asm/arch/pxa-regs.h>
> #include <asm/arch/regs-uart.h>
> #include <asm/io.h>
> +#include <dm.h>
> +#include <dm/platform_data/serial_pxa.h>
> #include <linux/compiler.h>
> #include <serial.h>
> #include <watchdog.h>
>
> DECLARE_GLOBAL_DATA_PTR;
>
> -/*
> - * The numbering scheme differs here for PXA25x, PXA27x and PXA3xx so we can
> - * easily handle enabling of clock.
> - */
> -#ifdef CONFIG_CPU_MONAHANS
> -#define UART_CLK_BASE CKENA_21_BTUART
> -#define UART_CLK_REG CKENA
> -#define BTUART_INDEX 0
> -#define FFUART_INDEX 1
> -#define STUART_INDEX 2
> -#elif CONFIG_CPU_PXA25X
> -#define UART_CLK_BASE (1 << 4) /* HWUART */
> -#define UART_CLK_REG CKEN
> -#define HWUART_INDEX 0
> -#define STUART_INDEX 1
> -#define FFUART_INDEX 2
> -#define BTUART_INDEX 3
> -#else /* PXA27x */
> -#define UART_CLK_BASE CKEN5_STUART
> -#define UART_CLK_REG CKEN
> -#define STUART_INDEX 0
> -#define FFUART_INDEX 1
> -#define BTUART_INDEX 2
> -#endif
> -
> -/*
> - * Only PXA250 has HWUART, to avoid poluting the code with more macros,
> - * artificially introduce this.
> - */
> -#ifndef CONFIG_CPU_PXA25X
> -#define HWUART_INDEX 0xff
> -#endif
> -
> -static uint32_t pxa_uart_get_baud_divider(void)
> +static uint32_t pxa_uart_get_baud_divider(int baudrate)
> {
While at it:
/* Do we even need this check ? */
if (baudrate != 1200 || baudrate != 9600 || baudrate != 19200 ||
baudrate != 38400 || baudrate != 57600 || baudrate != 115200)
return 0;
return 921600 / baudrate;
> - if (gd->baudrate == 1200)
> + if (baudrate == 1200)
> return 768;
> - else if (gd->baudrate == 9600)
> + else if (baudrate == 9600)
> return 96;
> - else if (gd->baudrate == 19200)
> + else if (baudrate == 19200)
> return 48;
> - else if (gd->baudrate == 38400)
> + else if (baudrate == 38400)
> return 24;
> - else if (gd->baudrate == 57600)
> + else if (baudrate == 57600)
> return 16;
> - else if (gd->baudrate == 115200)
> + else if (baudrate == 115200)
> return 8;
> else /* Unsupported baudrate */
> return 0;
> }
>
> -static struct pxa_uart_regs *pxa_uart_index_to_regs(uint32_t uart_index)
> -{
> - switch (uart_index) {
> - case FFUART_INDEX: return (struct pxa_uart_regs *)FFUART_BASE;
> - case BTUART_INDEX: return (struct pxa_uart_regs *)BTUART_BASE;
> - case STUART_INDEX: return (struct pxa_uart_regs *)STUART_BASE;
> - case HWUART_INDEX: return (struct pxa_uart_regs *)HWUART_BASE;
> - default:
> - return NULL;
> - }
> -}
> -
> static void pxa_uart_toggle_clock(uint32_t uart_index, int enable)
> {
> uint32_t clk_reg, clk_offset, reg;
> @@ -110,20 +70,13 @@ static void pxa_uart_toggle_clock(uint32_t uart_index, int enable)
> /*
> * Enable clock and set baud rate, parity etc.
> */
> -void pxa_setbrg_dev(uint32_t uart_index)
> +void pxa_setbrg_common(struct pxa_uart_regs *uart_regs, int port, int baudrate)
> {
> - uint32_t divider = 0;
> - struct pxa_uart_regs *uart_regs;
> -
> - divider = pxa_uart_get_baud_divider();
> + uint32_t divider = pxa_uart_get_baud_divider(baudrate);
> if (!divider)
> hang();
>
> - uart_regs = pxa_uart_index_to_regs(uart_index);
> - if (!uart_regs)
> - hang();
> -
> - pxa_uart_toggle_clock(uart_index, 1);
> + pxa_uart_toggle_clock(port, 1);
>
> /* Disable interrupts and FIFOs */
> writel(0, &uart_regs->ier);
> @@ -139,13 +92,38 @@ void pxa_setbrg_dev(uint32_t uart_index)
> writel(IER_UUE, &uart_regs->ier);
> }
>
> +#ifndef CONFIG_DM_SERIAL
> +static struct pxa_uart_regs *pxa_uart_index_to_regs(uint32_t uart_index)
> +{
> + switch (uart_index) {
> + case FFUART_INDEX: return (struct pxa_uart_regs *)FFUART_BASE;
> + case BTUART_INDEX: return (struct pxa_uart_regs *)BTUART_BASE;
> + case STUART_INDEX: return (struct pxa_uart_regs *)STUART_BASE;
> + case HWUART_INDEX: return (struct pxa_uart_regs *)HWUART_BASE;
> + default:
> + return NULL;
> + }
> +}
> +
> +/*
> + * Enable clock and set baud rate, parity etc.
> + */
> +void pxa_setbrg_dev(uint32_t uart_index)
> +{
> + struct pxa_uart_regs *uart_regs = pxa_uart_index_to_regs(uart_index);
> + if (!uart_regs)
> + hang();
Can we handle the failure gracefully ?
> + pxa_setbrg_common(uart_regs, uart_index, gd->baudrate);
> +}
> +
> /*
> * Initialise the serial port with the given baudrate. The settings
> * are always 8 data bits, no parity, 1 stop bit, no start bits.
> */
> int pxa_init_dev(unsigned int uart_index)
> {
> - pxa_setbrg_dev (uart_index);
> + pxa_setbrg_dev(uart_index);
> return 0;
> }
>
> @@ -297,3 +275,80 @@ void pxa_serial_initialize(void)
> serial_register(&serial_stuart_device);
> #endif
> }
> +#endif /* CONFIG_DM_SERIAL */
> +
> +#ifdef CONFIG_DM_SERIAL
> +static int pxa_serial_probe(struct udevice *dev)
> +{
> + struct pxa_serial_platdata *plat = dev->platdata;
> +
> + pxa_setbrg_common((struct pxa_uart_regs *)(plat->base), plat->port,
The parenthesis around plat->base is superfluous .
> + plat->baudrate);
> + return 0;
> +}
[...]
> diff --git a/include/dm/platform_data/serial_pxa.h b/include/dm/platform_data/serial_pxa.h
> new file mode 100644
> index 0000000..f08fc13
> --- /dev/null
> +++ b/include/dm/platform_data/serial_pxa.h
> @@ -0,0 +1,56 @@
> +/*
> + * Copyright (c) 2016 Marcel Ziswiler <marcel.ziswiler at toradex.com>
> + *
> + * SPDX-License-Identifier: GPL-2.0+
> + */
> +
> +#ifndef __serial_pxa_h
> +#define __serial_pxa_h
This should be in caps.
> +/*
> + * The numbering scheme differs here for PXA25x, PXA27x and PXA3xx so we can
> + * easily handle enabling of clock.
> + */
> +#ifdef CONFIG_CPU_MONAHANS
Please drop the #define[tab]FOO and replace it with #define[space]FOO
> +#define UART_CLK_BASE CKENA_21_BTUART
> +#define UART_CLK_REG CKENA
> +#define BTUART_INDEX 0
> +#define FFUART_INDEX 1
> +#define STUART_INDEX 2
> +#elif CONFIG_CPU_PXA25X
> +#define UART_CLK_BASE (1 << 4) /* HWUART */
> +#define UART_CLK_REG CKEN
> +#define HWUART_INDEX 0
> +#define STUART_INDEX 1
> +#define FFUART_INDEX 2
> +#define BTUART_INDEX 3
> +#else /* PXA27x */
> +#define UART_CLK_BASE CKEN5_STUART
> +#define UART_CLK_REG CKEN
> +#define STUART_INDEX 0
> +#define FFUART_INDEX 1
> +#define BTUART_INDEX 2
> +#endif
Thanks!
--
Best regards,
Marek Vasut
More information about the U-Boot
mailing list