<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Aug 29, 2014 at 5:38 PM, Ville Syrjälä <span dir="ltr"><<a href="mailto:ville.syrjala@linux.intel.com" target="_blank">ville.syrjala@linux.intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">> Would grabbing<br>
> struct drm_plane_state from the atomic branch fix this for you?<br>
<br>
</div>That looks like it's meant to house the user requested coordinates<br>
rather than the clipped ones. What you could do is just shovel the<br>
drm_rects we use during the clipping into a new i915 specific plane<br>
config struct and pass that to both check and commit. We can later<br>
look into moving that stuff into some core struct if seems like a win<br>
for more than one driver.</blockquote></div><br></div><div class="gmail_extra">Creating a new intel_plane_config will also be better for merging, since if you depend upon drm_plane_state directly your patch will be blocked. Ofc that means we need to do a bit of refactoring once atomic has landed, but that shouldn't be too onerous really.<br>
</div><div class="gmail_extra">-Daniel<br>-- <br>Daniel Vetter<br>Software Engineer, Intel Corporation<br>+41 (0) 79 365 57 48 - <a href="http://blog.ffwll.ch" target="_blank">http://blog.ffwll.ch</a>
</div></div>