diff mbox series

arm64: dts: qcom: sc7280: remove hbr3 support on herobrine boards

Message ID 20230329233416.27152-1-quic_abhinavk@quicinc.com
State Accepted
Commit 75eab749e7aec0b7b515d7c50ed429ef4e1c5f3f
Headers show
Series arm64: dts: qcom: sc7280: remove hbr3 support on herobrine boards | expand

Commit Message

Abhinav Kumar March 29, 2023, 11:34 p.m. UTC
There are some interop issues seen across a few DP monitors with
HBR3 and herobrine boards where the DP display stays blank with hbr3.
This is still under investigation but in preparation for supporting
higher resolutions, its better to disable HBR3 till the issues are
root-caused as there is really no guarantee which monitors will show
the issue and which would not.

This can be enabled back after successful validation across more DP
sinks.

Signed-off-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
---
 arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Doug Anderson March 30, 2023, 2:24 p.m. UTC | #1
Hi,

On Wed, Mar 29, 2023 at 4:34 PM Abhinav Kumar <quic_abhinavk@quicinc.com> wrote:
>
> There are some interop issues seen across a few DP monitors with
> HBR3 and herobrine boards where the DP display stays blank with hbr3.
> This is still under investigation but in preparation for supporting
> higher resolutions, its better to disable HBR3 till the issues are
> root-caused as there is really no guarantee which monitors will show
> the issue and which would not.
>
> This can be enabled back after successful validation across more DP
> sinks.
>
> Signed-off-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
> ---
>  arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
> index b6137816f2f3..313083ec1f39 100644
> --- a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
> @@ -464,7 +464,7 @@ &mdss_dp {
>
>  &mdss_dp_out {
>         data-lanes = <0 1>;
> -       link-frequencies = /bits/ 64 <1620000000 2700000000 5400000000 8100000000>;
> +       link-frequencies = /bits/ 64 <1620000000 2700000000 5400000000>;

This seems OK to me.

Reviewed-by: Douglas Anderson <dianders@chromium.org>
Bjorn Andersson April 7, 2023, 5:41 p.m. UTC | #2
On Wed, 29 Mar 2023 16:34:16 -0700, Abhinav Kumar wrote:
> There are some interop issues seen across a few DP monitors with
> HBR3 and herobrine boards where the DP display stays blank with hbr3.
> This is still under investigation but in preparation for supporting
> higher resolutions, its better to disable HBR3 till the issues are
> root-caused as there is really no guarantee which monitors will show
> the issue and which would not.
> 
> [...]

Applied, thanks!

[1/1] arm64: dts: qcom: sc7280: remove hbr3 support on herobrine boards
      commit: 75eab749e7aec0b7b515d7c50ed429ef4e1c5f3f

Best regards,
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
index b6137816f2f3..313083ec1f39 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
@@ -464,7 +464,7 @@  &mdss_dp {
 
 &mdss_dp_out {
 	data-lanes = <0 1>;
-	link-frequencies = /bits/ 64 <1620000000 2700000000 5400000000 8100000000>;
+	link-frequencies = /bits/ 64 <1620000000 2700000000 5400000000>;
 };
 
 &mdss_mdp {