Re: [PATCH] drm/kms: fix fbdev blanking regression
From: James Simmons
Date: Wed Jan 06 2010 - 12:33:43 EST
> Commit 731b5a15a3b1474a41c2ca29b4c32b0f21bc852e (drm/kms: properly
> handle fbdev blanking) breaks console blanking on my laptop (GM45
> chipset). Instead of blanking the screen, it is dimmed and then the
> backlight starts "glowing" in the lower edges of the screen. The glowing
> then slowly spreads upwards. Does not look healthy at all...
>
> This patch reverts to the old behaviour of going directly to
> DPMS_STANDBY.
The current fbdev blank is correct. Normal blanking doesn't power down
the hsync or the vsync signals. If you look at fbdev drivers you will see
this type of behavior. If you want a full power down then a FB_BLANK_POWERDOWN
is needed. As for the blacklight that is a another problem not related to
this issue. The fbdev layer sends a event to the backlight layer to tell
it to power down. That powerdown happens for the backlight in all cases
except FB_BLANK_UNBLANK. How is your backlight setup. Is the backlight
handled via the DRI driver or acpi?
> Signed-off-by: Johan Hovold <jhovold@xxxxxxxxx>
> ---
> drivers/gpu/drm/drm_fb_helper.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c
> index 1b49fa0..9008583 100644
> --- a/drivers/gpu/drm/drm_fb_helper.c
> +++ b/drivers/gpu/drm/drm_fb_helper.c
> @@ -389,7 +389,7 @@ int drm_fb_helper_blank(int blank, struct fb_info *info)
> break;
> /* Display: Off; HSync: On, VSync: On */
> case FB_BLANK_NORMAL:
> - drm_fb_helper_off(info, DRM_MODE_DPMS_ON);
> + drm_fb_helper_off(info, DRM_MODE_DPMS_STANDBY);
> break;
> /* Display: Off; HSync: Off, VSync: On */
> case FB_BLANK_HSYNC_SUSPEND:
> --
> 1.6.6
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/