Parcourir la source

qcacmn: Fix core_ctl_set_boost "off" ref count

In current implementation, we may call core_ctl_set_boost(false) even
if its already been turned off an a previous occasion. This leads to
excessive prints in the kmsg "Error turning off boost. Boost already
turned off"

Fix the ref_count such that we donot call the API to turn
core_ctl_set_boost "off", if it is already off.

CRs-Fixed: 2034738
Change-Id: I6e0760c9e6778087f8de6e5329bcaf110c7b264b
Mohit Khanna il y a 8 ans
Parent
commit
5d6386f288
1 fichiers modifiés avec 8 ajouts et 7 suppressions
  1. 8 7
      hif/src/hif_napi.c

+ 8 - 7
hif/src/hif_napi.c

@@ -1559,14 +1559,15 @@ int hif_napi_cpu_blacklist(struct qca_napi_data *napid,
 		}
 		break;
 	case BLACKLIST_OFF:
-		if (ref_count)
+		if (ref_count) {
 			ref_count--;
-		rc = 0;
-		if (ref_count == 0) {
-			rc = hif_napi_core_ctl_set_boost(false);
-			NAPI_DEBUG("boost_off() returns %d - refcnt=%d",
-				   rc, ref_count);
-			hif_napi_bl_irq(napid, false);
+			rc = 0;
+			if (ref_count == 0) {
+				rc = hif_napi_core_ctl_set_boost(false);
+				NAPI_DEBUG("boost_off() returns %d - refcnt=%d",
+					   rc, ref_count);
+				hif_napi_bl_irq(napid, false);
+			}
 		}
 		break;
 	default: