diff options
| author | Tom Rix <trix@redhat.com> | 2020-09-04 11:06:47 -0700 |
|---|---|---|
| committer | Wolfram Sang <wsa@kernel.org> | 2020-09-21 11:45:43 +0200 |
| commit | 97c93e06f506107362f1630aa25ec3e3ae71d070 (patch) | |
| tree | 1d26f1dd1a394a901c25f8b48aaa66c5531a6dc1 /tools/perf/scripts/python/bin/stackcollapse-report | |
| parent | i2c: stm32: Simplify with dev_err_probe() (diff) | |
| download | linux-97c93e06f506107362f1630aa25ec3e3ae71d070.tar.gz linux-97c93e06f506107362f1630aa25ec3e3ae71d070.zip | |
i2c: amd_mp2: handle num is 0 input for i2c_amd_xfer
clang static analyzer reports this problem
i2c-amd-mp2-plat.c:174:9: warning: Branch condition evaluates
to a garbage value
return err ? err : num;
^~~
err is not initialized, it depends on the being set in the
transfer loop which will not happen if num is 0. Surveying
other master_xfer() implementations show all handle a 0 num.
Because returning 0 is expected, initialize err to 0.
Signed-off-by: Tom Rix <trix@redhat.com>
Acked-by: Elie Morisse <syniurge@gmail.com>
Signed-off-by: Wolfram Sang <wsa@kernel.org>
Diffstat (limited to 'tools/perf/scripts/python/bin/stackcollapse-report')
0 files changed, 0 insertions, 0 deletions
