Re: [PATCH v2 1/3] firmware: qcom: scm: add a video command for state setting
From: Stephen Boyd <hidden>
Date: 2016-11-10 01:52:56
Also in:
linux-arm-msm, linux-remoteproc, lkml
On 11/07, Stanimir Varbanov wrote:
quoted hunk ↗ jump to hunk
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.
+ if (ret)
+ return ret;
+
+ ret = __qcom_scm_video_set_state(__scm->dev, state, spare);
+
+ qcom_scm_clk_disable();
+
+ return ret;
+}
+EXPORT_SYMBOL(qcom_scm_video_set_state);
+
static int qcom_scm_probe(struct platform_device *pdev)
{
struct qcom_scm *scm;-- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html