[Intel-gfx] [PATCH 8/8] drm/i915: protect FBC functions with FBC checks

Chris Wilson chris at chris-wilson.co.uk
Fri Jul 3 11:50:31 PDT 2015


On Fri, Jul 03, 2015 at 03:40:54PM -0300, Paulo Zanoni wrote:
> From: Paulo Zanoni <paulo.r.zanoni at intel.com>
> 
> Now all the functions called by other files check whether FBC has been
> initialized. This allows us to drop the checks on the static
> functions.
> 
> v2:
>  - s/HAS_FBC/dev_priv->display.enable_fbc/ everywhere but the init
>    function (Chris).
> 
> Suggested-by: Chris Wilson <chris at chris-wilson.co.uk>
> Signed-off-by: Paulo Zanoni <paulo.r.zanoni at intel.com>
> ---
>  drivers/gpu/drm/i915/intel_fbc.c | 27 ++++++++++++++++++---------
>  1 file changed, 18 insertions(+), 9 deletions(-)
> 
> 
> I'm not 100% sure this is what Chris was asking for v2, so please clarify if
> this is not what we want.

Does the trick for me,
Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list