From b3e7113283cca3300ad5badc2e6f6d697e84a86a Mon Sep 17 00:00:00 2001 From: Deepak Kumar Date: Thu, 30 Mar 2017 12:26:35 +0530 Subject: msm: kgsl: Remove incorrect check to cap AB vote to IB vote For dual channel DDR, IB vote from client is half of actual IB vote as IB vote calculation on client side doesn't consider number of DDR channels. ICB driver takes care of multiplying the client IB vote with number of DDR channels. This change removes the AB capping check to avoid the scenarios where AB vote > actual IB vote/2 but gets capped to actual IB vote/2 because client side IB vote is half of actual IB vote. Removal of this check will not impact single channel DDR targets because of the way AB value is calculated. In case of honest BW voting, AB will always be less than IB as AB calculation doesn't consider RAM wait value. In case of unhonest vote, AB value is always caluclated as some percentage of IB vote and this percentage value is always <=100%. Change-Id: Icdca6118f6605665979a1bead35ba3ef631d50e8 Signed-off-by: Deepak Kumar --- drivers/gpu/msm/kgsl_pwrctrl.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/gpu/msm/kgsl_pwrctrl.c b/drivers/gpu/msm/kgsl_pwrctrl.c index e4c431546d2a..a06af7a22595 100644 --- a/drivers/gpu/msm/kgsl_pwrctrl.c +++ b/drivers/gpu/msm/kgsl_pwrctrl.c @@ -150,9 +150,6 @@ static void _ab_buslevel_update(struct kgsl_pwrctrl *pwr, *ab = pwr->bus_ab_mbytes; else *ab = (pwr->bus_percent_ab * max_bw) / 100; - - if (*ab > ib) - *ab = ib; } /** -- cgit v1.2.3