On Thu, Jan 24, 2013 at 11:05:21PM +0100, Daniel Vetter wrote:
> On Thu, Jan 24, 2013 at 03:29:42PM +0200, ville.syrjala at linux.intel.com wrote:
> > From: Ville Syrj?l? <ville.syrjala at linux.intel.com>
> >
> > When passing the DP/HDMI/SDVO registers to the encoder init functions,
> > include the VLV specific offset in the value.
> >
> > Signed-off-by: Ville Syrj?l? <ville.syrjala at linux.intel.com>
> > ---
> > drivers/gpu/drm/i915/intel_display.c | 18 +++++++++---------
> > 1 file changed, 9 insertions(+), 9 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> > index 26df9e3..60b5112 100644
> > --- a/drivers/gpu/drm/i915/intel_display.c
> > +++ b/drivers/gpu/drm/i915/intel_display.c
> > @@ -8230,20 +8230,20 @@ static void intel_setup_outputs(struct drm_device *dev)
> > int found;
> >
> > /* Check for built-in panel first. Shares lanes with HDMI on SDVOC */
> > - if (I915_READ(DP_C) & DP_DETECTED)
> > - intel_dp_init(dev, DP_C, PORT_C);
> > + if (I915_READ(VLV_DISPLAY_BASE + DP_C) & DP_DETECTED)
> > + intel_dp_init(dev, VLV_DISPLAY_BASE + DP_C, PORT_C);
> >
> > - if (I915_READ(SDVOB) & PORT_DETECTED) {
> > + if (I915_READ(VLV_DISPLAY_BASE + SDVOB) & PORT_DETECTED) {
> > /* SDVOB multiplex with HDMIB */
> > - found = intel_sdvo_init(dev, SDVOB, true);
> > + found = intel_sdvo_init(dev, VLV_DISPLAY_BASE + SDVOB, true);
>
> I think we should kill the intel_sdvo_init copy&pasta here beforehand. Or
> notice that we have a big problem and need to fixup sdvo support ...
I think Jesse something like that lined up. Should we wait for that, or
do you want me to make a quick fix?
--
Ville Syrj?l?
Intel OTC
