diff mbox

4.1-rc4 display-port on radeon not working

Message ID CADnq5_Mu4MC9Rqx6Xp_U=b6+1VzGs6x3qvkTWkYyLCb4zYW29Q@mail.gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Alex Deucher May 20, 2015, 10:04 p.m. UTC 
On Wed, May 20, 2015 at 12:57 PM, Malte Schröder <malte@tnxip.de> wrote:
> On 05/20/15 17:37, Alex Deucher wrote:
>> On Tue, May 19, 2015 at 6:03 PM, Malte Schröder <malte@tnxip.de> wrote:
>>> Hi,
>>> with 4.1-rc display-port on my Tahiti XT stopped working. I used the
>>> firmware from http://people.freedesktop.org/~agd5f/radeon_ucode/ as of
>>> today.
>>> Note: the display is an Asus PB278, it used to have problems with
>>> display port negotiation, but those disappeared like a year ago.
>>>
>>
>> Does booting with radeon.auxch=0 on the kernel command line in grub
>> help?  Can you bisect?
>>
>> Alex
>>
>
> Yes, radeon.auxch=0 helps. As for bisecting, I can try, but I won't have
> time for that until next week.

No need to bisect.  if auxch=0 helps, I know what commit broke it.
Does the attached patch help?

Alex


>
> --
> Gruß
> Malte Schröder
> ---------------------------------------------------------------------
> MalteSch@gmx.de
>

Comments

Malte Schröder May 21, 2015, 7:50 a.m. UTC | #1 
On 05/21/15 00:04, Alex Deucher wrote:
> On Wed, May 20, 2015 at 12:57 PM, Malte Schröder <malte@tnxip.de> wrote:
>> On 05/20/15 17:37, Alex Deucher wrote:
>>> On Tue, May 19, 2015 at 6:03 PM, Malte Schröder <malte@tnxip.de> wrote:
>>>> Hi,
>>>> with 4.1-rc display-port on my Tahiti XT stopped working. I used the
>>>> firmware from http://people.freedesktop.org/~agd5f/radeon_ucode/ as of
>>>> today.
>>>> Note: the display is an Asus PB278, it used to have problems with
>>>> display port negotiation, but those disappeared like a year ago.
>>>>
>>>
>>> Does booting with radeon.auxch=0 on the kernel command line in grub
>>> help?  Can you bisect?
>>>
>>> Alex
>>>
>>
>> Yes, radeon.auxch=0 helps. As for bisecting, I can try, but I won't have
>> time for that until next week.
> 
> No need to bisect.  if auxch=0 helps, I know what commit broke it.
> Does the attached patch help?

Yes, this patch does fix the problem for me :)

> 
> Alex
diff mbox

Patch

From e3e5efb1e3f9d3f16a6521c56cde7a7b1c83a2e8 Mon Sep 17 00:00:00 2001
From: Alex Deucher <alexander.deucher@amd.com>
Date: Wed, 20 May 2015 17:58:49 -0400
Subject: [PATCH] drm/radeon: fix error flag checking in native aux path

That atom table does not check these bits.  Fixes aux
regressions on some boards.

Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Cc: stable@vger.kernel.org
---
 drivers/gpu/drm/radeon/radeon_dp_auxch.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/gpu/drm/radeon/radeon_dp_auxch.c b/drivers/gpu/drm/radeon/radeon_dp_auxch.c
index bf1fecc..fcbd60b 100644
--- a/drivers/gpu/drm/radeon/radeon_dp_auxch.c
+++ b/drivers/gpu/drm/radeon/radeon_dp_auxch.c
@@ -30,8 +30,6 @@ 
 			    AUX_SW_RX_HPD_DISCON |	     \
 			    AUX_SW_RX_PARTIAL_BYTE |	     \
 			    AUX_SW_NON_AUX_MODE |	     \
-			    AUX_SW_RX_MIN_COUNT_VIOL |	     \
-			    AUX_SW_RX_INVALID_STOP |	     \
 			    AUX_SW_RX_SYNC_INVALID_L |	     \
 			    AUX_SW_RX_SYNC_INVALID_H |	     \
 			    AUX_SW_RX_INVALID_START |	     \
-- 
1.8.3.1