On Wed, 18 Jul 2018 13:50:54 +0200, Mark Brown wrote: > > On Tue, Jul 17, 2018 at 05:42:51PM +0200, Jerome Brunet wrote: > > > + return !status ? IRQ_NONE : IRQ_HANDLED; > > Please just write normal conditional statements, it makes things a lot > easier to read (the ! isn't helping either here). ... and there is the standard IRQ_RETVAL() macro for this purpose. Takashi -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html