Message ID | 20101113182626.GC1795@bicker |
---|---|
State | Superseded |
Headers | show |
Dan Carpenter wrote: > platform_get_irq() can return negatives so the rtc->irq should be signed > to handle it. > > Signed-off-by: Dan Carpenter <error27@gmail.com> Acked-By: Lars-Peter Clausen <lars@metafoo.de> Thanks. > > diff --git a/drivers/rtc/rtc-jz4740.c b/drivers/rtc/rtc-jz4740.c > index 2e16f72..cfcd2da 100644 > --- a/drivers/rtc/rtc-jz4740.c > +++ b/drivers/rtc/rtc-jz4740.c > @@ -42,7 +42,7 @@ struct jz4740_rtc { > > struct rtc_device *rtc; > > - unsigned int irq; > + int irq; > > spinlock_t lock; > };
diff --git a/drivers/rtc/rtc-jz4740.c b/drivers/rtc/rtc-jz4740.c index 2e16f72..cfcd2da 100644 --- a/drivers/rtc/rtc-jz4740.c +++ b/drivers/rtc/rtc-jz4740.c @@ -42,7 +42,7 @@ struct jz4740_rtc { struct rtc_device *rtc; - unsigned int irq; + int irq; spinlock_t lock; };
platform_get_irq() can return negatives so the rtc->irq should be signed to handle it. Signed-off-by: Dan Carpenter <error27@gmail.com>