Re: [PATCH 0/1] Documentation update

From: Takashi Iwai
Date: Thu May 14 2020 - 05:21:02 EST


On Thu, 14 May 2020 11:07:15 +0200,
changjoon.lee@xxxxxxx wrote:
>
> From: ChangJoon Lee <changjoon.lee@xxxxxxx>
>
> Hi, ALSA Teams,
>
> This is documentation update purpose patch.
> This patch does NOT alter any of operations.
>
> This patch allows an application engineer to understand why they received negative return
> such as -EIO, -EBADFD, -ENOSYS, -ESTRPIPE and etc which are not fully documented on documentation site.
>
> This patch also exposes Expected Status of PCM based on bad_pcm_state() with P_STATE to help engineer to understand API behavior to the documentation site.
>
> TODO :
> We may need improve consistency of return value especially for snd_pcm_hw_params_current().
> It returns -EBADFD when PCM is not setup while other API returns -EIO in this case.

Could you put those explanations rather into the patch description,
not (only) in the cover letter?


thanks,

Takashi