[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Tue, 10 Oct 2017 12:03:56 +0200
From: Benjamin Gaignard <benjamin.gaignard@...aro.org>
To: Archit Taneja <architt@...eaurora.org>
Cc: Andrzej Hajda <a.hajda@...sung.com>,
Laurent Pinchart <Laurent.pinchart@...asonboard.com>,
David Airlie <airlied@...ux.ie>,
Daniel Vetter <daniel.vetter@...el.com>,
Jani Nikula <jani.nikula@...ux.intel.com>,
Sean Paul <seanpaul@...omium.org>,
Yannick Fertre <yannick.fertre@...com>,
Philippe Cornu <philippe.cornu@...com>,
"dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 0/5] Simplify panel bridge cleanup
2017-10-10 5:36 GMT+02:00 Archit Taneja <architt@...eaurora.org>:
>
>
> 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@...eaurora.org>
>
> 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
Powered by blists - more mailing lists