On Tue, Feb 15, 2022 at 08:32:05PM +0200, Ville Syrjala wrote: > @@ -2788,8 +2788,9 @@ static void intel_crtc_readout_derived_state(struct intel_crtc_state *crtc_state > /* Populate the "user" mode with full numbers */ > drm_mode_copy(mode, pipe_mode); > intel_mode_from_crtc_timings(mode, mode); > - mode->hdisplay = crtc_state->pipe_src_w << crtc_state->bigjoiner; > - mode->vdisplay = crtc_state->pipe_src_h; > + mode->hdisplay = drm_rect_width(&crtc_state->pipe_src) * > + (hweight8(crtc_state->bigjoiner_pipes) ?: 1); That hweight() stuff was supposed to be in one of the later patches btw. Looks like I accidentally squashed it here when splitting/reordering stuff. > + mode->vdisplay = drm_rect_height(&crtc_state->pipe_src); > > /* Derive per-pipe timings in case bigjoiner is used */ > intel_bigjoiner_adjust_timings(crtc_state, pipe_mode); -- Ville Syrjälä Intel
