diff mbox series

[PATCHv2] drm/i915/display/dp: Remove support for UHBR13.5

Message ID 20240228144350.3184930-1-arun.r.murthy@intel.com (mailing list archive)
State New, archived
Headers show
Series [PATCHv2] drm/i915/display/dp: Remove support for UHBR13.5 | expand

Commit Message

Murthy, Arun R Feb. 28, 2024, 2:43 p.m. UTC
UHBR13.5 is not supported in MTL and also the DP2.1 spec says UHBR13.5
is optional. Hence removing UHBR135 from the supported link rates.

Fixes: https://gitlab.freedesktop.org/drm/intel/-/issues/8686

v2: Reframed the commit message and added link to the the issue.

Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>
---
 drivers/gpu/drm/i915/display/intel_dp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jani Nikula Feb. 28, 2024, 3:06 p.m. UTC | #1
On Wed, 28 Feb 2024, Arun R Murthy <arun.r.murthy@intel.com> wrote:
> UHBR13.5 is not supported in MTL and also the DP2.1 spec says UHBR13.5
> is optional. Hence removing UHBR135 from the supported link rates.
>
> Fixes: https://gitlab.freedesktop.org/drm/intel/-/issues/8686

Fixes: trailer is for commits. And trailers should be together, with no
blank lines or patch changelogs in between.

>
> v2: Reframed the commit message and added link to the the issue.
>
> Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>

Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/8686
Fixes: 62618c7f117e ("drm/i915/mtl: C20 PLL programming")
Cc: <stable@vger.kernel.org> # v6.5+
Reviewed-by: Jani Nikula <jani.nikula@intel.com>

There's no need to add /display/ in every subject prefix. drm/i915/dp
will do just fine.

Can be fixed while applying.

> ---
>  drivers/gpu/drm/i915/display/intel_dp.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
> index 6ece2c563c7a..c11d9055981f 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -498,7 +498,7 @@ intel_dp_set_source_rates(struct intel_dp *intel_dp)
>  	/* The values must be in increasing order */
>  	static const int mtl_rates[] = {
>  		162000, 216000, 243000, 270000, 324000, 432000, 540000, 675000,
> -		810000,	1000000, 1350000, 2000000,
> +		810000,	1000000, 2000000,
>  	};
>  	static const int icl_rates[] = {
>  		162000, 216000, 270000, 324000, 432000, 540000, 648000, 810000,
diff mbox series

Patch

diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
index 6ece2c563c7a..c11d9055981f 100644
--- a/drivers/gpu/drm/i915/display/intel_dp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp.c
@@ -498,7 +498,7 @@  intel_dp_set_source_rates(struct intel_dp *intel_dp)
 	/* The values must be in increasing order */
 	static const int mtl_rates[] = {
 		162000, 216000, 243000, 270000, 324000, 432000, 540000, 675000,
-		810000,	1000000, 1350000, 2000000,
+		810000,	1000000, 2000000,
 	};
 	static const int icl_rates[] = {
 		162000, 216000, 270000, 324000, 432000, 540000, 648000, 810000,