On 7/24/20 7:52 PM, Stanimir Varbanov wrote:

Hi Lina,

On 7/24/20 7:28 PM, Lina Iyer wrote:

quoted

On Fri, Jul 24 2020 at 03:03 -0600, Rajendra Nayak wrote:

quoted

Hi Maulik/Lina,

On 7/23/2020 11:36 PM, Stanimir Varbanov wrote:

quoted

Hi Rajendra,

After applying 2,3 and 4/5 patches on linaro-integration v5.8-rc2 I see
below messages on db845:

qcom-venus aa00000.video-codec: dev_pm_opp_set_rate: failed to find
current OPP for freq 533000097 (-34)

^^^ This one is new.

qcom_rpmh TCS Busy, retrying RPMH message send: addr=0x30000

^^^ and this message is annoying, can we make it pr_debug in rpmh?

How annoyingly often do you see this message?

I haven't gig deeply but on every driver pm_runtime_suspend (after
applying Rajendra's patches). And I guess it comes after a call to
dev_pm_opp_set_rate(dev, 0).

Or it might be when the driver is switching off opp_pmdomain.

IMO this is too often.

quoted

Usually, this is an indication of bad system state either on remote
processors in the SoC or in Linux itself. On a smooth sailing build you
should not see this 'warning'.

quoted

Would you be fine with moving this message to a pr_debug? Its currently
a pr_info_ratelimited()

I would rather not, moving this out of sight will mask a lot serious
issues that otherwise bring attention to the developers.

--Lina

-- 
regards,
Stan