From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugzilla-daemon@freedesktop.org Subject: [Bug 101900] No HDMI HBR audio on Polaris (no TrueHD, no Atmos, no Neo:X, no HD Master audio) and static noise in sound when LPCM on amdgpu Xorg driver Date: Wed, 07 Mar 2018 19:23:57 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0031839523==" Return-path: Received: from culpepper.freedesktop.org (culpepper.freedesktop.org [131.252.210.165]) by gabe.freedesktop.org (Postfix) with ESMTP id DBD6C6E1E5 for ; Wed, 7 Mar 2018 19:23:56 +0000 (UTC) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============0031839523== Content-Type: multipart/alternative; boundary="15204506361.DaFBbD6F.27438" Content-Transfer-Encoding: 7bit --15204506361.DaFBbD6F.27438 Date: Wed, 7 Mar 2018 19:23:56 +0000 MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://bugs.freedesktop.org/ Auto-Submitted: auto-generated https://bugs.freedesktop.org/show_bug.cgi?id=3D101900 --- Comment #35 from Direx --- (In reply to Harry Wentland from comment #34) > DC seems to take channel_count/audio_count to refer to the actual number = of > channels whereas the CEA EDID extension and our HW represent it as the > number of channels-1. So as not to break other uses of this count (such as > check_audio_bandwidth_hdmi()) this patch adds 1 when we get the count from > the EDID. >=20 > Can someone give it a spin? It's based on the latest amd-staging-drm-next > branch but should apply on other branches as well. Just tested it on a fresh amd-staging-drm-next and it is also working. All audio formats are working correctly. *thumbs up* Could this be backported to 4.15 / 4.16 as it actually is a simple bug fix? @lethalwp: I think your issue is unrelated to this one. I get that too occasionally, b= ut it has nothing to do with the wrong channel count in this bug. --=20 You are receiving this mail because: You are the assignee for the bug.= --15204506361.DaFBbD6F.27438 Date: Wed, 7 Mar 2018 19:23:56 +0000 MIME-Version: 1.0 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://bugs.freedesktop.org/ Auto-Submitted: auto-generated

Comme= nt # 35 on bug 10190= 0 from Direx 
(In reply to Harry Wentland from comment #34)
> DC seems to take channel_count/audio_count to re=
fer to the actual number of
> channels whereas the CEA EDID extension and our HW represent it as the
> number of channels-1. So as not to break other uses of this count (suc=
h as
> check_audio_bandwidth_hdmi()) this patch adds 1 when we get the count =
from
> the EDID.
>=20
> Can someone give it a spin? It's based on the latest amd-staging-drm-n=
ext
> branch but should apply on other branches as well.

Just tested it on a fresh amd-staging-drm-next and it is also working. All
audio formats are working correctly. *thumbs up*

Could this be backported to 4.15 / 4.16 as it actually is a simple bug fix?


@lethalwp:

I think your issue is unrelated to this one. I get that too occasionally, b=
ut
it has nothing to do with the wrong channel count in this bug.

You are receiving this mail because:
  • You are the assignee for the bug.
= --15204506361.DaFBbD6F.27438-- --===============0031839523== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== --===============0031839523==--