Re: [PATCH v4] drm/i915: rename i915_init_power_well to init_power_domains_init

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2013/10/28 Imre Deak <imre.deak@xxxxxxxxx>:
> 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@xxxxxxxxx>

>
> Signed-off-by: Imre Deak <imre.deak@xxxxxxxxx>
> ---
>  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@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Paulo Zanoni
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux