From: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
To: Konrad Dybcio <konrad.dybcio@linaro.org>,
Jagadeesh Kona <quic_jkona@quicinc.com>,
Bjorn Andersson <andersson@kernel.org>,
Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@kernel.org>,
Stanimir Varbanov <stanimir.k.varbanov@gmail.com>,
Vikash Garodia <quic_vgarodia@quicinc.com>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Ulf Hansson <ulf.hansson@linaro.org>,
"Rafael J . Wysocki" <rafael@kernel.org>,
Kevin Hilman <khilman@kernel.org>, Pavel Machek <pavel@ucw.cz>,
Len Brown <len.brown@intel.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Andy Gross <agross@kernel.org>,
Dmitry Baryshkov <dmitry.baryshkov@linaro.org>,
Abel Vesa <abel.vesa@linaro.org>
Cc: linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-media@vger.kernel.org,
linux-pm@vger.kernel.org, Taniya Das <quic_tdas@quicinc.com>,
Satya Priya Kakitapalli <quic_skakitap@quicinc.com>,
Imran Shaik <quic_imrashai@quicinc.com>,
Ajit Pandey <quic_ajipan@quicinc.com>
Subject: Re: [PATCH V5 RESEND 5/5] venus: pm_helpers: Use dev_pm_genpd_set_hwmode to switch GDSC mode on V6
Date: Wed, 1 May 2024 10:14:56 +0100 [thread overview]
Message-ID: <ef194e5c-f136-4dba-bfe0-2c6439892e34@linaro.org> (raw)
In-Reply-To: <d36c1163-a3f0-4034-a430-91986e5bbce8@linaro.org>
On 30/04/2024 21:01, Konrad Dybcio wrote:
> On 24.04.2024 11:50 AM, Bryan O'Donoghue wrote:
>> On 24/04/2024 10:45, Jagadeesh Kona wrote:
>>>
>>> Thanks Bryan for testing this series. Can you please confirm if this issue is observed in every run or only seen during the first run? Also please let me know on which platform this issue is observed?
>>>
>>> Thanks,
>>> Jagadeesh
>>
>> rb5/sm8250
>>
>> My observation was on a previous _boot_ the stuttering was worse. There is in the video capture three times that I count where the video halts briefly, I guess we need to vote or set an OPP so the firmware knows not to power-collapse quite so aggressively.
>
> We seem to be having some qualcomm-wide variance on perf/pwr usage on some
> odd boots.. Any chance you could try like 5 times and see if it was a fluke?
>
> Konrad
Sure.
The first time I tried it, it was much worse.
The second time, captured in the video is only noticeable because I was
*looking* for this specific error i.e. I don't think I would have
noticed the error on the second run, had I not seen the first run.
I'll find some time to do 5x with and 5x without.
---
bod
next prev parent reply other threads:[~2024-05-01 9:15 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-13 15:20 [PATCH V5 RESEND 0/5] PM: domains: Add control for switching back and forth to HW control Jagadeesh Kona
2024-04-13 15:20 ` [PATCH V5 RESEND 1/5] PM: domains: Allow devices attached to genpd to be managed by HW Jagadeesh Kona
2024-04-13 15:20 ` [PATCH V5 RESEND 2/5] PM: domains: Add the domain HW-managed mode to the summary Jagadeesh Kona
2024-04-13 15:20 ` [PATCH V5 RESEND 3/5] clk: qcom: gdsc: Add set and get hwmode callbacks to switch GDSC mode Jagadeesh Kona
2024-04-23 23:48 ` Bryan O'Donoghue
2024-04-24 9:47 ` Jagadeesh Kona
2024-04-24 9:55 ` Bryan O'Donoghue
2024-04-24 10:27 ` Jagadeesh Kona
2024-04-24 12:22 ` Konrad Dybcio
2024-04-24 12:25 ` Konrad Dybcio
2024-04-13 15:20 ` [PATCH V5 RESEND 4/5] clk: qcom: Use HW_CTRL_TRIGGER flag to switch video GDSC to HW mode Jagadeesh Kona
2024-04-22 22:54 ` Konrad Dybcio
2024-04-24 9:48 ` Jagadeesh Kona
2024-04-13 15:20 ` [PATCH V5 RESEND 5/5] venus: pm_helpers: Use dev_pm_genpd_set_hwmode to switch GDSC mode on V6 Jagadeesh Kona
2024-04-14 12:39 ` Bryan O'Donoghue
2024-04-14 14:52 ` Jagadeesh Kona
2024-04-24 0:16 ` Bryan O'Donoghue
2024-04-24 9:45 ` Jagadeesh Kona
2024-04-24 9:50 ` Bryan O'Donoghue
2024-04-30 20:01 ` Konrad Dybcio
2024-05-01 9:14 ` Bryan O'Donoghue [this message]
2024-05-10 13:01 ` Bryan O'Donoghue
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ef194e5c-f136-4dba-bfe0-2c6439892e34@linaro.org \
--to=bryan.odonoghue@linaro.org \
--cc=abel.vesa@linaro.org \
--cc=agross@kernel.org \
--cc=andersson@kernel.org \
--cc=dmitry.baryshkov@linaro.org \
--cc=gregkh@linuxfoundation.org \
--cc=khilman@kernel.org \
--cc=konrad.dybcio@linaro.org \
--cc=len.brown@intel.com \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=mturquette@baylibre.com \
--cc=pavel@ucw.cz \
--cc=quic_ajipan@quicinc.com \
--cc=quic_imrashai@quicinc.com \
--cc=quic_jkona@quicinc.com \
--cc=quic_skakitap@quicinc.com \
--cc=quic_tdas@quicinc.com \
--cc=quic_vgarodia@quicinc.com \
--cc=rafael@kernel.org \
--cc=sboyd@kernel.org \
--cc=stanimir.k.varbanov@gmail.com \
--cc=ulf.hansson@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).