lkml.org 
[lkml]   [2015]   [Nov]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v4 13/24] hwmon: pwm-fan: use pwm_get/set_default_xxx() helpers where appropriate
    Date
    pwm_set/get_default_xxx() helpers have been introduced to differentiate
    the default PWM states (those retrieved through DT, PWM lookup table or
    statically assigned by the driver) and the current ones.
    Make use of those helpers where appropriate.

    Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
    ---
    drivers/hwmon/pwm-fan.c | 15 +++++++++------
    1 file changed, 9 insertions(+), 6 deletions(-)

    diff --git a/drivers/hwmon/pwm-fan.c b/drivers/hwmon/pwm-fan.c
    index 105b964..989d7b4 100644
    --- a/drivers/hwmon/pwm-fan.c
    +++ b/drivers/hwmon/pwm-fan.c
    @@ -47,8 +47,9 @@ static int __set_pwm(struct pwm_fan_ctx *ctx, unsigned long pwm)
    if (ctx->pwm_value == pwm)
    goto exit_set_pwm_err;

    - duty = DIV_ROUND_UP(pwm * (pwm_get_period((ctx->pwm)) - 1), MAX_PWM);
    - ret = pwm_config(ctx->pwm, duty, pwm_get_period((ctx->pwm)));
    + duty = DIV_ROUND_UP(pwm * (pwm_get_default_period((ctx->pwm)) - 1),
    + MAX_PWM);
    + ret = pwm_config(ctx->pwm, duty, pwm_get_default_period((ctx->pwm)));
    if (ret)
    goto exit_set_pwm_err;

    @@ -234,10 +235,11 @@ static int pwm_fan_probe(struct platform_device *pdev)
    platform_set_drvdata(pdev, ctx);

    /* Set duty cycle to maximum allowed */
    - duty_cycle = pwm_get_period((ctx->pwm)) - 1;
    + duty_cycle = pwm_get_default_period((ctx->pwm)) - 1;
    ctx->pwm_value = MAX_PWM;

    - ret = pwm_config(ctx->pwm, duty_cycle, pwm_get_period((ctx->pwm)));
    + ret = pwm_config(ctx->pwm, duty_cycle,
    + pwm_get_default_period((ctx->pwm)));
    if (ret) {
    dev_err(&pdev->dev, "Failed to configure PWM\n");
    return ret;
    @@ -309,9 +311,10 @@ static int pwm_fan_resume(struct device *dev)
    if (ctx->pwm_value == 0)
    return 0;

    - duty = DIV_ROUND_UP(ctx->pwm_value * (pwm_get_period((ctx->pwm)) - 1),
    + duty = DIV_ROUND_UP(ctx->pwm_value *
    + (pwm_get_default_period((ctx->pwm)) - 1),
    MAX_PWM);
    - ret = pwm_config(ctx->pwm, duty, pwm_get_period((ctx->pwm)));
    + ret = pwm_config(ctx->pwm, duty, pwm_get_default_period((ctx->pwm)));
    if (ret)
    return ret;
    return pwm_enable(ctx->pwm);
    --
    2.1.4


    \
     
     \ /
      Last update: 2015-11-16 10:21    [W:5.223 / U:3.508 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site