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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <41490c02-d87d-5ae1-9004-328e511a5656@xs4all.nl>
Date:   Mon, 9 Jan 2017 14:04:21 +0100
From:   Hans Verkuil <hverkuil@...all.nl>
To:     Stanimir Varbanov <stanimir.varbanov@...aro.org>,
        Mauro Carvalho Chehab <mchehab@...nel.org>
Cc:     Andy Gross <andy.gross@...aro.org>,
        Bjorn Andersson <bjorn.andersson@...aro.org>,
        Stephen Boyd <sboyd@...eaurora.org>,
        Srinivas Kandagatla <srinivas.kandagatla@...aro.org>,
        linux-media@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-arm-msm@...r.kernel.org
Subject: Re: [PATCH v4 8/9] media: venus: hfi: add Venus HFI files

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@...aro.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.
> 

What is the status of this patchset?

Regards,

	Hans