On Sat  7.Aug'10 at 11:01:35 +0100, Chris Wilson wrote:
>
> v2: Upgrade failure messages to DRM_ERROR on the suggestion of
> Eric Anholt. We do not expect to hit these conditions as they reflect
> programming errors, so if we do we want to be notified.

My dmesg from 2.6.36-rc5 is showing one of the messages introduced
in this patch. So I am notifying it :-)

> --- a/drivers/gpu/drm/i915/intel_lvds.c
> +++ b/drivers/gpu/drm/i915/intel_lvds.c
> @@ -96,7 +96,7 @@ static u32 intel_lvds_get_max_backlight(struct drm_device 
> *dev)
>  static void intel_lvds_set_power(struct drm_device *dev, bool on)
>  {
>       struct drm_i915_private *dev_priv = dev->dev_private;
> -     u32 pp_status, ctl_reg, status_reg, lvds_reg;
> +     u32 ctl_reg, status_reg, lvds_reg;
>  
>       if (HAS_PCH_SPLIT(dev)) {
>               ctl_reg = PCH_PP_CONTROL;
> @@ -114,9 +114,8 @@ static void intel_lvds_set_power(struct drm_device *dev, 
> bool on)
>  
>               I915_WRITE(ctl_reg, I915_READ(ctl_reg) |
>                          POWER_TARGET_ON);
> -             do {
> -                     pp_status = I915_READ(status_reg);
> -             } while ((pp_status & PP_ON) == 0);
> +             if (wait_for(I915_READ(status_reg) & PP_ON, 1000, 0))
> +                     DRM_ERROR("timed out waiting to enable LVDS pipe");


[    0.954724] ACPI: Battery Slot [BAT0] (battery present)
[    2.266008] [drm:intel_lvds_set_power] *ERROR* timed out waiting to enable 
LVDS pipeConsole: switching to colour frame buffer device 160x50
[    2.274860] fb0: inteldrmfb frame buffer device

But I have this 1+ sec gap in there for as long as I remember.

Full dmesg is here:
http://www.aei.mpg.de/~crmafra/dmesg-2.6.36-rc5.txt

Is there anything else I can provide to help this?
I would love to get rid of the 1 sec delay there :-)
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to