Am 22.10.2015 um 18:27 schrieb Alex Deucher:
> The section for UVD 2 and older was not updated
> when HEVC support was added. Reported by Kano
> on irc.
>
> Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
> Cc: mesa-sta...@lists.freedesktop.org
> ---
> src/gallium/drivers/radeon/radeon_video.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/radeon/radeon_video.c
> b/src/gallium/drivers/radeon/radeon_video.c
> index 3a1834b..722f446 100644
> --- a/src/gallium/drivers/radeon/radeon_video.c
> +++ b/src/gallium/drivers/radeon/radeon_video.c
> @@ -237,8 +237,9 @@ int rvid_get_video_param(struct pipe_screen *screen,
> enum pipe_video_format codec = u_reduce_video_profile(profile);
> switch (param) {
> case PIPE_VIDEO_CAP_SUPPORTED:
> - /* no support for MPEG4 */
> + /* no support for MPEG4, HEVC */
> return codec != PIPE_VIDEO_FORMAT_MPEG4 &&
> + codec != PIPE_VIDEO_FORMAT_HEVC &&
> /* FIXME: VC-1 simple/main profile is broken */
> profile != PIPE_VIDEO_PROFILE_VC1_SIMPLE &&
> profile != PIPE_VIDEO_PROFILE_VC1_MAIN;
>
Wouldn't it be safer to use a positive list instead? I could imagine
even more getting added sometime in the future...
Roland
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
