[Intel-gfx] [PATCH 03/29] drm/i915: add Haswell DIP controls registers

[Eugeni Dodonov]

On Tue, Apr 17, 2012 at 07:12, Daniel Vetter <daniel at ffwll.ch> wrote:

> On Fri, Apr 13, 2012 at 05:08:39PM -0300, Eugeni Dodonov wrote:
> > Haswell has different DIP control registers and offsets.
> >
> > Signed-off-by: Eugeni Dodonov <eugeni.dodonov at intel.com>
>
> I've read a bit through Bspec wrt dip writing and it looks like hsw is
> rather differen from previous chips:
> - with have a data reg for every type of dip
> - the bits in the ctl reg moved around completely
>
> ... so I guess this patch and the follow-on one are pretty bogus. One
> thing I've noticed is that intel_infoframe_index and intel_infoframe_flags
> have way too generic names, they're only useful to frob the dip ctl reg on
> pre-hsw afaics. I think we should rename them to i9xx_infoframe_ctl_inde
> and _flags or something similar.
>

Yep, I only did the minimally-required stuff to have HDMI output, so the
ones willing to use Haswell could have anything on their screen besides 'No
Signal' :).

But yes, those patches should receive more care for full hdmi and DIP
support going forward.

-- 
Eugeni Dodonov
<http://eugeni.dodonov.net/>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20120417/8afeafc5/attachment.html>