[Mesa-dev] [PATCH] gallium/radeon: use explicit drm_major, drm_minor check

[Alex Deucher]

On Wed, Mar 2, 2016 at 11:48 AM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> Just like everywhere else in the radeon codebase.
>
> Cc: Marek Olšák <marek.olsak at amd.com>
> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>

NACK, amdgpu has major version 3.

> ---
>
> Spotted while grepping for drm_minor - i.e. which version will be used
> for the next round.
>
> -Emil
>
>  src/gallium/drivers/radeon/radeon_vce.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/radeon/radeon_vce.c b/src/gallium/drivers/radeon/radeon_vce.c
> index 41603b3..b66ac9f 100644
> --- a/src/gallium/drivers/radeon/radeon_vce.c
> +++ b/src/gallium/drivers/radeon/radeon_vce.c
> @@ -404,7 +404,7 @@ struct pipe_video_codec *rvce_create_encoder(struct pipe_context *context,
>
>         if (rscreen->info.drm_major == 3)
>                 enc->use_vm = true;
> -       if ((rscreen->info.drm_major > 2) || (rscreen->info.drm_minor >= 42))
> +       if (rscreen->info.drm_major == 2 && rscreen->info.drm_minor >= 42)
>                 enc->use_vui = true;
>         if (rscreen->info.family >= CHIP_TONGA &&
>               rscreen->info.family != CHIP_STONEY)
> --
> 2.6.2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev