diff options
author | Ralf Baechle <ralf@linux-mips.org> | 2007-08-22 11:03:52 -0400 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-08-25 02:31:13 -0400 |
commit | e3efb05468128e834cf17d492822333c6e189ae4 (patch) | |
tree | 04aaf93bdfa6210dac5116d9e6359e663fff9fb8 | |
parent | c2cb71fafb4d514fbb8c9a8d663486a8f0400afa (diff) |
sgiseeq: Fix return type of sgiseeq_remove
The driver remove method needs to return an int not void. This was just
never noticed because usually this driver is not being built as a module.
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
-rw-r--r-- | drivers/net/sgiseeq.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/net/sgiseeq.c b/drivers/net/sgiseeq.c index 384b4685e977..0fb74cb51c4b 100644 --- a/drivers/net/sgiseeq.c +++ b/drivers/net/sgiseeq.c | |||
@@ -726,7 +726,7 @@ err_out: | |||
726 | return err; | 726 | return err; |
727 | } | 727 | } |
728 | 728 | ||
729 | static void __exit sgiseeq_remove(struct platform_device *pdev) | 729 | static int __exit sgiseeq_remove(struct platform_device *pdev) |
730 | { | 730 | { |
731 | struct net_device *dev = platform_get_drvdata(pdev); | 731 | struct net_device *dev = platform_get_drvdata(pdev); |
732 | struct sgiseeq_private *sp = netdev_priv(dev); | 732 | struct sgiseeq_private *sp = netdev_priv(dev); |
@@ -735,6 +735,8 @@ static void __exit sgiseeq_remove(struct platform_device *pdev) | |||
735 | free_page((unsigned long) sp->srings); | 735 | free_page((unsigned long) sp->srings); |
736 | free_netdev(dev); | 736 | free_netdev(dev); |
737 | platform_set_drvdata(pdev, NULL); | 737 | platform_set_drvdata(pdev, NULL); |
738 | |||
739 | return 0; | ||
738 | } | 740 | } |
739 | 741 | ||
740 | static struct platform_driver sgiseeq_driver = { | 742 | static struct platform_driver sgiseeq_driver = { |