On 7/24/20 7:52 PM, Stanimir Varbanov wrote: > 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). Or it might be when the driver is switching off opp_pmdomain. > > 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
