Santiago,
>> +
>> +/* Struct list for digital video presets */
>> +static const struct tvp7002_preset_definition tvp7002_presets[] = {
>> + {
>> + V4L2_DV_720P60,
>> + tvp7002_parms_720P60,
>> + V4L2_COLORSPACE_REC709,
>> + V4L2_FIELD_SEQ_TB,
Why don't we set this to V4L2_FIELD_NONE? That is what we used in
internal releases and is the requirement. Same for all of Progressive
frame formats below.
>> + 1,
>> + 0x2EE,
>> + 135,
>> + 153
>> + },
>> + {
>> + V4L2_DV_1080I60,
>> + tvp7002_parms_1080I60,
>> + V4L2_COLORSPACE_REC709,
>> + V4L2_FIELD_INTERLACED,
>> + 0,
>> + 0x465,
>> + 181,
>> + 205
>> + },
>> + {
>> + V4L2_DV_1080I50,
>> + tvp7002_parms_1080I50,
>> + V4L2_COLORSPACE_REC709,
>> + V4L2_FIELD_INTERLACED,
>> + 0,
>> + 0x465,
>> + 217,
>> + 245
>> + },
>> + {
>> + V4L2_DV_720P50,
>> + tvp7002_parms_720P50,
>> + V4L2_COLORSPACE_REC709,
>> + V4L2_FIELD_SEQ_TB,
>> + 1,
>> + 0x2EE,
>> + 163,
>> + 183
>> + },
>> + {
>> + V4L2_DV_1080P60,
>> + tvp7002_parms_1080P60,
>> + V4L2_COLORSPACE_REC709,
>> + V4L2_FIELD_SEQ_TB,
>> + 1,
>> + 0x465,
>> + 90,
>> + 102
>> + },
>> + {
>> + V4L2_DV_480P59_94,
>> + tvp7002_parms_480P,
>> + V4L2_COLORSPACE_SMPTE170M,
>> + V4L2_FIELD_SEQ_TB,
>> + 1,
>> + 0x20D,
>> + 0xffff,
>> + 0xffff
>> + },
>> + {
>> + V4L2_DV_576P50,
>> + tvp7002_parms_576P,
>> + V4L2_COLORSPACE_SMPTE170M,
>> + V4L2_FIELD_SEQ_TB,
>> + 1,
>> + 0x271,
>> + 0xffff,
>> + 0xffff
>> + }
>> +};
>> +
Murali
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
