return -EINVAL;
        }
 
-       for (i = ARRAY_SIZE(lp8725_buck_uA) - 1 ; i >= 0; i--) {
+       for (i = ARRAY_SIZE(lp8725_buck_uA) - 1; i >= 0; i--) {
                if (lp8725_buck_uA[i] >= min_uA &&
                        lp8725_buck_uA[i] <= max_uA)
                        return lp872x_update_bits(lp, addr,
        struct regulator_dev *rdev;
        int i, ret;
 
-       for (i = 0 ; i < lp->num_regulators ; i++) {
+       for (i = 0; i < lp->num_regulators; i++) {
                desc = (lp->chipid == LP8720) ? &lp8720_regulator_desc[i] :
                                                &lp8725_regulator_desc[i];
 
        struct regulator_dev *rdev;
        int i;
 
-       for (i = 0 ; i < lp->num_regulators ; i++) {
+       for (i = 0; i < lp->num_regulators; i++) {
                rdev = *(lp->regulators + i);
                regulator_unregister(rdev);
        }
                goto out;
 
        for (i = 0; i < num_matches; i++) {
-               pdata->regulator_data[i].id = (int)match[i].driver_data;
+               pdata->regulator_data[i].id =
+                               (enum lp872x_regulator_id)match[i].driver_data;
                pdata->regulator_data[i].init_data = match[i].init_data;
 
                /* Operation mode configuration for buck/buck1/buck2 */