diff options
author | Liu Ying <victor.liu@nxp.com> | 2018-11-28 16:54:08 +0800 |
---|---|---|
committer | Bryan O'Donoghue <bryan.odonoghue@linaro.org> | 2019-05-01 17:43:28 +0100 |
commit | 271bc9444d036ac1b7cc3eadc14697a1ca4a27b6 (patch) | |
tree | 75b779043f939e62454d3fa8b72f564dbe1ff16c | |
parent | 045f717b4d27566246e6db29322679972321b2bc (diff) |
MLK-20486 drm/imx: ldb: Cleanup enc&conn in ->unbind() if necessary
When the master imx-drm-core binding fails, component_bind_all()
in imx-drm-core ->bind() callback will unbind all bound components
first and then call drm_mode_config_cleanup(). Since the encoder
and connector(located in imx_ldb.imx_ldb_channel) are freed after
the ldb ->unbind() callback, drm_mode_config_cleanup() would
accidentally access the freed encoder and connector again. To fix
this issue, we should cleanup the encoder and connector, i.e.,
remove them from the global encoder and connector lists, in the
->unbind() callback, so that, drm_mode_config_cleanup() won't find
them again in the lists. However, we have to make sure they exist
before the cleanup in the ->unbind() callback, because imx-drm-core
->unbind() calls drm_mode_config_cleanup() first and then unbinds
all components via component_unbind_all(). Moreover, the connector
isn't created at the first place if a bridge exists, so the check
before the cleanup makes sense for this reason as well.
Signed-off-by: Liu Ying <victor.liu@nxp.com>
(cherry picked from commit 6d3fd1316d67ff0e62fae4067b61fd3f2809dfa5)
-rw-r--r-- | drivers/gpu/drm/imx/imx-ldb.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/drivers/gpu/drm/imx/imx-ldb.c b/drivers/gpu/drm/imx/imx-ldb.c index 02ea203d4ce8..5027efd3d8b1 100644 --- a/drivers/gpu/drm/imx/imx-ldb.c +++ b/drivers/gpu/drm/imx/imx-ldb.c @@ -1491,6 +1491,14 @@ static void imx_ldb_unbind(struct device *dev, struct device *master, if (channel->panel) drm_panel_detach(channel->panel); + /* make sure the connector exists, and then cleanup */ + if (channel->connector.dev) + imx_drm_connector_destroy(&channel->connector); + + /* make sure the encoder exists, and then cleanup */ + if (channel->encoder.dev) + imx_drm_encoder_destroy(&channel->encoder); + kfree(channel->edid); i2c_put_adapter(channel->ddc); } |