Re: [PATCH] ASoC: tas2783-sdw: power the Function up before preparing the port

[Robin Everaars]

Andrey,

thanks. One clarification about my negative v1 result: it was not an
active-playback resume test. The controlled pre-suspend tone had completed
before system suspend. The PipeWire sink was then idle for much longer than
WirePlumber's default five-second node-suspend timeout. After wake I started a
new pw-play process.

I did not trace the ALSA ioctls in that run, so I cannot prove exactly how
PipeWire reactivated the PCM. The test was not designed to call
snd_pcm_resume() on a running stream and should not be used as evidence for
that path.

That makes the missing b627da430357 on v7.1.7 the more relevant difference for
my result, although I have not proved that attribution yet. I agree that the
next useful test is the exact v2 series on broonie/sound for-next rather than
another cycle on v7.1.7.

I will keep the cases separate:

  - an active hw stream recovered only with snd_pcm_resume();
  - an idle PipeWire sink followed by fresh playback after resume;

  - a hw stream left PREPARED across suspend, then started without another
    prepare, to confirm the remaining gap described in the cover letter.

I will mask the resume rebind during each attributable cycle and keep the same
pre/post acoustic measurement.

Please send the register-readout module off-list. I can capture DP1
PrepareCtrl, PrepareStatus, ChannelEn and PDE23 around each path, especially if
one still fails on for-next.

Thanks,
Robin

Attachment: publickey - robineveraars@pm.me - 0x8B6BA132.asc
Description: application/pgp-keys

Attachment: signature.asc
Description: OpenPGP digital signature