[Bug 88493] No hdmi audio an agd5f 3.20-wip since consolidate audio_get_pin() functions

[bugzilla-daemon@freedesktop.org]

[-- Attachment #1.1: Type: text/plain, Size: 1217 bytes --]

https://bugs.freedesktop.org/show_bug.cgi?id=88493

--- Comment #10 from Andy Furniss <adf.lists@gmail.com> ---
(In reply to Alex Deucher from comment #1)
> Are you sure that's the right commit?  I don't see any functional change. 
> It just switches a direct function call to a function pointer.

I played a bit more - trying to see if there was any real change or maybe some
unlucky h/w timing/order issue.

There is a functional change, I guess seeded in a preceding commit.

Testing with debugging like -

diff --git a/drivers/gpu/drm/radeon/dce6_afmt.c
b/drivers/gpu/drm/radeon/dce6_afmt.c
index 96f298c..266f32a 100644
--- a/drivers/gpu/drm/radeon/dce6_afmt.c
+++ b/drivers/gpu/drm/radeon/dce6_afmt.c
@@ -75,6 +75,7 @@ struct r600_audio_pin *dce6_audio_get_pin(struct
radeon_device *rdev)
 {
        int i;

+        DRM_ERROR("[ADF] in dce6_audio_get_pin\n");
        dce6_afmt_get_connected_pins(rdev);

        for (i = 0; i < rdev->audio.num_pins; i++) {

On the bad commit there is no output after doing 

xrandr --output HDMI-0 --auto 

on the commit before there is.

On the bad I do get the output with a second use of xrandr.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[-- Attachment #1.2: Type: text/html, Size: 2140 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel