diff options
| author | Tom Rix <[email protected]> | 2021-05-21 21:48:05 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <[email protected]> | 2021-07-22 14:01:56 +0200 |
| commit | 544ee7306d9e3a7ff675e87f151723ff30efd9a1 (patch) | |
| tree | d02911d2a271d47304fd08252e518d583fee717f /tools/perf/scripts/python | |
| parent | f89aa0d174b3e7f2988a976125c3a67a07b71040 (diff) | |
media: atomisp: improve error handling in gc2235_detect()
Static analysis reports this representative problem
atomisp-gc2235.c:867:20: warning: The right operand
of '|' is a garbage value
id = ((high << 8) | low);
^ ~~~
When gc2235_read_reg() fails, its return val is never written.
For gc2235_detect(), high and low are or-ed and compared
with GC2235_ID, 0x2235. Initialize both to 0 and skip
checking the read returns, it's errors are not passed up, only
-ENODEV is.
Link: https://lore.kernel.org/linux-media/[email protected]
Signed-off-by: Tom Rix <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python')
0 files changed, 0 insertions, 0 deletions