https://bugs.freedesktop.org/show_bug.cgi?id=67530
--- Comment #5 from Christian K?nig <deathsimple at vodafone.de> ---
(In reply to comment #4)
> Created attachment 83741 [details] [review]
> use more common defaults for the level
>
> This patch adds more common defaults as level. Imho they are likely not
> correct for all the hardware.
Not bad for a first try, but I have some minor comments:
> return true;
> case PIPE_VIDEO_CAP_SUPPORTS_PROGRESSIVE:
> return true;
>+ case PIPE_VIDEO_CAP_MAX_LEVEL:
>+ switch (profile) {
>+ case PIPE_VIDEO_PROFILE_MPEG1:
The new code has a different indention, the radeon code uses tabs instead of
spaces, please fix.
Additional to that you need to fix all drivers to handle the new CAP, just
"grep -R PIPE_VIDEO_CAP_SUPPORTED src/gallium/drivers" to find all those
places.
With that fixed please send the patch to the mesa maillinglist for review.
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130807/86b09142/attachment.html>
