On Fri, May 16, 2014 at 7:25 AM, Lee, Chon Ming <chon.ming.lee@xxxxxxxxx> wrote: >> > Cherryview display allow the primary plane to be position at any >> > location similiar to sprite plane for certain port. So, this shouldn't need to >> check here. >> > >> > And the width/height doesn't need to cover the whole screen. >> >> In that case, wouldn't it make sense (at least when you want to expose those >> features) to *not* use primary plane helpers for that hw? >> >> IMHO, the primary plane helpers should be for "traditional" crtcs which do >> not have these features.. >> > I don't have the usage model for that windowing the primary plane now. Let say a primary plane at the beginning is using in traditional way, but if it want to switch to window mode, it might not make sense to create another plane to handle this right? I think what Rob means is that for now we should just use the primary plane helpers everywhere for easier conversion. Then later on we can enable the primary plane position feature on chv. So doing things step-by-step. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel