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@osg.samsung.com
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>