From: Michal Piotrowski <michal.k.k.piotrowski@gmail.com>
To: Andrew Morton <akpm@osdl.org>, Frodo Looijaard <frodol@dds.nl>,
Philip Edelbrock <phil@netroedge.com>,
Mark Studebaker <mdsxyz123@yahoo.com>,
lm-sensors@lm-sensors.org, LKML <linux-kernel@vger.kernel.org>
Subject: [RFC][PATCH] hwmon:fix sparse warnings + error handling
Date: Sun, 20 Aug 2006 22:44:30 +0200 [thread overview]
Message-ID: <44E8C9AE.3060307@gmail.com> (raw)
Hi,
This patch fixes 56 sparse "ignoring return value of 'device_create_file'" warnings. It also adds error handling.
w83627hf.c | 96 ++++++++++++++++++++++++++++++++++++++++++++++++++-----------
1 file changed, 80 insertions(+), 16 deletions(-)
Regards,
Michal
--
Michal K. K. Piotrowski
LTG - Linux Testers Group
(http://www.stardust.webpages.pl/ltg/wiki/)
Signed-off-by: Michal Piotrowski <michal.k.k.piotrowski@gmail.com>
diff -uprN -X linux-work/Documentation/dontdiff linux-work-clean/drivers/hwmon/w83627hf.c linux-work/drivers/hwmon/w83627hf.c
--- linux-work-clean/drivers/hwmon/w83627hf.c 2006-08-20 22:02:40.000000000 +0200
+++ linux-work/drivers/hwmon/w83627hf.c 2006-08-20 22:27:14.000000000 +0200
@@ -513,9 +513,21 @@ static DEVICE_ATTR(in0_max, S_IRUGO | S_
#define device_create_file_in(client, offset) \
do { \
-device_create_file(&client->dev, &dev_attr_in##offset##_input); \
-device_create_file(&client->dev, &dev_attr_in##offset##_min); \
-device_create_file(&client->dev, &dev_attr_in##offset##_max); \
+ err = device_create_file(&client->dev, &dev_attr_in##offset##_input); \
+ if (err) {\
+ hwmon_device_unregister(data->class_dev); \
+ return err; \
+ } \
+ err = device_create_file(&client->dev, &dev_attr_in##offset##_min); \
+ if (err) {\
+ hwmon_device_unregister(data->class_dev); \
+ return err; \
+ } \
+ err = device_create_file(&client->dev, &dev_attr_in##offset##_max); \
+ if (err) {\
+ hwmon_device_unregister(data->class_dev); \
+ return err; \
+ } \
} while (0)
#define show_fan_reg(reg) \
@@ -577,8 +589,16 @@ sysfs_fan_min_offset(3);
#define device_create_file_fan(client, offset) \
do { \
-device_create_file(&client->dev, &dev_attr_fan##offset##_input); \
-device_create_file(&client->dev, &dev_attr_fan##offset##_min); \
+ err = device_create_file(&client->dev, &dev_attr_fan##offset##_input); \
+ if (err) {\
+ hwmon_device_unregister(data->class_dev); \
+ return err; \
+ } \
+ err = device_create_file(&client->dev, &dev_attr_fan##offset##_min); \
+ if (err) {\
+ hwmon_device_unregister(data->class_dev); \
+ return err; \
+ } \
} while (0)
#define show_temp_reg(reg) \
@@ -657,9 +677,21 @@ sysfs_temp_offsets(3);
#define device_create_file_temp(client, offset) \
do { \
-device_create_file(&client->dev, &dev_attr_temp##offset##_input); \
-device_create_file(&client->dev, &dev_attr_temp##offset##_max); \
-device_create_file(&client->dev, &dev_attr_temp##offset##_max_hyst); \
+ err = device_create_file(&client->dev, &dev_attr_temp##offset##_input); \
+ if (err) {\
+ hwmon_device_unregister(data->class_dev); \
+ return err; \
+ } \
+ err = device_create_file(&client->dev, &dev_attr_temp##offset##_max); \
+ if (err) {\
+ hwmon_device_unregister(data->class_dev); \
+ return err; \
+ } \
+ err = device_create_file(&client->dev, &dev_attr_temp##offset##_max_hyst); \
+ if (err) {\
+ hwmon_device_unregister(data->class_dev); \
+ return err; \
+ } \
} while (0)
static ssize_t
@@ -670,7 +702,11 @@ show_vid_reg(struct device *dev, struct
}
static DEVICE_ATTR(cpu0_vid, S_IRUGO, show_vid_reg, NULL);
#define device_create_file_vid(client) \
-device_create_file(&client->dev, &dev_attr_cpu0_vid)
+err = device_create_file(&client->dev, &dev_attr_cpu0_vid); \
+if (err) { \
+ hwmon_device_unregister(data->class_dev); \
+ return err; \
+} \
static ssize_t
show_vrm_reg(struct device *dev, struct device_attribute *attr, char *buf)
@@ -692,7 +728,11 @@ store_vrm_reg(struct device *dev, struct
}
static DEVICE_ATTR(vrm, S_IRUGO | S_IWUSR, show_vrm_reg, store_vrm_reg);
#define device_create_file_vrm(client) \
-device_create_file(&client->dev, &dev_attr_vrm)
+err = device_create_file(&client->dev, &dev_attr_vrm); \
+if (err) {\
+ hwmon_device_unregister(data->class_dev); \
+ return err; \
+} \
static ssize_t
show_alarms_reg(struct device *dev, struct device_attribute *attr, char *buf)
@@ -702,7 +742,11 @@ show_alarms_reg(struct device *dev, stru
}
static DEVICE_ATTR(alarms, S_IRUGO, show_alarms_reg, NULL);
#define device_create_file_alarms(client) \
-device_create_file(&client->dev, &dev_attr_alarms)
+err = device_create_file(&client->dev, &dev_attr_alarms); \
+if (err) {\
+ hwmon_device_unregister(data->class_dev); \
+ return err; \
+} \
#define show_beep_reg(REG, reg) \
static ssize_t show_beep_##reg (struct device *dev, struct device_attribute *attr, char *buf) \
@@ -767,8 +811,16 @@ sysfs_beep(MASK, mask);
#define device_create_file_beep(client) \
do { \
-device_create_file(&client->dev, &dev_attr_beep_enable); \
-device_create_file(&client->dev, &dev_attr_beep_mask); \
+ err = device_create_file(&client->dev, &dev_attr_beep_enable); \
+ if (err) {\
+ hwmon_device_unregister(data->class_dev); \
+ return err; \
+ } \
+ err = device_create_file(&client->dev, &dev_attr_beep_mask); \
+ if (err) {\
+ hwmon_device_unregister(data->class_dev); \
+ return err; \
+ } \
} while (0)
static ssize_t
@@ -838,7 +890,11 @@ sysfs_fan_div(3);
#define device_create_file_fan_div(client, offset) \
do { \
-device_create_file(&client->dev, &dev_attr_fan##offset##_div); \
+ err = device_create_file(&client->dev, &dev_attr_fan##offset##_div); \
+ if (err) {\
+ hwmon_device_unregister(data->class_dev); \
+ return err; \
+ } \
} while (0)
static ssize_t
@@ -897,7 +953,11 @@ sysfs_pwm(3);
#define device_create_file_pwm(client, offset) \
do { \
-device_create_file(&client->dev, &dev_attr_pwm##offset); \
+ err = device_create_file(&client->dev, &dev_attr_pwm##offset); \
+ if (err) {\
+ hwmon_device_unregister(data->class_dev); \
+ return err; \
+ } \
} while (0)
static ssize_t
@@ -973,7 +1033,11 @@ sysfs_sensor(3);
#define device_create_file_sensor(client, offset) \
do { \
-device_create_file(&client->dev, &dev_attr_temp##offset##_type); \
+ err = device_create_file(&client->dev, &dev_attr_temp##offset##_type); \
+ if (err) {\
+ hwmon_device_unregister(data->class_dev); \
+ return err; \
+ } \
} while (0)
next reply other threads:[~2006-08-20 20:43 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-08-20 20:44 Michal Piotrowski [this message]
2006-08-20 21:49 ` [lm-sensors] [RFC][PATCH] hwmon:fix sparse warnings + error handling jim.cromie
2006-08-20 22:12 ` Michal Piotrowski
2006-08-20 22:28 ` Alan Cox
2006-08-21 2:30 ` [lm-sensors] " Mark M. Hoffman
2006-08-21 9:11 ` Jean Delvare
2006-08-21 16:46 ` Grant Coady
2006-08-21 8:04 ` Jean Delvare
2006-08-22 0:43 ` Dmitry Torokhov
2006-08-22 11:34 ` Mark M. Hoffman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=44E8C9AE.3060307@gmail.com \
--to=michal.k.k.piotrowski@gmail.com \
--cc=akpm@osdl.org \
--cc=frodol@dds.nl \
--cc=linux-kernel@vger.kernel.org \
--cc=lm-sensors@lm-sensors.org \
--cc=mdsxyz123@yahoo.com \
--cc=phil@netroedge.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox