[Freedreno] [PATCH v4 13/24] drm/bridge: lt9611uxc: Switch to devm MIPI-DSI helpers

[Maxime Ripard]

Let's switch to the new devm MIPI-DSI function to register and attach
our secondary device.

Signed-off-by: Maxime Ripard <max...@cerno.tech>
---
 drivers/gpu/drm/bridge/lontium-lt9611uxc.c | 38 +++++-----------------
 1 file changed, 8 insertions(+), 30 deletions(-)

diff --git a/drivers/gpu/drm/bridge/lontium-lt9611uxc.c 
b/drivers/gpu/drm/bridge/lontium-lt9611uxc.c
index 3cac16db970f..e5083bdf4c89 100644
--- a/drivers/gpu/drm/bridge/lontium-lt9611uxc.c
+++ b/drivers/gpu/drm/bridge/lontium-lt9611uxc.c
@@ -257,17 +257,18 @@ static struct mipi_dsi_device 
*lt9611uxc_attach_dsi(struct lt9611uxc *lt9611uxc,
        const struct mipi_dsi_device_info info = { "lt9611uxc", 0, NULL };
        struct mipi_dsi_device *dsi;
        struct mipi_dsi_host *host;
+       struct device *dev = lt9611uxc->dev;
        int ret;
 
        host = of_find_mipi_dsi_host_by_node(dsi_node);
        if (!host) {
-               dev_err(lt9611uxc->dev, "failed to find dsi host\n");
+               dev_err(dev, "failed to find dsi host\n");
                return ERR_PTR(-EPROBE_DEFER);
        }
 
-       dsi = mipi_dsi_device_register_full(host, &info);
+       dsi = devm_mipi_dsi_device_register_full(dev, host, &info);
        if (IS_ERR(dsi)) {
-               dev_err(lt9611uxc->dev, "failed to create dsi device\n");
+               dev_err(dev, "failed to create dsi device\n");
                return dsi;
        }
 
@@ -276,10 +277,9 @@ static struct mipi_dsi_device *lt9611uxc_attach_dsi(struct 
lt9611uxc *lt9611uxc,
        dsi->mode_flags = MIPI_DSI_MODE_VIDEO | MIPI_DSI_MODE_VIDEO_SYNC_PULSE |
                          MIPI_DSI_MODE_VIDEO_HSE;
 
-       ret = mipi_dsi_attach(dsi);
+       ret = devm_mipi_dsi_attach(dev, dsi);
        if (ret < 0) {
-               dev_err(lt9611uxc->dev, "failed to attach dsi to host\n");
-               mipi_dsi_device_unregister(dsi);
+               dev_err(dev, "failed to attach dsi to host\n");
                return ERR_PTR(ret);
        }
 
@@ -352,19 +352,6 @@ static int lt9611uxc_connector_init(struct drm_bridge 
*bridge, struct lt9611uxc
        return drm_connector_attach_encoder(&lt9611uxc->connector, 
bridge->encoder);
 }
 
-static void lt9611uxc_bridge_detach(struct drm_bridge *bridge)
-{
-       struct lt9611uxc *lt9611uxc = bridge_to_lt9611uxc(bridge);
-
-       if (lt9611uxc->dsi1) {
-               mipi_dsi_detach(lt9611uxc->dsi1);
-               mipi_dsi_device_unregister(lt9611uxc->dsi1);
-       }
-
-       mipi_dsi_detach(lt9611uxc->dsi0);
-       mipi_dsi_device_unregister(lt9611uxc->dsi0);
-}
-
 static int lt9611uxc_bridge_attach(struct drm_bridge *bridge,
                                   enum drm_bridge_attach_flags flags)
 {
@@ -385,19 +372,11 @@ static int lt9611uxc_bridge_attach(struct drm_bridge 
*bridge,
        /* Attach secondary DSI, if specified */
        if (lt9611uxc->dsi1_node) {
                lt9611uxc->dsi1 = lt9611uxc_attach_dsi(lt9611uxc, 
lt9611uxc->dsi1_node);
-               if (IS_ERR(lt9611uxc->dsi1)) {
-                       ret = PTR_ERR(lt9611uxc->dsi1);
-                       goto err_unregister_dsi0;
-               }
+               if (IS_ERR(lt9611uxc->dsi1))
+                       return PTR_ERR(lt9611uxc->dsi1);
        }
 
        return 0;
-
-err_unregister_dsi0:
-       mipi_dsi_detach(lt9611uxc->dsi0);
-       mipi_dsi_device_unregister(lt9611uxc->dsi0);
-
-       return ret;
 }
 
 static enum drm_mode_status
@@ -541,7 +520,6 @@ static struct edid *lt9611uxc_bridge_get_edid(struct 
drm_bridge *bridge,
 
 static const struct drm_bridge_funcs lt9611uxc_bridge_funcs = {
        .attach = lt9611uxc_bridge_attach,
-       .detach = lt9611uxc_bridge_detach,
        .mode_valid = lt9611uxc_bridge_mode_valid,
        .mode_set = lt9611uxc_bridge_mode_set,
        .detect = lt9611uxc_bridge_detect,
-- 
2.31.1