From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nicolas Ferre Subject: Re: [PATCH] spi/atmel: fix simple_return.cocci warnings Date: Fri, 17 Oct 2014 11:05:44 +0200 Message-ID: <5440DBE8.90207@atmel.com> References: <543feff0.QhS1UtLsJt/o5948%fengguang.wu@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Cc: , , , Wenyou Yang To: Fengguang Wu , Mark Brown Return-path: In-Reply-To: <543feff0.QhS1UtLsJt/o5948%fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> Sender: linux-spi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: On 16/10/2014 18:18, Fengguang Wu : > drivers/spi/spi-atmel.c:1518:1-4: WARNING: end returns can be simpified and declaration on line 1514 can be dropped > > Simplify a trivial if-return sequence. Possibly combine with a > preceding function call. > Generated by: scripts/coccinelle/misc/simple_return.cocci > > CC: Wenyou Yang > Signed-off-by: Fengguang Wu > --- > > Please take the patch only if it's a positive warning. Thanks! > > spi-atmel.c | 7 +------ > 1 file changed, 1 insertion(+), 6 deletions(-) > > --- a/drivers/spi/spi-atmel.c > +++ b/drivers/spi/spi-atmel.c > @@ -1511,15 +1511,10 @@ static int atmel_spi_runtime_resume(stru > { > struct spi_master *master = dev_get_drvdata(dev); > struct atmel_spi *as = spi_master_get_devdata(master); > - int ret; > > pinctrl_pm_select_default_state(dev); > > - ret = clk_prepare_enable(as->clk); > - if (ret) > - return ret; > - > - return 0; > + return clk_prepare_enable(as->clk); Indeed: Acked-by: Nicolas Ferre > } > #endif > > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- Nicolas Ferre -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html