<div class="gmail_quote">On Sat, May 12, 2012 at 3:22 PM, Daniel Vetter <span dir="ltr"><<a href="mailto:daniel.vetter@ffwll.ch" target="_blank">daniel.vetter@ffwll.ch</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

At least the worst offenders:<br>
- SDVO specifies that the encoder should compute the ecc. Testing also<br>
  shows that we must not send the ecc field, so copy the dip_infoframe<br>
  struct to a temporay place and avoid the ecc field. This way the avi<br>
  infoframe is exactly 17 bytes long, which agrees with what the spec<br>
  mandates as a minimal storage capacity (with the ecc field it would<br>
  be 18 bytes).<br>
- Only 17 when sending the avi infoframe. The SDVO spec explicitly<br>
  says that sending more data than what the device announces results<br>
  in undefined behaviour.<br>
- Add __attribute__((packed)) to the avi and spd infoframes, for<br>
  otherwise they're wrongly aligned. Noticed because the avi infoframe<br>
  ended up being 18 bytes large instead of 17. We haven't noticed this<br>
  yet because we don't use the uint16_t fields yet (which are the only<br>
  ones that would be wrongly aligned).<br>
<br>
This regression has been introduce by<br>
<br>
3c17fe4b8f40a112a85758a9ab2aebf772bdd647 is the first bad commit<br>
commit 3c17fe4b8f40a112a85758a9ab2aebf772bdd647<br>
Author: David Härdeman <<a href="mailto:david@hardeman.nu">david@hardeman.nu</a>><br>
Date:   Fri Sep 24 21:44:32 2010 +0200<br>
<br>
    i915: enable AVI infoframe for intel_hdmi.c [v4]<br>
<br>
Patch tested on my g33 with a sdvo hdmi adaptor.<br>
<br>
Bugzilla: <a href="https://bugzilla.kernel.org/show_bug.cgi?id=25732" target="_blank">https://bugzilla.kernel.org/show_bug.cgi?id=25732</a><br>
Signed-Off-by: Daniel Vetter <<a href="mailto:daniel.vetter@ffwll.ch">daniel.vetter@ffwll.ch</a>><br></blockquote><div><br></div><div>This looks correct, and the math matches the 17 bytes we need. So for the patch itself:</div>

<div>Reviewed-by: Eugeni Dodonov <<a href="mailto:eugeni.dodonov@intel.com">eugeni.dodonov@intel.com</a>></div><div><br></div><div>(But I haven't tested it myself as I don't have such a machine).</div><div>
<br>
</div></div>-- <br>Eugeni Dodonov<a href="http://eugeni.dodonov.net/" target="_blank"><br></a><br>