|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v3 13/18] xen/serial: remove serial_dt_irq
Hi,
As it's part of the "REST" category. Do I need a second ack here?
Regards,
On 04/08/2014 03:44 PM, Julien Grall wrote:
> This function was only used for ARM IRQ routing which has been removed in an
> earlier patch.
>
> Signed-off-by: Julien Grall <julien.grall@xxxxxxxxxx>
> Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
> CC: Keir Fraser <keir@xxxxxxx>
>
> ---
> Changes in v2:
> - Specify in the commit this patch is ARM specific
> ---
> xen/drivers/char/exynos4210-uart.c | 8 --------
> xen/drivers/char/ns16550.c | 11 -----------
> xen/drivers/char/omap-uart.c | 8 --------
> xen/drivers/char/pl011.c | 8 --------
> xen/drivers/char/serial.c | 9 ---------
> xen/include/xen/serial.h | 5 -----
> 6 files changed, 49 deletions(-)
>
> diff --git a/xen/drivers/char/exynos4210-uart.c
> b/xen/drivers/char/exynos4210-uart.c
> index d49e1fe..370539c 100644
> --- a/xen/drivers/char/exynos4210-uart.c
> +++ b/xen/drivers/char/exynos4210-uart.c
> @@ -275,13 +275,6 @@ static int __init exynos4210_uart_irq(struct serial_port
> *port)
> return uart->irq.irq;
> }
>
> -static const struct dt_irq __init *exynos4210_uart_dt_irq(struct serial_port
> *port)
> -{
> - struct exynos4210_uart *uart = port->uart;
> -
> - return &uart->irq;
> -}
> -
> static const struct vuart_info *exynos4210_vuart_info(struct serial_port
> *port)
> {
> struct exynos4210_uart *uart = port->uart;
> @@ -299,7 +292,6 @@ static struct uart_driver __read_mostly
> exynos4210_uart_driver = {
> .putc = exynos4210_uart_putc,
> .getc = exynos4210_uart_getc,
> .irq = exynos4210_uart_irq,
> - .dt_irq_get = exynos4210_uart_dt_irq,
> .vuart_info = exynos4210_vuart_info,
> };
>
> diff --git a/xen/drivers/char/ns16550.c b/xen/drivers/char/ns16550.c
> index 44e13b7..cc86921 100644
> --- a/xen/drivers/char/ns16550.c
> +++ b/xen/drivers/char/ns16550.c
> @@ -717,14 +717,6 @@ static int __init ns16550_irq(struct serial_port *port)
> return ((uart->irq > 0) ? uart->irq : -1);
> }
>
> -#ifdef HAS_DEVICE_TREE
> -static const struct dt_irq __init *ns16550_dt_irq(struct serial_port *port)
> -{
> - struct ns16550 *uart = port->uart;
> - return &uart->dt_irq;
> -}
> -#endif
> -
> #ifdef CONFIG_ARM
> static const struct vuart_info *ns16550_vuart_info(struct serial_port *port)
> {
> @@ -744,9 +736,6 @@ static struct uart_driver __read_mostly ns16550_driver = {
> .putc = ns16550_putc,
> .getc = ns16550_getc,
> .irq = ns16550_irq,
> -#ifdef HAS_DEVICE_TREE
> - .dt_irq_get = ns16550_dt_irq,
> -#endif
> #ifdef CONFIG_ARM
> .vuart_info = ns16550_vuart_info,
> #endif
> diff --git a/xen/drivers/char/omap-uart.c b/xen/drivers/char/omap-uart.c
> index 49ae1a4..b8da509 100644
> --- a/xen/drivers/char/omap-uart.c
> +++ b/xen/drivers/char/omap-uart.c
> @@ -262,13 +262,6 @@ static int __init omap_uart_irq(struct serial_port *port)
> return ((uart->irq.irq > 0) ? uart->irq.irq : -1);
> }
>
> -static const struct dt_irq __init *omap_uart_dt_irq(struct serial_port *port)
> -{
> - struct omap_uart *uart = port->uart;
> -
> - return &uart->irq;
> -}
> -
> static const struct vuart_info *omap_vuart_info(struct serial_port *port)
> {
> struct omap_uart *uart = port->uart;
> @@ -286,7 +279,6 @@ static struct uart_driver __read_mostly omap_uart_driver
> = {
> .putc = omap_uart_putc,
> .getc = omap_uart_getc,
> .irq = omap_uart_irq,
> - .dt_irq_get = omap_uart_dt_irq,
> .vuart_info = omap_vuart_info,
> };
>
> diff --git a/xen/drivers/char/pl011.c b/xen/drivers/char/pl011.c
> index 90bf0c6..459e686 100644
> --- a/xen/drivers/char/pl011.c
> +++ b/xen/drivers/char/pl011.c
> @@ -189,13 +189,6 @@ static int __init pl011_irq(struct serial_port *port)
> return ((uart->irq.irq > 0) ? uart->irq.irq : -1);
> }
>
> -static const struct dt_irq __init *pl011_dt_irq(struct serial_port *port)
> -{
> - struct pl011 *uart = port->uart;
> -
> - return &uart->irq;
> -}
> -
> static const struct vuart_info *pl011_vuart(struct serial_port *port)
> {
> struct pl011 *uart = port->uart;
> @@ -213,7 +206,6 @@ static struct uart_driver __read_mostly pl011_driver = {
> .putc = pl011_putc,
> .getc = pl011_getc,
> .irq = pl011_irq,
> - .dt_irq_get = pl011_dt_irq,
> .vuart_info = pl011_vuart,
> };
>
> diff --git a/xen/drivers/char/serial.c b/xen/drivers/char/serial.c
> index 9b006f2..44026b1 100644
> --- a/xen/drivers/char/serial.c
> +++ b/xen/drivers/char/serial.c
> @@ -500,15 +500,6 @@ int __init serial_irq(int idx)
> return -1;
> }
>
> -const struct dt_irq __init *serial_dt_irq(int idx)
> -{
> - if ( (idx >= 0) && (idx < ARRAY_SIZE(com)) &&
> - com[idx].driver && com[idx].driver->dt_irq_get )
> - return com[idx].driver->dt_irq_get(&com[idx]);
> -
> - return NULL;
> -}
> -
> const struct vuart_info *serial_vuart_info(int idx)
> {
> if ( (idx >= 0) && (idx < ARRAY_SIZE(com)) &&
> diff --git a/xen/include/xen/serial.h b/xen/include/xen/serial.h
> index f38c9b7..9f4451b 100644
> --- a/xen/include/xen/serial.h
> +++ b/xen/include/xen/serial.h
> @@ -81,8 +81,6 @@ struct uart_driver {
> int (*getc)(struct serial_port *, char *);
> /* Get IRQ number for this port's serial line: returns -1 if none. */
> int (*irq)(struct serial_port *);
> - /* Get IRQ device node for this port's serial line: returns NULL if
> none. */
> - const struct dt_irq *(*dt_irq_get)(struct serial_port *);
> /* Get serial information */
> const struct vuart_info *(*vuart_info)(struct serial_port *);
> };
> @@ -135,9 +133,6 @@ void serial_end_log_everything(int handle);
> /* Return irq number for specified serial port (identified by index). */
> int serial_irq(int idx);
>
> -/* Return irq device node for specified serial port (identified by index). */
> -const struct dt_irq *serial_dt_irq(int idx);
> -
> /* Retrieve basic UART information to emulate it (base address, size...) */
> const struct vuart_info* serial_vuart_info(int idx);
>
>
--
Julien Grall
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |