[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



On Thu, 2014-04-17 at 11:22 +0100, Julien Grall wrote:

I don't think so, if I thought there was any chance of this affecting
x86 I might want to see something from an x86 person, but I don't think
that's likely in this case.

> 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);
> >  
> > 
> 
> 



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.