<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <p><br>
    </p>
    <br>
    <div class="moz-cite-prefix">On 2018年07月11日 15:19, Takashi Iwai
      wrote:<br>
    </div>
    <blockquote type="cite" cite="mid:s5hfu0qp7cv.wl-tiwai@suse.de">
      <pre wrap="">On Tue, 10 Jul 2018 13:21:00 +0200,
Takashi Iwai wrote:
</pre>
      <blockquote type="cite">
        <pre wrap="">
</pre>
        <blockquote type="cite">
          <pre wrap="">revert the fix of amdgpu suspend issue, audio issue also can be observed.
</pre>
        </blockquote>
        <pre wrap="">
Did you check the behavior with the single AMD GPU hardware?
If confirmed, we can forget about vga_switcheroo.
</pre>
      </blockquote>
      <pre wrap="">
... and taking a look back at the recent changes, I guess it can be
the forced runtime PM enablement, not directly with vga_switcheroo
action itself.
</pre>
    </blockquote>
    <br>
    Yeah, the function vga_switcheroo_set_dynamic_switch() has
    discarded, so there is no way GFX driver to control audio power. 
    However, keep in mind, current audio is bound to iGPU, that mean the
    issue should be nothing about <br>
    vgaswtichreoo. since current audio pci bus is different from dGPU,
    that means the pci_bus_set_current_state() in
    vga_switcheroo_runtime_suspend() and pci_wakeup_bus() in
    vga_switcheroo_runtime_resume() could not touch the audio pci power
    state from dGPU instance.<br>
    <br>
    This is a feedback got from our OEM developer, it is the overview of
    audio detect process.<br>
    <br>
    <font size="2"><span style="font-size:11pt;">> First, the kernel
        audio driver will be triggered to read ELD, if the <br>
        >> ELD is valid, it will report a jack event (on or
        available) to sound <br>
        >> core driver; the pulseaudio subscribe all jack events,
        if it is told <br>
        >> that the hdmi jack is plugged in (on), the pulseaudio
        will set this <br>
        >> port to available, then the pa-card or pa-sink has
        available port, it <br>
        >> can be selected (manually, some daemons or policy in <br>
        >> /usr/share/pulseaudio/alsa-mixer/) as default output
        card/default sink.<br>
        <br>
        If the description is correct. I think there are maybe two
        problems.<br>
        <br>
        1. audio will auto power off after setup device link duo to
        usage_count=0.<br>
        2. duo to audio is power down, it could not get the HDMI jack
        insert event.<br>
        <br>
        How do you think?<br>
      </span></font><br>
    <blockquote type="cite" cite="mid:s5hfu0qp7cv.wl-tiwai@suse.de">
      <pre wrap="">
Jim, could you tell me which PCI devices are handled as vga_switcheroo
audio client?  The kernel should show all messages "xxx: Handle
vga_switcheroo audio client". 
</pre>
    </blockquote>
    <br>
    [    4.311095] snd_hda_intel 0000:06:00.1: enabling device (0000
    -> 0002)<br>
    [    4.314286] snd_hda_intel 0000:06:00.1: Handle vga_switcheroo
    audio client<br>
    [    4.314822] snd_hda_intel 0000:06:00.6: enabling device (0000
    -> 0002)<br>
    <br>
    01:00.0 VGA compatible controller [0300]: Advanced Micro Devices,
    Inc. [AMD/ATI] Device [1002:699f] (rev c3)<br>
    06:00.0 VGA compatible controller [0300]: Advanced Micro Devices,
    Inc. [AMD/ATI] Device [1002:15dd] (rev d1)<br>
    06:00.1 Audio device [0403]: Advanced Micro Devices, Inc. [AMD/ATI]
    Device [1002:15de]<br>
    <br>
    Thanks<br>
    JimQu<br>
    <blockquote type="cite" cite="mid:s5hfu0qp7cv.wl-tiwai@suse.de">
      <pre wrap="">
At best, give the full dmesg output and the lspci -nv output.


thanks,

Takashi
</pre>
    </blockquote>
    <br>
  </body>
</html>