[DPU] drm: add definitions for DP Audio/Video compliance tests

Submitted by chandanu@codeaurora.org on Jan. 2, 2019, 7:15 a.m.

Details

Message ID 1546413325-11673-2-git-send-email-chandanu@codeaurora.org
State New
Headers show
Series "drm: add definitions for DP Audio/Video compliance tests" ( rev: 1 ) in Freedreno

Not browsing as part of any series.

Commit Message

chandanu@codeaurora.org Jan. 2, 2019, 7:15 a.m.
Signed-off-by: Chandan Uddaraju <chandanu@codeaurora.org>
---
 include/drm/drm_dp_helper.h | 13 +++++++++++++
 1 file changed, 13 insertions(+)

Patch hide | download patch | download mbox

diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h
index 5736c94..e688e05 100644
--- a/include/drm/drm_dp_helper.h
+++ b/include/drm/drm_dp_helper.h
@@ -556,6 +556,8 @@ 
 # define DP_TEST_LINK_EDID_READ		    (1 << 2)
 # define DP_TEST_LINK_PHY_TEST_PATTERN	    (1 << 3) /* DPCD >= 1.1 */
 # define DP_TEST_LINK_FAUX_PATTERN	    (1 << 4) /* DPCD >= 1.2 */
+# define DP_TEST_LINK_AUDIO_PATTERN         (1 << 5) /* DPCD >= 1.2 */
+# define DP_TEST_LINK_AUDIO_DISABLED_VIDEO  (1 << 6) /* DPCD >= 1.2 */
 
 #define DP_TEST_LINK_RATE		    0x219
 # define DP_LINK_RATE_162		    (0x6)
@@ -604,6 +606,7 @@ 
 # define DP_COLOR_FORMAT_RGB                (0 << 1)
 # define DP_COLOR_FORMAT_YCbCr422           (1 << 1)
 # define DP_COLOR_FORMAT_YCbCr444           (2 << 1)
+# define DP_TEST_DYNAMIC_RANGE_VESA         (0 << 3)
 # define DP_TEST_DYNAMIC_RANGE_CEA          (1 << 3)
 # define DP_TEST_YCBCR_COEFFICIENTS         (1 << 4)
 # define DP_YCBCR_COEFFICIENTS_ITU601       (0 << 4)
@@ -653,6 +656,16 @@ 
 
 #define DP_TEST_SINK			    0x270
 # define DP_TEST_SINK_START		    (1 << 0)
+#define DP_TEST_AUDIO_MODE		    0x271
+#define DP_TEST_AUDIO_PATTERN_TYPE	    0x272
+#define DP_TEST_AUDIO_PERIOD_CH1	    0x273
+#define DP_TEST_AUDIO_PERIOD_CH2	    0x274
+#define DP_TEST_AUDIO_PERIOD_CH3	    0x275
+#define DP_TEST_AUDIO_PERIOD_CH4	    0x276
+#define DP_TEST_AUDIO_PERIOD_CH5	    0x277
+#define DP_TEST_AUDIO_PERIOD_CH6	    0x278
+#define DP_TEST_AUDIO_PERIOD_CH7	    0x279
+#define DP_TEST_AUDIO_PERIOD_CH8	    0x27A
 
 #define DP_FEC_STATUS			    0x280    /* 1.4 */
 # define DP_FEC_DECODE_EN_DETECTED	    (1 << 0)

Comments

On Tue, Jan 01, 2019 at 11:15:25PM -0800, Chandan Uddaraju wrote:

This seems fine to me. Could you please:
- delete the cover letter
- move the cover letter description into the patch commit msg
- strip DPU from the subject prefix
- send it to dri-devel, maintaining the cc's you have here

With that,

Reviewed-by: Sean Paul <sean@poorly.run>

Sean

> Signed-off-by: Chandan Uddaraju <chandanu@codeaurora.org>
> ---
>  include/drm/drm_dp_helper.h | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h
> index 5736c94..e688e05 100644
> --- a/include/drm/drm_dp_helper.h
> +++ b/include/drm/drm_dp_helper.h
> @@ -556,6 +556,8 @@
>  # define DP_TEST_LINK_EDID_READ		    (1 << 2)
>  # define DP_TEST_LINK_PHY_TEST_PATTERN	    (1 << 3) /* DPCD >= 1.1 */
>  # define DP_TEST_LINK_FAUX_PATTERN	    (1 << 4) /* DPCD >= 1.2 */
> +# define DP_TEST_LINK_AUDIO_PATTERN         (1 << 5) /* DPCD >= 1.2 */
> +# define DP_TEST_LINK_AUDIO_DISABLED_VIDEO  (1 << 6) /* DPCD >= 1.2 */
>  
>  #define DP_TEST_LINK_RATE		    0x219
>  # define DP_LINK_RATE_162		    (0x6)
> @@ -604,6 +606,7 @@
>  # define DP_COLOR_FORMAT_RGB                (0 << 1)
>  # define DP_COLOR_FORMAT_YCbCr422           (1 << 1)
>  # define DP_COLOR_FORMAT_YCbCr444           (2 << 1)
> +# define DP_TEST_DYNAMIC_RANGE_VESA         (0 << 3)
>  # define DP_TEST_DYNAMIC_RANGE_CEA          (1 << 3)
>  # define DP_TEST_YCBCR_COEFFICIENTS         (1 << 4)
>  # define DP_YCBCR_COEFFICIENTS_ITU601       (0 << 4)
> @@ -653,6 +656,16 @@
>  
>  #define DP_TEST_SINK			    0x270
>  # define DP_TEST_SINK_START		    (1 << 0)
> +#define DP_TEST_AUDIO_MODE		    0x271
> +#define DP_TEST_AUDIO_PATTERN_TYPE	    0x272
> +#define DP_TEST_AUDIO_PERIOD_CH1	    0x273
> +#define DP_TEST_AUDIO_PERIOD_CH2	    0x274
> +#define DP_TEST_AUDIO_PERIOD_CH3	    0x275
> +#define DP_TEST_AUDIO_PERIOD_CH4	    0x276
> +#define DP_TEST_AUDIO_PERIOD_CH5	    0x277
> +#define DP_TEST_AUDIO_PERIOD_CH6	    0x278
> +#define DP_TEST_AUDIO_PERIOD_CH7	    0x279
> +#define DP_TEST_AUDIO_PERIOD_CH8	    0x27A
>  
>  #define DP_FEC_STATUS			    0x280    /* 1.4 */
>  # define DP_FEC_DECODE_EN_DETECTED	    (1 << 0)
> -- 
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project
> 
> _______________________________________________
> Freedreno mailing list
> Freedreno@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/freedreno
On 2019-01-22 11:05, Sean Paul wrote:
> On Tue, Jan 01, 2019 at 11:15:25PM -0800, Chandan Uddaraju wrote:
> 
> This seems fine to me. Could you please:
> - delete the cover letter
> - move the cover letter description into the patch commit msg
> - strip DPU from the subject prefix
> - send it to dri-devel, maintaining the cc's you have here
> 
> With that,
> 
> Reviewed-by: Sean Paul <sean@poorly.run>
> 
> Sean
> 

Hello Sean,
Addressed your comments and uploaded new patch.

thanks
Chandan

>> Signed-off-by: Chandan Uddaraju <chandanu@codeaurora.org>
>> ---
>>  include/drm/drm_dp_helper.h | 13 +++++++++++++
>>  1 file changed, 13 insertions(+)
>> 
>> diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h
>> index 5736c94..e688e05 100644
>> --- a/include/drm/drm_dp_helper.h
>> +++ b/include/drm/drm_dp_helper.h
>> @@ -556,6 +556,8 @@
>>  # define DP_TEST_LINK_EDID_READ		    (1 << 2)
>>  # define DP_TEST_LINK_PHY_TEST_PATTERN	    (1 << 3) /* DPCD >= 1.1 */
>>  # define DP_TEST_LINK_FAUX_PATTERN	    (1 << 4) /* DPCD >= 1.2 */
>> +# define DP_TEST_LINK_AUDIO_PATTERN         (1 << 5) /* DPCD >= 1.2 
>> */
>> +# define DP_TEST_LINK_AUDIO_DISABLED_VIDEO  (1 << 6) /* DPCD >= 1.2 
>> */
>> 
>>  #define DP_TEST_LINK_RATE		    0x219
>>  # define DP_LINK_RATE_162		    (0x6)
>> @@ -604,6 +606,7 @@
>>  # define DP_COLOR_FORMAT_RGB                (0 << 1)
>>  # define DP_COLOR_FORMAT_YCbCr422           (1 << 1)
>>  # define DP_COLOR_FORMAT_YCbCr444           (2 << 1)
>> +# define DP_TEST_DYNAMIC_RANGE_VESA         (0 << 3)
>>  # define DP_TEST_DYNAMIC_RANGE_CEA          (1 << 3)
>>  # define DP_TEST_YCBCR_COEFFICIENTS         (1 << 4)
>>  # define DP_YCBCR_COEFFICIENTS_ITU601       (0 << 4)
>> @@ -653,6 +656,16 @@
>> 
>>  #define DP_TEST_SINK			    0x270
>>  # define DP_TEST_SINK_START		    (1 << 0)
>> +#define DP_TEST_AUDIO_MODE		    0x271
>> +#define DP_TEST_AUDIO_PATTERN_TYPE	    0x272
>> +#define DP_TEST_AUDIO_PERIOD_CH1	    0x273
>> +#define DP_TEST_AUDIO_PERIOD_CH2	    0x274
>> +#define DP_TEST_AUDIO_PERIOD_CH3	    0x275
>> +#define DP_TEST_AUDIO_PERIOD_CH4	    0x276
>> +#define DP_TEST_AUDIO_PERIOD_CH5	    0x277
>> +#define DP_TEST_AUDIO_PERIOD_CH6	    0x278
>> +#define DP_TEST_AUDIO_PERIOD_CH7	    0x279
>> +#define DP_TEST_AUDIO_PERIOD_CH8	    0x27A
>> 
>>  #define DP_FEC_STATUS			    0x280    /* 1.4 */
>>  # define DP_FEC_DECODE_EN_DETECTED	    (1 << 0)
>> --
>> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora 
>> Forum,
>> a Linux Foundation Collaborative Project
>> 
>> _______________________________________________
>> Freedreno mailing list
>> Freedreno@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/freedreno