projects / openwrt / staging / blogic.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 5d0c100) | patch
sh_eth: kill redundant check in the probe() method
authorSergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Sun, 31 Dec 2017 18:41:36 +0000 (21:41 +0300)
committerDavid S. Miller <davem@davemloft.net>
Wed, 3 Jan 2018 15:21:35 +0000 (10:21 -0500)
commit4282fc47c01262fdda55a9d63a4e25173fd9afb4
treeaf2fd522bbc6d9d22f7d34de40fdd1efeca5473ctree | snapshot
parent5d0c100c228b51ce2c14bdbc845ca446e0494689commit | diff
sh_eth: kill redundant check in the probe() method

Browsing thru the driver disassembly, I noticed that gcc was  able to
figure  out  that the 'ndev' pointer is always non-NULL when calling
free_netdev()  on the probe() method's  error path and  thus skip that
redundant NULL check... gcc is smart, be like gcc! :-)

Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/renesas/sh_eth.c diff | blob | history
John Crispins staging tree