summaryrefslogtreecommitdiffstats
path: root/drivers/net/ethernet/cadence
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2017-07-04 16:09:59 +0100
committerDavid S. Miller <davem@davemloft.net>2017-07-05 09:09:36 +0100
commit42af627b400f32e5b17023f39c4fc24107ca8c3d (patch)
treeb3271dbe5e090aed93882e15b47a289496470f08 /drivers/net/ethernet/cadence
parent6d3f06a0042ebd59a5e9d4ba6e8a85596901e140 (diff)
downloadlinux-42af627b400f32e5b17023f39c4fc24107ca8c3d.tar.bz2
net: macb: remove extraneous return when MACB_EXT_DESC is defined
When macro MACB_EXT_DESC is defined we end up with two identical return statements and just one is sufficient. Remove the extra return. Detected by CoverityScan, CID#1449361 ("Structurally dead code") Signed-off-by: Colin Ian King <colin.king@canonical.com> Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/cadence')
-rw-r--r--drivers/net/ethernet/cadence/macb_main.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/net/ethernet/cadence/macb_main.c b/drivers/net/ethernet/cadence/macb_main.c
index 41e5711544fc..e69ebdd65658 100644
--- a/drivers/net/ethernet/cadence/macb_main.c
+++ b/drivers/net/ethernet/cadence/macb_main.c
@@ -146,7 +146,6 @@ static unsigned int macb_adj_dma_desc_idx(struct macb *bp, unsigned int desc_idx
default:
break;
}
- return desc_idx;
#endif
return desc_idx;
}