Re: [Intel-gfx] [PATCH] drm/i915/display: Put glk_cdclk_table
Quoting Ville Syrjälä (2019-09-11 11:36:53) > On Wed, Sep 11, 2019 at 08:47:27AM +0100, Chris Wilson wrote: > > Commit 736da8112fee ("drm/i915: Use literal representation of cdclk > > tables") pushed the cdclk logic into tables, adding glk_cdclk_table but > > not using yet: > > > > drivers/gpu/drm/i915/display/intel_cdclk.c:1173:38: error: > > ‘glk_cdclk_table’ defined but not used [-Werror=unused-const-variable=] > > > > Fixes: 736da8112fee ("drm/i915: Use literal representation of cdclk tables") > > Signed-off-by: Chris Wilson > > Cc: Ville Syrjälä > > Cc: Matt Roper > > Cc: Jani Nikula > > --- > > drivers/gpu/drm/i915/display/intel_cdclk.c | 5 - > > 1 file changed, 4 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c > > b/drivers/gpu/drm/i915/display/intel_cdclk.c > > index 618a93bad0a8..13779b6029f5 100644 > > --- a/drivers/gpu/drm/i915/display/intel_cdclk.c > > +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c > > @@ -2511,7 +2511,10 @@ void intel_init_cdclk_hooks(struct drm_i915_private > > *dev_priv) > > dev_priv->display.set_cdclk = bxt_set_cdclk; > > dev_priv->display.modeset_calc_cdclk = bxt_modeset_calc_cdclk; > > dev_priv->display.calc_voltage_level = bxt_calc_voltage_level; > > - dev_priv->cdclk.table = bxt_cdclk_table; > > + if (IS_GEMINILAKE(dev_priv)) > > + dev_priv->cdclk.table = glk_cdclk_table; > > + else > > + dev_priv->cdclk.table = bxt_cdclk_table; > > Whoops. > > Reviewed-by: Ville Syrjälä Pushed to silence the compiler. I'll let Matt check if this was all that was truly required :) -Chris ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH] drm/i915/display: Put glk_cdclk_table
On Wed, Sep 11, 2019 at 08:47:27AM +0100, Chris Wilson wrote: > Commit 736da8112fee ("drm/i915: Use literal representation of cdclk > tables") pushed the cdclk logic into tables, adding glk_cdclk_table but > not using yet: > > drivers/gpu/drm/i915/display/intel_cdclk.c:1173:38: error: ‘glk_cdclk_table’ > defined but not used [-Werror=unused-const-variable=] > > Fixes: 736da8112fee ("drm/i915: Use literal representation of cdclk tables") > Signed-off-by: Chris Wilson > Cc: Ville Syrjälä > Cc: Matt Roper > Cc: Jani Nikula > --- > drivers/gpu/drm/i915/display/intel_cdclk.c | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c > b/drivers/gpu/drm/i915/display/intel_cdclk.c > index 618a93bad0a8..13779b6029f5 100644 > --- a/drivers/gpu/drm/i915/display/intel_cdclk.c > +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c > @@ -2511,7 +2511,10 @@ void intel_init_cdclk_hooks(struct drm_i915_private > *dev_priv) > dev_priv->display.set_cdclk = bxt_set_cdclk; > dev_priv->display.modeset_calc_cdclk = bxt_modeset_calc_cdclk; > dev_priv->display.calc_voltage_level = bxt_calc_voltage_level; > - dev_priv->cdclk.table = bxt_cdclk_table; > + if (IS_GEMINILAKE(dev_priv)) > + dev_priv->cdclk.table = glk_cdclk_table; > + else > + dev_priv->cdclk.table = bxt_cdclk_table; Whoops. Reviewed-by: Ville Syrjälä > } else if (IS_GEN9_BC(dev_priv)) { > dev_priv->display.set_cdclk = skl_set_cdclk; > dev_priv->display.modeset_calc_cdclk = skl_modeset_calc_cdclk; > -- > 2.23.0 -- Ville Syrjälä Intel ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
[Intel-gfx] [PATCH] drm/i915/display: Put glk_cdclk_table
Commit 736da8112fee ("drm/i915: Use literal representation of cdclk tables") pushed the cdclk logic into tables, adding glk_cdclk_table but not using yet: drivers/gpu/drm/i915/display/intel_cdclk.c:1173:38: error: ‘glk_cdclk_table’ defined but not used [-Werror=unused-const-variable=] Fixes: 736da8112fee ("drm/i915: Use literal representation of cdclk tables") Signed-off-by: Chris Wilson Cc: Ville Syrjälä Cc: Matt Roper Cc: Jani Nikula --- drivers/gpu/drm/i915/display/intel_cdclk.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c index 618a93bad0a8..13779b6029f5 100644 --- a/drivers/gpu/drm/i915/display/intel_cdclk.c +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c @@ -2511,7 +2511,10 @@ void intel_init_cdclk_hooks(struct drm_i915_private *dev_priv) dev_priv->display.set_cdclk = bxt_set_cdclk; dev_priv->display.modeset_calc_cdclk = bxt_modeset_calc_cdclk; dev_priv->display.calc_voltage_level = bxt_calc_voltage_level; - dev_priv->cdclk.table = bxt_cdclk_table; + if (IS_GEMINILAKE(dev_priv)) + dev_priv->cdclk.table = glk_cdclk_table; + else + dev_priv->cdclk.table = bxt_cdclk_table; } else if (IS_GEN9_BC(dev_priv)) { dev_priv->display.set_cdclk = skl_set_cdclk; dev_priv->display.modeset_calc_cdclk = skl_modeset_calc_cdclk; -- 2.23.0 ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx