Re: [PATCH v26 28/33] ALSA: usb-audio: qcom: Introduce QC USB SND offloading support

[Wesley Cheng <quic_wcheng@quicinc.com>]

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