From mboxrd@z Thu Jan  1 00:00:00 1970
From: David Miller <davem@davemloft.net>
Subject: Re: [PATCH net-next] net: caif: fix return type of ndo_start_xmit
 function
Date: Sat, 29 Sep 2018 11:38:02 -0700 (PDT)
Message-ID: <20180929.113802.1016843680571572182.davem@davemloft.net>
References: <20180926092705.28880-1-yuehaibing@huawei.com>
Mime-Version: 1.0
Content-Type: Text/Plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Cc: dmitry.tarnyagin@lockless.no, linux-kernel@vger.kernel.org,
        netdev@vger.kernel.org
To: yuehaibing@huawei.com
Return-path: <linux-kernel-owner@vger.kernel.org>
In-Reply-To: <20180926092705.28880-1-yuehaibing@huawei.com>
Sender: linux-kernel-owner@vger.kernel.org
List-Id: netdev.vger.kernel.org

From: YueHaibing <yuehaibing@huawei.com>
Date: Wed, 26 Sep 2018 17:27:05 +0800

> @@ -1014,7 +1014,7 @@ static int cfhsi_xmit(struct sk_buff *skb, struct net_device *dev)
>  	int prio;
>  
>  	if (!dev)
> -		return -EINVAL;
> +		return NETDEV_TX_BUSY;
>  
>  	cfhsi = netdev_priv(dev);
>  

Even though the return type of ndo_start_xmit is netdev_tx_t, negative error codes are
still allowed I believe.

Look, reviewing these are pretty stressful for me, because you aren't documenting your
changes and in many cases the transformations look incorrect.

I'm tossing the rest of your changes in this area for now, sorry.

Please double check your work and resubmit this at some time in the not-too-near
future.

Thank you.