diff options
| author | Arnd Bergmann <[email protected]> | 2018-09-26 08:51:01 -0400 |
|---|---|---|
| committer | Mauro Carvalho Chehab <[email protected]> | 2018-10-05 11:04:39 -0400 |
| commit | 146c45ef1fec3e3217b78c5a606efe41150301ea (patch) | |
| tree | 7bf10052bebedea5c508f9769b23444e85f22513 /tools/perf/scripts/python/export-to-postgresql.py | |
| parent | 370d8e2a7570adb46ab5d90b19292c5c40129e24 (diff) | |
media: ov9650: avoid maybe-uninitialized warnings
The regmap change causes multiple warnings like
drivers/media/i2c/ov9650.c: In function 'ov965x_g_volatile_ctrl':
drivers/media/i2c/ov9650.c:889:29: error: 'reg2' may be used uninitialized in this function [-Werror=maybe-uninitialized]
exposure = ((reg2 & 0x3f) << 10) | (reg1 << 2) |
~~~~~~~~~~~~~~~^~~~~~
It is apparently hard for the compiler to see here if ov965x_read()
returned successfully or not. Besides, we have a v4l2_dbg() statement
that prints an uninitialized value if regmap_read() fails.
Adding an 'else' clause avoids the ambiguity.
Fixes: 361f3803adfe ("media: ov9650: use SCCB regmap")
Signed-off-by: Arnd Bergmann <[email protected]>
Reviewed-by: Akinobu Mita <[email protected]>
Signed-off-by: Sakari Ailus <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/export-to-postgresql.py')
0 files changed, 0 insertions, 0 deletions