linux-fpga.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] usb: misc: fix boolconv.cocci warnings
@ 2018-11-21 17:33 Julia Lawall
  2018-11-23 15:10 ` Greg KH
  0 siblings, 1 reply; 3+ messages in thread
From: Julia Lawall @ 2018-11-21 17:33 UTC (permalink / raw)
  To: Anatolij Gustschin
  Cc: linux-usb, linux-spi, linux-fpga, linux-kernel, gregkh, broonie,
	atull, mdf, kbuild-all

From: kbuild test robot <fengguang.wu@intel.com>

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 <agust@denx.de>
Signed-off-by: kbuild test robot <fengguang.wu@intel.com>
Signd-off-by: Julia Lawall <julia.lawall@lip6.fr>

---

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);

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] usb: misc: fix boolconv.cocci warnings
  2018-11-21 17:33 [PATCH] usb: misc: fix boolconv.cocci warnings Julia Lawall
@ 2018-11-23 15:10 ` Greg KH
  2018-11-23 15:29   ` Anatolij Gustschin
  0 siblings, 1 reply; 3+ messages in thread
From: Greg KH @ 2018-11-23 15:10 UTC (permalink / raw)
  To: Julia Lawall
  Cc: Anatolij Gustschin, linux-usb, linux-spi, linux-fpga,
	linux-kernel, broonie, atull, mdf, kbuild-all

On Wed, Nov 21, 2018 at 06:33:51PM +0100, Julia Lawall wrote:
> From: kbuild test robot <fengguang.wu@intel.com>
> 
> 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

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] usb: misc: fix boolconv.cocci warnings
  2018-11-23 15:10 ` Greg KH
@ 2018-11-23 15:29   ` Anatolij Gustschin
  0 siblings, 0 replies; 3+ messages in thread
From: Anatolij Gustschin @ 2018-11-23 15:29 UTC (permalink / raw)
  To: Greg KH
  Cc: Julia Lawall, linux-usb, linux-spi, linux-fpga, linux-kernel,
	broonie, atull, mdf, kbuild-all

On Fri, 23 Nov 2018 16:10:54 +0100
Greg KH gregkh@linuxfoundation.org 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

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-11-24  1:55 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-11-21 17:33 [PATCH] usb: misc: fix boolconv.cocci warnings Julia Lawall
2018-11-23 15:10 ` Greg KH
2018-11-23 15:29   ` Anatolij Gustschin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).