Re: [PATCH 3/3] regulator: s2mps11: Optimize the regulator description macro

From: amit daniel kachhap
Date: Wed Jul 09 2014 - 23:25:11 EST

Next message: Daeseok Youn: "[PATCH V2] staging: dgap: removes redundant null check and change paramter for dgap_maxcps_room()"
Previous message: amit daniel kachhap: "Re: [PATCH 2/3] regulator: s2mpa01: Optimize the regulator description macro"
In reply to: Amit Daniel Kachhap: "[PATCH 3/3] regulator: s2mps11: Optimize the regulator description macro"
Next in thread: Amit Daniel Kachhap: "[PATCH 2/3] regulator: s2mpa01: Optimize the regulator description macro"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Adding MFD maintainers

On Tue, Jul 8, 2014 at 5:58 PM, Amit Daniel Kachhap
<amit.daniel@xxxxxxxxxxx> wrote:
> This patch makes the regulator description macro take minimum and
> steps voltage as parameter. In this way many repeated macros can be
> removed. Now these macros are repeated only if the the LDO/BUCK ctrl
> registers have non-linear positions. The good thing is these ctrl registers
> are mostly linear so they are not passed as parameters.
>
> This patch reduces the code size and also allow easy addition of more
> s2mpxxx PMIC drivers which differs a lot in minimum/step voltages.
>
> Signed-off-by: Amit Daniel Kachhap <amit.daniel@xxxxxxxxxxx>
> ---
> drivers/regulator/s2mps11.c | 261 +++++++++++++++-----------------------------
> 1 file changed, 86 insertions(+), 175 deletions(-)
>
> diff --git a/drivers/regulator/s2mps11.c b/drivers/regulator/s2mps11.c
> index 2098c3e..ea94f47 100644
> --- a/drivers/regulator/s2mps11.c
> +++ b/drivers/regulator/s2mps11.c
> @@ -250,28 +250,14 @@ static struct regulator_ops s2mps11_buck_ops = {
> .set_ramp_delay = s2mps11_set_ramp_delay,
> };
>
> -#define regulator_desc_s2mps11_ldo1(num) { \
> +#define regulator_desc_s2mps11_ldo(num, min, step) { \
> .name = "LDO"#num, \
> .id = S2MPS11_LDO##num, \
> .ops = &s2mps11_ldo_ops, \
> .type = REGULATOR_VOLTAGE, \
> .owner = THIS_MODULE, \
> - .min_uV = MIN_800_MV, \
> - .uV_step = STEP_50_MV, \
> - .n_voltages = S2MPS11_LDO_N_VOLTAGES, \
> - .vsel_reg = S2MPS11_REG_L1CTRL + num - 1, \
> - .vsel_mask = S2MPS11_LDO_VSEL_MASK, \
> - .enable_reg = S2MPS11_REG_L1CTRL + num - 1, \
> - .enable_mask = S2MPS11_ENABLE_MASK \
> -}
> -#define regulator_desc_s2mps11_ldo2(num) { \
> - .name = "LDO"#num, \
> - .id = S2MPS11_LDO##num, \
> - .ops = &s2mps11_ldo_ops, \
> - .type = REGULATOR_VOLTAGE, \
> - .owner = THIS_MODULE, \
> - .min_uV = MIN_800_MV, \
> - .uV_step = STEP_25_MV, \
> + .min_uV = min, \
> + .uV_step = step, \
> .n_voltages = S2MPS11_LDO_N_VOLTAGES, \
> .vsel_reg = S2MPS11_REG_L1CTRL + num - 1, \
> .vsel_mask = S2MPS11_LDO_VSEL_MASK, \
> @@ -311,14 +297,14 @@ static struct regulator_ops s2mps11_buck_ops = {
> .enable_mask = S2MPS11_ENABLE_MASK \
> }
>
> -#define regulator_desc_s2mps11_buck6_8(num) { \
> +#define regulator_desc_s2mps11_buck6_10(num, min, step) { \
> .name = "BUCK"#num, \
> .id = S2MPS11_BUCK##num, \
> .ops = &s2mps11_buck_ops, \
> .type = REGULATOR_VOLTAGE, \
> .owner = THIS_MODULE, \
> - .min_uV = MIN_600_MV, \
> - .uV_step = STEP_6_25_MV, \
> + .min_uV = min, \
> + .uV_step = step, \
> .n_voltages = S2MPS11_BUCK_N_VOLTAGES, \
> .ramp_delay = S2MPS11_RAMP_DELAY, \
> .vsel_reg = S2MPS11_REG_B6CTRL2 + (num - 6) * 2, \
> @@ -327,87 +313,55 @@ static struct regulator_ops s2mps11_buck_ops = {
> .enable_mask = S2MPS11_ENABLE_MASK \
> }
>
> -#define regulator_desc_s2mps11_buck9 { \
> - .name = "BUCK9", \
> - .id = S2MPS11_BUCK9, \
> - .ops = &s2mps11_buck_ops, \
> - .type = REGULATOR_VOLTAGE, \
> - .owner = THIS_MODULE, \
> - .min_uV = MIN_3000_MV, \
> - .uV_step = STEP_25_MV, \
> - .n_voltages = S2MPS11_BUCK_N_VOLTAGES, \
> - .ramp_delay = S2MPS11_RAMP_DELAY, \
> - .vsel_reg = S2MPS11_REG_B9CTRL2, \
> - .vsel_mask = S2MPS11_BUCK_VSEL_MASK, \
> - .enable_reg = S2MPS11_REG_B9CTRL1, \
> - .enable_mask = S2MPS11_ENABLE_MASK \
> -}
> -
> -#define regulator_desc_s2mps11_buck10 { \
> - .name = "BUCK10", \
> - .id = S2MPS11_BUCK10, \
> - .ops = &s2mps11_buck_ops, \
> - .type = REGULATOR_VOLTAGE, \
> - .owner = THIS_MODULE, \
> - .min_uV = MIN_750_MV, \
> - .uV_step = STEP_12_5_MV, \
> - .n_voltages = S2MPS11_BUCK_N_VOLTAGES, \
> - .ramp_delay = S2MPS11_RAMP_DELAY, \
> - .vsel_reg = S2MPS11_REG_B10CTRL2, \
> - .vsel_mask = S2MPS11_BUCK_VSEL_MASK, \
> - .enable_reg = S2MPS11_REG_B10CTRL1, \
> - .enable_mask = S2MPS11_ENABLE_MASK \
> -}
> -
> static const struct regulator_desc s2mps11_regulators[] = {
> - regulator_desc_s2mps11_ldo2(1),
> - regulator_desc_s2mps11_ldo1(2),
> - regulator_desc_s2mps11_ldo1(3),
> - regulator_desc_s2mps11_ldo1(4),
> - regulator_desc_s2mps11_ldo1(5),
> - regulator_desc_s2mps11_ldo2(6),
> - regulator_desc_s2mps11_ldo1(7),
> - regulator_desc_s2mps11_ldo1(8),
> - regulator_desc_s2mps11_ldo1(9),
> - regulator_desc_s2mps11_ldo1(10),
> - regulator_desc_s2mps11_ldo2(11),
> - regulator_desc_s2mps11_ldo1(12),
> - regulator_desc_s2mps11_ldo1(13),
> - regulator_desc_s2mps11_ldo1(14),
> - regulator_desc_s2mps11_ldo1(15),
> - regulator_desc_s2mps11_ldo1(16),
> - regulator_desc_s2mps11_ldo1(17),
> - regulator_desc_s2mps11_ldo1(18),
> - regulator_desc_s2mps11_ldo1(19),
> - regulator_desc_s2mps11_ldo1(20),
> - regulator_desc_s2mps11_ldo1(21),
> - regulator_desc_s2mps11_ldo2(22),
> - regulator_desc_s2mps11_ldo2(23),
> - regulator_desc_s2mps11_ldo1(24),
> - regulator_desc_s2mps11_ldo1(25),
> - regulator_desc_s2mps11_ldo1(26),
> - regulator_desc_s2mps11_ldo2(27),
> - regulator_desc_s2mps11_ldo1(28),
> - regulator_desc_s2mps11_ldo1(29),
> - regulator_desc_s2mps11_ldo1(30),
> - regulator_desc_s2mps11_ldo1(31),
> - regulator_desc_s2mps11_ldo1(32),
> - regulator_desc_s2mps11_ldo1(33),
> - regulator_desc_s2mps11_ldo1(34),
> - regulator_desc_s2mps11_ldo1(35),
> - regulator_desc_s2mps11_ldo1(36),
> - regulator_desc_s2mps11_ldo1(37),
> - regulator_desc_s2mps11_ldo1(38),
> + regulator_desc_s2mps11_ldo(1, MIN_800_MV, STEP_25_MV),
> + regulator_desc_s2mps11_ldo(2, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(3, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(4, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(5, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(6, MIN_800_MV, STEP_25_MV),
> + regulator_desc_s2mps11_ldo(7, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(8, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(9, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(10, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(11, MIN_800_MV, STEP_25_MV),
> + regulator_desc_s2mps11_ldo(12, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(13, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(14, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(15, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(16, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(17, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(18, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(19, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(20, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(21, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(22, MIN_800_MV, STEP_25_MV),
> + regulator_desc_s2mps11_ldo(23, MIN_800_MV, STEP_25_MV),
> + regulator_desc_s2mps11_ldo(24, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(25, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(26, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(27, MIN_800_MV, STEP_25_MV),
> + regulator_desc_s2mps11_ldo(28, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(29, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(30, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(31, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(32, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(33, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(34, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(35, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(36, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(37, MIN_800_MV, STEP_50_MV),
> + regulator_desc_s2mps11_ldo(38, MIN_800_MV, STEP_50_MV),
> regulator_desc_s2mps11_buck1_4(1),
> regulator_desc_s2mps11_buck1_4(2),
> regulator_desc_s2mps11_buck1_4(3),
> regulator_desc_s2mps11_buck1_4(4),
> regulator_desc_s2mps11_buck5,
> - regulator_desc_s2mps11_buck6_8(6),
> - regulator_desc_s2mps11_buck6_8(7),
> - regulator_desc_s2mps11_buck6_8(8),
> - regulator_desc_s2mps11_buck9,
> - regulator_desc_s2mps11_buck10,
> + regulator_desc_s2mps11_buck6_10(6, MIN_600_MV, STEP_6_25_MV),
> + regulator_desc_s2mps11_buck6_10(7, MIN_600_MV, STEP_6_25_MV),
> + regulator_desc_s2mps11_buck6_10(8, MIN_600_MV, STEP_6_25_MV),
> + regulator_desc_s2mps11_buck6_10(9, MIN_3000_MV, STEP_25_MV),
> + regulator_desc_s2mps11_buck6_10(10, MIN_750_MV, STEP_12_5_MV),
> };
>
> static int s2mps14_regulator_enable(struct regulator_dev *rdev)
> @@ -467,56 +421,29 @@ static struct regulator_ops s2mps14_reg_ops = {
> .set_suspend_disable = s2mps14_regulator_set_suspend_disable,
> };
>
> -#define regulator_desc_s2mps14_ldo1(num) { \
> - .name = "LDO"#num, \
> - .id = S2MPS14_LDO##num, \
> - .ops = &s2mps14_reg_ops, \
> - .type = REGULATOR_VOLTAGE, \
> - .owner = THIS_MODULE, \
> - .min_uV = MIN_800_MV, \
> - .uV_step = STEP_25_MV, \
> - .n_voltages = S2MPS14_LDO_N_VOLTAGES, \
> - .vsel_reg = S2MPS14_REG_L1CTRL + num - 1, \
> - .vsel_mask = S2MPS14_LDO_VSEL_MASK, \
> - .enable_reg = S2MPS14_REG_L1CTRL + num - 1, \
> - .enable_mask = S2MPS14_ENABLE_MASK \
> -}
> -#define regulator_desc_s2mps14_ldo2(num) { \
> - .name = "LDO"#num, \
> - .id = S2MPS14_LDO##num, \
> - .ops = &s2mps14_reg_ops, \
> - .type = REGULATOR_VOLTAGE, \
> - .owner = THIS_MODULE, \
> - .min_uV = MIN_1800_MV, \
> - .uV_step = STEP_25_MV, \
> - .n_voltages = S2MPS14_LDO_N_VOLTAGES, \
> - .vsel_reg = S2MPS14_REG_L1CTRL + num - 1, \
> - .vsel_mask = S2MPS14_LDO_VSEL_MASK, \
> - .enable_reg = S2MPS14_REG_L1CTRL + num - 1, \
> - .enable_mask = S2MPS14_ENABLE_MASK \
> -}
> -#define regulator_desc_s2mps14_ldo3(num) { \
> +#define regulator_desc_s2mps14_ldo(num, min, step) { \
> .name = "LDO"#num, \
> .id = S2MPS14_LDO##num, \
> .ops = &s2mps14_reg_ops, \
> .type = REGULATOR_VOLTAGE, \
> .owner = THIS_MODULE, \
> - .min_uV = MIN_800_MV, \
> - .uV_step = STEP_12_5_MV, \
> + .min_uV = min, \
> + .uV_step = step, \
> .n_voltages = S2MPS14_LDO_N_VOLTAGES, \
> .vsel_reg = S2MPS14_REG_L1CTRL + num - 1, \
> .vsel_mask = S2MPS14_LDO_VSEL_MASK, \
> .enable_reg = S2MPS14_REG_L1CTRL + num - 1, \
> .enable_mask = S2MPS14_ENABLE_MASK \
> }
> -#define regulator_desc_s2mps14_buck1235(num) { \
> +
> +#define regulator_desc_s2mps14_buck(num, min, step) { \
> .name = "BUCK"#num, \
> .id = S2MPS14_BUCK##num, \
> .ops = &s2mps14_reg_ops, \
> .type = REGULATOR_VOLTAGE, \
> .owner = THIS_MODULE, \
> - .min_uV = MIN_600_MV, \
> - .uV_step = STEP_6_25_MV, \
> + .min_uV = min, \
> + .uV_step = step, \
> .n_voltages = S2MPS14_BUCK_N_VOLTAGES, \
> .linear_min_sel = S2MPS14_BUCK1235_START_SEL, \
> .ramp_delay = S2MPS14_BUCK_RAMP_DELAY, \
> @@ -525,54 +452,38 @@ static struct regulator_ops s2mps14_reg_ops = {
> .enable_reg = S2MPS14_REG_B1CTRL1 + (num - 1) * 2, \
> .enable_mask = S2MPS14_ENABLE_MASK \
> }
> -#define regulator_desc_s2mps14_buck4(num) { \
> - .name = "BUCK"#num, \
> - .id = S2MPS14_BUCK##num, \
> - .ops = &s2mps14_reg_ops, \
> - .type = REGULATOR_VOLTAGE, \
> - .owner = THIS_MODULE, \
> - .min_uV = MIN_1400_MV, \
> - .uV_step = STEP_12_5_MV, \
> - .n_voltages = S2MPS14_BUCK_N_VOLTAGES, \
> - .linear_min_sel = S2MPS14_BUCK4_START_SEL, \
> - .ramp_delay = S2MPS14_BUCK_RAMP_DELAY, \
> - .vsel_reg = S2MPS14_REG_B1CTRL2 + (num - 1) * 2, \
> - .vsel_mask = S2MPS14_BUCK_VSEL_MASK, \
> - .enable_reg = S2MPS14_REG_B1CTRL1 + (num - 1) * 2, \
> - .enable_mask = S2MPS14_ENABLE_MASK \
> -}
>
> static const struct regulator_desc s2mps14_regulators[] = {
> - regulator_desc_s2mps14_ldo3(1),
> - regulator_desc_s2mps14_ldo3(2),
> - regulator_desc_s2mps14_ldo1(3),
> - regulator_desc_s2mps14_ldo1(4),
> - regulator_desc_s2mps14_ldo3(5),
> - regulator_desc_s2mps14_ldo3(6),
> - regulator_desc_s2mps14_ldo1(7),
> - regulator_desc_s2mps14_ldo2(8),
> - regulator_desc_s2mps14_ldo3(9),
> - regulator_desc_s2mps14_ldo3(10),
> - regulator_desc_s2mps14_ldo1(11),
> - regulator_desc_s2mps14_ldo2(12),
> - regulator_desc_s2mps14_ldo2(13),
> - regulator_desc_s2mps14_ldo2(14),
> - regulator_desc_s2mps14_ldo2(15),
> - regulator_desc_s2mps14_ldo2(16),
> - regulator_desc_s2mps14_ldo2(17),
> - regulator_desc_s2mps14_ldo2(18),
> - regulator_desc_s2mps14_ldo1(19),
> - regulator_desc_s2mps14_ldo1(20),
> - regulator_desc_s2mps14_ldo1(21),
> - regulator_desc_s2mps14_ldo3(22),
> - regulator_desc_s2mps14_ldo1(23),
> - regulator_desc_s2mps14_ldo2(24),
> - regulator_desc_s2mps14_ldo2(25),
> - regulator_desc_s2mps14_buck1235(1),
> - regulator_desc_s2mps14_buck1235(2),
> - regulator_desc_s2mps14_buck1235(3),
> - regulator_desc_s2mps14_buck4(4),
> - regulator_desc_s2mps14_buck1235(5),
> + regulator_desc_s2mps14_ldo(1, MIN_800_MV, STEP_12_5_MV),
> + regulator_desc_s2mps14_ldo(2, MIN_800_MV, STEP_12_5_MV),
> + regulator_desc_s2mps14_ldo(3, MIN_800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(4, MIN_800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(5, MIN_800_MV, STEP_12_5_MV),
> + regulator_desc_s2mps14_ldo(6, MIN_800_MV, STEP_12_5_MV),
> + regulator_desc_s2mps14_ldo(7, MIN_800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(8, MIN_1800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(9, MIN_800_MV, STEP_12_5_MV),
> + regulator_desc_s2mps14_ldo(10, MIN_800_MV, STEP_12_5_MV),
> + regulator_desc_s2mps14_ldo(11, MIN_800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(12, MIN_1800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(13, MIN_1800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(14, MIN_1800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(15, MIN_1800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(16, MIN_1800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(17, MIN_1800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(18, MIN_1800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(19, MIN_800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(20, MIN_800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(21, MIN_800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(22, MIN_800_MV, STEP_12_5_MV),
> + regulator_desc_s2mps14_ldo(23, MIN_800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(24, MIN_1800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_ldo(25, MIN_1800_MV, STEP_25_MV),
> + regulator_desc_s2mps14_buck(1, MIN_600_MV, STEP_6_25_MV),
> + regulator_desc_s2mps14_buck(2, MIN_600_MV, STEP_6_25_MV),
> + regulator_desc_s2mps14_buck(3, MIN_600_MV, STEP_6_25_MV),
> + regulator_desc_s2mps14_buck(4, MIN_1400_MV, STEP_12_5_MV),
> + regulator_desc_s2mps14_buck(5, MIN_600_MV, STEP_6_25_MV),
> };
>
> static int s2mps14_pmic_enable_ext_control(struct s2mps11_info *s2mps11,
> --
> 1.9.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Next message: Daeseok Youn: "[PATCH V2] staging: dgap: removes redundant null check and change paramter for dgap_maxcps_room()"
Previous message: amit daniel kachhap: "Re: [PATCH 2/3] regulator: s2mpa01: Optimize the regulator description macro"
In reply to: Amit Daniel Kachhap: "[PATCH 3/3] regulator: s2mps11: Optimize the regulator description macro"
Next in thread: Amit Daniel Kachhap: "[PATCH 2/3] regulator: s2mpa01: Optimize the regulator description macro"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]