[Intel-gfx] [PATCH] drm/i915: fix comment referencing imaginary functions

Chris Wilson chris at chris-wilson.co.uk
Fri Oct 21 12:28:41 UTC 2016


On Fri, Oct 21, 2016 at 02:16:46PM +0200, Arkadiusz Hiler wrote:
> On Wed, Aug 24, 2016 at 05:03:11PM +0100, Matthew Auld wrote:
> > The comment which documents the proper usage of the *_FW family of macros makes
> > reference to intel_uncore_forcewake_irq{unlock, lock}, which is just
> > confusing, seeing as such a set of functions don't even exist and never have
> > for that matter(according to git). Let's fix that by replacing them with
> > intel_uncore_forcewake_{get, put}.
> > 
> > Cc: Chris Wilson <chris at chris-wilson.co.uk>
> > Signed-off-by: Matthew Auld <matthew.auld at intel.com>
> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler at intel.com>

The downside is that this now doesn't mention the locking required to
prevent machine hangs on some platforms.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list