Message ID | 20190310150609.5178-1-axel.lin@ingics.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | regulator: palmas: Remove *rdev[PALMAS_NUM_REGS] from struct palmas_pmic | expand |
On 10/03/19 8:36 PM, Axel Lin wrote: > This driver is using devm_regulator_register() so it is not necessary > to save *rdev for clean up. Actually the pmic->rdev[id] is not used now. Reviewed-by: Keerthy <j-keerthy@ti.com> > > Signed-off-by: Axel Lin <axel.lin@ingics.com> > --- > drivers/regulator/palmas-regulator.c | 12 ------------ > include/linux/mfd/palmas.h | 1 - > 2 files changed, 13 deletions(-) > > diff --git a/drivers/regulator/palmas-regulator.c b/drivers/regulator/palmas-regulator.c > index 7fb9e8dd834e..f13c7c8b1061 100644 > --- a/drivers/regulator/palmas-regulator.c > +++ b/drivers/regulator/palmas-regulator.c > @@ -991,9 +991,6 @@ static int palmas_ldo_registration(struct palmas_pmic *pmic, > return PTR_ERR(rdev); > } > > - /* Save regulator for cleanup */ > - pmic->rdev[id] = rdev; > - > /* Initialise sleep/init values from platform data */ > if (pdata) { > reg_init = pdata->reg_init[id]; > @@ -1101,9 +1098,6 @@ static int tps65917_ldo_registration(struct palmas_pmic *pmic, > return PTR_ERR(rdev); > } > > - /* Save regulator for cleanup */ > - pmic->rdev[id] = rdev; > - > /* Initialise sleep/init values from platform data */ > if (pdata) { > reg_init = pdata->reg_init[id]; > @@ -1288,9 +1282,6 @@ static int palmas_smps_registration(struct palmas_pmic *pmic, > pdev_name); > return PTR_ERR(rdev); > } > - > - /* Save regulator for cleanup */ > - pmic->rdev[id] = rdev; > } > > return 0; > @@ -1395,9 +1386,6 @@ static int tps65917_smps_registration(struct palmas_pmic *pmic, > pdev_name); > return PTR_ERR(rdev); > } > - > - /* Save regulator for cleanup */ > - pmic->rdev[id] = rdev; > } > > return 0; > diff --git a/include/linux/mfd/palmas.h b/include/linux/mfd/palmas.h > index 75e5c8ff85fc..c34d5f0d34d7 100644 > --- a/include/linux/mfd/palmas.h > +++ b/include/linux/mfd/palmas.h > @@ -553,7 +553,6 @@ struct palmas_pmic { > struct palmas *palmas; > struct device *dev; > struct regulator_desc desc[PALMAS_NUM_REGS]; > - struct regulator_dev *rdev[PALMAS_NUM_REGS]; > struct mutex mutex; > > int smps123; >
diff --git a/drivers/regulator/palmas-regulator.c b/drivers/regulator/palmas-regulator.c index 7fb9e8dd834e..f13c7c8b1061 100644 --- a/drivers/regulator/palmas-regulator.c +++ b/drivers/regulator/palmas-regulator.c @@ -991,9 +991,6 @@ static int palmas_ldo_registration(struct palmas_pmic *pmic, return PTR_ERR(rdev); } - /* Save regulator for cleanup */ - pmic->rdev[id] = rdev; - /* Initialise sleep/init values from platform data */ if (pdata) { reg_init = pdata->reg_init[id]; @@ -1101,9 +1098,6 @@ static int tps65917_ldo_registration(struct palmas_pmic *pmic, return PTR_ERR(rdev); } - /* Save regulator for cleanup */ - pmic->rdev[id] = rdev; - /* Initialise sleep/init values from platform data */ if (pdata) { reg_init = pdata->reg_init[id]; @@ -1288,9 +1282,6 @@ static int palmas_smps_registration(struct palmas_pmic *pmic, pdev_name); return PTR_ERR(rdev); } - - /* Save regulator for cleanup */ - pmic->rdev[id] = rdev; } return 0; @@ -1395,9 +1386,6 @@ static int tps65917_smps_registration(struct palmas_pmic *pmic, pdev_name); return PTR_ERR(rdev); } - - /* Save regulator for cleanup */ - pmic->rdev[id] = rdev; } return 0; diff --git a/include/linux/mfd/palmas.h b/include/linux/mfd/palmas.h index 75e5c8ff85fc..c34d5f0d34d7 100644 --- a/include/linux/mfd/palmas.h +++ b/include/linux/mfd/palmas.h @@ -553,7 +553,6 @@ struct palmas_pmic { struct palmas *palmas; struct device *dev; struct regulator_desc desc[PALMAS_NUM_REGS]; - struct regulator_dev *rdev[PALMAS_NUM_REGS]; struct mutex mutex; int smps123;
This driver is using devm_regulator_register() so it is not necessary to save *rdev for clean up. Actually the pmic->rdev[id] is not used now. Signed-off-by: Axel Lin <axel.lin@ingics.com> --- drivers/regulator/palmas-regulator.c | 12 ------------ include/linux/mfd/palmas.h | 1 - 2 files changed, 13 deletions(-)