linux-usb.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] net: usb: remove duplicate assignment
@ 2022-03-31 12:08 Qing Wang
  2022-03-31 12:58 ` Bjørn Mork
  2022-03-31 15:46 ` Jakub Kicinski
  0 siblings, 2 replies; 3+ messages in thread
From: Qing Wang @ 2022-03-31 12:08 UTC (permalink / raw)
  To: Bjørn Mork, David S. Miller, Jakub Kicinski, netdev,
	linux-usb, linux-kernel
  Cc: Wang Qing

From: Wang Qing <wangqing@vivo.com>

netdev_alloc_skb() has assigned ssi->netdev to skb->dev if successed,
no need to repeat assignment.

Signed-off-by: Wang Qing <wangqing@vivo.com>
---
 drivers/net/usb/qmi_wwan.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/net/usb/qmi_wwan.c b/drivers/net/usb/qmi_wwan.c
index 3353e76..17cf0d1
--- a/drivers/net/usb/qmi_wwan.c
+++ b/drivers/net/usb/qmi_wwan.c
@@ -190,7 +190,6 @@ static int qmimux_rx_fixup(struct usbnet *dev, struct sk_buff *skb)
 		skbn = netdev_alloc_skb(net, pkt_len + LL_MAX_HEADER);
 		if (!skbn)
 			return 0;
-		skbn->dev = net;
 
 		switch (skb->data[offset + qmimux_hdr_sz] & 0xf0) {
 		case 0x40:
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] net: usb: remove duplicate assignment
  2022-03-31 12:08 [PATCH] net: usb: remove duplicate assignment Qing Wang
@ 2022-03-31 12:58 ` Bjørn Mork
  2022-03-31 15:46 ` Jakub Kicinski
  1 sibling, 0 replies; 3+ messages in thread
From: Bjørn Mork @ 2022-03-31 12:58 UTC (permalink / raw)
  To: Qing Wang
  Cc: David S. Miller, Jakub Kicinski, netdev, linux-usb, linux-kernel

Qing Wang <wangqing@vivo.com> writes:

> netdev_alloc_skb() has assigned ssi->netdev to skb->dev if successed,
> no need to repeat assignment.

Thanks.

Acked-by: Bjørn Mork <bjorn@mork.no>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] net: usb: remove duplicate assignment
  2022-03-31 12:08 [PATCH] net: usb: remove duplicate assignment Qing Wang
  2022-03-31 12:58 ` Bjørn Mork
@ 2022-03-31 15:46 ` Jakub Kicinski
  1 sibling, 0 replies; 3+ messages in thread
From: Jakub Kicinski @ 2022-03-31 15:46 UTC (permalink / raw)
  To: Qing Wang
  Cc: Bjørn Mork, David S. Miller, netdev, linux-usb, linux-kernel

On Thu, 31 Mar 2022 05:08:14 -0700 Qing Wang wrote:
> From: Wang Qing <wangqing@vivo.com>
> 
> netdev_alloc_skb() has assigned ssi->netdev to skb->dev if successed,
> no need to repeat assignment.
> 
> Signed-off-by: Wang Qing <wangqing@vivo.com>

# Form letter - net-next is closed

We have already sent the networking pull request for 5.18
and therefore net-next is closed for new drivers, features,
code refactoring and optimizations. We are currently accepting
bug fixes only.

Please repost when net-next reopens after 5.18-rc1 is cut.

RFC patches sent for review only are obviously welcome at any time.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-03-31 15:47 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-31 12:08 [PATCH] net: usb: remove duplicate assignment Qing Wang
2022-03-31 12:58 ` Bjørn Mork
2022-03-31 15:46 ` Jakub Kicinski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).