On Sun, 1 Jul 2012 11:39:40 +0800
Hunt Xu <mhuntxu at gmail.com> wrote:
> Commit 0136db586c028f71e7cc21cc183064ff0d5919c8 merges rc6 information
> into the power group. However, when compiled with CONFIG_PM not set,
> modprobing i915 would taint since power_group_name is defined as NULL.
>
> This patch makes these rc6 in sysfs functions conditional upon the
> definition of the CONFIG_PM macro to avoid the above-mentioned problem.
>
Thanks for the patch, the only problem is we want this information even
when CONFIG_PM is not set as we currently don't control our rc6 state
based on that. Unfortunately it also doesn't make too much sense to have
CONFIG_PM control the rc6 info.
Do you have any other ideas to make this work?
> Signed-off-by: Hunt Xu <mhuntxu+intelgfx at gmail.com>
> ---y,
> drivers/gpu/drm/i915/i915_sysfs.c | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/i915_sysfs.c b/drivers/gpu/drm/i915/i915_sysfs.c
> index 79f8344..6d4b001 100644
> --- a/drivers/gpu/drm/i915/i915_sysfs.c
> +++ b/drivers/gpu/drm/i915/i915_sysfs.c
> @@ -31,6 +31,7 @@
> #include <linux/sysfs.h>
> #include "i915_drv.h"
>
> +#ifdef CONFIG_PM
> static u32 calc_residency(struct drm_device *dev, const u32 reg)
> {
> struct drm_i915_private *dev_priv = dev->dev_private;
> @@ -109,3 +110,14 @@ void i915_teardown_sysfs(struct drm_device *dev)
> {
> sysfs_unmerge_group(&dev->primary->kdev.kobj, &rc6_attr_group);
> }
> +#else
> +void i915_setup_sysfs(struct drm_device *dev)
> +{
> + return;
> +}
> +
> +void i915_teardown_sysfs(struct drm_device *dev)
> +{
> + return;
> +}
> +#endif /* CONFIG_PM */
--
Ben Widawsky, Intel Open Source Technology Center
