staging: [media] mn88472: simplify NULL tests
authorEva Rachel Retuya <eraretuya@gmail.com>
Sat, 27 Feb 2016 12:39:22 +0000 (20:39 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 12 Mar 2016 06:09:09 +0000 (22:09 -0800)
Replace direct comparisons to NULL i.e. 'x == NULL' with '!x' for
consistency. Coccinelle semantic patch used:

@@
identifier func;
expression x;
statement Z;
@@

x = func(...);

if (
(
+ !
x
- == NULL
|
+ !
- NULL ==
x
)
   ) Z

Signed-off-by: Eva Rachel Retuya <eraretuya@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/media/mn88472/mn88472.c

index c2f2a63ef3ce14f1e62e1564de1a7c95e9c7b2ca..7ea749cf19f992d6e7cc67a5bd36ea94bde4ae87 100644 (file)
@@ -456,7 +456,7 @@ static int mn88472_probe(struct i2c_client *client,
        }
 
        dev = kzalloc(sizeof(*dev), GFP_KERNEL);
-       if (dev == NULL) {
+       if (!dev) {
                ret = -ENOMEM;
                goto err;
        }
@@ -483,7 +483,7 @@ static int mn88472_probe(struct i2c_client *client,
         * 0x1a and 0x1c, in order to get own I2C client for each register page.
         */
        dev->client[1] = i2c_new_dummy(client->adapter, 0x1a);
-       if (dev->client[1] == NULL) {
+       if (!dev->client[1]) {
                ret = -ENODEV;
                dev_err(&client->dev, "I2C registration failed\n");
                if (ret)
@@ -497,7 +497,7 @@ static int mn88472_probe(struct i2c_client *client,
        i2c_set_clientdata(dev->client[1], dev);
 
        dev->client[2] = i2c_new_dummy(client->adapter, 0x1c);
-       if (dev->client[2] == NULL) {
+       if (!dev->client[2]) {
                ret = -ENODEV;
                dev_err(&client->dev, "2nd I2C registration failed\n");
                if (ret)
This page took 0.040893 seconds and 5 git commands to generate.