diff options
author | Julia Lawall <julia@diku.dk> | 2010-05-28 16:58:46 -0400 |
---|---|---|
committer | Samuel Ortiz <sameo@linux.intel.com> | 2010-08-12 05:27:19 -0400 |
commit | 25fe24f884015ba2e1e39376deb055bbbdc5ba83 (patch) | |
tree | 75aa1f9cdefabc453908f150a66cf8be7ff4df14 /drivers/mfd | |
parent | c0d4010e11e23d6f55f9bdda63b1974f35af71bf (diff) |
mfd: kzalloc doesn't return ERR_PTR
Use !x rather than IS_ERR(x) to test the result of kzalloc.
The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)
// <smpl>
@@
expression x,E;
@@
x = \(kmalloc\|kzalloc\|kcalloc\)(...)
... when != x = E
- IS_ERR(x)
+ !x
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
Diffstat (limited to 'drivers/mfd')
-rw-r--r-- | drivers/mfd/abx500-core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mfd/abx500-core.c b/drivers/mfd/abx500-core.c index 3b3b97ec32a..f12720dbe12 100644 --- a/drivers/mfd/abx500-core.c +++ b/drivers/mfd/abx500-core.c | |||
@@ -36,7 +36,7 @@ int abx500_register_ops(struct device *dev, struct abx500_ops *ops) | |||
36 | struct abx500_device_entry *dev_entry; | 36 | struct abx500_device_entry *dev_entry; |
37 | 37 | ||
38 | dev_entry = kzalloc(sizeof(struct abx500_device_entry), GFP_KERNEL); | 38 | dev_entry = kzalloc(sizeof(struct abx500_device_entry), GFP_KERNEL); |
39 | if (IS_ERR(dev_entry)) { | 39 | if (!dev_entry) { |
40 | dev_err(dev, "register_ops kzalloc failed"); | 40 | dev_err(dev, "register_ops kzalloc failed"); |
41 | return -ENOMEM; | 41 | return -ENOMEM; |
42 | } | 42 | } |