[PATCH 2/6] interconnect: qcom: icc-rpm: Don't skip small votes

From: Konrad Dybcio

Date: Wed Sep 09 2026 - 14:53:31 EST


From: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx>

Small votes (such as 1 or 100) can end up being rounded down to 0
midway through the calculations. Use qcom_bw_div() when dividing to
prevent that.

Assisted-by: LLM
Fixes: 30c8fa3ec61a ("interconnect: qcom: Add MSM8916 interconnect provider driver")
Signed-off-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx>
---
drivers/interconnect/qcom/icc-rpm.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/interconnect/qcom/icc-rpm.c b/drivers/interconnect/qcom/icc-rpm.c
index ca4c8a944755..f571f002ccbd 100644
--- a/drivers/interconnect/qcom/icc-rpm.c
+++ b/drivers/interconnect/qcom/icc-rpm.c
@@ -298,25 +298,25 @@ static u64 qcom_icc_calc_rate(struct qcom_icc_provider *qp, struct qcom_icc_node
u64 agg_avg_rate, agg_peak_rate, agg_rate;

if (qn->channels)
- agg_avg_rate = div_u64(qn->sum_avg[ctx], qn->channels);
+ agg_avg_rate = qcom_bw_div(qn->sum_avg[ctx], qn->channels);
else
agg_avg_rate = qn->sum_avg[ctx];

if (qn->ab_coeff) {
agg_avg_rate = agg_avg_rate * qn->ab_coeff;
- agg_avg_rate = div_u64(agg_avg_rate, 100);
+ agg_avg_rate = qcom_bw_div(agg_avg_rate, 100);
}

if (qn->ib_coeff) {
agg_peak_rate = qn->max_peak[ctx] * 100;
- agg_peak_rate = div_u64(agg_peak_rate, qn->ib_coeff);
+ agg_peak_rate = qcom_bw_div(agg_peak_rate, qn->ib_coeff);
} else {
agg_peak_rate = qn->max_peak[ctx];
}

agg_rate = max_t(u64, agg_avg_rate, agg_peak_rate);

- return div_u64(agg_rate, qn->buswidth);
+ return qcom_bw_div(agg_rate, qn->buswidth);
}

/**

--
2.55.0