diff options
author | Tang Bin <tangbin@cmss.chinamobile.com> | 2022-09-13 09:37:00 +0300 |
---|---|---|
committer | Stanimir Varbanov <stanimir.varbanov@linaro.org> | 2022-10-25 10:38:56 +0300 |
commit | 0f6e8d8c94a82e85e1b9b62a7671990740dc6f70 (patch) | |
tree | e45e83ab79dd70698f74569e8acc5856fc280d08 /drivers/media/platform/qcom/venus/pm_helpers.c | |
parent | 1eee6bb9b64bb3a914433bb7ec739d2e67cba5bd (diff) | |
download | linux-0f6e8d8c94a82e85e1b9b62a7671990740dc6f70.tar.xz |
venus: pm_helpers: Fix error check in vcodec_domains_get()
In the function vcodec_domains_get(), dev_pm_domain_attach_by_name()
may return NULL in some cases, so IS_ERR() doesn't meet the
requirements. Thus fix it.
Fixes: 7482a983dea3 ("media: venus: redesign clocks and pm domains control")
Signed-off-by: Tang Bin <tangbin@cmss.chinamobile.com>
Diffstat (limited to 'drivers/media/platform/qcom/venus/pm_helpers.c')
-rw-r--r-- | drivers/media/platform/qcom/venus/pm_helpers.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/media/platform/qcom/venus/pm_helpers.c b/drivers/media/platform/qcom/venus/pm_helpers.c index c93d2906e4c7..48c9084bb4db 100644 --- a/drivers/media/platform/qcom/venus/pm_helpers.c +++ b/drivers/media/platform/qcom/venus/pm_helpers.c @@ -869,8 +869,8 @@ static int vcodec_domains_get(struct venus_core *core) for (i = 0; i < res->vcodec_pmdomains_num; i++) { pd = dev_pm_domain_attach_by_name(dev, res->vcodec_pmdomains[i]); - if (IS_ERR(pd)) - return PTR_ERR(pd); + if (IS_ERR_OR_NULL(pd)) + return PTR_ERR(pd) ? : -ENODATA; core->pmdomains[i] = pd; } |