[Intel-gfx] [PATCH 1/6] drm: Renaming DP training vswing/pre-emph defines

Daniel Vetter daniel at ffwll.ch
Tue Aug 5 13:15:35 CEST 2014


On Tue, Aug 05, 2014 at 04:38:17PM +0530, sonika.jindal at intel.com wrote:
> From: Sonika Jindal <sonika.jindal at intel.com>
> 
> Renaming defines to have levels instead of nominal values.
> 
> Signed-off-by: Sonika Jindal <sonika.jindal at intel.com>

You can't split up patches like this since this will break compilation.
For larger stuff (and imo this is right above the cutoff) you first need
to add the new functions/defines, then convert everyone over. And only
when all the drivers are converted can we apply the patch to remove the
old functions/defines.
-Daniel

> ---
>  include/drm/drm_dp_helper.h |   16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h
> index a21568b..70f362b 100644
> --- a/include/drm/drm_dp_helper.h
> +++ b/include/drm/drm_dp_helper.h
> @@ -190,16 +190,16 @@
>  # define DP_TRAIN_VOLTAGE_SWING_MASK	    0x3
>  # define DP_TRAIN_VOLTAGE_SWING_SHIFT	    0
>  # define DP_TRAIN_MAX_SWING_REACHED	    (1 << 2)
> -# define DP_TRAIN_VOLTAGE_SWING_400	    (0 << 0)
> -# define DP_TRAIN_VOLTAGE_SWING_600	    (1 << 0)
> -# define DP_TRAIN_VOLTAGE_SWING_800	    (2 << 0)
> -# define DP_TRAIN_VOLTAGE_SWING_1200	    (3 << 0)
> +# define DP_TRAIN_VOLTAGE_SWING_LEVEL_0     (0 << 0)
> +# define DP_TRAIN_VOLTAGE_SWING_LEVEL_1     (1 << 0)
> +# define DP_TRAIN_VOLTAGE_SWING_LEVEL_2     (2 << 0)
> +# define DP_TRAIN_VOLTAGE_SWING_LEVEL_3     (3 << 0)
>  
>  # define DP_TRAIN_PRE_EMPHASIS_MASK	    (3 << 3)
> -# define DP_TRAIN_PRE_EMPHASIS_0	    (0 << 3)
> -# define DP_TRAIN_PRE_EMPHASIS_3_5	    (1 << 3)
> -# define DP_TRAIN_PRE_EMPHASIS_6	    (2 << 3)
> -# define DP_TRAIN_PRE_EMPHASIS_9_5	    (3 << 3)
> +# define DP_TRAIN_PRE_EMPHASIS_LEVEL_0  (0 << 3)
> +# define DP_TRAIN_PRE_EMPHASIS_LEVEL_1  (1 << 3)
> +# define DP_TRAIN_PRE_EMPHASIS_LEVEL_2  (2 << 3)
> +# define DP_TRAIN_PRE_EMPHASIS_LEVEL_3  (3 << 3)
>  
>  # define DP_TRAIN_PRE_EMPHASIS_SHIFT	    3
>  # define DP_TRAIN_MAX_PRE_EMPHASIS_REACHED  (1 << 5)
> -- 
> 1.7.10.4
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch



More information about the Intel-gfx mailing list