regulator: axp20x: Use rdev_get_id at appropriate places
authorAxel Lin <axel.lin@ingics.com>
Tue, 19 Mar 2019 03:36:08 +0000 (11:36 +0800)
committerMark Brown <broonie@kernel.org>
Tue, 19 Mar 2019 11:54:58 +0000 (11:54 +0000)
Use rdev_get_id() to simplify the code a bit.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
Acked-by: Chen-Yu Tsai <wens@csie.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/axp20x-regulator.c

index 8a366f6..07e10c8 100644 (file)
@@ -367,16 +367,14 @@ static const int axp209_dcdc2_ldo3_slew_rates[] = {
 static int axp20x_set_ramp_delay(struct regulator_dev *rdev, int ramp)
 {
        struct axp20x_dev *axp20x = rdev_get_drvdata(rdev);
-       const struct regulator_desc *desc;
+       int id = rdev_get_id(rdev);
        u8 reg, mask, enable, cfg = 0xff;
        const int *slew_rates;
        int rate_count = 0;
 
-       desc = rdev->desc;
-
        switch (axp20x->variant) {
        case AXP209_ID:
-               if (desc->id == AXP20X_DCDC2) {
+               if (id == AXP20X_DCDC2) {
                        slew_rates = axp209_dcdc2_ldo3_slew_rates;
                        rate_count = ARRAY_SIZE(axp209_dcdc2_ldo3_slew_rates);
                        reg = AXP20X_DCDC2_LDO3_V_RAMP;
@@ -388,7 +386,7 @@ static int axp20x_set_ramp_delay(struct regulator_dev *rdev, int ramp)
                        break;
                }
 
-               if (desc->id == AXP20X_LDO3) {
+               if (id == AXP20X_LDO3) {
                        slew_rates = axp209_dcdc2_ldo3_slew_rates;
                        rate_count = ARRAY_SIZE(axp209_dcdc2_ldo3_slew_rates);
                        reg = AXP20X_DCDC2_LDO3_V_RAMP;
@@ -435,13 +433,11 @@ static int axp20x_set_ramp_delay(struct regulator_dev *rdev, int ramp)
 static int axp20x_regulator_enable_regmap(struct regulator_dev *rdev)
 {
        struct axp20x_dev *axp20x = rdev_get_drvdata(rdev);
-       const struct regulator_desc *desc;
-
-       desc = rdev->desc;
+       int id = rdev_get_id(rdev);
 
        switch (axp20x->variant) {
        case AXP209_ID:
-               if ((desc->id == AXP20X_LDO3) &&
+               if ((id == AXP20X_LDO3) &&
                    rdev->constraints && rdev->constraints->soft_start) {
                        int v_out;
                        int ret;