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 at gmail.com>
>
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;
>