On Sun, Oct 02, 2011 at 09:20:32PM -0700, Keith Packard wrote: > On Sun, 2 Oct 2011 18:27:12 -0700, Ben Widawsky <[email protected]> wrote: > > > +static void blt_ring_begin2(struct intel_ring_buffer *ring) > > +{ > > + if (!ring->private) > > + return; > > + > > + intel_ring_emit(ring, MI_LOAD_REGISTER_IMM(1)); > > + intel_ring_emit(ring, 0x2209c); > > + intel_ring_emit(ring, 0x20002); > > + intel_ring_emit(ring, MI_FLUSH_DW); > > + intel_ring_emit(ring, 0x2209c); > > + intel_ring_emit(ring, MI_LOAD_REGISTER_IMM(1)); > > + intel_ring_emit(ring, 0x2209c); > > + intel_ring_emit(ring, 0x20000); > > + intel_ring_emit(ring, MI_LOAD_REGISTER_IMM(1)); > > + intel_ring_emit(ring, 0x2209c); > > + intel_ring_emit(ring, 0x20000); > > + intel_ring_emit(ring, MI_NOOP); > > +} > > + > > Surely you can provide symbolic names here. >
Of course. I lazily just copied the docs hoping someone would come up with some reason not to bother going further. Assuming nobody has any complains, I'll fix it up. Ben
pgpW2ZLYxoEbE.pgp
Description: PGP signature
_______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
