On Thu, Dec 08, 2016 at 08:35:48AM +0000, Chris Wilson wrote: > On Wed, Dec 07, 2016 at 07:28:07PM +0200, ville.syrjala@xxxxxxxxxxxxxxx wrote: > > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > > > Replace the use of crtc->mode.h/vdisplay with the more appropriate > > config->pipe_src_w/h. > > I'll ask the dumb question: wouldn't pipe_src be the fb dimensions? It > doesn't seem obvious to be they are the CRTC dimensions. Yes, they make > sense from the pov this is the source rectangle that the CRTC is pulling > pixels from. Hmm, hopefully I've answered my own question. Indeed you have. The PIPESRC register name is a little confusing. "Canvas size" or something along those lines might be less confusing. > > Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > -Chris > > -- > Chris Wilson, Intel Open Source Technology Centre -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
