Hi, On 11/10/2016 03:52 AM, Stephen Boyd wrote: > On 11/07, Stanimir Varbanov wrote: >> diff --git a/drivers/firmware/qcom_scm.c b/drivers/firmware/qcom_scm.c >> index d95c70227c05..7e364691a87c 100644 >> --- a/drivers/firmware/qcom_scm.c >> +++ b/drivers/firmware/qcom_scm.c >> @@ -320,6 +320,22 @@ bool qcom_scm_is_available(void) >> } >> EXPORT_SYMBOL(qcom_scm_is_available); >> >> +int qcom_scm_video_set_state(u32 state, u32 spare) >> +{ >> + int ret; >> + >> + ret = qcom_scm_clk_enable(); > > Do we need clk control for this? Usually it's only required for > crypto engine things, and turning on video doesn't sound like it > uses crypto. I don't think downstream android kernel does this. Correct, the crypto clk is not needed. -- regards, Stan -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html