Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH v3 0/2] Intra-refresh period control
To:     Stanimir Varbanov <stanimir.varbanov@linaro.org>,
        linux-media@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-arm-msm@vger.kernel.org
Cc:     Ezequiel Garcia <ezequiel@collabora.com>,
        nicolas.dufresne@collabora.com
References: <20210302095340.3584204-1-stanimir.varbanov@linaro.org>
 <3480b6ee-f442-57be-473f-c90966584d40@linaro.org>
From:   Hans Verkuil <hverkuil-cisco@xs4all.nl>
Message-ID: <29fc026d-6ba3-dfc3-d57b-1dabce765dca@xs4all.nl>
Date:   Tue, 13 Apr 2021 17:57:05 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Firefox/78.0 Thunderbird/78.9.0
MIME-Version: 1.0
In-Reply-To: <3480b6ee-f442-57be-473f-c90966584d40@linaro.org>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

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