diff mbox series

can: usb: ucan: modify unregister_netdev to unregister_candev

Message ID 20221024110033.727542-1-dzm91@hust.edu.cn (mailing list archive)
State Awaiting Upstream
Delegated to: Netdev Maintainers
Headers show
Series can: usb: ucan: modify unregister_netdev to unregister_candev | expand

Checks

Context Check Description
netdev/tree_selection success Series ignored based on subject

Commit Message

Dongliang Mu Oct. 24, 2022, 11 a.m. UTC 
>From API pairing, modify unregister_netdev to unregister_candev since
the registeration function is register_candev. Actually, they are the
same.

Signed-off-by: Dongliang Mu <dzm91@hust.edu.cn>
---
 drivers/net/can/usb/ucan.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Marc Kleine-Budde Oct. 24, 2022, 1:54 p.m. UTC | #1 
On 24.10.2022 19:00:30, Dongliang Mu wrote:
> From API pairing, modify unregister_netdev to unregister_candev since
> the registeration function is register_candev. Actually, they are the
            ^ typo
> same.
> 
> Signed-off-by: Dongliang Mu <dzm91@hust.edu.cn>

Fixed while applying.

Thanks,
Marc
Dongliang Mu Oct. 25, 2022, 2:27 a.m. UTC | #2 
> On Oct 24, 2022, at 21:54, Marc Kleine-Budde <mkl@pengutronix.de> wrote:
> 
> On 24.10.2022 19:00:30, Dongliang Mu wrote:
>> From API pairing, modify unregister_netdev to unregister_candev since
>> the registeration function is register_candev. Actually, they are the
>            ^ typo

:(

>> same.
>> 
>> Signed-off-by: Dongliang Mu <dzm91@hust.edu.cn>
> 
> Fixed while applying.

You mean it is already done in your own tree? If yes, that’s fine.

> 
> Thanks,
> Marc
> 
> -- 
> Pengutronix e.K.                 | Marc Kleine-Budde           |
> Embedded Linux                   | https://www.pengutronix.de  |
> Vertretung West/Dortmund         | Phone: +49-231-2826-924     |
> Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917-5555 |
Marc Kleine-Budde Oct. 25, 2022, 7:13 a.m. UTC | #3 
On 25.10.2022 10:27:12, Dongliang Mu wrote:
> > Fixed while applying.
> 
> You mean it is already done in your own tree? If yes, that’s fine.

ACK

Marc
diff mbox series

Patch

diff --git a/drivers/net/can/usb/ucan.c b/drivers/net/can/usb/ucan.c
index 7c35f50fda4e..60f730094e5d 100644
--- a/drivers/net/can/usb/ucan.c
+++ b/drivers/net/can/usb/ucan.c
@@ -1581,7 +1581,7 @@  static void ucan_disconnect(struct usb_interface *intf)
 	usb_set_intfdata(intf, NULL);
 
 	if (up) {
-		unregister_netdev(up->netdev);
+		unregister_candev(up->netdev);
 		free_candev(up->netdev);
 	}
 }