linux-kernel - Re: [PATCH] usb: chipidea: usbmisc_imx: fix returnvar.cocci warnings

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20150325062916.GB8235@shlinux2>
Date:	Wed, 25 Mar 2015 14:29:18 +0800
From:	Peter Chen <peter.chen@...escale.com>
To:	Julia Lawall <julia.lawall@...6.fr>
CC:	kbuild test robot <fengguang.wu@...el.com>, <kbuild@...org>,
	<linux-usb@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
	<gregkh@...uxfoundation.org>
Subject: Re: [PATCH] usb: chipidea: usbmisc_imx: fix returnvar.cocci warnings

On Wed, Mar 25, 2015 at 07:23:52AM +0100, Julia Lawall wrote:
> Please take a look.
> 

It is strange why I did not receive it.
The patch is ok.

Signed-off-by: Peter Chen <peter.chen@...escale.com>

> thanks,
> julia
> 
> On Wed, 25 Mar 2015, kbuild test robot wrote:
> 
> > TO: Peter Chen <peter.chen@...escale.com>
> > CC: "Greg Kroah-Hartman" <gregkh@...uxfoundation.org>
> > CC: linux-usb@...r.kernel.org
> > CC: linux-kernel@...r.kernel.org
> > 
> > drivers/usb/chipidea/usbmisc_imx.c:277:5-8: Unneeded variable: "ret". Return "0" on line 297
> > 
> > 
> >  Removes unneeded variable used to store return value.
> > 
> > Generated by: scripts/coccinelle/misc/returnvar.cocci
> > 
> > CC: Peter Chen <peter.chen@...escale.com>
> > Signed-off-by: Fengguang Wu <fengguang.wu@...el.com>
> > ---
> > 
> > Please take the patch only if it's a positive warning. Thanks!
> > 
> >  usbmisc_imx.c |    3 +--
> >  1 file changed, 1 insertion(+), 2 deletions(-)
> > 
> > --- a/drivers/usb/chipidea/usbmisc_imx.c
> > +++ b/drivers/usb/chipidea/usbmisc_imx.c
> > @@ -274,7 +274,6 @@ static int usbmisc_imx6sx_init(struct im
> >  	unsigned long flags;
> >  	struct imx_usbmisc *usbmisc = dev_get_drvdata(data->dev);
> >  	u32 val;
> > -	int ret = 0;
> >  
> >  	usbmisc_imx6q_init(data);
> >  
> > @@ -294,7 +293,7 @@ static int usbmisc_imx6sx_init(struct im
> >  		spin_unlock_irqrestore(&usbmisc->lock, flags);
> >  	}
> >  
> > -	return ret;
> > +	return 0;
> >  }
> >  
> >  static int usbmisc_vf610_init(struct imx_usbmisc_data *data)
> > 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-usb" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

-- 

Best Regards,
Peter Chen
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/