From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> Date: Sun, 29 Oct 2017 19:45:07 +0100 A goto statement jumped to a target which followed a condition check immediately without the specification of useful actions between. Thus remove such unnecessary source code at the end of this function. Signed-off-by: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> --- drivers/net/wireless/ti/wlcore/main.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/net/wireless/ti/wlcore/main.c b/drivers/net/wireless/ti/wlcore/main.c index 48380d48ef09..efea811c1c83 100644 --- a/drivers/net/wireless/ti/wlcore/main.c +++ b/drivers/net/wireless/ti/wlcore/main.c @@ -4085,9 +4085,6 @@ static int wlcore_set_beacon_template(struct wl1271 *wl, min_rate); end_bcn: dev_kfree_skb(beacon); - if (ret < 0) - goto out; - out: return ret; } -- 2.14.3 -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html