]> git.baikalelectronics.ru Git - kernel.git/commit
drm/i915: Don't store panning coordinates as 16.16 fixed point
authorMatt Roper <matthew.d.roper@intel.com>
Tue, 18 Nov 2014 02:10:38 +0000 (18:10 -0800)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 19 Nov 2014 13:56:53 +0000 (14:56 +0100)
commita44650ec058a5eb0dcf5e2fa7379c4fd7348aa3a
treed35a6ca1dbd444115ded842f45e43ce2cab91891
parentd09915ef8c6565395840f9114daa929c2d32eed9
drm/i915: Don't store panning coordinates as 16.16 fixed point

When using the universal plane interface, the source rectangle
coordinates define the panning offset for the primary plane, which needs
to be stored in crtc->{x,y}.  The original universal plane code
negelected to set these panning offset fields, which was partially
remedied in:

        commit b20f8ba1b1f30124aae52240f1adf91598a13298
        Author: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
        Date:   Wed Sep 24 14:20:22 2014 -0300

            drm/i915: Merge of visible and !visible paths for primary planes

However the plane source coordinates are provided in 16.16 fixed point
format and the above commit forgot to convert back to integer
coordinates before saving the values.  When we replace
intel_pipe_set_base() with plane->funcs->update_plane() in a future
patch, this bug becomes visible via the set_config entrypoint as well as
update_plane.

Cc: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
Testcase: igt/kms_plane
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_display.c