From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756018AbdJJDhD (ORCPT ); Mon, 9 Oct 2017 23:37:03 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:44278 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754906AbdJJDhC (ORCPT ); Mon, 9 Oct 2017 23:37:02 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 43AAA6073E Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=architt@codeaurora.org Subject: Re: [PATCH v2 0/5] Simplify panel bridge cleanup To: Benjamin Gaignard , a.hajda@samsung.com, Laurent.pinchart@ideasonboard.com, airlied@linux.ie, daniel.vetter@intel.com, jani.nikula@linux.intel.com, seanpaul@chromium.org, yannick.fertre@st.com, philippe.cornu@st.com Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org References: <1506936755-23625-1-git-send-email-benjamin.gaignard@linaro.org> From: Archit Taneja Message-ID: Date: Tue, 10 Oct 2017 09:06:53 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <1506936755-23625-1-git-send-email-benjamin.gaignard@linaro.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.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 Feel free to queue to drm-misc-next. 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