On Fri, Nov 16, 2012 at 12:20:41PM +0530, Tushar Behera wrote: > No need to check whether unsigned variable is less than 0. > > CC: Konrad Rzeszutek Wilk <[email protected]>
Acked-by: Konrad Rzeszutek Wilk <[email protected]> > CC: Jeremy Fitzhardinge <[email protected]> > CC: [email protected] > CC: [email protected] > Signed-off-by: Tushar Behera <[email protected]> > --- > drivers/xen/events.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/xen/events.c b/drivers/xen/events.c > index 4293c57..cadd7d1 100644 > --- a/drivers/xen/events.c > +++ b/drivers/xen/events.c > @@ -216,7 +216,7 @@ static void xen_irq_info_pirq_init(unsigned irq, > */ > static unsigned int evtchn_from_irq(unsigned irq) > { > - if (unlikely(WARN(irq < 0 || irq >= nr_irqs, "Invalid irq %d!\n", irq))) > + if (unlikely(WARN(irq >= nr_irqs, "Invalid irq %d!\n", irq))) > return 0; > > return info_for_irq(irq)->evtchn; > -- > 1.7.4.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
