pwm_config/enable/disable() have been deprecated and should be replaced
by pwm_apply_state().

Signed-off-by: Boris Brezillon <boris.brezillon at free-electrons.com>
---
 drivers/input/misc/pwm-beeper.c | 46 +++++++++++++++++++++++++++--------------
 1 file changed, 31 insertions(+), 15 deletions(-)

diff --git a/drivers/input/misc/pwm-beeper.c b/drivers/input/misc/pwm-beeper.c
index f2261ab..36c9897 100644
--- a/drivers/input/misc/pwm-beeper.c
+++ b/drivers/input/misc/pwm-beeper.c
@@ -34,7 +34,7 @@ static int pwm_beeper_event(struct input_dev *input,
 {
        int ret = 0;
        struct pwm_beeper *beeper = input_get_drvdata(input);
-       unsigned long period;
+       struct pwm_state pstate;

        if (type != EV_SND || value < 0)
                return -EINVAL;
@@ -49,19 +49,22 @@ static int pwm_beeper_event(struct input_dev *input,
                return -EINVAL;
        }

+       pwm_get_state(beeper->pwm, &pstate);
+
        if (value == 0) {
-               pwm_disable(beeper->pwm);
+               pstate.enabled = false;
        } else {
-               period = HZ_TO_NANOSECONDS(value);
-               ret = pwm_config(beeper->pwm, period / 2, period);
-               if (ret)
-                       return ret;
-               ret = pwm_enable(beeper->pwm);
-               if (ret)
-                       return ret;
-               beeper->period = period;
+               pstate.enabled = false;
+               pstate.period = HZ_TO_NANOSECONDS(value);
+               pstate.duty_cycle = pstate.period / 2;
        }

+       ret = pwm_apply_state(beeper->pwm, &pstate);
+       if (ret)
+               return ret;
+
+       beeper->period = value ? pstate.period : 0;
+
        return 0;
 }

@@ -132,10 +135,13 @@ err_free:
 static int pwm_beeper_remove(struct platform_device *pdev)
 {
        struct pwm_beeper *beeper = platform_get_drvdata(pdev);
+       struct pwm_state pstate;

        input_unregister_device(beeper->input);

-       pwm_disable(beeper->pwm);
+       pwm_get_state(beeper->pwm, &pstate);
+       pstate.enabled = false;
+       pwm_apply_state(beeper->pwm, &pstate);
        pwm_free(beeper->pwm);

        kfree(beeper);
@@ -147,8 +153,13 @@ static int __maybe_unused pwm_beeper_suspend(struct device 
*dev)
 {
        struct pwm_beeper *beeper = dev_get_drvdata(dev);

-       if (beeper->period)
-               pwm_disable(beeper->pwm);
+       if (beeper->period) {
+               struct pwm_state pstate;
+
+               pwm_get_state(beeper->pwm, &pstate);
+               pstate.enabled = false;
+               pwm_apply_state(beeper->pwm, &pstate);
+       }

        return 0;
 }
@@ -158,8 +169,13 @@ static int __maybe_unused pwm_beeper_resume(struct device 
*dev)
        struct pwm_beeper *beeper = dev_get_drvdata(dev);

        if (beeper->period) {
-               pwm_config(beeper->pwm, beeper->period / 2, beeper->period);
-               pwm_enable(beeper->pwm);
+               struct pwm_state pstate;
+
+               pwm_get_state(beeper->pwm, &pstate);
+               pstate.period = beeper->period;
+               pstate.duty_cycle = beeper->period / 2;
+               pstate.enabled = true;
+               pwm_apply_state(beeper->pwm, &pstate);
        }

        return 0;
-- 
2.5.0

Reply via email to