Re: [Intel-gfx] [PATCH] drm/i915/dsi: Dont forget to clean up the connector on error (v2)

2020-06-02 Thread Souza, Jose
On Fri, 2020-05-22 at 13:26 -0700, Vivek Kasireddy wrote:
> If an error is encountered during the DSI initialization setup, the
> drm connector object also needs to be cleaned up along with the encoder.
> The error can happen due to a missing mode in the VBT or for other
> reasons.
> 
> v2: Rephrase the commit message to make it more clear.
> 

Reviewed-by: José Roberto de Souza 

> Cc: Jani Nikula 
> Cc: Vandita Kulkarni 
> Signed-off-by: Vivek Kasireddy 
> ---
>  drivers/gpu/drm/i915/display/icl_dsi.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/i915/display/icl_dsi.c 
> b/drivers/gpu/drm/i915/display/icl_dsi.c
> index 4fec5bd64920..f93f72463df5 100644
> --- a/drivers/gpu/drm/i915/display/icl_dsi.c
> +++ b/drivers/gpu/drm/i915/display/icl_dsi.c
> @@ -1954,6 +1954,7 @@ void icl_dsi_init(struct drm_i915_private *dev_priv)
>   return;
>  
>  err:
> + drm_connector_cleanup(connector);
>   drm_encoder_cleanup(>base);
>   kfree(intel_dsi);
>   kfree(intel_connector);
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx


[Intel-gfx] [PATCH] drm/i915/dsi: Dont forget to clean up the connector on error (v2)

2020-05-22 Thread Vivek Kasireddy
If an error is encountered during the DSI initialization setup, the
drm connector object also needs to be cleaned up along with the encoder.
The error can happen due to a missing mode in the VBT or for other
reasons.

v2: Rephrase the commit message to make it more clear.

Cc: Jani Nikula 
Cc: Vandita Kulkarni 
Signed-off-by: Vivek Kasireddy 
---
 drivers/gpu/drm/i915/display/icl_dsi.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/i915/display/icl_dsi.c 
b/drivers/gpu/drm/i915/display/icl_dsi.c
index 4fec5bd64920..f93f72463df5 100644
--- a/drivers/gpu/drm/i915/display/icl_dsi.c
+++ b/drivers/gpu/drm/i915/display/icl_dsi.c
@@ -1954,6 +1954,7 @@ void icl_dsi_init(struct drm_i915_private *dev_priv)
return;
 
 err:
+   drm_connector_cleanup(connector);
drm_encoder_cleanup(>base);
kfree(intel_dsi);
kfree(intel_connector);
-- 
2.21.1

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