Checkpatch output:
WARNING: Block comments use a trailing */ on a separate line
+        * - turn off */
WARNING: Missing a blank line after declarations
+       int ret;
+       ret = i2c_add_driver(&pm860x_driver);
total: 0 errors, 2 warnings, 1283 lines checked
Signed-off-by: Lee Jones <lee.jones@linaro.org>
                        chip->osc_status);
 
        mutex_lock(&chip->osc_lock);
-       /*Update voting status */
+       /* Update voting status */
        chip->osc_vote &= ~(client);
-       /* If reference group is off and this is the last client to release
-        * - turn off */
+       /*
+        * If reference group is off and this is the last client to release
+        * - turn off
+        */
        if ((chip->osc_status != PM8606_REF_GP_OSC_OFF) &&
                        (chip->osc_vote == REF_GP_NO_CLIENTS)) {
                chip->osc_status = PM8606_REF_GP_OSC_UNKNOWN;
 static int __init pm860x_i2c_init(void)
 {
        int ret;
+
        ret = i2c_add_driver(&pm860x_driver);
        if (ret != 0)
                pr_err("Failed to register 88PM860x I2C driver: %d\n", ret);