[Intel-gfx] [PATCH 05/15] drm/i915: Add locking around chv_phy_control_init()

[Deepak]

On 07/09/2015 02:15 AM, ville.syrjala at linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
>
> dev_priv->chv_phy_control is protected by the power_domains->lock
> elsewhere, so also grab it when initializing chv_phy_control.
>
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> ---
>   drivers/gpu/drm/i915/intel_runtime_pm.c | 2 ++
>   1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/intel_runtime_pm.c b/drivers/gpu/drm/i915/intel_runtime_pm.c
> index 2142ae6..dab1da9 100644
> --- a/drivers/gpu/drm/i915/intel_runtime_pm.c
> +++ b/drivers/gpu/drm/i915/intel_runtime_pm.c
> @@ -1678,7 +1678,9 @@ void intel_power_domains_init_hw(struct drm_i915_private *dev_priv)
>   	power_domains->initializing = true;
>   
>   	if (IS_CHERRYVIEW(dev)) {
> +		mutex_lock(&power_domains->lock);
>   		chv_phy_control_init(dev_priv);
> +		mutex_unlock(&power_domains->lock);
>   	} else if (IS_VALLEYVIEW(dev)) {
>   		mutex_lock(&power_domains->lock);
>   		vlv_cmnlane_wa(dev_priv);
Looks fine to me :)
Reviewed-by: Deepak S <deepak.s at linux.intel.com>