[v3,0/2] Intra-refresh period control

Message

Stanimir Varbanov March 2, 2021, 9:53 a.m. UTC

Hi,

This series add a new intra-refresh period control for encoders. The
series is a continuation of [1]. Comments addressed:
 * A typo in .rst (Hans)
 * Clarified the relationship with CYCLIC_INTRA_REFRESH_MB (Hans)

Comments are welcome!

regards,
Stan

[1] https://www.spinics.net/lists/linux-media/msg183019.html

Stanimir Varbanov (2):
  media: v4l2-ctrls: Add intra-refresh period control
  venus: venc: Add support for intra-refresh period

 .../media/v4l/ext-ctrls-codec.rst             | 12 ++++++++
 drivers/media/platform/qcom/venus/core.h      |  1 +
 drivers/media/platform/qcom/venus/venc.c      | 28 +++++++++++++++++++
 .../media/platform/qcom/venus/venc_ctrls.c    | 13 ++++-----
 drivers/media/v4l2-core/v4l2-ctrls.c          |  2 ++
 include/uapi/linux/v4l2-controls.h            |  1 +
 6 files changed, 50 insertions(+), 7 deletions(-)

Comments

Stanimir Varbanov April 13, 2021, 3:49 p.m. UTC | #1

Hi Hans,

Any comments?

On 3/2/21 11:53 AM, Stanimir Varbanov wrote:
> Hi,
> 
> This series add a new intra-refresh period control for encoders. The
> series is a continuation of [1]. Comments addressed:
>  * A typo in .rst (Hans)
>  * Clarified the relationship with CYCLIC_INTRA_REFRESH_MB (Hans)
> 
> Comments are welcome!
> 
> regards,
> Stan
> 
> [1] https://www.spinics.net/lists/linux-media/msg183019.html
> 
> Stanimir Varbanov (2):
>   media: v4l2-ctrls: Add intra-refresh period control
>   venus: venc: Add support for intra-refresh period
> 
>  .../media/v4l/ext-ctrls-codec.rst             | 12 ++++++++
>  drivers/media/platform/qcom/venus/core.h      |  1 +
>  drivers/media/platform/qcom/venus/venc.c      | 28 +++++++++++++++++++
>  .../media/platform/qcom/venus/venc_ctrls.c    | 13 ++++-----
>  drivers/media/v4l2-core/v4l2-ctrls.c          |  2 ++
>  include/uapi/linux/v4l2-controls.h            |  1 +
>  6 files changed, 50 insertions(+), 7 deletions(-)
>

Hans Verkuil April 13, 2021, 3:57 p.m. UTC | #2

On 13/04/2021 17:49, Stanimir Varbanov wrote:
> Hi Hans,
> 
> Any comments?

Thanks for the reminder, I replied to the patch.

Regards,

	Hans

> 
> On 3/2/21 11:53 AM, Stanimir Varbanov wrote:
>> Hi,
>>
>> This series add a new intra-refresh period control for encoders. The
>> series is a continuation of [1]. Comments addressed:
>>  * A typo in .rst (Hans)
>>  * Clarified the relationship with CYCLIC_INTRA_REFRESH_MB (Hans)
>>
>> Comments are welcome!
>>
>> regards,
>> Stan
>>
>> [1] https://www.spinics.net/lists/linux-media/msg183019.html
>>
>> Stanimir Varbanov (2):
>>   media: v4l2-ctrls: Add intra-refresh period control
>>   venus: venc: Add support for intra-refresh period
>>
>>  .../media/v4l/ext-ctrls-codec.rst             | 12 ++++++++
>>  drivers/media/platform/qcom/venus/core.h      |  1 +
>>  drivers/media/platform/qcom/venus/venc.c      | 28 +++++++++++++++++++
>>  .../media/platform/qcom/venus/venc_ctrls.c    | 13 ++++-----
>>  drivers/media/v4l2-core/v4l2-ctrls.c          |  2 ++
>>  include/uapi/linux/v4l2-controls.h            |  1 +
>>  6 files changed, 50 insertions(+), 7 deletions(-)
>>
>