drop the unnecessary low variable initialization and make the return
more straightforward.

Signed-off-by: Dong Aisheng <aisheng.d...@nxp.com>
---
 drivers/devfreq/devfreq.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/devfreq/devfreq.c b/drivers/devfreq/devfreq.c
index ce569bd9adfa..7231fe6862a2 100644
--- a/drivers/devfreq/devfreq.c
+++ b/drivers/devfreq/devfreq.c
@@ -351,7 +351,7 @@ static int devfreq_set_target(struct devfreq *devfreq, 
unsigned long new_freq,
 {
        struct devfreq_freqs freqs;
        unsigned long cur_freq;
-       int err = 0;
+       int err;
 
        if (devfreq->profile->get_cur_freq)
                devfreq->profile->get_cur_freq(devfreq->dev.parent, &cur_freq);
@@ -392,7 +392,7 @@ static int devfreq_set_target(struct devfreq *devfreq, 
unsigned long new_freq,
        if (devfreq->suspend_freq)
                devfreq->resume_freq = new_freq;
 
-       return err;
+       return 0;
 }
 
 /**
-- 
2.25.1

Reply via email to