regulator: pv88080: Update regulator for PV88080 BB silicon support
authorEric Jeong <eric.jeong.opensource@diasemi.com>
Mon, 26 Sep 2016 07:33:27 +0000 (16:33 +0900)
committerMark Brown <broonie@kernel.org>
Mon, 26 Sep 2016 15:59:08 +0000 (08:59 -0700)
Three files are modified, the driver, header file and the binding document.

Updates for the regulator source file include and .of_match_table entry
and node match checking in the probe() function for a compatible pv88080
silicon type. A new "HVBUCK" is added in source file and added
regsiter definition in header file for pv88080 bb silicion.
The binding documentation changes have been made to reflect these updates.

Signed-off-by: Eric Jeong <eric.jeong.opensource@diasemi.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Documentation/devicetree/bindings/regulator/pv88080.txt
drivers/regulator/pv88080-regulator.c
drivers/regulator/pv88080-regulator.h

index 38a614210dcb157f66d20da89c54043a9a1a1790..e6e4b9c82d8902a1a4a5f806fd4c0b2b5f891437 100644 (file)
@@ -1,22 +1,28 @@
 * Powerventure Semiconductor PV88080 Voltage Regulator
 
 Required properties:
-- compatible: "pvs,pv88080".
-- reg: I2C slave address, usually 0x49.
+- compatible: Must be one of the following, depending on the
+  silicon version:
+       - "pvs,pv88080" (DEPRECATED)
+
+       - "pvs,pv88080-aa" for PV88080 AA or AB silicon
+       - "pvs,pv88080-ba" for PV88080 BA or BB silicon
+  NOTE: The use of the compatibles with no silicon version is deprecated.
+- reg: I2C slave address, usually 0x49
 - interrupts: the interrupt outputs of the controller
 - regulators: A node that houses a sub-node for each regulator within the
   device. Each sub-node is identified using the node's name, with valid
   values listed below. The content of each sub-node is defined by the
   standard binding for regulators; see regulator.txt.
-  BUCK1, BUCK2, and BUCK3.
+  BUCK1, BUCK2, BUCK3 and HVBUCK.
 
 Optional properties:
 - Any optional property defined in regulator.txt
 
-Example
+Example:
 
        pmic: pv88080@49 {
-               compatible = "pvs,pv88080";
+               compatible = "pvs,pv88080-ba";
                reg = <0x49>;
                interrupt-parent = <&gpio>;
                interrupts = <24 24>;
@@ -45,5 +51,12 @@ Example
                                regulator-min-microamp  = <1496000>;
                                regulator-max-microamp  = <4189000>;
                        };
+
+                       HVBUCK {
+                               regulator-name = "hvbuck";
+                               regulator-min-microvolt = <   5000>;
+                               regulator-max-microvolt = <1275000>;
+                       };
                };
        };
+
index 81950bdb1cc426b8bbb20e017860084055e0f717..954a20eeb26f89362be0a4274d364b22a8db3783 100644 (file)
@@ -16,6 +16,7 @@
 #include <linux/err.h>
 #include <linux/i2c.h>
 #include <linux/module.h>
+#include <linux/of.h>
 #include <linux/init.h>
 #include <linux/slab.h>
 #include <linux/regulator/driver.h>
@@ -26,7 +27,7 @@
 #include <linux/regulator/of_regulator.h>
 #include "pv88080-regulator.h"
 
-#define PV88080_MAX_REGULATORS 3
+#define PV88080_MAX_REGULATORS 4
 
 /* PV88080 REGULATOR IDs */
 enum {
@@ -34,6 +35,12 @@ enum {
        PV88080_ID_BUCK1,
        PV88080_ID_BUCK2,
        PV88080_ID_BUCK3,
+       PV88080_ID_HVBUCK,
+};
+
+enum pv88080_types {
+       TYPE_PV88080_AA,
+       TYPE_PV88080_BA,
 };
 
 struct pv88080_regulator {
@@ -42,7 +49,8 @@ struct pv88080_regulator {
        unsigned int n_current_limits;
        const int *current_limits;
        unsigned int limit_mask;
-       unsigned int conf;
+       unsigned int mode_reg;
+       unsigned int limit_reg;
        unsigned int conf2;
        unsigned int conf5;
 };
@@ -51,6 +59,8 @@ struct pv88080 {
        struct device *dev;
        struct regmap *regmap;
        struct regulator_dev *rdev[PV88080_MAX_REGULATORS];
+       unsigned long type;
+       const struct pv88080_compatible_regmap *regmap_config;
 };
 
 struct pv88080_buck_voltage {
@@ -59,6 +69,30 @@ struct pv88080_buck_voltage {
        int uV_step;
 };
 
+struct pv88080_buck_regmap {
+       /* REGS */
+       int buck_enable_reg;
+       int buck_vsel_reg;
+       int buck_mode_reg;
+       int buck_limit_reg;
+       int buck_vdac_range_reg;
+       int buck_vrange_gain_reg;
+       /* MASKS */
+       int buck_enable_mask;
+       int buck_vsel_mask;
+       int buck_limit_mask;
+};
+
+struct pv88080_compatible_regmap {
+       /* BUCK1, 2, 3 */
+       struct pv88080_buck_regmap buck_regmap[PV88080_MAX_REGULATORS-1];
+       /* HVBUCK */
+       int hvbuck_enable_reg;
+       int hvbuck_vsel_reg;
+       int hvbuck_enable_mask;
+       int hvbuck_vsel_mask;
+};
+
 static const struct regmap_config pv88080_regmap_config = {
        .reg_bits = 8,
        .val_bits = 8,
@@ -89,13 +123,111 @@ static const struct pv88080_buck_voltage pv88080_buck_vol[2] = {
        },
 };
 
+static const struct pv88080_compatible_regmap pv88080_aa_regs = {
+       /* BUCK1 */
+       .buck_regmap[0] = {
+               .buck_enable_reg      = PV88080AA_REG_BUCK1_CONF0,
+               .buck_vsel_reg        = PV88080AA_REG_BUCK1_CONF0,
+               .buck_mode_reg        = PV88080AA_REG_BUCK1_CONF1,
+               .buck_limit_reg       = PV88080AA_REG_BUCK1_CONF1,
+               .buck_vdac_range_reg  = PV88080AA_REG_BUCK1_CONF2,
+               .buck_vrange_gain_reg = PV88080AA_REG_BUCK1_CONF5,
+               .buck_enable_mask     = PV88080_BUCK1_EN,
+               .buck_vsel_mask       = PV88080_VBUCK1_MASK,
+               .buck_limit_mask      = PV88080_BUCK1_ILIM_MASK,
+       },
+       /* BUCK2 */
+       .buck_regmap[1] = {
+               .buck_enable_reg      = PV88080AA_REG_BUCK2_CONF0,
+               .buck_vsel_reg        = PV88080AA_REG_BUCK2_CONF0,
+               .buck_mode_reg        = PV88080AA_REG_BUCK2_CONF1,
+               .buck_limit_reg       = PV88080AA_REG_BUCK2_CONF1,
+               .buck_vdac_range_reg  = PV88080AA_REG_BUCK2_CONF2,
+               .buck_vrange_gain_reg = PV88080AA_REG_BUCK2_CONF5,
+               .buck_enable_mask         = PV88080_BUCK2_EN,
+               .buck_vsel_mask       = PV88080_VBUCK2_MASK,
+               .buck_limit_mask      = PV88080_BUCK2_ILIM_MASK,
+       },
+       /* BUCK3 */
+       .buck_regmap[2] = {
+               .buck_enable_reg          = PV88080AA_REG_BUCK3_CONF0,
+               .buck_vsel_reg        = PV88080AA_REG_BUCK3_CONF0,
+               .buck_mode_reg        = PV88080AA_REG_BUCK3_CONF1,
+               .buck_limit_reg       = PV88080AA_REG_BUCK3_CONF1,
+               .buck_vdac_range_reg  = PV88080AA_REG_BUCK3_CONF2,
+               .buck_vrange_gain_reg = PV88080AA_REG_BUCK3_CONF5,
+               .buck_enable_mask         = PV88080_BUCK3_EN,
+               .buck_vsel_mask       = PV88080_VBUCK3_MASK,
+               .buck_limit_mask      = PV88080_BUCK3_ILIM_MASK,
+       },
+       /* HVBUCK */
+       .hvbuck_enable_reg            = PV88080AA_REG_HVBUCK_CONF2,
+       .hvbuck_vsel_reg          = PV88080AA_REG_HVBUCK_CONF1,
+       .hvbuck_enable_mask       = PV88080_HVBUCK_EN,
+       .hvbuck_vsel_mask         = PV88080_VHVBUCK_MASK,
+};
+
+static const struct pv88080_compatible_regmap pv88080_ba_regs = {
+       /* BUCK1 */
+       .buck_regmap[0] = {
+               .buck_enable_reg          = PV88080BA_REG_BUCK1_CONF0,
+               .buck_vsel_reg        = PV88080BA_REG_BUCK1_CONF0,
+               .buck_mode_reg        = PV88080BA_REG_BUCK1_CONF1,
+               .buck_limit_reg       = PV88080BA_REG_BUCK1_CONF1,
+               .buck_vdac_range_reg  = PV88080BA_REG_BUCK1_CONF2,
+               .buck_vrange_gain_reg = PV88080BA_REG_BUCK1_CONF5,
+               .buck_enable_mask     = PV88080_BUCK1_EN,
+               .buck_vsel_mask       = PV88080_VBUCK1_MASK,
+               .buck_limit_mask          = PV88080_BUCK1_ILIM_MASK,
+       },
+       /* BUCK2 */
+       .buck_regmap[1] = {
+               .buck_enable_reg          = PV88080BA_REG_BUCK2_CONF0,
+               .buck_vsel_reg        = PV88080BA_REG_BUCK2_CONF0,
+               .buck_mode_reg        = PV88080BA_REG_BUCK2_CONF1,
+               .buck_limit_reg       = PV88080BA_REG_BUCK2_CONF1,
+               .buck_vdac_range_reg  = PV88080BA_REG_BUCK2_CONF2,
+               .buck_vrange_gain_reg = PV88080BA_REG_BUCK2_CONF5,
+               .buck_enable_mask         = PV88080_BUCK2_EN,
+               .buck_vsel_mask       = PV88080_VBUCK2_MASK,
+               .buck_limit_mask          = PV88080_BUCK2_ILIM_MASK,
+       },
+       /* BUCK3 */
+       .buck_regmap[2] = {
+               .buck_enable_reg          = PV88080BA_REG_BUCK3_CONF0,
+               .buck_vsel_reg        = PV88080BA_REG_BUCK3_CONF0,
+               .buck_mode_reg        = PV88080BA_REG_BUCK3_CONF1,
+               .buck_limit_reg       = PV88080BA_REG_BUCK3_CONF1,
+               .buck_vdac_range_reg  = PV88080BA_REG_BUCK3_CONF2,
+               .buck_vrange_gain_reg = PV88080BA_REG_BUCK3_CONF5,
+               .buck_enable_mask         = PV88080_BUCK3_EN,
+               .buck_vsel_mask       = PV88080_VBUCK3_MASK,
+               .buck_limit_mask          = PV88080_BUCK3_ILIM_MASK,
+       },
+       /* HVBUCK */
+       .hvbuck_enable_reg            = PV88080BA_REG_HVBUCK_CONF2,
+       .hvbuck_vsel_reg          = PV88080BA_REG_HVBUCK_CONF1,
+       .hvbuck_enable_mask       = PV88080_HVBUCK_EN,
+       .hvbuck_vsel_mask                 = PV88080_VHVBUCK_MASK,
+};
+
+#ifdef CONFIG_OF
+static const struct of_device_id pv88080_dt_ids[] = {
+       { .compatible = "pvs,pv88080",    .data = (void *)TYPE_PV88080_AA },
+       { .compatible = "pvs,pv88080-aa", .data = (void *)TYPE_PV88080_AA },
+       { .compatible = "pvs,pv88080-ba", .data = (void *)TYPE_PV88080_BA },
+       {},
+};
+MODULE_DEVICE_TABLE(of, pv88080_dt_ids);
+#endif
+
 static unsigned int pv88080_buck_get_mode(struct regulator_dev *rdev)
 {
        struct pv88080_regulator *info = rdev_get_drvdata(rdev);
        unsigned int data;
        int ret, mode = 0;
 
-       ret = regmap_read(rdev->regmap, info->conf, &data);
+       ret = regmap_read(rdev->regmap, info->mode_reg, &data);
        if (ret < 0)
                return ret;
 
@@ -136,7 +268,7 @@ static int pv88080_buck_set_mode(struct regulator_dev *rdev,
                return -EINVAL;
        }
 
-       return regmap_update_bits(rdev->regmap, info->conf,
+       return regmap_update_bits(rdev->regmap, info->mode_reg,
                                        PV88080_BUCK1_MODE_MASK, val);
 }
 
@@ -151,7 +283,7 @@ static int pv88080_set_current_limit(struct regulator_dev *rdev, int min,
                if (min <= info->current_limits[i]
                        && max >= info->current_limits[i]) {
                                return regmap_update_bits(rdev->regmap,
-                                       info->conf,
+                                       info->limit_reg,
                                        info->limit_mask,
                                        i << PV88080_BUCK1_ILIM_SHIFT);
                }
@@ -166,7 +298,7 @@ static int pv88080_get_current_limit(struct regulator_dev *rdev)
        unsigned int data;
        int ret;
 
-       ret = regmap_read(rdev->regmap, info->conf, &data);
+       ret = regmap_read(rdev->regmap, info->limit_reg, &data);
        if (ret < 0)
                return ret;
 
@@ -187,6 +319,15 @@ static struct regulator_ops pv88080_buck_ops = {
        .get_current_limit = pv88080_get_current_limit,
 };
 
+static struct regulator_ops pv88080_hvbuck_ops = {
+       .enable = regulator_enable_regmap,
+       .disable = regulator_disable_regmap,
+       .is_enabled = regulator_is_enabled_regmap,
+       .set_voltage_sel = regulator_set_voltage_sel_regmap,
+       .get_voltage_sel = regulator_get_voltage_sel_regmap,
+       .list_voltage = regulator_list_voltage_linear,
+};
+
 #define PV88080_BUCK(chip, regl_name, min, step, max, limits_array) \
 {\
        .desc   =       {\
@@ -200,17 +341,25 @@ static struct regulator_ops pv88080_buck_ops = {
                .min_uV = min, \
                .uV_step = step, \
                .n_voltages = ((max) - (min))/(step) + 1, \
-               .enable_reg = PV88080_REG_##regl_name##_CONF0, \
-               .enable_mask = PV88080_##regl_name##_EN, \
-               .vsel_reg = PV88080_REG_##regl_name##_CONF0, \
-               .vsel_mask = PV88080_V##regl_name##_MASK, \
        },\
        .current_limits = limits_array, \
        .n_current_limits = ARRAY_SIZE(limits_array), \
-       .limit_mask = PV88080_##regl_name##_ILIM_MASK, \
-       .conf = PV88080_REG_##regl_name##_CONF1, \
-       .conf2 = PV88080_REG_##regl_name##_CONF2, \
-       .conf5 = PV88080_REG_##regl_name##_CONF5, \
+}
+
+#define PV88080_HVBUCK(chip, regl_name, min, step, max) \
+{\
+       .desc   =       {\
+               .id = chip##_ID_##regl_name,\
+               .name = __stringify(chip##_##regl_name),\
+               .of_match = of_match_ptr(#regl_name),\
+               .regulators_node = of_match_ptr("regulators"),\
+               .type = REGULATOR_VOLTAGE,\
+               .owner = THIS_MODULE,\
+               .ops = &pv88080_hvbuck_ops,\
+               .min_uV = min, \
+               .uV_step = step, \
+               .n_voltages = ((max) - (min))/(step) + 1, \
+       },\
 }
 
 static struct pv88080_regulator pv88080_regulator_info[] = {
@@ -220,6 +369,7 @@ static struct pv88080_regulator pv88080_regulator_info[] = {
                pv88080_buck23_limits),
        PV88080_BUCK(PV88080, BUCK3, 600000, 6250, 1393750,
                pv88080_buck23_limits),
+       PV88080_HVBUCK(PV88080, HVBUCK, 0, 5000, 1275000),
 };
 
 static irqreturn_t pv88080_irq_handler(int irq, void *data)
@@ -280,6 +430,8 @@ static int pv88080_i2c_probe(struct i2c_client *i2c,
 {
        struct regulator_init_data *init_data = dev_get_platdata(&i2c->dev);
        struct pv88080 *chip;
+       const struct pv88080_compatible_regmap *regmap_config;
+       const struct of_device_id *match;
        struct regulator_config config = { };
        int i, error, ret;
        unsigned int conf2, conf5;
@@ -297,6 +449,17 @@ static int pv88080_i2c_probe(struct i2c_client *i2c,
                return error;
        }
 
+       if (i2c->dev.of_node) {
+               match = of_match_node(pv88080_dt_ids, i2c->dev.of_node);
+               if (!match) {
+                       dev_err(chip->dev, "Failed to get of_match_node\n");
+                       return -EINVAL;
+               }
+               chip->type = (unsigned long)match->data;
+       } else {
+               chip->type = id->driver_data;
+       }
+
        i2c_set_clientdata(i2c, chip);
 
        if (i2c->irq != 0) {
@@ -336,31 +499,58 @@ static int pv88080_i2c_probe(struct i2c_client *i2c,
                                "Failed to update mask reg: %d\n", ret);
                        return ret;
                }
-
        } else {
                dev_warn(chip->dev, "No IRQ configured\n");
        }
 
+       switch (chip->type) {
+       case TYPE_PV88080_AA:
+               chip->regmap_config = &pv88080_aa_regs;
+               break;
+       case TYPE_PV88080_BA:
+               chip->regmap_config = &pv88080_ba_regs;
+               break;
+       }
+
+       regmap_config = chip->regmap_config;
        config.dev = chip->dev;
        config.regmap = chip->regmap;
 
-       for (i = 0; i < PV88080_MAX_REGULATORS; i++) {
+       /* Registeration for BUCK1, 2, 3 */
+       for (i = 0; i < PV88080_MAX_REGULATORS-1; i++) {
                if (init_data)
                        config.init_data = &init_data[i];
 
+               pv88080_regulator_info[i].limit_reg
+                       = regmap_config->buck_regmap[i].buck_limit_reg;
+               pv88080_regulator_info[i].limit_mask
+                       = regmap_config->buck_regmap[i].buck_limit_mask;
+               pv88080_regulator_info[i].mode_reg
+                       = regmap_config->buck_regmap[i].buck_mode_reg;
+               pv88080_regulator_info[i].conf2
+                       = regmap_config->buck_regmap[i].buck_vdac_range_reg;
+               pv88080_regulator_info[i].conf5
+                       = regmap_config->buck_regmap[i].buck_vrange_gain_reg;
+               pv88080_regulator_info[i].desc.enable_reg
+                       = regmap_config->buck_regmap[i].buck_enable_reg;
+               pv88080_regulator_info[i].desc.enable_mask
+                       = regmap_config->buck_regmap[i].buck_enable_mask;
+               pv88080_regulator_info[i].desc.vsel_reg
+                       = regmap_config->buck_regmap[i].buck_vsel_reg;
+               pv88080_regulator_info[i].desc.vsel_mask
+                       = regmap_config->buck_regmap[i].buck_vsel_mask;
+
                ret = regmap_read(chip->regmap,
-                       pv88080_regulator_info[i].conf2, &conf2);
+                               pv88080_regulator_info[i].conf2, &conf2);
                if (ret < 0)
                        return ret;
-
                conf2 = ((conf2 >> PV88080_BUCK_VDAC_RANGE_SHIFT) &
                        PV88080_BUCK_VDAC_RANGE_MASK);
 
                ret = regmap_read(chip->regmap,
-                       pv88080_regulator_info[i].conf5, &conf5);
+                               pv88080_regulator_info[i].conf5, &conf5);
                if (ret < 0)
                        return ret;
-
                conf5 = ((conf5 >> PV88080_BUCK_VRANGE_GAIN_SHIFT) &
                        PV88080_BUCK_VRANGE_GAIN_MASK);
 
@@ -383,23 +573,38 @@ static int pv88080_i2c_probe(struct i2c_client *i2c,
                }
        }
 
+       pv88080_regulator_info[PV88080_ID_HVBUCK].desc.enable_reg
+               = regmap_config->hvbuck_enable_reg;
+       pv88080_regulator_info[PV88080_ID_HVBUCK].desc.enable_mask
+               = regmap_config->hvbuck_enable_mask;
+       pv88080_regulator_info[PV88080_ID_HVBUCK].desc.vsel_reg
+               = regmap_config->hvbuck_vsel_reg;
+       pv88080_regulator_info[PV88080_ID_HVBUCK].desc.vsel_mask
+               = regmap_config->hvbuck_vsel_mask;
+
+       /* Registeration for HVBUCK */
+       if (init_data)
+               config.init_data = &init_data[PV88080_ID_HVBUCK];
+
+       config.driver_data = (void *)&pv88080_regulator_info[PV88080_ID_HVBUCK];
+       chip->rdev[PV88080_ID_HVBUCK] = devm_regulator_register(chip->dev,
+               &pv88080_regulator_info[PV88080_ID_HVBUCK].desc, &config);
+       if (IS_ERR(chip->rdev[PV88080_ID_HVBUCK])) {
+               dev_err(chip->dev, "Failed to register PV88080 regulator\n");
+               return PTR_ERR(chip->rdev[PV88080_ID_HVBUCK]);
+       }
+
        return 0;
 }
 
 static const struct i2c_device_id pv88080_i2c_id[] = {
-       {"pv88080", 0},
+       { "pv88080",    TYPE_PV88080_AA },
+       { "pv88080-aa", TYPE_PV88080_AA },
+       { "pv88080-ba", TYPE_PV88080_BA },
        {},
 };
 MODULE_DEVICE_TABLE(i2c, pv88080_i2c_id);
 
-#ifdef CONFIG_OF
-static const struct of_device_id pv88080_dt_ids[] = {
-       { .compatible = "pvs,pv88080", .data = &pv88080_i2c_id[0] },
-       {},
-};
-MODULE_DEVICE_TABLE(of, pv88080_dt_ids);
-#endif
-
 static struct i2c_driver pv88080_regulator_driver = {
        .driver = {
                .name = "pv88080",
index 5e9afde606f4036efaeaa540ffded3ef56fee66e..ae25ff360e3db7ac726406274a6548fb00cf675d 100644 (file)
 #define __PV88080_REGISTERS_H__
 
 /* System Control and Event Registers */
-#define        PV88080_REG_EVENT_A                     0x04
-#define        PV88080_REG_MASK_A                      0x09
-#define        PV88080_REG_MASK_B                      0x0a
-#define        PV88080_REG_MASK_C                      0x0b
-
-/* Regulator Registers */
-#define        PV88080_REG_BUCK1_CONF0                 0x27
-#define        PV88080_REG_BUCK1_CONF1                 0x28
-#define        PV88080_REG_BUCK1_CONF2                 0x59
-#define        PV88080_REG_BUCK1_CONF5                 0x5c
-#define        PV88080_REG_BUCK2_CONF0                 0x29
-#define        PV88080_REG_BUCK2_CONF1                 0x2a
-#define        PV88080_REG_BUCK2_CONF2                 0x61
-#define        PV88080_REG_BUCK2_CONF5                 0x64
-#define        PV88080_REG_BUCK3_CONF0                 0x2b
-#define        PV88080_REG_BUCK3_CONF1                 0x2c
-#define        PV88080_REG_BUCK3_CONF2                 0x69
-#define        PV88080_REG_BUCK3_CONF5                 0x6c
+#define        PV88080_REG_EVENT_A                             0x04
+#define        PV88080_REG_MASK_A                              0x09
+#define        PV88080_REG_MASK_B                              0x0A
+#define        PV88080_REG_MASK_C                              0x0B
+
+/* Regulator Registers - rev. AA */
+#define PV88080AA_REG_HVBUCK_CONF1             0x2D
+#define PV88080AA_REG_HVBUCK_CONF2             0x2E
+#define        PV88080AA_REG_BUCK1_CONF0               0x27
+#define        PV88080AA_REG_BUCK1_CONF1               0x28
+#define        PV88080AA_REG_BUCK1_CONF2               0x59
+#define        PV88080AA_REG_BUCK1_CONF5               0x5C
+#define        PV88080AA_REG_BUCK2_CONF0               0x29
+#define        PV88080AA_REG_BUCK2_CONF1               0x2A
+#define        PV88080AA_REG_BUCK2_CONF2               0x61
+#define        PV88080AA_REG_BUCK2_CONF5               0x64
+#define        PV88080AA_REG_BUCK3_CONF0               0x2B
+#define        PV88080AA_REG_BUCK3_CONF1               0x2C
+#define        PV88080AA_REG_BUCK3_CONF2               0x69
+#define        PV88080AA_REG_BUCK3_CONF5               0x6C
+
+/* Regulator Registers - rev. BA */
+#define        PV88080BA_REG_HVBUCK_CONF1              0x33
+#define        PV88080BA_REG_HVBUCK_CONF2              0x34
+#define        PV88080BA_REG_BUCK1_CONF0               0x2A
+#define        PV88080BA_REG_BUCK1_CONF1               0x2C
+#define        PV88080BA_REG_BUCK1_CONF2               0x5A
+#define        PV88080BA_REG_BUCK1_CONF5               0x5D
+#define        PV88080BA_REG_BUCK2_CONF0               0x2D
+#define        PV88080BA_REG_BUCK2_CONF1               0x2F
+#define        PV88080BA_REG_BUCK2_CONF2               0x63
+#define        PV88080BA_REG_BUCK2_CONF5               0x66
+#define        PV88080BA_REG_BUCK3_CONF0               0x30
+#define        PV88080BA_REG_BUCK3_CONF1               0x32
+#define        PV88080BA_REG_BUCK3_CONF2               0x6C
+#define        PV88080BA_REG_BUCK3_CONF5               0x6F
 
 /* PV88080_REG_EVENT_A (addr=0x04) */
 #define        PV88080_E_VDD_FLT                               0x01
-#define        PV88080_E_OVER_TEMP                     0x02
+#define        PV88080_E_OVER_TEMP                             0x02
 
 /* PV88080_REG_MASK_A (addr=0x09) */
 #define        PV88080_M_VDD_FLT                               0x01
-#define        PV88080_M_OVER_TEMP                     0x02
+#define        PV88080_M_OVER_TEMP                             0x02
 
-/* PV88080_REG_BUCK1_CONF0 (addr=0x27) */
+/* PV88080_REG_BUCK1_CONF0 (addr=0x27|0x2A) */
 #define        PV88080_BUCK1_EN                                0x80
-#define PV88080_VBUCK1_MASK                    0x7F
-/* PV88080_REG_BUCK2_CONF0 (addr=0x29) */
+#define PV88080_VBUCK1_MASK                            0x7F
+
+/* PV88080_REG_BUCK2_CONF0 (addr=0x29|0x2D) */
 #define        PV88080_BUCK2_EN                                0x80
-#define PV88080_VBUCK2_MASK                    0x7F
-/* PV88080_REG_BUCK3_CONF0 (addr=0x2b) */
+#define PV88080_VBUCK2_MASK                            0x7F
+
+/* PV88080_REG_BUCK3_CONF0 (addr=0x2B|0x30) */
 #define        PV88080_BUCK3_EN                                0x80
-#define PV88080_VBUCK3_MASK                    0x7F
+#define PV88080_VBUCK3_MASK                            0x7F
 
-/* PV88080_REG_BUCK1_CONF1 (addr=0x28) */
-#define PV88080_BUCK1_ILIM_SHIFT                       2
+/* PV88080_REG_BUCK1_CONF1 (addr=0x28|0x2C) */
+#define PV88080_BUCK1_ILIM_SHIFT               2
 #define PV88080_BUCK1_ILIM_MASK                        0x0C
 #define PV88080_BUCK1_MODE_MASK                        0x03
 
-/* PV88080_REG_BUCK2_CONF1 (addr=0x2a) */
-#define PV88080_BUCK2_ILIM_SHIFT                       2
+/* PV88080_REG_BUCK2_CONF1 (addr=0x2A|0x2F) */
+#define PV88080_BUCK2_ILIM_SHIFT               2
 #define PV88080_BUCK2_ILIM_MASK                        0x0C
 #define PV88080_BUCK2_MODE_MASK                        0x03
 
-/* PV88080_REG_BUCK3_CONF1 (addr=0x2c) */
-#define PV88080_BUCK3_ILIM_SHIFT                       2
+/* PV88080_REG_BUCK3_CONF1 (addr=0x2C|0x32) */
+#define PV88080_BUCK3_ILIM_SHIFT               2
 #define PV88080_BUCK3_ILIM_MASK                        0x0C
 #define PV88080_BUCK3_MODE_MASK                        0x03
 
 #define        PV88080_BUCK_MODE_AUTO                  0x01
 #define        PV88080_BUCK_MODE_SYNC                  0x02
 
-/* PV88080_REG_BUCK2_CONF2 (addr=0x61) */
-/* PV88080_REG_BUCK3_CONF2 (addr=0x69) */
-#define PV88080_BUCK_VDAC_RANGE_SHIFT                  7
-#define PV88080_BUCK_VDAC_RANGE_MASK                   0x01
+/* PV88080_REG_HVBUCK_CONF1 (addr=0x2D|0x33) */
+#define PV88080_VHVBUCK_MASK                   0xFF
+
+/* PV88080_REG_HVBUCK_CONF1 (addr=0x2E|0x34) */
+#define PV88080_HVBUCK_EN                              0x01
+
+/* PV88080_REG_BUCK2_CONF2 (addr=0x61|0x63) */
+/* PV88080_REG_BUCK3_CONF2 (addr=0x69|0x6C) */
+#define PV88080_BUCK_VDAC_RANGE_SHIFT  7
+#define PV88080_BUCK_VDAC_RANGE_MASK   0x01
 
-#define PV88080_BUCK_VDAC_RANGE_1                      0x00
-#define PV88080_BUCK_VDAC_RANGE_2                      0x01
+#define PV88080_BUCK_VDAC_RANGE_1              0x00
+#define PV88080_BUCK_VDAC_RANGE_2              0x01
 
-/* PV88080_REG_BUCK2_CONF5 (addr=0x64) */
-/* PV88080_REG_BUCK3_CONF5 (addr=0x6c) */
-#define PV88080_BUCK_VRANGE_GAIN_SHIFT                 0
-#define PV88080_BUCK_VRANGE_GAIN_MASK                  0x01
+/* PV88080_REG_BUCK2_CONF5 (addr=0x64|0x66) */
+/* PV88080_REG_BUCK3_CONF5 (addr=0x6C|0x6F) */
+#define PV88080_BUCK_VRANGE_GAIN_SHIFT 0
+#define PV88080_BUCK_VRANGE_GAIN_MASK  0x01
 
-#define PV88080_BUCK_VRANGE_GAIN_1                     0x00
-#define PV88080_BUCK_VRANGE_GAIN_2                     0x01
+#define PV88080_BUCK_VRANGE_GAIN_1             0x00
+#define PV88080_BUCK_VRANGE_GAIN_2             0x01
 
 #endif /* __PV88080_REGISTERS_H__ */