On 11/03/2014 04:02 PM, Daniel Vetter wrote:
On Thu, Oct 30, 2014 at 04:39:37PM +0000, Tvrtko Ursulin wrote:
Just a sideline comment here for now ;-)
+/* intel_dma.c */
+int i915_st_set_pages(struct sg_table **st, struct page **pvec, int num_pages);
Comments all over should tell you that i915_dma.c is super legacy
territory and really you should never touch it. I think i915_gem.c is a
better place.
Didn't spot any comments to this effect, in fact it looks like a
mish-mash of stuff which is exactly why it looked appropriate for this.
:) That said I don't mind moving it.
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
