Re: [PATCH 27/29] drm/amd/dal: Correctly interpret rotation as bit set

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Feb 11, 2016 at 7:20 PM, Harry Wentland <harry.wentland@xxxxxxx> wrote:
> Signed-off-by: Harry Wentland <harry.wentland@xxxxxxx>
> Reviewed-by: Mykola Lysenko <mykola.lysenko@xxxxxxx>
> ---
>  drivers/gpu/drm/amd/dal/amdgpu_dm/amdgpu_dm_types.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/dal/amdgpu_dm/amdgpu_dm_types.c b/drivers/gpu/drm/amd/dal/amdgpu_dm/amdgpu_dm_types.c
> index 1cc9fd1054ab..da6c0116aa1a 100644
> --- a/drivers/gpu/drm/amd/dal/amdgpu_dm/amdgpu_dm_types.c
> +++ b/drivers/gpu/drm/amd/dal/amdgpu_dm/amdgpu_dm_types.c
> @@ -366,16 +366,16 @@ static bool fill_rects_from_plane_state(
>         surface->clip_rect = surface->dst_rect;
>
>         switch (state->rotation) {
> -       case DRM_ROTATE_0:
> +       case BIT(DRM_ROTATE_0):
>                 surface->rotation = ROTATION_ANGLE_0;
>                 break;
> -       case DRM_ROTATE_90:
> +       case BIT(DRM_ROTATE_90):
>                 surface->rotation = ROTATION_ANGLE_90;
>                 break;
> -       case DRM_ROTATE_180:
> +       case BIT(DRM_ROTATE_180):
>                 surface->rotation = ROTATION_ANGLE_180;
>                 break;
> -       case DRM_ROTATE_270:
> +       case BIT(DRM_ROTATE_270):
>                 surface->rotation = ROTATION_ANGLE_270;
>                 break;
>         default:
> --
> 2.1.4
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

One minor suggestion for the V2 of this patch-set, please squash
patches 27-29 into the previous patches (relevant to the code they
change). No point in reviewing code of earlier patches that is later
going to be changed by a patch in the same patch-set.

Thanks,

     Oded
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux