Message ID | 20200426094228.23829-1-yanaijie@huawei.com (mailing list archive) |
---|---|
State | Not Applicable |
Headers | show |
Series | i2c: powermac: use true,false for bool variable | expand |
Context | Check | Description |
---|---|---|
snowpatch_ozlabs/apply_patch | success | Successfully applied on branch powerpc/merge (54dc28ff5e0b3585224d49a31b53e030342ca5c3) |
snowpatch_ozlabs/build-ppc64le | success | Build succeeded |
snowpatch_ozlabs/build-ppc64be | success | Build succeeded |
snowpatch_ozlabs/build-ppc64e | success | Build succeeded |
snowpatch_ozlabs/build-pmac32 | warning | Upstream build failed, couldn't test patch |
snowpatch_ozlabs/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 8 lines checked |
snowpatch_ozlabs/needsstable | success | Patch has no Fixes tags |
On Sun, Apr 26, 2020 at 05:42:28PM +0800, Jason Yan wrote: > In i2c_powermac_register_devices(), variable 'found_onyx' is bool and > assigned '0' and 'true' in different places. Use 'false' instead of '0'. > This fixes the following coccicheck warning: > > drivers/i2c/busses/i2c-powermac.c:318:6-16: WARNING: Assignment of 0/1 > to bool variable > > Signed-off-by: Jason Yan <yanaijie@huawei.com> Applied to for-next, thanks!
diff --git a/drivers/i2c/busses/i2c-powermac.c b/drivers/i2c/busses/i2c-powermac.c index d565714c1f13..00a6fd42c1ae 100644 --- a/drivers/i2c/busses/i2c-powermac.c +++ b/drivers/i2c/busses/i2c-powermac.c @@ -315,7 +315,7 @@ static void i2c_powermac_register_devices(struct i2c_adapter *adap, { struct i2c_client *newdev; struct device_node *node; - bool found_onyx = 0; + bool found_onyx = false; /* * In some cases we end up with the via-pmu node itself, in this
In i2c_powermac_register_devices(), variable 'found_onyx' is bool and assigned '0' and 'true' in different places. Use 'false' instead of '0'. This fixes the following coccicheck warning: drivers/i2c/busses/i2c-powermac.c:318:6-16: WARNING: Assignment of 0/1 to bool variable Signed-off-by: Jason Yan <yanaijie@huawei.com> --- drivers/i2c/busses/i2c-powermac.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)