From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugzilla-daemon@freedesktop.org Subject: [Bug 106597] [vga_switcheroo] commit 07f4f97d7b4bf325d9f558c5b58230387e4e57e0 breaks dpm on Alienware 15R3 Date: Thu, 24 May 2018 13:07:17 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0886327049==" Return-path: Received: from culpepper.freedesktop.org (culpepper.freedesktop.org [131.252.210.165]) by gabe.freedesktop.org (Postfix) with ESMTP id 7B20D6E63F for ; Thu, 24 May 2018 13:07:17 +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 --===============0886327049== Content-Type: multipart/alternative; boundary="15271672370.568b2.32111" Content-Transfer-Encoding: 7bit --15271672370.568b2.32111 Date: Thu, 24 May 2018 13:07:17 +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=3D106597 --- Comment #26 from Lukas Wunner --- So hda_call_codec_suspend() tries to set the codec's power state to D3 by calling hda_set_power_state(). The return value of that is the 32 bit respo= nse detailed on page 151 ff. of the HDA spec. hda_codec_runtime_suspend() then checks if the PS-ClkStopOk bit (bit 9) is set. Only then does it allow susp= end of the codec. That bit is not set. It's also not set for the other two codecs of your mac= hine (in the Intel HDA controller). And there's another oddity, the 32 bit response should contain the actual p= ower state in bits 7:4, but those bits are always cleared, for all codecs and regardless whether the codec was put into D0 or D3. In other words, the cod= ecs all remain in D0 even if we tell them to go to D3. Moreover, bits 0:3 of the response should contain the last power state set,= but those bits are always either set to D0 or D1. But according to the power capabilities of the AMD HDA controller, it only supports D0 and D3, not D1.= So this looks totally bogus. Let me test the debug patch on my own machine and check what the response l= ooks like there. --=20 You are receiving this mail because: You are the assignee for the bug.= --15271672370.568b2.32111 Date: Thu, 24 May 2018 13:07:17 +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 # 26 on bug 10659= 7 from Lukas Wunner
So hda_call_codec_suspend() tries to set the codec's power sta=
te to D3 by
calling hda_set_power_state(). The return value of that is the 32 bit respo=
nse
detailed on page 151 ff. of the HDA spec. hda_codec_runtime_suspend() then
checks if the PS-ClkStopOk bit (bit 9) is set. Only then does it allow susp=
end
of the codec.

That bit is not set. It's also not set for the other two codecs of your mac=
hine
(in the Intel HDA controller).

And there's another oddity, the 32 bit response should contain the actual p=
ower
state in bits 7:4, but those bits are always cleared, for all codecs and
regardless whether the codec was put into D0 or D3. In other words, the cod=
ecs
all remain in D0 even if we tell them to go to D3.

Moreover, bits 0:3 of the response should contain the last power state set,=
 but
those bits are always either set to D0 or D1. But according to the power
capabilities of the AMD HDA controller, it only supports D0 and D3, not D1.=
 So
this looks totally bogus.

Let me test the debug patch on my own machine and check what the response l=
ooks
like there.


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