From: Jeff Garzik Date: Tue, 17 Jul 2007 04:01:09 +0000 (-0400) Subject: netdrvr: natsemi: Fix device removal bug X-Git-Tag: v2.6.22.11~10 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=c9a06c0099d53aab5853a52076a756c967c121a5;p=thirdparty%2Fkernel%2Fstable.git netdrvr: natsemi: Fix device removal bug 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 Cc: Karsten Keil Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/net/natsemi.c b/drivers/net/natsemi.c index 460a08718c69f..41f68ec3a52e6 100644 --- a/drivers/net/natsemi.c +++ b/drivers/net/natsemi.c @@ -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);