Re: [PATCH v2 0/5] Simplify panel bridge cleanup

[Daniel Vetter]

On Mon, Oct 02, 2017 at 11:32:30AM +0200, Benjamin Gaignard wrote:
> The goal of this series is to simplify driver code when they need to clean up
> a previously allocated panel bridge.
> Few drivers have "is_panel_bridge" flag to be able to distinguish a
> drm_panel_bridge from "simple" drm_bridge.
> To remove this flag I propose to
> - let drm_panel_bridge_remove() check if the bridge provided in parameter is
>   really a drm_panel_bridge.
> - add drm_of_panel_bridge_remove() to remove a bridge given DT port and
>   endpoint
> Finally that allow to remove drm_bridge structure and "is_panel_bridge" flag
> from stm driver internal structure.
> 
> version 2:
> - does the same for vc4 and dw-mipi-dsi

Sweet. Ack on the entire series from my side. Since you have commit rights
I'd wait a bit more for feedback from driver maintainers, then push the
entire pile.

Cheers, Daniel

> 
> Benjamin Gaignard (5):
>   drm/bridge: make drm_panel_bridge_remove more robust
>   drm/drm_of: add drm_of_panel_bridge_remove function
>   drm/stm: ltdc: remove bridge from driver internal structure
>   drm/vc4: remove bridge from driver internal structure
>   drm/bridge/synopsys: dsi :remove is_panel_bridge
> 
>  drivers/gpu/drm/bridge/panel.c                | 10 +++++++-
>  drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c |  5 +---
>  drivers/gpu/drm/drm_of.c                      | 33 +++++++++++++++++++++++++++
>  drivers/gpu/drm/stm/ltdc.c                    | 16 ++++---------
>  drivers/gpu/drm/stm/ltdc.h                    |  2 --
>  drivers/gpu/drm/vc4/vc4_dpi.c                 | 17 +++++---------
>  include/drm/drm_of.h                          |  8 +++++++
>  7 files changed, 62 insertions(+), 29 deletions(-)
> 
> -- 
> 2.7.4
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch