rtc: pcf2127: fix a bug when not specify interrupts property

Fix a bug when not specify interrupts property in dts
as follows,
    rtc-pcf2127-i2c 1-0051: failed to request alarm irq
    rtc-pcf2127-i2c: probe of 1-0051 failed with error -22

Signed-off-by: Biwen Li <biwen.li@nxp.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Link: https://lore.kernel.org/r/20200915073213.12779-1-biwen.li@oss.nxp.com

diff --git a/drivers/rtc/rtc-pcf2127.c b/drivers/rtc/rtc-pcf2127.c
index ed6316992cbb89c0e25a0651003ed00a911302b6..07a5630ec841fcb0ecd44d8de3a28b7c518976bb 100644 (file)
--- a/drivers/rtc/rtc-pcf2127.c
+++ b/drivers/rtc/rtc-pcf2127.c
@@ -559,7 +559,7 @@ static int pcf2127_probe(struct device *dev, struct regmap *regmap,
        pcf2127->rtc->set_start_time = true; /* Sets actual start to 1970 */
        pcf2127->rtc->uie_unsupported = 1;
 
-       if (alarm_irq >= 0) {
+       if (alarm_irq > 0) {
                ret = devm_request_threaded_irq(dev, alarm_irq, NULL,
                                                pcf2127_rtc_irq,
                                                IRQF_TRIGGER_LOW | IRQF_ONESHOT,
@@ -570,7 +570,7 @@ static int pcf2127_probe(struct device *dev, struct regmap *regmap,
                }
        }
 
-       if (alarm_irq >= 0 || device_property_read_bool(dev, "wakeup-source")) {
+       if (alarm_irq > 0 || device_property_read_bool(dev, "wakeup-source")) {
                device_init_wakeup(dev, true);
                pcf2127->rtc->ops = &pcf2127_rtc_alrm_ops;
        }