From mboxrd@z Thu Jan 1 00:00:00 1970 From: Your Name Subject: [PATCH v4 04/12] drm: bridge: analogix: Destroy connector & encoder when unbinding Date: Thu, 6 Apr 2017 20:22:15 +0800 Message-ID: <1491481343-11025-5-git-send-email-you@example.com> References: <1491481343-11025-1-git-send-email-you@example.com> Return-path: In-Reply-To: <1491481343-11025-1-git-send-email-you@example.com> Sender: linux-kernel-owner@vger.kernel.org To: linux-kernel@vger.kernel.org Cc: briannorris@chromium.org, dianders@chromium.org, tfiga@chromium.org, seanpaul@chromium.org, zyw@rock-chips.com, mark.yao@rock-chips.com, Jeffy Chen , Your Name , Marek Szyprowski , Yakir Yang , Tomeu Vizoso , dri-devel@lists.freedesktop.org, David Airlie , Archit Taneja List-Id: dri-devel@lists.freedesktop.org From: Jeffy Chen Normally we do this in drm_mode_config_cleanup. But: 1/ analogix dp's connector is allocated in bind, and freed after unbind. So we need to destroy it in unbind to avoid further access. 2/ the drm bridge is attached in bind, and detached in encoder cleanup. So we need to destroy encoder in unbind. Signed-off-by: Jeffy Chen Signed-off-by: Your Name --- Changes in v4: Address Andrzej Hajda 's comments. Changes in v3: None Changes in v2: None drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c index d05ade4..4c758ed 100644 --- a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c +++ b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c @@ -1439,6 +1439,8 @@ void analogix_dp_unbind(struct device *dev, struct device *master, struct analogix_dp_device *dp = dev_get_drvdata(dev); analogix_dp_bridge_disable(dp->bridge); + dp->connector.funcs->destroy(&dp->connector); + dp->encoder->funcs->destroy(dp->encoder); if (dp->plat_data->panel) { if (drm_panel_unprepare(dp->plat_data->panel)) -- 2.1.4