On Sun, Dec 18, 2016 at 01:36:28PM -0800, Rodrigo Vivi wrote: > Let's take usage of IS_LP to simplify the gem stolen > initialization as suggest by Tvrtko. > > Also assume that all new LP platforms follows the chv+ > and others bdw+. > > Broxton and Geminilake were probably using the incorrect path > so far. > > Cc: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@xxxxxxxxx> > Cc: Imre Deak <imre.deak@xxxxxxxxx> > Cc: Mika Kuoppala <mika.kuoppala@xxxxxxxxx> > Cc: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx> > --- > drivers/gpu/drm/i915/i915_gem_stolen.c | 9 ++++----- > 1 file changed, 4 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_gem_stolen.c b/drivers/gpu/drm/i915/i915_gem_stolen.c > index 3d24227..98deaa7 100644 > --- a/drivers/gpu/drm/i915/i915_gem_stolen.c > +++ b/drivers/gpu/drm/i915/i915_gem_stolen.c > @@ -454,13 +454,12 @@ int i915_gem_init_stolen(struct drm_i915_private *dev_priv) > &reserved_size); > break; > default: > - if (IS_BROADWELL(dev_priv) || > - IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv)) > - bdw_get_stolen_reserved(dev_priv, &reserved_base, > + if (IS_LP(dev_priv)) > + chv_get_stolen_reserved(dev_priv, &reserved_base, > &reserved_size); > else > - chv_get_stolen_reserved(dev_priv, &reserved_base, > - &reserved_size); > + bdw_get_stolen_reserved(dev_priv, &reserved_base, > + &reserved_size); > break; > } > > -- > 1.9.1 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
