Gerd Hoffmann
2018-Dec-12 13:21 UTC
[PATCH v2 10/18] drm/qxl: move qxl_primary_apply_cursor to correct place
The cursor must be set again after creating the primary surface. Also drop the error message. Signed-off-by: Gerd Hoffmann <kraxel at redhat.com> --- drivers/gpu/drm/qxl/qxl_display.c | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/qxl/qxl_display.c b/drivers/gpu/drm/qxl/qxl_display.c index 1dde3b2ecb..a4bdcca715 100644 --- a/drivers/gpu/drm/qxl/qxl_display.c +++ b/drivers/gpu/drm/qxl/qxl_display.c @@ -533,7 +533,6 @@ static void qxl_primary_atomic_update(struct drm_plane *plane, .x2 = plane->state->fb->width, .y2 = plane->state->fb->height }; - int ret; bool same_shadow = false; if (old_state->fb) { @@ -554,16 +553,13 @@ static void qxl_primary_atomic_update(struct drm_plane *plane, if (!same_shadow) qxl_io_destroy_primary(qdev); bo_old->is_primary = false; - - ret = qxl_primary_apply_cursor(plane); - if (ret) - DRM_ERROR( - "could not set cursor after creating primary"); } if (!bo->is_primary) { - if (!same_shadow) + if (!same_shadow) { qxl_io_create_primary(qdev, 0, bo); + qxl_primary_apply_cursor(plane); + } bo->is_primary = true; } -- 2.9.3
Reasonably Related Threads
- [PATCH 1/6] drm/qxl: move qxl_primary_apply_cursor to correct place
- [PATCH 4/6] drm/qxl: use shadow bo directly
- [PATCH v2 13/18] drm/qxl: use shadow bo directly
- [PATCH v2 11/18] drm/qxl: drop unused offset parameter from qxl_io_create_primary()
- [PATCH 2/6] drm/qxl: drop unused offset parameter from qxl_io_create_primary()