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

[Benjamin Gaignard]

2017-10-10 5:36 GMT+02:00 Archit Taneja <architt@xxxxxxxxxxxxxx>:
>
>
> On 10/02/2017 03:02 PM, 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
>
> For the series:
>
> Reviewed-by: Archit Taneja <architt@xxxxxxxxxxxxxx>
>
> Feel free to queue to drm-misc-next.

Pushed thanks

>
> Thanks,
> Archit
>
>>
>> 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(-)
>>
>
> --
> Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
> a Linux Foundation Collaborative Project



-- 
Benjamin Gaignard

Graphic Study Group

Linaro.org â Open source software for ARM SoCs

Follow Linaro: Facebook | Twitter | Blog