<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:ville.syrjala@linux.intel.com" title="Ville Syrjala <ville.syrjala@linux.intel.com>"> <span class="fn">Ville Syrjala</span></a>
</span> changed
<a class="bz_bug_link
bz_status_REOPENED "
title="REOPENED - [BDW][Audio][HD-A Display] DP/HDMI audio playback abnormal"
href="https://bugs.freedesktop.org/show_bug.cgi?id=91410">bug 91410</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">CC</td>
<td>
</td>
<td>tiwai@suse.de
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_REOPENED "
title="REOPENED - [BDW][Audio][HD-A Display] DP/HDMI audio playback abnormal"
href="https://bugs.freedesktop.org/show_bug.cgi?id=91410#c25">Comment # 25</a>
on <a class="bz_bug_link
bz_status_REOPENED "
title="REOPENED - [BDW][Audio][HD-A Display] DP/HDMI audio playback abnormal"
href="https://bugs.freedesktop.org/show_bug.cgi?id=91410">bug 91410</a>
from <span class="vcard"><a class="email" href="mailto:ville.syrjala@linux.intel.com" title="Ville Syrjala <ville.syrjala@linux.intel.com>"> <span class="fn">Ville Syrjala</span></a>
</span></b>
<pre>(In reply to Jim Bride from <a href="show_bug.cgi?id=91410#c24">comment #24</a>)
<span class="quote">> I'm not so sure that this is a race per se. From the brief look I took at
> the HDA driver, it seems that the audio code believes that CDCLK stays at a
> fixed value and thus it only confirms the value during its initial init and
> when a resume operation occurs. Since CDCLK can be changed at various times
> on some platforms, it seems like what we need is some sort of a callback
> mechanism to notify the audio driver when CDCLK gets set to a new value so
> that they can update their M/N values appropriately. Alternately, although
> I'd leave this determination to folks that are more familiar with the audio
> driver, if there's a good spot where they can call
> snd_hdac_get_display_clk() periodically and update their M/N values if the
> new value differs from the previous value that they have then that would
> work too. Thoughts?</span >
We have the .pin_eld_notify() hook which could trigger the audio driver to
re-query the cdclk. If the port is enabled, cdclk is currently guaranteed to
stay at a fixed frequency between the .pin_eld_notify() calls from the port
enable and disable.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
<li>You are on the CC list for the bug.</li>
</ul>
</body>
</html>