[Intel-gfx] [PATCH] drm/i915: Eliminate dead code in intel_sanitize_enable_ppgtt()
Imre Deak
imre.deak at intel.com
Mon Jul 17 12:00:30 UTC 2017
On Wed, Jun 01, 2016 at 06:04:40PM +0100, Chris Wilson wrote:
> On Wed, Jun 01, 2016 at 05:01:13PM +0100, Damien Lespiau wrote:
> > We exit early if has_aliasing_ppgtt is 0, so towards the end of the
> > function has_aliasing_ppgtt can only be 1.
> >
> > Also:
> >
> > if (foo)
> > return 1;
> > else
> > return 0;
> >
> > when foo is already a bool is really just:
> >
> > return foo;
> >
> > Signed-off-by: Damien Lespiau <damien.lespiau at intel.com>
> > ---
> > drivers/gpu/drm/i915/i915_gem_gtt.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c
> > index 4668477..6aae4b8 100644
> > --- a/drivers/gpu/drm/i915/i915_gem_gtt.c
> > +++ b/drivers/gpu/drm/i915/i915_gem_gtt.c
> > @@ -161,7 +161,7 @@ int intel_sanitize_enable_ppgtt(struct drm_i915_private *dev_priv,
> > if (INTEL_GEN(dev_priv) >= 8 && i915.enable_execlists)
> > return has_full_48bit_ppgtt ? 3 : 2;
> > else
> > - return has_aliasing_ppgtt ? 1 : 0;
> > + return has_aliasing_ppgtt;
>
> /* full-ppgtt doesn't yet work reliably in legacy ringbuffer mode */
> if (!i915.enable_execlists)
> return 1;
>
> return has_full_48bit_ppgtt ? 3 : 2;
This fell through the cracks. We'd need it to improve the output from
static checkers.
--Imre
>
> --
> Chris Wilson, Intel Open Source Technology Centre
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
More information about the Intel-gfx
mailing list