| Message ID | 20201024163509.22246-1-stanimir.varbanov@linaro.org |
|---|---|
| State | Accepted |
| Commit | 4568ea3fa95d33cc10e8938949ae9a875046e553 |
| Headers | show |
| Series | venus: venc: Fix default capture format | expand |
Hello: This patch was applied to qcom/linux.git (refs/heads/for-next): On Sat, 24 Oct 2020 19:35:09 +0300 you wrote: > The default codec after driver open is set to be H264 but the > instance format for capture is wrongly set to H263. Correct this > to H264. > > For regular applications this is not a big issue because they set > the format through S_FMT but for example v4l2-complience does not. > > [...] Here is the summary with links: - venus: venc: Fix default capture format https://git.kernel.org/qcom/c/4568ea3fa95d You are awesome, thank you! -- Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/patchwork/pwbot.html
diff --git a/drivers/media/platform/qcom/venus/venc.c b/drivers/media/platform/qcom/venus/venc.c index 178e034769e8..4aedde172c83 100644 --- a/drivers/media/platform/qcom/venus/venc.c +++ b/drivers/media/platform/qcom/venus/venc.c @@ -1029,7 +1029,7 @@ static int m2m_queue_init(void *priv, struct vb2_queue *src_vq, static void venc_inst_init(struct venus_inst *inst) { - inst->fmt_cap = &venc_formats[2]; + inst->fmt_cap = &venc_formats[3]; inst->fmt_out = &venc_formats[0]; inst->width = 1280; inst->height = ALIGN(720, 32);
The default codec after driver open is set to be H264 but the instance format for capture is wrongly set to H263. Correct this to H264. For regular applications this is not a big issue because they set the format through S_FMT but for example v4l2-complience does not. Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org> --- drivers/media/platform/qcom/venus/venc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)