Hi Shuah, Thank you for the patch. On Thursday 11 February 2016 16:41:19 Shuah Khan wrote: > Add Audio Function Entities > > Signed-off-by: Shuah Khan <shuahkh@xxxxxxxxxxxxxxx> > --- > Documentation/DocBook/media/v4l/media-types.xml | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/Documentation/DocBook/media/v4l/media-types.xml > b/Documentation/DocBook/media/v4l/media-types.xml index 3730967..924a604 > 100644 > --- a/Documentation/DocBook/media/v4l/media-types.xml > +++ b/Documentation/DocBook/media/v4l/media-types.xml > @@ -113,6 +113,18 @@ > decoder. > </entry> > </row> > + <row> > + <entry><constant>MEDIA_ENT_F_AUDIO_CAPTURE</constant></entry> > + <entry>Audio Capture Function Entity.</entry> > + </row> > + <row> > + <entry><constant>MEDIA_ENT_F_AUDIO_PLAYBACK</constant></entry> > + <entry>Audio Playback Function Entity.</entry> > + </row> I think this deserves a longer description. From the name and short description I'm not sure what the capture and playback functions are. > + <row> > + <entry><constant>MEDIA_ENT_F_AUDIO_MIXER</constant></entry> > + <entry>Audio Mixer Function Entity.</entry> > + </row> Entities can implement multiple functions, so function descriptions shouldn't refer to entity this way. MEDIA_ENT_F_AUDIO_MIXER doesn't mean that the entity is an audio mixer, it means that the entity implements the audio mixer function. If you want to keep the description short you could just write "Audio mixer". A longer description could be "The entity can mix multiple input audio streams to produce an output audio stream.". > </tbody> > </tgroup> > </table> -- Regards, Laurent Pinchart _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
