lkml.org 
[lkml]   [2019]   [Dec]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL 4.19 027/177] media: venus: Fix occasionally failures to suspend
    Date
    From: Stanimir Varbanov <stanimir.varbanov@linaro.org>

    [ Upstream commit 8dbebb2bd01e6f36e9a215dcde99ace70408f2c8 ]

    Failure to suspend (venus_suspend_3xx) happens when the system
    is fresh booted and loading venus driver. This happens once and
    after reload the venus driver modules the problem disrepair.

    Fix the failure by skipping the check for WFI and IDLE bits if
    PC_READY is on in control status register.

    Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org>
    Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/media/platform/qcom/venus/hfi_venus.c | 6 ++++++
    1 file changed, 6 insertions(+)

    diff --git a/drivers/media/platform/qcom/venus/hfi_venus.c b/drivers/media/platform/qcom/venus/hfi_venus.c
    index 124085556b94b..fbcc67c10993f 100644
    --- a/drivers/media/platform/qcom/venus/hfi_venus.c
    +++ b/drivers/media/platform/qcom/venus/hfi_venus.c
    @@ -1484,6 +1484,7 @@ static int venus_suspend_3xx(struct venus_core *core)
    {
    struct venus_hfi_device *hdev = to_hfi_priv(core);
    struct device *dev = core->dev;
    + u32 ctrl_status;
    bool val;
    int ret;

    @@ -1499,6 +1500,10 @@ static int venus_suspend_3xx(struct venus_core *core)
    return -EINVAL;
    }

    + ctrl_status = venus_readl(hdev, CPU_CS_SCIACMDARG0);
    + if (ctrl_status & CPU_CS_SCIACMDARG0_PC_READY)
    + goto power_off;
    +
    /*
    * Power collapse sequence for Venus 3xx and 4xx versions:
    * 1. Check for ARM9 and video core to be idle by checking WFI bit
    @@ -1523,6 +1528,7 @@ static int venus_suspend_3xx(struct venus_core *core)
    if (ret)
    return ret;

    +power_off:
    mutex_lock(&hdev->lock);

    ret = venus_power_off(hdev);
    --
    2.20.1
    \
     
     \ /
      Last update: 2019-12-10 22:46    [W:4.022 / U:0.388 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site