lkml.org 
[lkml]   [2018]   [May]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v2 23/29] venus: helpers: add a helper to return opb buffer sizes
On Tue, May 15, 2018 at 5:02 PM Stanimir Varbanov
<stanimir.varbanov@linaro.org> wrote:
>
> Add a helper function to return current output picture buffer size.
> OPB sizes can vary depending on the selected decoder output(s).
>
> Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org>
> ---
> drivers/media/platform/qcom/venus/core.h | 10 ++++++++++
> drivers/media/platform/qcom/venus/helpers.c | 15 +++++++++++++++
> drivers/media/platform/qcom/venus/helpers.h | 1 +
> 3 files changed, 26 insertions(+)
>
> diff --git a/drivers/media/platform/qcom/venus/core.h b/drivers/media/platform/qcom/venus/core.h
> index 255292899204..4d6c05f156c4 100644
> --- a/drivers/media/platform/qcom/venus/core.h
> +++ b/drivers/media/platform/qcom/venus/core.h
> @@ -234,6 +234,11 @@ struct venus_buffer {
> * @num_output_bufs: holds number of output buffers
> * @input_buf_size holds input buffer size
> * @output_buf_size: holds output buffer size
> + * @output2_buf_size: holds secondary decoder output buffer size
> + * @dpb_buftype: decoded picture buffer type
> + * @dpb_fmt: decodec picture buffre raw format

typo: s/decodec/decoded/ and s/buffre/buffer/

> + * @opb_buftype: output picture buffer type
> + * @opb_fmt: output picture buffer raw format
> * @reconfig: a flag raised by decoder when the stream resolution changed
> * @reconfig_width: holds the new width
> * @reconfig_height: holds the new height
> @@ -282,6 +287,11 @@ struct venus_inst {
> unsigned int num_output_bufs;
> unsigned int input_buf_size;
> unsigned int output_buf_size;
> + unsigned int output2_buf_size;
> + u32 dpb_buftype;
> + u32 dpb_fmt;

These 2 don't seem to be used.

Best regards,
Tomasz

\
 
 \ /
  Last update: 2018-05-31 11:22    [W:1.563 / U:0.036 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site