Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 7bdee1c1 authored by Deva Ramasubramanian's avatar Deva Ramasubramanian Committed by Gerrit - the friendly Code Review server
Browse files

msm: vidc: Lower priority of log messages 



Lower the priority of log messages printed out by driver when unhandled
packets are encountered.  Also if there is a failiure during power
collapse, the respective log messages are now warnings as failiure to
power collapse isn't fatal.

Change-Id: Ie298442caae41309d02e7deaf13f954b3e78c096
Signed-off-by: default avatarDeva Ramasubramanian <dramasub@codeaurora.org>
parent 63875384

drivers/media/platform/msm/vidc/hfi_response_handler.c

+4 −4
Original line number Original line Diff line number Diff line
@@ -858,8 +858,8 @@ static void hfi_process_session_prop_info(msm_vidc_callback callback, 
		callback(SESSION_PROPERTY_INFO, &cmd_done);

		callback(SESSION_PROPERTY_INFO, &cmd_done);

		break;

		break;

	default:

	default:

		dprintk(VIDC_ERR,

		dprintk(VIDC_DBG,

				"hal_process_session_prop_info: unknown_prop_id: %d\n",

				"hal_process_session_prop_info: unknown_prop_id: %x\n",

				pkt->rg_property_data[0]);

				pkt->rg_property_data[0]);

		break;

		break;

	}

	}
@@ -1320,8 +1320,8 @@ static void hfi_process_sys_property_info( 
		hfi_process_sys_get_prop_image_version(pkt);

		hfi_process_sys_get_prop_image_version(pkt);

		break;

		break;

	default:

	default:

		dprintk(VIDC_ERR,

		dprintk(VIDC_DBG,

				"hfi_process_sys_property_info: unknown_prop_id: %d\n",

				"hfi_process_sys_property_info: unknown_prop_id: %x\n",

				pkt->rg_property_data[0]);

				pkt->rg_property_data[0]);

	}

	}

}

}

drivers/media/platform/msm/vidc/venus_hfi.c

+3 −3
Original line number Original line Diff line number Diff line
@@ -1060,12 +1060,12 @@ static inline int venus_hfi_power_off(struct venus_hfi_device *device) 
	/*Temporarily enable clocks to make TZ call.*/

	/*Temporarily enable clocks to make TZ call.*/

	rc = venus_hfi_clk_enable(device);

	rc = venus_hfi_clk_enable(device);

	if (rc) {

	if (rc) {

		dprintk(VIDC_ERR, "Failed to enable clocks before TZ call\n");

		dprintk(VIDC_WARN, "Failed to enable clocks before TZ call\n");

		return rc;

		return rc;

	}

	}

	rc = venus_hfi_tzbsp_set_video_state(TZBSP_VIDEO_STATE_SUSPEND);

	rc = venus_hfi_tzbsp_set_video_state(TZBSP_VIDEO_STATE_SUSPEND);

	if (rc) {

	if (rc) {

		dprintk(VIDC_ERR, "Failed to suspend video core %d\n", rc);

		dprintk(VIDC_WARN, "Failed to suspend video core %d\n", rc);

		venus_hfi_clk_disable(device);

		venus_hfi_clk_disable(device);

		return rc;

		return rc;

	}

	}
@@ -1073,7 +1073,7 @@ static inline int venus_hfi_power_off(struct venus_hfi_device *device) 
	venus_hfi_iommu_detach(device);

	venus_hfi_iommu_detach(device);

	rc = regulator_disable(venus_hfi_get_regulator(device, "venus"));

	rc = regulator_disable(venus_hfi_get_regulator(device, "venus"));

	if (rc) {

	if (rc) {

		dprintk(VIDC_ERR, "Failed to disable GDSC, %d\n", rc);

		dprintk(VIDC_WARN, "Failed to disable GDSC, %d\n", rc);

		return rc;

		return rc;

	}

	}