mbox series

[0/1,SRU,Jammy,OEM-5.14] The display output on type-c hub doesn't work on ADL platform

Message ID 20220127023453.262136-1-acelan.kao@canonical.com
Headers show
Series The display output on type-c hub doesn't work on ADL platform | expand

Message

AceLan Kao Jan. 27, 2022, 2:34 a.m. UTC
From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>

BugLink: https://bugs.launchpad.net/bugs/1959061

[Impact]
The display output(DP/HDMI) on the USB type-c hub doesn't work. No display
output device could be found after the monitor is plugged into the hub.

[Fix]
Intel provide a new patch to fix this issue on ADL platforms.

[Test]
Verified on ADL platform with different type-c hubs, the display output
works.

[Where problems could occur]
The TCSS_DDI_STATUS register is indexed by tc_port not by the FIA port
index. So, this patch is more like a fix and only affect ADL platforms.
And as the commit log describes, the value is wrong on TC#3/4 ports,
so it won't affect the origin working ports.

Imre Deak (1):
  UBUNTU: SAUCE: drm/i915/adlp: Fix TypeC PHY-ready status readout

 drivers/gpu/drm/i915/display/intel_tc.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Tim Gardner Jan. 27, 2022, 1:09 p.m. UTC | #1
Acked-by: Tim Gardner <tim.gardner@canonical.com>

This patch should disappear once its merged upstream and bubbles down 
via stable.

On 1/26/22 7:34 PM, AceLan Kao wrote:
> From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/1959061
> 
> [Impact]
> The display output(DP/HDMI) on the USB type-c hub doesn't work. No display
> output device could be found after the monitor is plugged into the hub.
> 
> [Fix]
> Intel provide a new patch to fix this issue on ADL platforms.
> 
> [Test]
> Verified on ADL platform with different type-c hubs, the display output
> works.
> 
> [Where problems could occur]
> The TCSS_DDI_STATUS register is indexed by tc_port not by the FIA port
> index. So, this patch is more like a fix and only affect ADL platforms.
> And as the commit log describes, the value is wrong on TC#3/4 ports,
> so it won't affect the origin working ports.
> 
> Imre Deak (1):
>    UBUNTU: SAUCE: drm/i915/adlp: Fix TypeC PHY-ready status readout
> 
>   drivers/gpu/drm/i915/display/intel_tc.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
Timo Aaltonen Jan. 31, 2022, 12:42 p.m. UTC | #2
On 27.1.2022 4.34, AceLan Kao wrote:
> From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/1959061
> 
> [Impact]
> The display output(DP/HDMI) on the USB type-c hub doesn't work. No display
> output device could be found after the monitor is plugged into the hub.
> 
> [Fix]
> Intel provide a new patch to fix this issue on ADL platforms.
> 
> [Test]
> Verified on ADL platform with different type-c hubs, the display output
> works.
> 
> [Where problems could occur]
> The TCSS_DDI_STATUS register is indexed by tc_port not by the FIA port
> index. So, this patch is more like a fix and only affect ADL platforms.
> And as the commit log describes, the value is wrong on TC#3/4 ports,
> so it won't affect the origin working ports.
> 
> Imre Deak (1):
>    UBUNTU: SAUCE: drm/i915/adlp: Fix TypeC PHY-ready status readout
> 
>   drivers/gpu/drm/i915/display/intel_tc.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 

applied to oem-5.14, thanks