diff options
| author | Linux Build Service Account <lnxbuild@localhost> | 2016-10-22 11:26:54 -0700 |
|---|---|---|
| committer | Gerrit - the friendly Code Review server <code-review@localhost> | 2016-10-22 11:26:53 -0700 |
| commit | dac0f016e77c589d249e6297cb36aa9920fa6277 (patch) | |
| tree | 5256e7af3300bae2eeab3e23cf13f45e9e2b542f | |
| parent | 3320f03611573a955335d21df736b48412c8f902 (diff) | |
| parent | 237cd9fbf6c4f1fe14e628b27d95cd56d31ddaf9 (diff) | |
Merge "msm: kgsl: Remove use of uninitialized variable"
| -rw-r--r-- | drivers/devfreq/governor_bw_vbif.c | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/drivers/devfreq/governor_bw_vbif.c b/drivers/devfreq/governor_bw_vbif.c index da1eefb8c94e..33e144b653d0 100644 --- a/drivers/devfreq/governor_bw_vbif.c +++ b/drivers/devfreq/governor_bw_vbif.c @@ -78,15 +78,13 @@ static int devfreq_vbif_ev_handler(struct devfreq *devfreq, case DEVFREQ_GOV_START: mutex_lock(&df_lock); df = devfreq; - if (df->profile->get_dev_status) - ret = df->profile->get_dev_status(df->dev.parent, - &stat); + if (df->profile->get_dev_status && + !df->profile->get_dev_status(df->dev.parent, &stat) && + stat.private_data) + dev_ab = stat.private_data; else - ret = 0; - if (ret || !stat.private_data) pr_warn("Device doesn't take AB votes!\n"); - else - dev_ab = stat.private_data; + mutex_unlock(&df_lock); ret = devfreq_vbif_update_bw(0, 0); |