Re: [PATCH 1/2] watchdog: orion_wdt: Propagate -EPROBE_DEFER from IRQ lookup
Bui Duc Phuc <[email protected]>
| Newsgroups | gmane.linux.watchdog,gmane.linux.kernel,gmane.linux.ports.arm.kernel |
|---|---|
| Message-ID | <CAABR9nEys_RZqpYEqhq+LGwWc1Fan3cN8mLSrejM+8vAnFOn0A@mail.gmail.com> |
Hi Guenter, Thank you for your review. > > + if (irq == -EPROBE_DEFER) > > + return irq; > > I think this should be > > if (irq < 0 && irq != -ENXIO) > return irq; > Agreed, that's better. -EINVAL also deserves to be returned, not just -EPROBE_DEFER only -ENXIO really means "no IRQ". Will send v2 soon. Best regards, Phuc