To prepare for operating as a standalone DP bridge with the DRM device
implemented in the PL, move registration of the AUX bus to bridge attach
time, as that's the earliest point when a DRM device is available.

The DRM device pointer stored in zynqmp_dp isn't used anymore, drop it.

Signed-off-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
---
 drivers/gpu/drm/xlnx/zynqmp_dp.c | 41 +++++++++++++++++++-------------
 1 file changed, 24 insertions(+), 17 deletions(-)

diff --git a/drivers/gpu/drm/xlnx/zynqmp_dp.c b/drivers/gpu/drm/xlnx/zynqmp_dp.c
index 72fe3b7fb78e..e40ddfd27ff0 100644
--- a/drivers/gpu/drm/xlnx/zynqmp_dp.c
+++ b/drivers/gpu/drm/xlnx/zynqmp_dp.c
@@ -275,7 +275,6 @@ struct zynqmp_dp_config {
  * struct zynqmp_dp - Xilinx DisplayPort core
  * @dev: device structure
  * @dpsub: Display subsystem
- * @drm: DRM core
  * @iomem: device I/O memory for register access
  * @reset: reset controller
  * @irq: irq
@@ -296,7 +295,6 @@ struct zynqmp_dp_config {
 struct zynqmp_dp {
        struct device *dev;
        struct zynqmp_dpsub *dpsub;
-       struct drm_device *drm;
        void __iomem *iomem;
        struct reset_control *reset;
        int irq;
@@ -1056,7 +1054,7 @@ static int zynqmp_dp_aux_init(struct zynqmp_dp *dp)
 
        dp->aux.name = "ZynqMP DP AUX";
        dp->aux.dev = dp->dev;
-       dp->aux.drm_dev = dp->drm;
+       dp->aux.drm_dev = dp->bridge.dev;
        dp->aux.transfer = zynqmp_dp_aux_transfer;
 
        return drm_dp_aux_register(&dp->aux);
@@ -1282,14 +1280,35 @@ static int zynqmp_dp_bridge_attach(struct drm_bridge 
*bridge,
        struct zynqmp_dp *dp = bridge_to_dp(bridge);
        int ret;
 
+       /* Initialize and register the AUX adapter. */
+       ret = zynqmp_dp_aux_init(dp);
+       if (ret) {
+               dev_err(dp->dev, "failed to initialize DP aux\n");
+               return ret;
+       }
+
        if (dp->next_bridge) {
                ret = drm_bridge_attach(bridge->encoder, dp->next_bridge,
                                        bridge, flags);
                if (ret < 0)
-                       return ret;
+                       goto error;
        }
 
+       /* Now that initialisation is complete, enable interrupts. */
+       zynqmp_dp_write(dp, ZYNQMP_DP_INT_EN, ZYNQMP_DP_INT_ALL);
+
        return 0;
+
+error:
+       zynqmp_dp_aux_cleanup(dp);
+       return ret;
+}
+
+static void zynqmp_dp_bridge_detach(struct drm_bridge *bridge)
+{
+       struct zynqmp_dp *dp = bridge_to_dp(bridge);
+
+       zynqmp_dp_aux_cleanup(dp);
 }
 
 static int zynqmp_dp_bridge_mode_valid(struct drm_bridge *bridge,
@@ -1494,6 +1513,7 @@ static struct edid *zynqmp_dp_bridge_get_edid(struct 
drm_bridge *bridge,
 
 static const struct drm_bridge_funcs zynqmp_dp_bridge_funcs = {
        .attach = zynqmp_dp_bridge_attach,
+       .detach = zynqmp_dp_bridge_detach,
        .mode_valid = zynqmp_dp_bridge_mode_valid,
        .atomic_enable = zynqmp_dp_bridge_atomic_enable,
        .atomic_disable = zynqmp_dp_bridge_atomic_disable,
@@ -1593,7 +1613,6 @@ int zynqmp_dp_drm_init(struct zynqmp_dpsub *dpsub)
 {
        struct zynqmp_dp *dp = dpsub->dp;
        struct drm_bridge *bridge = &dp->bridge;
-       int ret;
 
        dp->config.misc0 &= ~ZYNQMP_DP_MAIN_STREAM_MISC0_SYNC_LOCK;
        zynqmp_dp_set_format(dp, NULL, ZYNQMP_DPSUB_FORMAT_RGB, 8);
@@ -1605,16 +1624,6 @@ int zynqmp_dp_drm_init(struct zynqmp_dpsub *dpsub)
        bridge->type = DRM_MODE_CONNECTOR_DisplayPort;
        dpsub->bridge = bridge;
 
-       /* Initialize and register the AUX adapter. */
-       ret = zynqmp_dp_aux_init(dp);
-       if (ret) {
-               dev_err(dp->dev, "failed to initialize DP aux\n");
-               return ret;
-       }
-
-       /* Now that initialisation is complete, enable interrupts. */
-       zynqmp_dp_write(dp, ZYNQMP_DP_INT_EN, ZYNQMP_DP_INT_ALL);
-
        return 0;
 }
 
@@ -1632,7 +1641,6 @@ int zynqmp_dp_probe(struct zynqmp_dpsub *dpsub, struct 
drm_device *drm)
        dp->dev = &pdev->dev;
        dp->dpsub = dpsub;
        dp->status = connector_status_disconnected;
-       dp->drm = drm;
 
        INIT_DELAYED_WORK(&dp->hpd_work, zynqmp_dp_hpd_work_func);
 
@@ -1718,7 +1726,6 @@ void zynqmp_dp_remove(struct zynqmp_dpsub *dpsub)
        disable_irq(dp->irq);
 
        cancel_delayed_work_sync(&dp->hpd_work);
-       zynqmp_dp_aux_cleanup(dp);
 
        zynqmp_dp_write(dp, ZYNQMP_DP_TRANSMITTER_ENABLE, 0);
        zynqmp_dp_write(dp, ZYNQMP_DP_INT_DS, 0xffffffff);
-- 
Regards,

Laurent Pinchart

Reply via email to