diff mbox series

[v1,1/2] Revert "usb: gadget: u_ether: Re-attach netif device to mirror detachment"

Message ID 20240606210436.54100-2-ftoth@exalondelft.nl (mailing list archive)
State Accepted
Commit 24bf27b92b1c6a322faa88977de2207aa8c26509
Headers show
Series "usb: gadget: u_ether: revert netif_device_detach change | expand

Commit Message

Ferry Toth June 6, 2024, 9:02 p.m. UTC
This reverts commit 76c945730cdffb572c7767073cc6515fd3f646b4.

Prerequisite revert for the reverting of the original commit f49449fbc21e.

Fixes: 76c945730cdf ("usb: gadget: u_ether: Re-attach netif device to mirror detachment")
Fixes: f49449fbc21e ("usb: gadget: u_ether: Replace netif_stop_queue with netif_device_detach")
Reported-by: Ferry Toth <fntoth@gmail.com>
Cc: stable@vger.kernel.org
---
 drivers/usb/gadget/function/u_ether.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Greg KH June 20, 2024, 5:35 p.m. UTC | #1
On Thu, Jun 06, 2024 at 11:02:31PM +0200, Ferry Toth wrote:
> This reverts commit 76c945730cdffb572c7767073cc6515fd3f646b4.
> 
> Prerequisite revert for the reverting of the original commit f49449fbc21e.
> 
> Fixes: 76c945730cdf ("usb: gadget: u_ether: Re-attach netif device to mirror detachment")
> Fixes: f49449fbc21e ("usb: gadget: u_ether: Replace netif_stop_queue with netif_device_detach")
> Reported-by: Ferry Toth <fntoth@gmail.com>
> Cc: stable@vger.kernel.org
> ---
>  drivers/usb/gadget/function/u_ether.c | 2 --
>  1 file changed, 2 deletions(-)

You have to sign-off on your changes, otherwise the tools will reject
them (as will I).  Please fix up for both of these and resend.

thanks,

greg k-h
Ferry Toth June 20, 2024, 7 p.m. UTC | #2
Hi Greg,

Op 20-06-2024 om 19:35 schreef Greg Kroah-Hartman:
> On Thu, Jun 06, 2024 at 11:02:31PM +0200, Ferry Toth wrote:
>> This reverts commit 76c945730cdffb572c7767073cc6515fd3f646b4.
>>
>> Prerequisite revert for the reverting of the original commit f49449fbc21e.
>>
>> Fixes: 76c945730cdf ("usb: gadget: u_ether: Re-attach netif device to mirror detachment")
>> Fixes: f49449fbc21e ("usb: gadget: u_ether: Replace netif_stop_queue with netif_device_detach")
>> Reported-by: Ferry Toth <fntoth@gmail.com>
>> Cc: stable@vger.kernel.org
>> ---
>>   drivers/usb/gadget/function/u_ether.c | 2 --
>>   1 file changed, 2 deletions(-)
> 
> You have to sign-off on your changes, otherwise the tools will reject
> them (as will I).  Please fix up for both of these and resend.

Oops, I knew that. Just didn't notice SoB wasn't auto added.
I'll that up, sorry.

> thanks,
> 
> greg k-h
diff mbox series

Patch

diff --git a/drivers/usb/gadget/function/u_ether.c b/drivers/usb/gadget/function/u_ether.c
index 11dd0b9e847f..aa0511c3a62c 100644
--- a/drivers/usb/gadget/function/u_ether.c
+++ b/drivers/usb/gadget/function/u_ether.c
@@ -1163,8 +1163,6 @@  struct net_device *gether_connect(struct gether *link)
 		if (netif_running(dev->net))
 			eth_start(dev, GFP_ATOMIC);
 
-		netif_device_attach(dev->net);
-
 	/* on error, disable any endpoints  */
 	} else {
 		(void) usb_ep_disable(link->out_ep);