[Xen-devel] Re: [PATCH] XEN: remove undefined functions

[Jeremy Fitzhardinge <jeremy@xxxxxxxx>]

On 08/25/09 03:01, Jaswinder Singh Rajput wrote:
> mk_pirq_info(), gsi_from_irq() and vector_from_irq() are static functions
> and no one is calling them.
>
> This fixed following compilation warnings :
>
>   drivers/xen/events.c:134: warning: âmk_pirq_infoâ defined but not used
>   drivers/xen/events.c:180: warning: âgsi_from_irqâ defined but not used
>   drivers/xen/events.c:190: warning: âvector_from_irqâ defined but not used
>
> Signed-off-by: Jaswinder Singh Rajput <jaswinderrajput@xxxxxxxxx>
>   

Hm, they're all about to become used.

    J
> ---
>  drivers/xen/events.c |   27 ---------------------------
>  1 files changed, 0 insertions(+), 27 deletions(-)
>
> diff --git a/drivers/xen/events.c b/drivers/xen/events.c
> index abad71b..d43957a 100644
> --- a/drivers/xen/events.c
> +++ b/drivers/xen/events.c
> @@ -131,13 +131,6 @@ static struct irq_info mk_virq_info(unsigned short 
> evtchn, unsigned short virq)
>                       .cpu = 0, .u.virq = virq };
>  }
>  
> -static struct irq_info mk_pirq_info(unsigned short evtchn,
> -                                 unsigned short gsi, unsigned short vector)
> -{
> -     return (struct irq_info) { .type = IRQT_PIRQ, .evtchn = evtchn,
> -                     .cpu = 0, .u.pirq = { .gsi = gsi, .vector = vector } };
> -}
> -
>  /*
>   * Accessors for packed IRQ information.
>   */
> @@ -177,26 +170,6 @@ static unsigned virq_from_irq(unsigned irq)
>       return info->u.virq;
>  }
>  
> -static unsigned gsi_from_irq(unsigned irq)
> -{
> -     struct irq_info *info = info_for_irq(irq);
> -
> -     BUG_ON(info == NULL);
> -     BUG_ON(info->type != IRQT_PIRQ);
> -
> -     return info->u.pirq.gsi;
> -}
> -
> -static unsigned vector_from_irq(unsigned irq)
> -{
> -     struct irq_info *info = info_for_irq(irq);
> -
> -     BUG_ON(info == NULL);
> -     BUG_ON(info->type != IRQT_PIRQ);
> -
> -     return info->u.pirq.vector;
> -}
> -
>  static enum xen_irq_type type_from_irq(unsigned irq)
>  {
>       return info_for_irq(irq)->type;
>   


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel