summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMahesh Kumar Sharma <smahesh@codeaurora.org>2021-08-05 10:41:06 -0700
committerGerrit - the friendly Code Review server <code-review@localhost>2021-08-11 08:58:56 -0700
commit91770b6061972b5b12f7b5fb66e46c36ea068bc8 (patch)
tree09ffba4a67750579d9619cd077fc5070d85ceefa
parentb5fa8d8109eacfee0751462c18954892a28ac343 (diff)
downloadmmrm-91770b6061972b5b12f7b5fb66e46c36ea068bc8.tar.gz
mmrm: Fix KW error
Initialize variable before use. Moved statement to correct block. Change-Id: I6effe4a8d2363596250bbd663e68f1c6bea07589 Signed-off-by: Mahesh Kumar Sharma <smahesh@codeaurora.org>
-rw-r--r--driver/src/mmrm_clk_rsrc_mgr_sw.c27
1 files changed, 13 insertions, 14 deletions
diff --git a/driver/src/mmrm_clk_rsrc_mgr_sw.c b/driver/src/mmrm_clk_rsrc_mgr_sw.c
index e55dfcc..4d9dc85 100644
--- a/driver/src/mmrm_clk_rsrc_mgr_sw.c
+++ b/driver/src/mmrm_clk_rsrc_mgr_sw.c
@@ -518,21 +518,20 @@ static int mmrm_sw_throttle_low_priority_client(
d_mpr_h("%s:csid(0x%x) name(%s)\n",
__func__, tbl_entry_throttle_client->clk_src_id,
tbl_entry_throttle_client->name);
- }
- d_mpr_h("%s:now_cur_ma(%llu) min_cur_ma(%llu) delta_cur(%d)\n",
- __func__, now_cur_ma, min_cur_ma, *delta_cur);
-
- if (!IS_ERR_OR_NULL(tbl_entry_throttle_client)
- && (now_cur_ma > min_cur_ma)
- && (now_cur_ma - min_cur_ma > *delta_cur)) {
- found_client_throttle = true;
- d_mpr_h("%s: Throttle client csid(0x%x) name(%s)\n",
- __func__, tbl_entry_throttle_client->clk_src_id,
- tbl_entry_throttle_client->name);
- d_mpr_h("%s:now_cur_ma %llu-min_cur_ma %llu>delta_cur %d\n",
+ d_mpr_h("%s:now_cur_ma(%llu) min_cur_ma(%llu) delta_cur(%d)\n",
__func__, now_cur_ma, min_cur_ma, *delta_cur);
- /* found client to throttle, break from here. */
- break;
+
+ if ((now_cur_ma > min_cur_ma)
+ && (now_cur_ma - min_cur_ma > *delta_cur)) {
+ found_client_throttle = true;
+ d_mpr_h("%s: Throttle client csid(0x%x) name(%s)\n",
+ __func__, tbl_entry_throttle_client->clk_src_id,
+ tbl_entry_throttle_client->name);
+ d_mpr_h("%s:now_cur_ma %llu-min_cur_ma %llu>delta_cur %d\n",
+ __func__, now_cur_ma, min_cur_ma, *delta_cur);
+ /* found client to throttle, break from here. */
+ break;
+ }
}
}