Hi Pierre,
On 8/30/2024 2:52 AM, Pierre-Louis Bossart wrote:
+/* Stream disable request timeout during USB device disconnect */ +#define DEV_RELEASE_WAIT_TIMEOUT 10000 /* in ms */
10s really? That seems rather large for a stream disable timeout...
Hmm, yes that is overkill, will adjust it accordingly.
+static struct snd_usb_platform_ops offload_ops = {
- .connect_cb = qc_usb_audio_offload_probe,
- .disconnect_cb = qc_usb_audio_offload_disconnect,
- .suspend_cb = qc_usb_audio_offload_suspend,
+};
You probably want to explain why there's no .resume_cb?
The comments mention also that the suspend_cb has to stop playback, but then who resumes playback :-)
I can add a comment. Ideally, the suspend_cb is only used for the case of PM suspend/system suspend. If usb autosuspend is enabled, then the QC offload driver will handle the voting based on the audio stream being active or not. Is there a use case where the ASoC layer re-opens any previously active audio streams so that userspace doesn't have to? Currently, I was under the assumption that the audio stream would have to be re-opened by the application.
Thanks
Wesley Cheng