From patchwork Mon Sep 28 16:44:28 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stanimir Varbanov X-Patchwork-Id: 255576 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI, SPF_HELO_NONE, SPF_PASS, USER_AGENT_GIT autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4ADFBC2D0A8 for ; Mon, 28 Sep 2020 16:45:00 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 079822100A for ; Mon, 28 Sep 2020 16:44:59 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="AXDUg7VQ" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726670AbgI1Qo7 (ORCPT ); Mon, 28 Sep 2020 12:44:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:32788 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726770AbgI1Qo4 (ORCPT ); Mon, 28 Sep 2020 12:44:56 -0400 Received: from mail-ej1-x643.google.com (mail-ej1-x643.google.com [IPv6:2a00:1450:4864:20::643]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1FEEEC0613CF for ; Mon, 28 Sep 2020 09:44:56 -0700 (PDT) Received: by mail-ej1-x643.google.com with SMTP id i26so9438864ejb.12 for ; Mon, 28 Sep 2020 09:44:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id; bh=O4wMisusw5zksx0GW6HdF+k+feJd8Jhmsn7fgUQP2rg=; b=AXDUg7VQm2DBTse4gsSS7l84FGvg7tqTAk6Usm/0qXjQIl3Sx7LsB3WtpUbIYfbBbk ydmmjNit0LhPwyh6cbpwkQYMnVSmWY72vyw7yfwWH16RqStWc23BeW1PSL+8Z4wMYqjn gYqE9TGTiZtMIMTyR70HtSkBrSHnTZN9G7XC1auAD1/oFuCbIUPPYUn0PCEAsnxsht17 DTxF98h3vVM05J3vaxx0U1czbxfKTyGkynZDnSlKoHA+jaCj+YoiAonRi+e8/R+nrI8I +jD8GDK231VaLDXrhjfu7L7d650QfbynIuHv6eyKZsbYwNI+yzR8UmxrRY7aEp+L+rSs sXbw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=O4wMisusw5zksx0GW6HdF+k+feJd8Jhmsn7fgUQP2rg=; b=JjcE1i+GYRDxu1ZngagmfAAZbICDL6vTiaYKbhLmjrORvzRaysUacOXiYPHGqnQ9Q8 fGruxwMAMBns8DfODm0fF/7rI09FbZDa9CP+42mm9bIlFuNzm+3CRLNy/s8WeVTT5GLP uhGARL4AVupW9PxjfyoLv6VrR3w9S75xhWzLT3XrOmYdcHj0twb2NVbY8x04nbEytCLt 5IRobJTcDy6Vcq6KIiQUphQASerlTxVp2kKACHXkWJ8LJWrwiDmVkYZzbOkbi7ZhNaoN 2EDY6tc+Xzh6X0czacwUK49T3Kq7JUQbKgR++AFx2P9uRpS+n4mVMXaDi3+PytW7NWZ5 hV/g== X-Gm-Message-State: AOAM533tABrpweLYOiW/WN6KQSiqXOvFeTmJNAU5+DG620LaBPMCk6tM HqkAAjhiJ8Nx2ESp3Jz50ZBTHjHyWe+QKuZN X-Google-Smtp-Source: ABdhPJwkyS+Tqx2QxSXh7hULUfFINZgVFH9lz8QmQ9qr7HLGRVs8KiNRS3tLvld55tjQGtB4y3+Xcg== X-Received: by 2002:a17:906:fb84:: with SMTP id lr4mr2605288ejb.282.1601311494510; Mon, 28 Sep 2020 09:44:54 -0700 (PDT) Received: from localhost.localdomain ([195.24.90.54]) by smtp.gmail.com with ESMTPSA id nh1sm1912594ejb.21.2020.09.28.09.44.52 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 28 Sep 2020 09:44:53 -0700 (PDT) From: Stanimir Varbanov To: linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Vikash Garodia , Alexandre Courbot , Mansur Alisha Shaik , Stanimir Varbanov Subject: [PATCH 0/3] Venus dynamic resolution change fixes Date: Mon, 28 Sep 2020 19:44:28 +0300 Message-Id: <20200928164431.21884-1-stanimir.varbanov@linaro.org> X-Mailer: git-send-email 2.17.1 Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org Hi all, Those three patches are needed to fix setting of LAST buffer flag during dynamic-resolution-change state. The first patch in this series fix the LAST buffer flag setting, the second unify the driver behavior no matter the event from firmware is sufficient or insufficient resources and the third one is moving the locking from buf_queue helper function to encoder and decoder buf_queue vb2 ops. Comments are welcome! Stanimir Varbanov (3): venus: vdec: Fix non reliable setting of LAST flag venus: vdec: Make decoder return LAST flag for sufficient event venus: helpers: Lock outside of buffer queue helper drivers/media/platform/qcom/venus/core.h | 5 +- drivers/media/platform/qcom/venus/helpers.c | 15 ++-- drivers/media/platform/qcom/venus/vdec.c | 92 +++++++++++++-------- drivers/media/platform/qcom/venus/venc.c | 11 ++- 4 files changed, 76 insertions(+), 47 deletions(-) Reviewed-by: Vikash Garodia