diff mbox series

venus: fix hw overload error log condition

Message ID 1617888900-857-1-git-send-email-mansur@codeaurora.org
State Accepted
Commit bcf6b264a806295cd4578aa7a03b3bcb15b1ab30
Headers show
Series venus: fix hw overload error log condition | expand

Commit Message

Mansur Alisha Shaik April 8, 2021, 1:35 p.m. UTC
In current video driver, frequency is calculated for all the
running video instances and check aganist maximum supported frequency.
If both calculated frequency and maximum supported frequency are same,
even then HW overload error is printed.
Fix this by printing error log only when frequency is greater than
maximum supported frequency.

Signed-off-by: Mansur Alisha Shaik <mansur@codeaurora.org>
---
 drivers/media/platform/qcom/venus/pm_helpers.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

patchwork-bot+linux-arm-msm@kernel.org May 26, 2021, 7:03 p.m. UTC | #1
Hello:

This patch was applied to qcom/linux.git (refs/heads/for-next):

On Thu,  8 Apr 2021 19:05:00 +0530 you wrote:
> In current video driver, frequency is calculated for all the

> running video instances and check aganist maximum supported frequency.

> If both calculated frequency and maximum supported frequency are same,

> even then HW overload error is printed.

> Fix this by printing error log only when frequency is greater than

> maximum supported frequency.

> 

> [...]


Here is the summary with links:
  - venus: fix hw overload error log condition
    https://git.kernel.org/qcom/c/bcf6b264a806

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
diff mbox series

Patch

diff --git a/drivers/media/platform/qcom/venus/pm_helpers.c b/drivers/media/platform/qcom/venus/pm_helpers.c
index dfe3ee8..9714ca7 100644
--- a/drivers/media/platform/qcom/venus/pm_helpers.c
+++ b/drivers/media/platform/qcom/venus/pm_helpers.c
@@ -1083,7 +1083,7 @@  static int load_scale_v4(struct venus_inst *inst)
 
 	freq = max(freq_core1, freq_core2);
 
-	if (freq >= table[0].freq) {
+	if (freq > table[0].freq) {
 		freq = table[0].freq;
 		dev_warn(dev, "HW is overloaded, needed: %lu max: %lu\n",
 			 freq, table[0].freq);