Searched hist:"17104 ca2713ba443f9a02a7c4090514ad1e46ee9" (Results 1 – 1 of 1) sorted by relevance
/openbmc/linux/drivers/iio/adc/ |
H A D | stmpe-adc.c | diff 17104ca2713ba443f9a02a7c4090514ad1e46ee9 Thu Mar 07 11:16:04 CST 2019 Nathan Chancellor <natechancellor@gmail.com> iio: adc: stmpe-adc: Shuffle an if statement around in stmpe_adc_isr
When building with -Wsometimes-uninitialized, Clang warns:
drivers/iio/adc/stmpe-adc.c:204:13: warning: variable 'data' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]
Clang can't tell that data will never be used uninitialized because the two if statements take care of all cases. Remove the first if statement and make it the else branch of the second one so that it is apparent to Clang that all cases are covered.
Link: https://github.com/ClangBuiltLinux/linux/issues/387 Suggested-by: Nick Desaulniers <ndesaulniers@google.com> Signed-off-by: Nathan Chancellor <natechancellor@gmail.com> Reviewed-by: NIck Desaulniers <ndesaulniers@google.com> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
|