Hi Fabrice, > + return irq_event ? irq_event : -ENODEV; Maybe -ENOENT instead of -ENODEV? I mean you have a dev_err there, so the driver core should probably also complain? You could also shorten the ternary operator to: return irq_event ? : -E; However, both are minor nits. If you prefer to keep the patch as is, fine with me. Regards, Wolfram