Atm the DPT object can accommodate only one VMA, so the VMA offset will
be always 0. Add an assert for this.

Cc: Juha-Pekka Heikkila <juhapekka.heikk...@gmail.com>
Signed-off-by: Imre Deak <imre.d...@intel.com>
Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikk...@gmail.com>
---
 drivers/gpu/drm/i915/display/skl_universal_plane.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/drivers/gpu/drm/i915/display/skl_universal_plane.c 
b/drivers/gpu/drm/i915/display/skl_universal_plane.c
index 724e7b04f3b63..f50282b60de44 100644
--- a/drivers/gpu/drm/i915/display/skl_universal_plane.c
+++ b/drivers/gpu/drm/i915/display/skl_universal_plane.c
@@ -993,6 +993,11 @@ static u32 skl_surf_address(const struct intel_plane_state 
*plane_state,
        u32 offset = plane_state->view.color_plane[color_plane].offset;
 
        if (intel_fb_uses_dpt(fb)) {
+               /*
+                * The DPT object contains only one vma, so the VMA's offset
+                * within the DPT is always 0.
+                */
+               WARN_ON(plane_state->dpt_vma->node.start);
                WARN_ON(offset & 0x1fffff);
                return offset >> 9;
        } else {
-- 
2.27.0

Reply via email to