| Message ID | 20201204100139.6216-1-stanimir.varbanov@linaro.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | Venus encoder improvements | expand |
On Fri, Dec 4, 2020 at 2:03 AM Stanimir Varbanov <stanimir.varbanov@linaro.org> wrote: > > Hello, > > Changes since v1: > * 1/4 - fixed error handling in hfi_session_deinit (Alex) > - keep venc_set_properties invocation from start_streaming (Dikshita) > * 2/4 - keep original mutex_lock (Alex) > * 3/4 - move msg queue inside if statement (Fritz) > - move rx_req setting before triggering soft interrupt (Alex) > * Add one more patch 4/4 to address comments for hfi_session_init > EINVAL return error code (Alex) > > The v1 can be found at [1]. > > regards, > Stan > > [1] https://www.spinics.net/lists/linux-media/msg181634.html > > Stanimir Varbanov (3): > venus: venc: Init the session only once in queue_setup > venus: Limit HFI sessions to the maximum supported > venus: hfi: Correct session init return error > > Vikash Garodia (1): > media: venus: request for interrupt from venus > > drivers/media/platform/qcom/venus/core.h | 1 + > drivers/media/platform/qcom/venus/hfi.c | 18 +++- > .../media/platform/qcom/venus/hfi_parser.c | 3 + > drivers/media/platform/qcom/venus/hfi_venus.c | 77 ++++++++++------- > drivers/media/platform/qcom/venus/vdec.c | 2 +- > drivers/media/platform/qcom/venus/venc.c | 85 ++++++++++++++----- > 6 files changed, 127 insertions(+), 59 deletions(-) > > -- > 2.17.1 > I haven't had a chance to review the code yet, I'll leave that for early next week. In the meantime I have tested the patches and found them to be working well. Tested-by: Fritz Koenig <frkoenig@chromium.org>
Hello: This series was applied to qcom/linux.git (refs/heads/for-next): On Fri, 4 Dec 2020 12:01:35 +0200 you wrote: > Hello, > > Changes since v1: > * 1/4 - fixed error handling in hfi_session_deinit (Alex) > - keep venc_set_properties invocation from start_streaming (Dikshita) > * 2/4 - keep original mutex_lock (Alex) > * 3/4 - move msg queue inside if statement (Fritz) > - move rx_req setting before triggering soft interrupt (Alex) > * Add one more patch 4/4 to address comments for hfi_session_init > EINVAL return error code (Alex) > > [...] Here is the summary with links: - [v2,1/4] venus: venc: Init the session only once in queue_setup https://git.kernel.org/qcom/c/5f2ca73dcca9 - [v2,2/4] venus: Limit HFI sessions to the maximum supported https://git.kernel.org/qcom/c/20891170f339 - [v2,3/4] media: venus: request for interrupt from venus https://git.kernel.org/qcom/c/7f339fdc3756 - [v2,4/4] venus: hfi: Correct session init return error https://git.kernel.org/qcom/c/e922a33e0228 You are awesome, thank you! -- Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/patchwork/pwbot.html
Hello, Changes since v1: * 1/4 - fixed error handling in hfi_session_deinit (Alex) - keep venc_set_properties invocation from start_streaming (Dikshita) * 2/4 - keep original mutex_lock (Alex) * 3/4 - move msg queue inside if statement (Fritz) - move rx_req setting before triggering soft interrupt (Alex) * Add one more patch 4/4 to address comments for hfi_session_init EINVAL return error code (Alex) The v1 can be found at [1]. regards, Stan [1] https://www.spinics.net/lists/linux-media/msg181634.html Stanimir Varbanov (3): venus: venc: Init the session only once in queue_setup venus: Limit HFI sessions to the maximum supported venus: hfi: Correct session init return error Vikash Garodia (1): media: venus: request for interrupt from venus drivers/media/platform/qcom/venus/core.h | 1 + drivers/media/platform/qcom/venus/hfi.c | 18 +++- .../media/platform/qcom/venus/hfi_parser.c | 3 + drivers/media/platform/qcom/venus/hfi_venus.c | 77 ++++++++++------- drivers/media/platform/qcom/venus/vdec.c | 2 +- drivers/media/platform/qcom/venus/venc.c | 85 ++++++++++++++----- 6 files changed, 127 insertions(+), 59 deletions(-)