On Thu, Jul 24, 2014 at 11:56:32AM +0200, Marc Kleine-Budde wrote: > >> @@ -696,6 +696,44 @@ static int imx_drm_platform_remove(struct platform_device *pdev) > >> return 0; > >> } > >> > >> +#if CONFIG_PM_SLEEP > > > > use #ifdef > > ...or remove #if/#ifdef and mark as __maybe_unused I personally do not like the idea of __maybe_unused. Will there be a compile error if I call some helper functions which are only available with CONFIG_PM_SLEEP? Shawn _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel