[Intel-gfx] [PATCH] i915: enable AVI infoframe for intel_hdmi.c [v2]
david at hardeman.nu
Fri Sep 10 01:13:15 PDT 2010
On Thu, September 9, 2010 23:58, Chris Wilson wrote:
> On Thu, 09 Sep 2010 23:00:01 +0200, David HÃ¤rdeman <david at hardeman.nu>
>> This is the second version which merges the infoframe code from
>> intel_hdmi.c and intel_sdvo.c, I hope this is something along the lines
>> Chris Wilson had in mind. Note that I'm assuming that the sdvo hardware
>> also stores a header ECC byte in the MSB of the first dword (haven't
>> any documentation for the sdvo).
> Pretty close. Using a callback function for writing the data is overkill,
> just call the common function to compute the avi csum, then pass the avi
> to the sdvo or hdmi handlers. It just makes following the logic that
> little bit easier.
> drm-intel-next has changed slightly since you wrote the patch, so can you
> respin it - just a couple of the macro names have changed to be
I'll respin the patch, but I still need to know if the intel_sdvo.c
hardware has an ECC byte as the MSB of the first dword or not.
More information about the Intel-gfx