On Fri, 10 Oct 2025 15:54:27 +0200, Andrew Lunn andrew@lunn.ch wrote:
- ret = usb_register(&rtl8152_driver);
- if (ret) {
usb_deregister_device_driver(&rtl8152_cfgselector_driver);
return ret;
- }
- return ret;
Now look at this code and think about it.
Sorry, these two 'return ret' are indeed redundant. I will submit a new patch version to fix this.