<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [ BSW regression] Call trace when starting the system"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=90012#c3">Comment # 3</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [ BSW regression] Call trace when starting the system"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=90012">bug 90012</a>
              from <span class="vcard"><a class="email" href="mailto:imre.deak@intel.com" title="Imre Deak <imre.deak@intel.com>"> <span class="fn">Imre Deak</span></a>
</span></b>
        <pre>(In reply to Jani Nikula from <a href="show_bug.cgi?id=90012#c2">comment #2</a>)
<span class="quote">> i915_audio_component_get_cdclk_freq is not implemented for hsw.</span >

On hsw it is but not on bsw. There is the corresponding get_display_clock_speed
hook defined for all platforms, so in theory this would work everywhere. Ville
suggested to Mika Kahola that he removes this WARN after the CDCLK patchset is
merged, I think that's the way to fix this.

<span class="quote">> I am not sure if haswell_set_bclk in the hda driver is the right thing to
> call on a bsw either.</span >

Yes, that looks incorrect. haswell_set_bclk() in practice only updates the
EM4/EM5 registers which - based on Bspec - only exist on hsw and bdw.

<span class="quote">> Imre?</span ></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>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>