RFC: V4L2 MPEG Encoding API: Part 2

[Michael Schimek <mschimek (at) gmx.at>]

On Tue, 2006-05-02 at 17:26 -0300, Mauro Carvalho Chehab wrote:
> > V4L2_CID_COMP_AUDIO_SAMPLING_FREQ_ENUM
> > Set audio recording frequency.
> > MENU: possible values: 0 = 44.1, 1 = 48, 2 = 32 kHz
> 
> > V4L2_CID_COMP_AUDIO_SAMPLING_FREQ
> > Set audio recording frequency.
> > INT: sampling frequency in Hz.
> Hmm... having an enum and a Hz control for the same stuff? wouldn't be
> better to have only one?
> 
> > V4L2_CID_COMP_AUDIO_L1_BITRATE:
> > Set audio bitrate for layer 1.
> > Set to INACTIVE if layer != 1.
...
> > V4L2_CID_COMP_AUDIO_BITRATE:
> > INT: Set audio bitrate to the nearest valid value for the current layer.
> 
> Same comment.

How about a new enum type?

V4L2_CID_COMP_AUDIO_SAMPLING_FREQ
ENUM: for example 32000, 44100, 48000

V4L2_CID_COMP_AUDIO_BITRATE:
ENUM: for example 32000, 64000, 128000, ...

Not MPEG specific, no "inactive" menu items, setting COMP_AUDIO_ENCODING
could just change the the enumeration of COMP_AUDIO_BITRATE, and it could
be up to drivers to implement such controls as INT if they support arbitrary
values. Granted a new control type wouldn't be upwards compatible.

Michael

--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request (at) redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list