drivers/atm: No need to return void
Subsystems (2)
PATCH 1/1 drivers/atm: No need to return void
The module_exit function has return-type void andpci_unregister_driver() returns void anyway.
Signed-off-by Tobias Klauser <[email protected]>
--- drivers/atm/ambassador.c | 4 ++-- drivers/atm/horizon.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-)
drivers/atm/ambassador.c +2 -2
@@ -2459,8 +2459,8 @@ static int __init amb_module_init (void) static void __exit amb_module_exit (void) {PRINTD (DBG_FLOW |DBG_INIT ,"cleanup_module" ); - -return pci_unregister_driver (&amb_driver); + +pci_unregister_driver (&amb_driver); }module_init (amb_module_init);
drivers/atm/horizon.c +2 -2
@@ -2945,8 +2945,8 @@ static int __init hrz_module_init (void) static void __exit hrz_module_exit (void) {PRINTD (DBG_FLOW ,"cleanup_module" ); - -return pci_unregister_driver (&hrz_driver); + +pci_unregister_driver (&hrz_driver); }module_init (hrz_module_init);
Signature · 6 lines
1.4.2 _______________________________________________ Kernel-janitors mailing list [email protected] https://lists.osdl.org/mailman/listinfo/kernel-janitors
drivers/atm/ambassador.c static int __init amb_module_init (void) · line 2463
Quoted 5 lines · Tobias Klauser
The
module_exitfunction has return-type void andpci_unregister_driver()returns void anyway.drivers/atm/ambassador.c +1
-1-return pci_unregister_driver (&amb_driver); +pci_unregister_driver (&amb_driver);
Logical continuation would be a patch so sparse(1) would start warning about such things.
_______________________________________________
Kernel-janitors mailing list
[email protected]
https://lists.osdl.org/mailman/listinfo/kernel-janitors