drm/bridge/synopsys: stop clobbering drvdata
Bridge drivers/helpers shouldn't be clobbering the drvdata, since a parent driver might need to own this. Instead, let's return our 'dw_mipi_dsi' object and have callers pass that back to us for removal. Signed-off-by: Brian Norris <briannorris@chromium.org> Reviewed-by: Matthias Kaehlcke <mka@chromium.org> Reviewed-by: Archit Taneja <architt@codeaurora.org> Acked-by: Philippe Cornu <philippe.cornu@st.com> Signed-off-by: Andrzej Hajda <a.hajda@samsung.com> Link: https://patchwork.freedesktop.org/patch/msgid/20171128010538.119114-1-briannorris@chromium.org
此提交包含在:
@@ -65,6 +65,7 @@ enum dsi_color {
|
||||
struct dw_mipi_dsi_stm {
|
||||
void __iomem *base;
|
||||
struct clk *pllref_clk;
|
||||
struct dw_mipi_dsi *dsi;
|
||||
};
|
||||
|
||||
static inline void dsi_write(struct dw_mipi_dsi_stm *dsi, u32 reg, u32 val)
|
||||
@@ -312,21 +313,24 @@ static int dw_mipi_dsi_stm_probe(struct platform_device *pdev)
|
||||
dw_mipi_dsi_stm_plat_data.base = dsi->base;
|
||||
dw_mipi_dsi_stm_plat_data.priv_data = dsi;
|
||||
|
||||
ret = dw_mipi_dsi_probe(pdev, &dw_mipi_dsi_stm_plat_data);
|
||||
if (ret) {
|
||||
platform_set_drvdata(pdev, dsi);
|
||||
|
||||
dsi->dsi = dw_mipi_dsi_probe(pdev, &dw_mipi_dsi_stm_plat_data);
|
||||
if (IS_ERR(dsi->dsi)) {
|
||||
DRM_ERROR("Failed to initialize mipi dsi host\n");
|
||||
clk_disable_unprepare(dsi->pllref_clk);
|
||||
return PTR_ERR(dsi->dsi);
|
||||
}
|
||||
|
||||
return ret;
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int dw_mipi_dsi_stm_remove(struct platform_device *pdev)
|
||||
{
|
||||
struct dw_mipi_dsi_stm *dsi = dw_mipi_dsi_stm_plat_data.priv_data;
|
||||
struct dw_mipi_dsi_stm *dsi = platform_get_drvdata(pdev);
|
||||
|
||||
clk_disable_unprepare(dsi->pllref_clk);
|
||||
dw_mipi_dsi_remove(pdev);
|
||||
dw_mipi_dsi_remove(dsi->dsi);
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
新增問題並參考
封鎖使用者