On Tue, 13 Feb 2018, Oscar Mateo <oscar.mateo@xxxxxxxxx> wrote:
> @@ -8979,6 +8701,7 @@ static void i830_init_clock_gating(struct drm_i915_private *dev_priv)
> void intel_init_clock_gating(struct drm_i915_private *dev_priv)
> {
> dev_priv->display.init_clock_gating(dev_priv);
> + intel_disp_workarounds_apply(dev_priv);
Please don't abbreviate display to disp, use the full word instead,
throughout the series. Rationale:
$ git grep disp_ -- drivers/gpu/drm/i915 | wc -l
8
$ git grep display_ -- drivers/gpu/drm/i915 | wc -l
752
> diff --git a/drivers/gpu/drm/i915/intel_workarounds.c b/drivers/gpu/drm/i915/intel_workarounds.c
> +static void bdw_disp_workarounds_apply(struct drm_i915_private *dev_priv)
If you want to shorten the names within intel_workarounds.c please
abbreviate workarounds to wa instead. I think it should be obvious
within that file. But I don't mind longer names here. It's the long
names that are used throughout the driver that I'd avoid.
BR,
Jani.
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
