lkml.org 
[lkml]   [2021]   [Jan]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v4 1/7] regulator: qcom-labibb: Switch voltage ops from linear_range to linear
    Date
    The LAB and IBB regulator have just one range and it is useless
    to use linear_range ops, as these are used to express multiple
    linear ranges.

    Switch list_voltage and map_voltage callbacks to *_linear instead.

    Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>
    ---
    drivers/regulator/qcom-labibb-regulator.c | 16 ++++++----------
    1 file changed, 6 insertions(+), 10 deletions(-)

    diff --git a/drivers/regulator/qcom-labibb-regulator.c b/drivers/regulator/qcom-labibb-regulator.c
    index 9f51c96f16fb..0fe0f6bce4cf 100644
    --- a/drivers/regulator/qcom-labibb-regulator.c
    +++ b/drivers/regulator/qcom-labibb-regulator.c
    @@ -59,8 +59,8 @@ static const struct regulator_ops qcom_labibb_ops = {
    .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_range,
    - .map_voltage = regulator_map_voltage_linear_range,
    + .list_voltage = regulator_list_voltage_linear,
    + .map_voltage = regulator_map_voltage_linear,
    };

    static const struct regulator_desc pmi8998_lab_desc = {
    @@ -76,10 +76,8 @@ static const struct regulator_desc pmi8998_lab_desc = {
    .off_on_delay = LABIBB_OFF_ON_DELAY,
    .owner = THIS_MODULE,
    .type = REGULATOR_VOLTAGE,
    - .linear_ranges = (struct linear_range[]) {
    - REGULATOR_LINEAR_RANGE(4600000, 0, 15, 100000),
    - },
    - .n_linear_ranges = 1,
    + .min_uV = 4600000,
    + .uV_step = 100000,
    .n_voltages = 16,
    .ops = &qcom_labibb_ops,
    };
    @@ -97,10 +95,8 @@ static const struct regulator_desc pmi8998_ibb_desc = {
    .off_on_delay = LABIBB_OFF_ON_DELAY,
    .owner = THIS_MODULE,
    .type = REGULATOR_VOLTAGE,
    - .linear_ranges = (struct linear_range[]) {
    - REGULATOR_LINEAR_RANGE(1400000, 0, 63, 100000),
    - },
    - .n_linear_ranges = 1,
    + .min_uV = 1400000,
    + .uV_step = 100000,
    .n_voltages = 64,
    .ops = &qcom_labibb_ops,
    };
    --
    2.30.0
    \
     
     \ /
      Last update: 2021-01-19 23:30    [W:4.163 / U:0.436 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site