# Re: fbdev: ssd1307fb: Unify init code and obtain hw specific bits from DT

- From: Dan Carpenter \<hidden\>
- Date: 2015-05-25 09:48:42 
- List: linux-fbdev
- Page: https://ratatoskr.run/linux-fbdev/2015/05/13466214
- Thread: https://ratatoskr.run/linux-fbdev/2015/05/13466211/t.md

```text
On Mon, May 25, 2015 at 10:30:38AM +0530, Sudip Mukherjee wrote:
> On Sat, May 23, 2015 at 08:32:45PM +0300, Dan Carpenter wrote:
> > Hello Thomas Niederprüm,
> 
> > But I also have a hard time remembering if | or << is higher precedence
> 
> man page of operator (man operator) will show the precedence.
> << is higher precendence than |

The point is that people should just use parenthesis when it's not
something that's obvious.

Also the more I think about it, writing "!par->com_seq & 0x1" is
nonsense.  If that's really what was intended then just say
"!par->com_seq".  I think it's a bug though and "!(par->com_seq & 0x1)"
is intended.

regards,
dan carpenter
```