drm/i915: don't forget the PCH backlight registers
authorPaulo Zanoni <paulo.r.zanoni@intel.com>
Sat, 14 Jul 2012 14:57:12 +0000 (11:57 -0300)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 20 Jul 2012 10:21:41 +0000 (12:21 +0200)
When we enable/disable the CPU backlight registers we can't forget to
enable/disable the PCH backlight registers. Since we're using the CPU
registers we should also unset the override bit.

Fixes a regression on the following commit:
  drm/i915: properly enable the blc controller on the right pipe

The commit just deleted the code that sets the PCH registers, so it
was relying on the values set by the BIOS. I told my BIOS to boot on
the DVI monitor instead of the LVDS panel, so I noticed the bug.

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_panel.c

index 58c7ee7238b891a2ea1cf75c0a440872f03a180e..10c7d39034e1c8c0a3e17491e17ffa3286b5f97a 100644 (file)
@@ -289,11 +289,17 @@ void intel_panel_disable_backlight(struct drm_device *dev)
        intel_panel_actually_set_backlight(dev, 0);
 
        if (INTEL_INFO(dev)->gen >= 4) {
-               uint32_t reg;
+               uint32_t reg, tmp;
 
                reg = HAS_PCH_SPLIT(dev) ? BLC_PWM_CPU_CTL2 : BLC_PWM_CTL2;
 
                I915_WRITE(reg, I915_READ(reg) & ~BLM_PWM_ENABLE);
+
+               if (HAS_PCH_SPLIT(dev)) {
+                       tmp = I915_READ(BLC_PWM_PCH_CTL1);
+                       tmp &= ~BLM_PCH_PWM_ENABLE;
+                       I915_WRITE(BLC_PWM_PCH_CTL1, tmp);
+               }
        }
 }
 
@@ -333,6 +339,13 @@ void intel_panel_enable_backlight(struct drm_device *dev,
                I915_WRITE(reg, tmp);
                POSTING_READ(reg);
                I915_WRITE(reg, tmp | BLM_PWM_ENABLE);
+
+               if (HAS_PCH_SPLIT(dev)) {
+                       tmp = I915_READ(BLC_PWM_PCH_CTL1);
+                       tmp |= BLM_PCH_PWM_ENABLE;
+                       tmp &= ~BLM_PCH_OVERRIDE_ENABLE;
+                       I915_WRITE(BLC_PWM_PCH_CTL1, tmp);
+               }
        }
 }