[Intel-gfx] [PATCH 4/5] drm/i915/icl: Skip init for an already enabled combo PHY

2018-11-02 Thread Imre Deak 
Bspec says we should skip the initialization of combo PHYs that are
already initialized. We'll need to reinit the PHYs more frequently
when exiting from DC6 (after the next patch), so let's make sure the
uninit sequence complies with the spec. For safety skip the init only if
all the PHY register fields have their expected values.

Bspec: 21257
Cc: Paulo Zanoni 
Cc: Ville Syrjälä 
Cc: José Roberto de Souza 
Cc: Rodrigo Vivi 
Signed-off-by: Imre Deak 
---
 drivers/gpu/drm/i915/intel_combo_phy.c | 6 ++
 1 file changed, 6 insertions(+)

diff --git a/drivers/gpu/drm/i915/intel_combo_phy.c 
b/drivers/gpu/drm/i915/intel_combo_phy.c
index 1522e2a25390..7b911702f834 100644
--- a/drivers/gpu/drm/i915/intel_combo_phy.c
+++ b/drivers/gpu/drm/i915/intel_combo_phy.c
@@ -220,6 +220,12 @@ void icl_combo_phys_init(struct drm_i915_private *dev_priv)
for (port = PORT_A; port <= PORT_B; port++) {
u32 val;
 
+   if (icl_combo_phy_verify_state(dev_priv, port)) {
+   DRM_DEBUG_DRIVER("Port-%c combo PHY already enabled, 
won't reprogram it.\n",
+port_name(port));
+   continue;
+   }
+
val = I915_READ(ICL_PHY_MISC(port));
val &= ~ICL_PHY_MISC_DE_IO_COMP_PWR_DOWN;
I915_WRITE(ICL_PHY_MISC(port), val);
-- 
2.13.2

___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [Intel-gfx] [PATCH 4/5] drm/i915/icl: Skip init for an already enabled combo PHY

2018-11-02 Thread Rodrigo Vivi 
On Fri, Nov 02, 2018 at 08:07:05PM +0200, Imre Deak wrote:
> Bspec says we should skip the initialization of combo PHYs that are
> already initialized. We'll need to reinit the PHYs more frequently
> when exiting from DC6 (after the next patch), so let's make sure the
> uninit sequence complies with the spec. For safety skip the init only if
> all the PHY register fields have their expected values.
> 
> Bspec: 21257
> Cc: Paulo Zanoni 
> Cc: Ville Syrjälä 
> Cc: José Roberto de Souza 
> Cc: Rodrigo Vivi 
> Signed-off-by: Imre Deak 
> ---
>  drivers/gpu/drm/i915/intel_combo_phy.c | 6 ++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/intel_combo_phy.c 
> b/drivers/gpu/drm/i915/intel_combo_phy.c
> index 1522e2a25390..7b911702f834 100644
> --- a/drivers/gpu/drm/i915/intel_combo_phy.c
> +++ b/drivers/gpu/drm/i915/intel_combo_phy.c
> @@ -220,6 +220,12 @@ void icl_combo_phys_init(struct drm_i915_private 
> *dev_priv)
>   for (port = PORT_A; port <= PORT_B; port++) {
>   u32 val;
>  
> + if (icl_combo_phy_verify_state(dev_priv, port)) {
> + DRM_DEBUG_DRIVER("Port-%c combo PHY already enabled, 
> won't reprogram it.\n",
> +  port_name(port));

Note since I didn't rv-b the previous patch:
I have checked the previous patch that introduces this and it is indeed correct:

"If PORT_COMP_DW0 Comp Init == 1b, skip the rest of this sequence since it is 
already initialized"

so,

Reviewed-by: Rodrigo Vivi 

> + continue;
> + }
> +
>   val = I915_READ(ICL_PHY_MISC(port));
>   val &= ~ICL_PHY_MISC_DE_IO_COMP_PWR_DOWN;
>   I915_WRITE(ICL_PHY_MISC(port), val);
> -- 
> 2.13.2
> 
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [Intel-gfx] [PATCH 4/5] drm/i915/icl: Skip init for an already enabled combo PHY

2018-11-02 Thread Souza, Jose 
On Fri, 2018-11-02 at 20:07 +0200, Imre Deak wrote:
> Bspec says we should skip the initialization of combo PHYs that are
> already initialized. We'll need to reinit the PHYs more frequently
> when exiting from DC6 (after the next patch), so let's make sure the
> uninit sequence complies with the spec. For safety skip the init only
> if
> all the PHY register fields have their expected values.
> 
> Bspec: 21257
> Cc: Paulo Zanoni 
> Cc: Ville Syrjälä 

Reviewed-by: José Roberto de Souza 

> Cc: José Roberto de Souza 
> Cc: Rodrigo Vivi 
> Signed-off-by: Imre Deak 
> ---
>  drivers/gpu/drm/i915/intel_combo_phy.c | 6 ++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/intel_combo_phy.c
> b/drivers/gpu/drm/i915/intel_combo_phy.c
> index 1522e2a25390..7b911702f834 100644
> --- a/drivers/gpu/drm/i915/intel_combo_phy.c
> +++ b/drivers/gpu/drm/i915/intel_combo_phy.c
> @@ -220,6 +220,12 @@ void icl_combo_phys_init(struct drm_i915_private
> *dev_priv)
>   for (port = PORT_A; port <= PORT_B; port++) {
>   u32 val;
>  
> + if (icl_combo_phy_verify_state(dev_priv, port)) {
> + DRM_DEBUG_DRIVER("Port-%c combo PHY already
> enabled, won't reprogram it.\n",
> +  port_name(port));
> + continue;
> + }
> +
>   val = I915_READ(ICL_PHY_MISC(port));
>   val &= ~ICL_PHY_MISC_DE_IO_COMP_PWR_DOWN;
>   I915_WRITE(ICL_PHY_MISC(port), val);
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx