Hi Hans, On 12/05/2016 02:05 PM, Hans Verkuil wrote: > On 12/01/2016 10:03 AM, Stanimir Varbanov wrote: >> Here is the implementation of Venus video accelerator low-level >> functionality. It contanins code which setup the registers and >> startup uthe processor, allocate and manipulates with the shared >> memory used for sending commands and receiving messages. >> >> Signed-off-by: Stanimir Varbanov <stanimir.varba...@linaro.org> >> --- >> drivers/media/platform/qcom/venus/hfi_venus.c | 1508 >> ++++++++++++++++++++++ >> drivers/media/platform/qcom/venus/hfi_venus.h | 23 + >> drivers/media/platform/qcom/venus/hfi_venus_io.h | 98 ++ >> 3 files changed, 1629 insertions(+) >> create mode 100644 drivers/media/platform/qcom/venus/hfi_venus.c >> create mode 100644 drivers/media/platform/qcom/venus/hfi_venus.h >> create mode 100644 drivers/media/platform/qcom/venus/hfi_venus_io.h >> >> diff --git a/drivers/media/platform/qcom/venus/hfi_venus.c >> b/drivers/media/platform/qcom/venus/hfi_venus.c >> new file mode 100644 >> index 000000000000..f004a9a80d85 >> --- /dev/null >> +++ b/drivers/media/platform/qcom/venus/hfi_venus.c >> @@ -0,0 +1,1508 @@ >> +static int venus_tzbsp_set_video_state(enum tzbsp_video_state state) >> +{ >> + return qcom_scm_video_set_state(state, 0); > > This functions doesn't seem to exist. Is there a prerequisite patch series > that > introduces this function?
yes, the patchset [1] is under review. -- regards, Stan [1] https://lkml.org/lkml/2016/11/7/533