Re: [PATCH 2/2] drm/i915: eDP DRRS limited to only one panel at a time

From: Sivakumar Thulasimani <sivakumar.thulasimani@intel.com>
To: Ramalingam C <ramalingam.c@intel.com>,
	intel-gfx@lists.freedesktop.org, rodrigo.vivi@intel.com,
	daniel@ffwll.ch
Cc: paulo.r.zanoni@intel.com
Subject: Re: [PATCH 2/2] drm/i915: eDP DRRS limited to only one panel at a time
Date: Tue, 24 Mar 2015 07:33:40 +0530	[thread overview]
Message-ID: <5510C5FC.2080906@intel.com> (raw)
In-Reply-To: <1427132524-27829-3-git-send-email-ramalingam.c@intel.com>

since drrs struct is used primarily in intel_dp.c file alone can it be 
moved to intel_dp ?

On 3/23/2015 11:12 PM, Ramalingam C wrote:
> In case of multiple eDP panels, only one can have
> the DRRS enabled on it.
>
> In future eDP DRRS will be extended for multiple panels.
>
> Signed-off-by: Ramalingam C <ramalingam.c@intel.com>
> ---
>   drivers/gpu/drm/i915/intel_dp.c |    5 +++++
>   1 file changed, 5 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
> index 0b26df9..ec40d19 100644
> --- a/drivers/gpu/drm/i915/intel_dp.c
> +++ b/drivers/gpu/drm/i915/intel_dp.c
> @@ -5311,6 +5311,11 @@ intel_dp_drrs_init(struct intel_connector *intel_connector,
>   		return NULL;
>   	}
>   
> +	if (dev_priv->drrs.type == SEAMLESS_DRRS_SUPPORT) {
> +		DRM_DEBUG_KMS("DRRS already enabled on previous connector\n");
> +		return NULL;
> +	}
> +
>   	INIT_DELAYED_WORK(&dev_priv->drrs.work, intel_edp_drrs_downclock_work);
>   
>   	mutex_init(&dev_priv->drrs.mutex);

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