RE: [PATCH v2] ASoC: amd: update pm_runtime enable sequence

From: Limonciello, Mario
Date: Fri Jun 23 2023 - 11:13:28 EST


[AMD Official Use Only - General]

> On Fri, Jun 23, 2023 at 08:40:15AM -0500, Limonciello, Mario wrote:
> > On 6/23/2023 4:41 PM, Arun Gopal Kondaveeti wrote:
> > > pm_runtime_allow() is not needed for ACP child platform devices.
> > > Replace pm_runtime_allow() with pm_runtime_mark_last_busy()
> > > & pm_runtime_set_active() in pm_runtime enable sequence for
> > > ACP child platform drivers.
>
> > Can you explain what prompted this?
>
> > Does this fix a particular bug, or is it just to correct
> > things? If it fixes a particular bug; I think it should be
> > split up across 5 patches (one for each APU) and then
> > appropriate Fixes tags applied for each on the code they're
> > fixing.
>
> I had already gone ahead and applied this before your mail - it seemed
> better to get it into 6.5 than leave it waiting longer. If there's a
> need for backports that can be handled through the stable process.

Sounds good, thanks!