[FFmpeg-devel] [PATCH][VAAPI][1/6] Add VA API formats

[Gwenole Beauchesne]

On Thu, 26 Feb 2009, Michael Niedermayer wrote:

>> @@ -76,6 +76,7 @@ const uint8_t ff_mpeg1_dc_scale_table[128]={
>>  };
>>
>>  const enum PixelFormat ff_pixfmt_list_420[] = {
>> +    PIX_FMT_VAAPI_VLD,
>>      PIX_FMT_YUV420P,
>>      PIX_FMT_NONE
>>  };
>
> considering that this list is used by some codecs that VAAPI does not support
> this isnt correct

Sure some codecs are not supported, but I thought this was your intent 
because ::get_format() has to check for it anyway. i.e. the API can 
specify support for a codec but the user has to check the HW actually 
supports it. So, even if we add the HW formats on per-codec, there are 
cases were it wouldn't be supported for real. e.g. VDPAU VC-1 in the past.