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

Reply via email to