Hi Dave, One for you? Begin forwarded message: Date: Wed, 31 Mar 2010 12:18:00 +0800 From: Changli Gao <xiaosuo@xxxxxxxxx> To: linux-next@xxxxxxxxxxxxxxx Cc: xiaosuo <xiaosuo@xxxxxxxxx> Subject: [PATCH] drm: build error when magic sysrq isn't enabled drm build error when magic sysrq isn't enabled sysrq_drm_fb_helper_restore_op is only valid when magic sysrq is enabled, but it is used whenever magic sysrq is enabled or not. Signed-off-by: Changli Gao <xiaosuo@xxxxxxxxx> ---- drivers/gpu/drm/drm_fb_helper.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c index 85cdf05..0e5b4f6 100644 --- a/drivers/gpu/drm/drm_fb_helper.c +++ b/drivers/gpu/drm/drm_fb_helper.c @@ -913,7 +913,9 @@ int drm_fb_helper_single_fb_probe(struct drm_device *dev, printk(KERN_INFO "registered panic notifier\n"); atomic_notifier_chain_register(&panic_notifier_list, &paniced); +#ifdef CONFIG_MAGIC_SYSRQ register_sysrq_key('v', &sysrq_drm_fb_helper_restore_op); +#endif } list_add(&fb_helper->kernel_fb_list, &kernel_fb_helper_list); return 0; @@ -927,7 +929,9 @@ void drm_fb_helper_free(struct drm_fb_helper *helper) printk(KERN_INFO "unregistered panic notifier\n"); atomic_notifier_chain_unregister(&panic_notifier_list, &paniced); +#ifdef CONFIG_MAGIC_SYSRQ unregister_sysrq_key('v', &sysrq_drm_fb_helper_restore_op); +#endif } drm_fb_helper_crtc_free(helper); fb_dealloc_cmap(&helper->fb->fbdev->cmap); -- Cheers, Stephen Rothwell sfr@xxxxxxxxxxxxxxxx http://www.canb.auug.org.au/~sfr/ -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html