[Intel-gfx] [PATCH 1/7] drm: Fix DP_TEST_COUNT_MASK

Thierry Reding thierry.reding at gmail.com
Mon Jul 27 04:23:44 PDT 2015


On Thu, Jul 23, 2015 at 04:34:58PM -0700, Rodrigo Vivi wrote:
> By Vesa's DP 1.2 Spec this counter has 4 bits [3:0].
> 
> This mask is wrong since when the counter was introduced by myself
> on commit ad9dc91b6e21266bfc6f466db4b95e10211f31ee
> Author: Rodrigo Vivi <rodrigo.vivi at gmail.com>
> Date:   Tue Sep 16 19:18:12 2014 -0400
> 
>     drm/i915: Fix Sink CRC
> 
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
> ---
>  include/drm/drm_dp_helper.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h
> index 2e86f64..94898f6 100644
> --- a/include/drm/drm_dp_helper.h
> +++ b/include/drm/drm_dp_helper.h
> @@ -420,7 +420,7 @@
>  
>  #define DP_TEST_SINK_MISC		    0x246
>  # define DP_TEST_CRC_SUPPORTED		    (1 << 5)
> -# define DP_TEST_COUNT_MASK		    0x7
> +# define DP_TEST_COUNT_MASK		    0xf

According to the specification this field should really be called
DP_TEST_CRC_COUNT_MASK, because it counts the number of times that the
TEST_CRC_* registers are updated. That could be a separate patch,
though.

Thierry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20150727/19f8722b/attachment.sig>


More information about the Intel-gfx mailing list