public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
From: Jani Nikula <jani.nikula@linux.intel.com>
To: Harry Wentland <harry.wentland@amd.com>,
	Manasi Navare <manasi.d.navare@intel.com>,
	intel-gfx@lists.freedesktop.org
Cc: dri-devel@lists.freedesktop.org, Rodrigo Vivi <rodrigo.vivi@intel.com>
Subject: Re: [PATCH 1/2] drm/dp: Add HBR3 support in existing DRM DP helpers
Date: Fri, 26 Jan 2018 13:38:54 +0200	[thread overview]
Message-ID: <87y3kkdf3l.fsf@intel.com> (raw)
In-Reply-To: <95d5534c-f558-b14d-96ed-01e380eb18c4@amd.com>

On Tue, 23 Jan 2018, Harry Wentland <harry.wentland@amd.com> wrote:
> On 2018-01-22 05:43 PM, Manasi Navare wrote:
>> Existing helpers add support upto HBR2. This patch
>> adds support for HBR3 rate (8.1 Gbps) introduced as
>> part of DP 1.4 specification.
>> 
>> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
>> Cc: Jani Nikula <jani.nikula@linux.intel.com>
>> Cc: dri-devel@lists.freedesktop.org
>> Signed-off-by: Manasi Navare <manasi.d.navare@intel.com>
>
> Both patches look right according to DP 1.4 spec.
>
> Series is
> Reviewed-by: Harry Wentland <harry.wentland@amd.com>

Both pushed to drm-misc-next, thanks for the patches and review.

BR,
Jani.

>
> Harry
>
>> ---
>>  drivers/gpu/drm/drm_dp_helper.c       | 4 ++++
>>  drivers/gpu/drm/drm_dp_mst_topology.c | 3 +++
>>  include/drm/drm_dp_helper.h           | 1 +
>>  3 files changed, 8 insertions(+)
>> 
>> diff --git a/drivers/gpu/drm/drm_dp_helper.c b/drivers/gpu/drm/drm_dp_helper.c
>> index adf79be..ffe14ec 100644
>> --- a/drivers/gpu/drm/drm_dp_helper.c
>> +++ b/drivers/gpu/drm/drm_dp_helper.c
>> @@ -146,6 +146,8 @@ u8 drm_dp_link_rate_to_bw_code(int link_rate)
>>  		return DP_LINK_BW_2_7;
>>  	case 540000:
>>  		return DP_LINK_BW_5_4;
>> +	case 810000:
>> +		return DP_LINK_BW_8_1;
>>  	}
>>  }
>>  EXPORT_SYMBOL(drm_dp_link_rate_to_bw_code);
>> @@ -161,6 +163,8 @@ int drm_dp_bw_code_to_link_rate(u8 link_bw)
>>  		return 270000;
>>  	case DP_LINK_BW_5_4:
>>  		return 540000;
>> +	case DP_LINK_BW_8_1:
>> +		return 810000;
>>  	}
>>  }
>>  EXPORT_SYMBOL(drm_dp_bw_code_to_link_rate);
>> diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu/drm/drm_dp_mst_topology.c
>> index 70dcfa5..36df7df 100644
>> --- a/drivers/gpu/drm/drm_dp_mst_topology.c
>> +++ b/drivers/gpu/drm/drm_dp_mst_topology.c
>> @@ -2087,6 +2087,9 @@ static bool drm_dp_get_vc_payload_bw(int dp_link_bw,
>>  	case DP_LINK_BW_5_4:
>>  		*out = 10 * dp_link_count;
>>  		break;
>> +	case DP_LINK_BW_8_1:
>> +		*out = 15 * dp_link_count;
>> +		break;
>>  	}
>>  	return true;
>>  }
>> diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h
>> index 9d3ce3b..c80fa92 100644
>> --- a/include/drm/drm_dp_helper.h
>> +++ b/include/drm/drm_dp_helper.h
>> @@ -334,6 +334,7 @@
>>  # define DP_LINK_BW_1_62		    0x06
>>  # define DP_LINK_BW_2_7			    0x0a
>>  # define DP_LINK_BW_5_4			    0x14    /* 1.2 */
>> +# define DP_LINK_BW_8_1			    0x1e    /* 1.4 */
>>  
>>  #define DP_LANE_COUNT_SET	            0x101
>>  # define DP_LANE_COUNT_MASK		    0x0f
>> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

      reply	other threads:[~2018-01-26 11:38 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-01-22 22:43 [PATCH 1/2] drm/dp: Add HBR3 support in existing DRM DP helpers Manasi Navare
2018-01-22 22:43 ` [PATCH 2/2] drm/dp: Add definitions for TPS4 bits and macros to check the support Manasi Navare
2018-01-22 23:03 ` ✓ Fi.CI.BAT: success for series starting with [1/2] drm/dp: Add HBR3 support in existing DRM DP helpers Patchwork
2018-01-23  6:03 ` ✗ Fi.CI.IGT: failure " Patchwork
2018-01-23 14:47 ` [PATCH 1/2] " Harry Wentland
2018-01-26 11:38   ` Jani Nikula [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87y3kkdf3l.fsf@intel.com \
    --to=jani.nikula@linux.intel.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=harry.wentland@amd.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=manasi.d.navare@intel.com \
    --cc=rodrigo.vivi@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox