> From: Ville Syrjälä <ville.syrj...@linux.intel.com>
> 
> The comments regarding PCH compatibility use long vs.
> short names inconsistently. Just use short names always.
> 
> Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
> ---
>  drivers/gpu/drm/i915/intel_pch.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_pch.c
> b/drivers/gpu/drm/i915/intel_pch.c
> index e2b2bbdc0714..94446cac6605 100644
> --- a/drivers/gpu/drm/i915/intel_pch.c
> +++ b/drivers/gpu/drm/i915/intel_pch.c
> @@ -25,7 +25,7 @@ intel_pch_type(const struct drm_i915_private
> *dev_priv, unsigned short id)
>               drm_dbg_kms(&dev_priv->drm, "Found PantherPoint
> PCH\n");
>               drm_WARN_ON(&dev_priv->drm,
>                           GRAPHICS_VER(dev_priv) != 6 &&
> !IS_IVYBRIDGE(dev_priv));
> -             /* PantherPoint is CPT compatible */
> +             /* PPT is CPT compatible */


This is not the code but only comment, I feel elaborated names makes it more 
readable than abbreviation.

Thanks and Regards,
Arun R Murthy
--------------------

Reply via email to