From 22d99df3403103ebacb384f1c573e5c40fc55182 Mon Sep 17 00:00:00 2001 From: Srinivas Girigowda Date: Thu, 5 Apr 2018 09:23:42 -0700 Subject: [PATCH] qcacmn: Fix function return type for epping_hard_start_xmit Fix function return type for epping_hard_start_xmit. Currently .ndo_start_xmit callback functions returns int, but the correct return type should be netdev_tx_t. Change-Id: I36d3cc886bfa0fd74a264f2791f09a251baab2ef CRs-Fixed: 2219223 --- utils/epping/src/epping_txrx.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/utils/epping/src/epping_txrx.c b/utils/epping/src/epping_txrx.c index cf0bfb3a74..1179742765 100644 --- a/utils/epping/src/epping_txrx.c +++ b/utils/epping/src/epping_txrx.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2014-2017 The Linux Foundation. All rights reserved. + * Copyright (c) 2014-2018 The Linux Foundation. All rights reserved. * * Previously licensed under the ISC license by Qualcomm Atheros, Inc. * @@ -144,7 +144,8 @@ end: } -static int epping_hard_start_xmit(struct sk_buff *skb, struct net_device *dev) +static netdev_tx_t epping_hard_start_xmit(struct sk_buff *skb, + struct net_device *dev) { epping_adapter_t *adapter; int ret = 0; @@ -153,12 +154,13 @@ static int epping_hard_start_xmit(struct sk_buff *skb, struct net_device *dev) if (NULL == adapter) { EPPING_LOG(QDF_TRACE_LEVEL_FATAL, "%s: EPPING adapter context is Null", __func__); + kfree_skb(skb); ret = -ENODEV; goto end; } ret = epping_tx_send(skb, adapter); end: - return ret; + return NETDEV_TX_OK; } static struct net_device_stats *epping_get_stats(struct net_device *dev)