[Intel-gfx] [PATCH v4] drm/i915: rename i915_init_power_well to init_power_domains_init
Paulo Zanoni
przanoni at gmail.com
Mon Oct 28 19:51:52 CET 2013
2013/10/28 Imre Deak <imre.deak at intel.com>:
> Similarly rename the other related functions in the power domain
> interface.
>
> Higher level driver code calling these functions knows only about power
> domains, not the underlying power wells which may be different on
> different platforms. Also these functions really init/cleanup/resume
> power domains and only through that all related power wells, so rename
> them accordingly.
>
> Note that I left i915_{request,release}_power_well as is, since that
> really changes the state only of a single power well (and is HSW
> specific). It should also get a better name once we make it more
> generic by controlling things through a new audio power domain.
>
> v4:
> - use intel prefix instead of i915 everywhere (Paulo)
> - use a $prefix_$block_$action format (Daniel)
Reviewed-by: Paulo Zanoni <paulo.r.zanoni at intel.com>
>
> Signed-off-by: Imre Deak <imre.deak at intel.com>
> ---
> drivers/gpu/drm/i915/i915_dma.c | 8 ++++----
> drivers/gpu/drm/i915/i915_drv.c | 2 +-
> drivers/gpu/drm/i915/intel_drv.h | 6 +++---
> drivers/gpu/drm/i915/intel_pm.c | 10 +++++-----
> 4 files changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_dma.c b/drivers/gpu/drm/i915/i915_dma.c
> index b722b35..0cab2d0 100644
> --- a/drivers/gpu/drm/i915/i915_dma.c
> +++ b/drivers/gpu/drm/i915/i915_dma.c
> @@ -1311,7 +1311,7 @@ static int i915_load_modeset_init(struct drm_device *dev)
> if (ret)
> goto cleanup_gem_stolen;
>
> - intel_init_power_well(dev);
> + intel_power_domains_init_hw(dev);
>
> /* Important: The output setup functions called by modeset_init need
> * working irqs for e.g. gmbus and dp aux transfers. */
> @@ -1640,7 +1640,7 @@ int i915_driver_load(struct drm_device *dev, unsigned long flags)
> }
>
> if (HAS_POWER_WELL(dev))
> - i915_init_power_well(dev);
> + intel_power_domains_init(dev);
>
> if (drm_core_check_feature(dev, DRIVER_MODESET)) {
> ret = i915_load_modeset_init(dev);
> @@ -1668,7 +1668,7 @@ int i915_driver_load(struct drm_device *dev, unsigned long flags)
>
> out_power_well:
> if (HAS_POWER_WELL(dev))
> - i915_remove_power_well(dev);
> + intel_power_domains_remove(dev);
> drm_vblank_cleanup(dev);
> out_gem_unload:
> if (dev_priv->mm.inactive_shrinker.scan_objects)
> @@ -1711,7 +1711,7 @@ int i915_driver_unload(struct drm_device *dev)
> * the power well is not enabled, so just enable it in case
> * we're going to unload/reload. */
> intel_display_set_init_power(dev, true);
> - i915_remove_power_well(dev);
> + intel_power_domains_remove(dev);
> }
>
> i915_teardown_sysfs(dev);
> diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
> index 770c9f8..a0804fa 100644
> --- a/drivers/gpu/drm/i915/i915_drv.c
> +++ b/drivers/gpu/drm/i915/i915_drv.c
> @@ -597,7 +597,7 @@ static int __i915_drm_thaw(struct drm_device *dev, bool restore_gtt_mappings)
> mutex_unlock(&dev->struct_mutex);
> }
>
> - intel_init_power_well(dev);
> + intel_power_domains_init_hw(dev);
>
> i915_restore_state(dev);
> intel_opregion_setup(dev);
> diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h
> index bf4394a..9d2624f 100644
> --- a/drivers/gpu/drm/i915/intel_drv.h
> +++ b/drivers/gpu/drm/i915/intel_drv.h
> @@ -823,15 +823,15 @@ bool intel_fbc_enabled(struct drm_device *dev);
> void intel_update_fbc(struct drm_device *dev);
> void intel_gpu_ips_init(struct drm_i915_private *dev_priv);
> void intel_gpu_ips_teardown(void);
> -int i915_init_power_well(struct drm_device *dev);
> -void i915_remove_power_well(struct drm_device *dev);
> +int intel_power_domains_init(struct drm_device *dev);
> +void intel_power_domains_remove(struct drm_device *dev);
> bool intel_display_power_enabled(struct drm_device *dev,
> enum intel_display_power_domain domain);
> void intel_display_power_get(struct drm_device *dev,
> enum intel_display_power_domain domain);
> void intel_display_power_put(struct drm_device *dev,
> enum intel_display_power_domain domain);
> -void intel_init_power_well(struct drm_device *dev);
> +void intel_power_domains_init_hw(struct drm_device *dev);
> void intel_set_power_well(struct drm_device *dev, bool enable);
> void intel_enable_gt_powersave(struct drm_device *dev);
> void intel_disable_gt_powersave(struct drm_device *dev);
> diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
> index d2a640c..a0c907f 100644
> --- a/drivers/gpu/drm/i915/intel_pm.c
> +++ b/drivers/gpu/drm/i915/intel_pm.c
> @@ -5677,7 +5677,7 @@ void i915_release_power_well(void)
> }
> EXPORT_SYMBOL_GPL(i915_release_power_well);
>
> -int i915_init_power_well(struct drm_device *dev)
> +int intel_power_domains_init(struct drm_device *dev)
> {
> struct drm_i915_private *dev_priv = dev->dev_private;
> struct i915_power_domains *power_domains = &dev_priv->power_domains;
> @@ -5692,12 +5692,12 @@ int i915_init_power_well(struct drm_device *dev)
> return 0;
> }
>
> -void i915_remove_power_well(struct drm_device *dev)
> +void intel_power_domains_remove(struct drm_device *dev)
> {
> hsw_pwr = NULL;
> }
>
> -static void intel_resume_power_well(struct drm_device *dev)
> +static void intel_power_domains_resume(struct drm_device *dev)
> {
> struct drm_i915_private *dev_priv = dev->dev_private;
> struct i915_power_domains *power_domains = &dev_priv->power_domains;
> @@ -5720,7 +5720,7 @@ static void intel_resume_power_well(struct drm_device *dev)
> * to be enabled, and it will only be disabled if none of the registers is
> * requesting it to be enabled.
> */
> -void intel_init_power_well(struct drm_device *dev)
> +void intel_power_domains_init_hw(struct drm_device *dev)
> {
> struct drm_i915_private *dev_priv = dev->dev_private;
>
> @@ -5729,7 +5729,7 @@ void intel_init_power_well(struct drm_device *dev)
>
> /* For now, we need the power well to be always enabled. */
> intel_display_set_init_power(dev, true);
> - intel_resume_power_well(dev);
> + intel_power_domains_resume(dev);
>
> /* We're taking over the BIOS, so clear any requests made by it since
> * the driver is in charge now. */
> --
> 1.8.4
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Paulo Zanoni
More information about the Intel-gfx
mailing list