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.olive...@intel.com> Cc: Imre Deak <imre.d...@intel.com> Cc: Mika Kuoppala <mika.kuopp...@intel.com> Cc: Tvrtko Ursulin <tvrtko.ursu...@intel.com> Signed-off-by: Rodrigo Vivi <rodrigo.v...@intel.com> --- 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@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx