Re: [PATCH 0/1] Documentation update

From: Takashi Iwai
Date: Thu May 14 2020 - 05:22:50 EST


On Thu, 14 May 2020 11:20:59 +0200,
Takashi Iwai wrote:
>
> 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?

Also, when you resubmit with some changes, it's helpful to add a
prefix like "v2" to the subject line, e.g.
[PATCH v2] Fix something ....

Additionally it's often helpful to mention concisely what you changed
from the previous submission.


thanks,

Takashi