Messages in this thread | | | Date | Sun, 04 Jan 2015 22:20:22 -0500 (EST) | Subject | Re: [PATCH] GMAC: fix simple_return.cocci warnings | From | David Miller <> |
| |
From: Joe Perches <joe@perches.com> Date: Fri, 02 Jan 2015 16:46:45 -0800
> On Sat, 2015-01-03 at 08:25 +0800, kbuild test robot wrote: >> drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c:425:1-4: WARNING: end returns can be simpified >> >> Simplify a trivial if-return sequence. Possibly combine with a >> preceding function call. >> Generated by: scripts/coccinelle/misc/simple_return.cocci >> >> CC: Roger Chen <roger.chen@rock-chips.com> >> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com> >> --- >> >> dwmac-rk.c | 6 +----- >> 1 file changed, 1 insertion(+), 5 deletions(-) >> >> --- a/drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c >> +++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c >> @@ -422,11 +422,7 @@ static int rk_gmac_init(struct platform_ >> if (ret) >> return ret; >> >> - ret = gmac_clk_enable(bsp_priv, true); >> - if (ret) >> - return ret; >> - >> - return 0; >> + return gmac_clk_enable(bsp_priv, true); > > I think this change is not particularly better. > > When the pattern is multiply repeated like: ... > I think it's better to not change the last > test in the sequence just to minimize overall > line count.
I think it's a wash and that both ways are about the same to me.
I won't apply this, sorry.
| |