diff options
| author | Nathan Chancellor <natechancellor@gmail.com> | 2019-09-26 21:31:32 -0700 |
|---|---|---|
| committer | Michael Bestas <mkbestas@lineageos.org> | 2020-02-02 01:40:54 +0200 |
| commit | 6a31f29c4f58e5c4797528e78c769ab00f9088ff (patch) | |
| tree | 4d8f9c57dd918504911bbb40cc6412114a095bc5 /tools/perf/scripts/python | |
| parent | caa14bd8a74606e212c074ceecc65f1f05064f72 (diff) | |
clk: qcom: clk-debug: Reorder if statement to avoid self assignment
Currently, this if statement is worded as:
if (a == 0xFF)
b = b;
else if (a) /* a != 0 and a != 0xFF */
b |= c;
else /* a == 0 */
b |= d;
This causes a clang warning:
../drivers/clk/qcom/clk-debug.c:182:11: warning: explicitly assigning
value of variable of type 'u32' (aka 'unsigned int') to itself
[-Wself-assign]
regval = regval;
~~~~~~ ^ ~~~~~~
1 warning generated.
This can be reworded as:
if (!a) /* a == 0 */
b |= d;
else if (a != 0xFF)
b |= c;
which is equivalent in meaning and avoids the warning.
Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
Diffstat (limited to 'tools/perf/scripts/python')
0 files changed, 0 insertions, 0 deletions
