On Thu, 16 Oct 2008 21:23:07 +0400, Sergei Shtylyov <sshtylyov@xxxxxxxxxxxxx> wrote: > >>>+ if ((dma_stat & 7) == 0 && > >>>+ (ctl & (TX4939IDE_INT_XFEREND | TX4939IDE_INT_HOST)) == > >>>+ (TX4939IDE_INT_XFEREND | TX4939IDE_INT_HOST)) > >> > >> Parens around & and | are hardly needed... > > > You mean more parens are needed? > > I mean less. :-) Well, I think all above parens are required. '&' and '|' are weaker than '==', no? --- Atsushi Nemoto