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

[Hans Verkuil]

On 12/05/2016 01:20 PM, Stanimir Varbanov wrote:
> 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.
> 

What is the status of this patchset?

Regards,

	Hans