On Tue, Aug 23, 2022 at 02:29:16PM +0300, Jouni Högander wrote: > Currently damage clips handling is broken for planes when using big > framebuffer + offset in case kms driver adjusts drm_plane_state.src > coords. This is because damage clips are using coords relative to > original coords from user-space. > > This patchset is fixing this by using original > coords from user-space instead of drm_plane_state.src when iterating > damage_clips. > > v2: Modify drm unit tests accordingly > > Cc: Daniel Vetter <daniel.vetter@xxxxxxxx> > Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> > Cc: Jani Nikula <jani.nikula@xxxxxxxxx> > Cc: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > Cc: José Roberto de Souza <jose.souza@xxxxxxxxx> > Cc: Mika Kahola <mika.kahola@xxxxxxxxx> > Cc: Maíra Canal <mairacanal@xxxxxxxxxx> > > Jouni Högander (4): > drm: Use original src rect while initializing damage iterator > drm/i915/display: Use original src in psr2 sel fetch area calculation > drm/i915/display: Use drm helper instead of own loop for damage clips > drm/tests: Set also mock plane src_x, src_y, src_w and src_h Do these need to be applied into the same tree, or can the drm vs. i915 stuff go in separately? > > drivers/gpu/drm/drm_damage_helper.c | 11 ++++++---- > drivers/gpu/drm/i915/display/intel_psr.c | 20 +++++++------------ > .../gpu/drm/tests/drm_damage_helper_test.c | 5 +++++ > 3 files changed, 19 insertions(+), 17 deletions(-) > > -- > 2.34.1 -- Ville Syrjälä Intel