On Mon, 16 Apr 2012 21:02:41 +0000 "Reese, Armin C" <armin.c.reese at intel.com> wrote: > Corrected my name in the patch (acreese -> Armin Reese) ... > > The attached patch file was updated to reflect reviewer's comments. The only change I did not make was using PAGE_MASK instead of DISP_BASEADDR_MASK. PAGE_MASK is CPU architecture dependent and I didn't want to tie our GPU addressing arch to that of the CPU. Looks good, thanks Armin. Reviewed-by: Jesse Barnes <jbarnes at virtuousgeek.org> -- Jesse Barnes, Intel Open Source Technology Center