On ti, 2017-05-09 at 14:08 +0000, Michal Wajdeczko wrote: > We are using some scratch registers in MMIO based send function. > Make their base and count flexible in preparation of upcoming > GuC firmware/hardware changes. While around, change cmd len > parameter verification from WARN_ON to GEM_BUG_ON as we don't > need this all the time. > > v2: call out WARN/GEM_BUG change in the commit msg (Daniele) > v3: don't overqualify the ints (Chris) > v4: rebase and use proper enum > > Signed-off-by: Michal Wajdeczko <michal.wajdec...@intel.com> > Suggested-by: Daniele Ceraolo Spurio <daniele.ceraolospu...@intel.com> > Cc: Daniele Ceraolo Spurio <daniele.ceraolospu...@intel.com> > Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com> > Cc: Chris Wilson <ch...@chris-wilson.co.uk> > Cc: Jani Nikula <jani.nik...@linux.intel.com> > Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospu...@intel.com>
I suppose this could've had "# v2" at the end, unless Daniele acked all the remaining changes. Anyway, looks good to me with the changes. Reviewed-by: Joonas Lahtinen <joonas.lahti...@linux.intel.com> Could use an A-b from Jani for disagree and commit stamp as he initially objected the way of implementing this. Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
