2018-11-21 19:51:50

by Julia Lawall

[permalink] [raw]
Subject: [PATCH] usb: misc: fix boolconv.cocci warnings

From: kbuild test robot <[email protected]>

drivers/usb/misc/ft232h-intf.c:816:27-32: WARNING: conversion to bool not needed here

Remove unneeded conversion to bool

Semantic patch information:
Relational and logical operators evaluate to bool,
explicit conversion is overly verbose and unneeded.

Generated by: scripts/coccinelle/misc/boolconv.cocci

Fixes: 0772a5bf580c ("usb: misc: add driver for FT232H based FPGA configuration devices")
CC: Anatolij Gustschin <[email protected]>
Signed-off-by: kbuild test robot <[email protected]>
Signd-off-by: Julia Lawall <[email protected]>

---

url: https://github.com/0day-ci/linux/commits/Anatolij-Gustschin/Add-support-for-ARRI-FPGA-configuration/20181120-161201
:::::: branch date: 17 hours ago
:::::: commit date: 17 hours ago

Please take the patch only if it's a positive warning. Thanks!

ft232h-intf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

--- a/drivers/usb/misc/ft232h-intf.c
+++ b/drivers/usb/misc/ft232h-intf.c
@@ -813,7 +813,7 @@ static int ftdi_mpsse_gpio_get(struct gp

dev_dbg(chip->parent, "%s: offset %d\n", __func__, offset);

- low = offset < 5 ? true : false;
+ low = offset < 5;

mutex_lock(&priv->ops_mutex);



2018-11-24 08:40:58

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH] usb: misc: fix boolconv.cocci warnings

On Wed, Nov 21, 2018 at 06:33:51PM +0100, Julia Lawall wrote:
> From: kbuild test robot <[email protected]>
>
> drivers/usb/misc/ft232h-intf.c:816:27-32: WARNING: conversion to bool not needed here
>
> Remove unneeded conversion to bool
>
> Semantic patch information:
> Relational and logical operators evaluate to bool,
> explicit conversion is overly verbose and unneeded.
>
> Generated by: scripts/coccinelle/misc/boolconv.cocci
>
> Fixes: 0772a5bf580c ("usb: misc: add driver for FT232H based FPGA configuration devices")

I can't find this commit anywhere, nor the file that this patch touches.

What tree is it in?

thanks,

greg k-h

2018-11-24 08:41:30

by Anatolij Gustschin

[permalink] [raw]
Subject: Re: [PATCH] usb: misc: fix boolconv.cocci warnings

On Fri, 23 Nov 2018 16:10:54 +0100
Greg KH [email protected] wrote:
...
>> Fixes: 0772a5bf580c ("usb: misc: add driver for FT232H based FPGA configuration devices")
>
>I can't find this commit anywhere, nor the file that this patch touches.
>
>What tree is it in?

This new driver was submitted to the lists and is not in mainline
tree yet. It was committed to some tree for build testing the
submitted patches, it seems.

Thanks,

Anatolij