For sure. Thanks, Oded.
I'll send out squashed patches later today.
Harry
On 2016-02-11 04:00 PM, Oded Gabbay wrote:
On Thu, Feb 11, 2016 at 7:20 PM, Harry Wentland harry.wentland@amd.com wrote:
Signed-off-by: Harry Wentland harry.wentland@amd.com Reviewed-by: Mykola Lysenko mykola.lysenko@amd.com
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@lists.freedesktop.org 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