mbox series

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

Message ID 20210302095340.3584204-1-stanimir.varbanov@linaro.org
Headers show
Series Intra-refresh period control | expand

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(-)

> 


-- 
regards,
Stan
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(-)

>>

>