[Intel-gfx] [PATCH v2 06/11] drm/i915: Name i915_runtime_pm structure in dev_priv as "rpm"

[Sagar Arun Kamble]

On 10/6/2017 6:10 PM, Chris Wilson wrote:
> Quoting Sagar Arun Kamble (2017-10-06 13:13:35)
>> We were using dev_priv->pm for runtime power management related state.
>> This patch renames it to "rpm" which looks more apt. Will be using pm
>> for state containing RPS/RC6 state in the next patch.
>>
>> Signed-off-by: Sagar Arun Kamble <sagar.a.kamble at intel.com>
>> Cc: Imre Deak <imre.deak at intel.com>
>> Cc: Chris Wilson <chris at chris-wilson.co.uk>
>> Cc: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
>> Reviewed-by: Radoslaw Szwichtenberg <radoslaw.szwichtenberg at intel.com>
>> Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
> Thinking about this again, rpm, pm are very close. How about if we used
> i915->runtime_pm and i915->gt_pm (or i915->gt.pm)? Imre, any thoughts?
rps.hw_lock/pcu_lock is used by display too, so I just kept it pm. 
should we pull rps.hw_lock/pcu_lock out into drm_i915_private
and then gt_pm would be good.
> -Chris