On Wed, 2018-07-18 at 17:39 +0200, Takashi Iwai wrote: > 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). Indeed. I'll correct this in a follow up patch > > ... and there is the standard IRQ_RETVAL() macro for this purpose. I was not aware of it. Thanks for the tip Takashi > > > 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