diff options
| author | Gustavo A. R. Silva <[email protected]> | 2019-10-22 15:25:22 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <[email protected]> | 2019-12-13 09:14:27 +0100 |
| commit | 0d962e061abcf1b9105f88fb850158b5887fbca3 (patch) | |
| tree | b48f377abac602b48a27a472f354251e5156e170 /tools/perf/scripts/python/bin | |
| parent | c4e8f760581b8607a1989acb8925be25d6628760 (diff) | |
media: i2c: adv748x: Fix unsafe macros
Enclose multiple macro parameters in parentheses in order to
make such macros safer and fix the Clang warning below:
drivers/media/i2c/adv748x/adv748x-afe.c:452:12: warning: operator '?:'
has lower precedence than '|'; '|' will be evaluated first
[-Wbitwise-conditional-parentheses]
ret = sdp_clrset(state, ADV748X_SDP_FRP, ADV748X_SDP_FRP_MASK, enable
? ctrl->val - 1 : 0);
Fixes: 3e89586a64df ("media: i2c: adv748x: add adv748x driver")
Reported-by: Dmitry Vyukov <[email protected]>
Signed-off-by: Gustavo A. R. Silva <[email protected]>
Reviewed-by: Kieran Bingham <[email protected]>
Signed-off-by: Hans Verkuil <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/bin')
0 files changed, 0 insertions, 0 deletions