[netdrvr] natsemi: Fix device removal bug
authorJeff Garzik <jeff@garzik.org>
Tue, 17 Jul 2007 04:01:09 +0000 (00:01 -0400)
committerJeff Garzik <jeff@garzik.org>
Tue, 17 Jul 2007 20:17:59 +0000 (16:17 -0400)
This episode illustrates how an overused warning can train people to
ignore that warning, which winds up hiding bugs.

The warning

drivers/net/natsemi.c: In function ‘natsemi_remove1’:
drivers/net/natsemi.c:3222: warning: ignoring return value of
‘device_create_file’, declared with attribute warn_unused_result

is oft-ignored, even though at close inspection one notices this occurs
in the /remove/ function, not normally where creation occurs.  A quick
s/create/remove/ and we are fixed, with the warning gone.

Signed-off-by: Jeff Garzik <jeff@garzik.org>
drivers/net/natsemi.c

index 3450051ae56b635c260f56d159314466eb0e5a0a..6bb48ba809649ca7e3ab940f42ba98ba1f9dc643 100644 (file)
@@ -671,7 +671,7 @@ static ssize_t natsemi_show_##_name(struct device *dev, \
 #define NATSEMI_CREATE_FILE(_dev, _name) \
          device_create_file(&_dev->dev, &dev_attr_##_name)
 #define NATSEMI_REMOVE_FILE(_dev, _name) \
-         device_create_file(&_dev->dev, &dev_attr_##_name)
+         device_remove_file(&_dev->dev, &dev_attr_##_name)
 
 NATSEMI_ATTR(dspcfg_workaround);