{ 0x0d, 0x00, 0x01, },  /* reset and use defaults */
                { 0x0d, 0x00, 0x00, },
                { 0x0a, 0x00, 0x21, },
-               { 0x21, 0x04, 0x00, },  /* full readout speed, no row/col skipping */
+               { 0x21, 0x04, 0x00, },  /* full readout spd, no row/col skip */
        };
 
        for (i = 0; i < ARRAY_SIZE(regs); i++)
                        break;
                default:
                        dev_info(&dev->intf->dev,
-                                "unknown Micron sensor detected: 0x%04x\n", id);
+                                "unknown Micron sensor detected: 0x%04x\n",
+                                id);
                        return 0;
                }
 
        struct i2c_client *client = &dev->i2c_client[dev->def_i2c_bus];
 
        dev->em28xx_sensor = EM28XX_NOSENSOR;
-       /* NOTE: these devices have the register auto incrementation disabled
-        * by default, so we have to use single byte reads !              */
+       /*
+        * NOTE: these devices have the register auto incrementation disabled
+        * by default, so we have to use single byte reads !
+        */
        for (i = 0; omnivision_sensor_addrs[i] != I2C_CLIENT_END; i++) {
                client->addr = omnivision_sensor_addrs[i];
                /* Read manufacturer ID from registers 0x1c-0x1d (BE) */
                subdev =
                     v4l2_i2c_new_subdev_board(&v4l2->v4l2_dev, adap,
                                               &ov2640_info, NULL);
-               if (subdev == NULL)
+               if (!subdev)
                        return -ENODEV;
 
                format.format.code = MEDIA_BUS_FMT_YUYV8_2X8;