On 18 Feb 2014, at 05:37, Justin Ruggles <[email protected]> wrote:
> On 02/17/2014 09:39 PM, Tim Walker wrote:
>> @@ -55,10 +55,11 @@ static const AVOption ac3_options[] = {
>> {"ltrt_surmixlev", "Lt/Rt Surround Mix Level",
>> OFFSET(ltrt_surround_mix_level), AV_OPT_TYPE_FLOAT, {.dbl = -1.0 }, -1.0,
>> 2.0, AC3ENC_PARAM},
>> {"loro_cmixlev", "Lo/Ro Center Mix Level", OFFSET(loro_center_mix_level),
>> AV_OPT_TYPE_FLOAT, {.dbl = -1.0 }, -1.0, 2.0, AC3ENC_PARAM},
>> {"loro_surmixlev", "Lo/Ro Surround Mix Level",
>> OFFSET(loro_surround_mix_level), AV_OPT_TYPE_FLOAT, {.dbl = -1.0 }, -1.0,
>> 2.0, AC3ENC_PARAM},
>> -{"dsurex_mode", "Dolby Surround EX Mode", OFFSET(dolby_surround_ex_mode),
>> AV_OPT_TYPE_INT, {.i64 = AC3ENC_OPT_NONE }, AC3ENC_OPT_NONE,
>> AC3ENC_OPT_MODE_ON, AC3ENC_PARAM, "dsurex_mode"},
>> +{"dsurex_mode", "Dolby Surround EX Mode", OFFSET(dolby_surround_ex_mode),
>> AV_OPT_TYPE_INT, {.i64 = AC3ENC_OPT_NONE }, AC3ENC_OPT_NONE,
>> AC3ENC_OPT_MODE_RESERVED, AC3ENC_PARAM, "dsurex_mode"},
>> {"notindicated", "Not Indicated (default)", 0, AV_OPT_TYPE_CONST,
>> {.i64 = AC3ENC_OPT_NOT_INDICATED }, INT_MIN, INT_MAX, AC3ENC_PARAM,
>> "dsurex_mode"},
>> {"on", "Dolby Surround EX Encoded", 0, AV_OPT_TYPE_CONST,
>> {.i64 = AC3ENC_OPT_MODE_ON }, INT_MIN, INT_MAX, AC3ENC_PARAM,
>> "dsurex_mode"},
>> {"off", "Not Dolby Surround EX Encoded", 0, AV_OPT_TYPE_CONST,
>> {.i64 = AC3ENC_OPT_MODE_OFF }, INT_MIN, INT_MAX, AC3ENC_PARAM,
>> "dsurex_mode"},
>> + {"dpliiz", "Dolby Pro Logic IIz-encoded", 0, AV_OPT_TYPE_CONST,
>> {.i64 = AC3ENC_OPT_MODE_RESERVED }, INT_MIN, INT_MAX, AC3ENC_PARAM,
>> "dsurex_mode"},
>> {"dheadphone_mode", "Dolby Headphone Mode", OFFSET(dolby_headphone_mode),
>> AV_OPT_TYPE_INT, {.i64 = AC3ENC_OPT_NONE }, AC3ENC_OPT_NONE,
>> AC3ENC_OPT_MODE_ON, AC3ENC_PARAM, "dheadphone_mode"},
>> {"notindicated", "Not Indicated (default)", 0, AV_OPT_TYPE_CONST,
>> {.i64 = AC3ENC_OPT_NOT_INDICATED }, INT_MIN, INT_MAX, AC3ENC_PARAM,
>> "dheadphone_mode"},
>> {"on", "Dolby Headphone Encoded", 0, AV_OPT_TYPE_CONST,
>> {.i64 = AC3ENC_OPT_MODE_ON }, INT_MIN, INT_MAX, AC3ENC_PARAM,
>> "dheadphone_mode"},
>
> Um, I don't quite get this patch. If A/52 says "Reserved" that means
> don't use it. How can that be taken to mean DPL2z?
>
> -Justin
No, the latest spec. from ATSC (2012-12-17) says it means DPL2z.
I suppose I should define AC3ENC_OPT_DSUREX_DPLIIZ instead of
AC3ENC_OPT_MODE_RESERVED.
Sounds good?
Tim
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
