Re: [PATCH v4 8/9] media: venus: hfi: add Venus HFI files

[Hans Verkuil]

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?

> +}

Regards,

	Hans