diff options
| author | Paulo Zanoni <paulo.r.zanoni@intel.com> | 2018-08-20 16:31:37 -0700 |
|---|---|---|
| committer | Paulo Zanoni <paulo.r.zanoni@intel.com> | 2018-08-24 12:49:07 -0700 |
| commit | 0229bfd42bbe7fd49d23b5f696c22241096b8847 (patch) | |
| tree | 647557e863c627486368dbac807d366f38f0bf08 /drivers/gpu/drm/i915/intel_runtime_pm.c | |
| parent | 99da0b35396f3907fa8594b554bf81904389c48c (diff) | |
| download | linux-stable-0229bfd42bbe7fd49d23b5f696c22241096b8847.tar.gz linux-stable-0229bfd42bbe7fd49d23b5f696c22241096b8847.tar.bz2 linux-stable-0229bfd42bbe7fd49d23b5f696c22241096b8847.zip | |
drm/i915: use for_each_power_well in lookup_power_well()
Use the nice helper function to make the implementation simpler.
v2: Rebase.
Cc: Imre Deak <imre.deak@intel.com>
Reviewed-by: José Roberto de Souza <jose.souza@intel.com> (v1)
Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180820233139.11936-3-paulo.r.zanoni@intel.com
Diffstat (limited to 'drivers/gpu/drm/i915/intel_runtime_pm.c')
| -rw-r--r-- | drivers/gpu/drm/i915/intel_runtime_pm.c | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/drivers/gpu/drm/i915/intel_runtime_pm.c b/drivers/gpu/drm/i915/intel_runtime_pm.c index dd5d1ba4a7f1..33dbbd9efe65 100644 --- a/drivers/gpu/drm/i915/intel_runtime_pm.c +++ b/drivers/gpu/drm/i915/intel_runtime_pm.c @@ -1087,16 +1087,11 @@ static struct i915_power_well * lookup_power_well(struct drm_i915_private *dev_priv, enum i915_power_well_id power_well_id) { - struct i915_power_domains *power_domains = &dev_priv->power_domains; - int i; - - for (i = 0; i < power_domains->power_well_count; i++) { - struct i915_power_well *power_well; + struct i915_power_well *power_well; - power_well = &power_domains->power_wells[i]; + for_each_power_well(dev_priv, power_well) if (power_well->desc->id == power_well_id) return power_well; - } /* * It's not feasible to add error checking code to the callers since @@ -1106,7 +1101,7 @@ lookup_power_well(struct drm_i915_private *dev_priv, * our driver. */ WARN(1, "Power well %d not defined for this platform\n", power_well_id); - return &power_domains->power_wells[0]; + return &dev_priv->power_domains.power_wells[0]; } #define BITS_SET(val, bits) (((val) & (bits)) == (bits)) |