summaryrefslogtreecommitdiffstats
path: root/drivers/char
diff options
context:
space:
mode:
authorThomas Meyer <thomas@m3y3r.de>2011-08-25 15:59:09 -0700
committerLinus Torvalds <torvalds@linux-foundation.org>2011-08-25 16:25:33 -0700
commit7e8aa048989bf7e0604996a3e2068fb1a81f81bd (patch)
tree58defa5a56ffa54b2a95fc857be487d318bf4dd3 /drivers/char
parent30ecad51849ae132dc6ef6ddb62d499c7257515b (diff)
downloadop-kernel-dev-7e8aa048989bf7e0604996a3e2068fb1a81f81bd.zip
op-kernel-dev-7e8aa048989bf7e0604996a3e2068fb1a81f81bd.tar.gz
drivers/char/msm_smd_pkt.c: don't use IS_ERR()
The various basic memory allocation function return NULL, not an ERR_PTR. The semantic patch that makes this change is available in scripts/coccinelle/null/eno.cocci. More information about semantic patching is available at http://coccinelle.lip6.fr/ Signed-off-by: Thomas Meyer <thomas@m3y3r.de> Cc: Niranjana Vishwanathapura <nvishwan@codeaurora.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/char')
-rw-r--r--drivers/char/msm_smd_pkt.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/char/msm_smd_pkt.c b/drivers/char/msm_smd_pkt.c
index b6f8a65..8eca55d 100644
--- a/drivers/char/msm_smd_pkt.c
+++ b/drivers/char/msm_smd_pkt.c
@@ -379,9 +379,8 @@ static int __init smd_pkt_init(void)
for (i = 0; i < NUM_SMD_PKT_PORTS; ++i) {
smd_pkt_devp[i] = kzalloc(sizeof(struct smd_pkt_dev),
GFP_KERNEL);
- if (IS_ERR(smd_pkt_devp[i])) {
- r = PTR_ERR(smd_pkt_devp[i]);
- pr_err("kmalloc() failed %d\n", r);
+ if (!smd_pkt_devp[i]) {
+ pr_err("kmalloc() failed\n");
goto clean_cdevs;
}
OpenPOWER on IntegriCloud