Merge "video: driver: Reduce log level for non-fatal logs"

driver/vidc/src/msm_vidc.c

@@ -488,7 +488,7 @@ int msm_vidc_dqbuf(void *instance, struct v4l2_buffer *b)
 	if (rc == -EAGAIN) {
 		goto unlock;
 	} else if (rc) {
-		i_vpr_e(inst, "%s: failed with %d\n", __func__, rc);
+		i_vpr_l(inst, "%s: failed with %d\n", __func__, rc);
 		goto unlock;
 	}
 

driver/vidc/src/venus_hfi.c

@@ -1460,10 +1460,6 @@ void __disable_unprepare_clks(struct msm_vidc_core *core)
 
 		clk_disable_unprepare(cl->clk);
 		cl->prev = 0;
-
-		if (__clk_is_enabled(cl->clk))
-			d_vpr_e("%s: clock %s not disabled\n",
-				__func__, cl->name);
 	}
 }
 
@@ -1521,10 +1517,6 @@ static int __prepare_enable_clks(struct msm_vidc_core *core)
 			__set_clk_rate(core, cl,
 					clk_round_rate(cl->clk, 0));
 
-		if (__clk_is_enabled(cl->clk))
-			d_vpr_e("%s: clock %s already enabled\n",
-				__func__, cl->name);
-
 		rc = clk_prepare_enable(cl->clk);
 		if (rc) {
 			d_vpr_e("Failed to enable clocks\n");
@@ -1812,10 +1804,6 @@ static int __disable_regulator(struct regulator_info *rinfo,
 		goto disable_regulator_failed;
 	}
 
-	if (regulator_is_enabled(rinfo->regulator))
-		d_vpr_e("%s: regulator %s not disabled\n",
-			__func__, rinfo->name);
-
 	return 0;
 disable_regulator_failed:
 
@@ -1844,10 +1832,6 @@ static int __enable_regulators(struct msm_vidc_core *core)
 	d_vpr_h("Enabling regulators\n");
 
 	venus_hfi_for_each_regulator(core, rinfo) {
-		if (regulator_is_enabled(rinfo->regulator))
-			d_vpr_e("%s: regulator %s already enabled\n",
-				__func__, rinfo->name);
-
 		rc = regulator_enable(rinfo->regulator);
 		if (rc) {
 			d_vpr_e("Failed to enable %s: %d\n",