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.varbanov@xxxxxxxxxx> >> --- >> 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 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html