[FFmpeg-devel] [PATCH] HWAccel infrastructure (take 7.1)

[Gwenole Beauchesne]

Le 24 f?vr. 09 ? 18:08, Reimar D?ffinger a ?crit :

> Not using per-codec PIX_FMTs seems more likely to me to just  
> complicate
> things, though I do not know for sure.

It turned out the changes to my local mplayer tree were rather small.  
See attachement + adding the extra codec_ids in the table.

However, I terribly dislike the pixfmt2imgfmt() name as it is now  
implied by the removal of codec information in pixfmt.

> In contrast, IMO per-level PIX_FMTs would have been a serious  
> detriment
> to future development and generally better hardware support (and I  
> still
> consider the usage of this stuff a serious misdesign in the  
> accelration
> APIs).

Actually, some manufacturers use level information for error checking.  
Intel VA API implementation actually doesn't care of level, neither  
does VDPAU IIRC. However, they do care somehow on correct profile  
selection, but only if you want to be more energy efficient, I'd guess.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: mplayer.hwaccel.minus.codec.pixfmt.patch
Type: application/octet-stream
Size: 2835 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090224/fe4f072a/attachment.obj>
-------------- next part --------------