On Thu, 14 May 2020 11:20:59 +0200, Takashi Iwai wrote: > > On Thu, 14 May 2020 11:07:15 +0200, > changjoon....@lge.com wrote: > > > > From: ChangJoon Lee <changjoon....@lge.com> > > > > 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
