Hi Lina, On 7/24/20 7:28 PM, Lina Iyer wrote: > On Fri, Jul 24 2020 at 03:03 -0600, Rajendra Nayak wrote: >> Hi Maulik/Lina, >> >> On 7/23/2020 11:36 PM, Stanimir Varbanov wrote: >>> 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). IMO this is too often. > 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'. > >> 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
