Lets now pass drm_connector to prepare_writeback_job since
drm_writeback_connector now resides within drm_connector.
It also make it uniform with params passed to other
drm_connector_helper_funcs.

Signed-off-by: Suraj Kandpal <suraj.kand...@intel.com>
---
 drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_wb.c | 4 ++--
 drivers/gpu/drm/drm_writeback.c                      | 2 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_writeback.c        | 4 +---
 drivers/gpu/drm/renesas/rcar-du/rcar_du_writeback.c  | 6 ++----
 drivers/gpu/drm/vkms/vkms_writeback.c                | 5 +----
 include/drm/drm_modeset_helper_vtables.h             | 2 +-
 6 files changed, 8 insertions(+), 15 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_wb.c 
b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_wb.c
index d02f5d20f3b1..bf1ecf5d3027 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_wb.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_wb.c
@@ -144,8 +144,8 @@ static int amdgpu_dm_wb_prepare_job(struct drm_connector 
*connector,
        return r;
 }
 
-static void amdgpu_dm_wb_cleanup_job(struct drm_writeback_connector *connector,
-                               struct drm_writeback_job *job)
+static void amdgpu_dm_wb_cleanup_job(struct drm_connector *connector,
+                                    struct drm_writeback_job *job)
 {
        struct amdgpu_bo *rbo;
        int r;
diff --git a/drivers/gpu/drm/drm_writeback.c b/drivers/gpu/drm/drm_writeback.c
index 32a8e6498585..a2cb66f70ed1 100644
--- a/drivers/gpu/drm/drm_writeback.c
+++ b/drivers/gpu/drm/drm_writeback.c
@@ -451,7 +451,7 @@ void drm_writeback_cleanup_job(struct drm_writeback_job 
*job)
                connector->helper_private;
 
        if (job->prepared && funcs->cleanup_writeback_job)
-               funcs->cleanup_writeback_job(wb_connector, job);
+               funcs->cleanup_writeback_job(connector, job);
 
        if (job->fb)
                drm_framebuffer_put(job->fb);
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_writeback.c 
b/drivers/gpu/drm/msm/disp/dpu1/dpu_writeback.c
index 26a93c3cc454..03e63b6c5351 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_writeback.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_writeback.c
@@ -96,11 +96,9 @@ static int dpu_wb_conn_prepare_job(struct drm_connector 
*connector,
        return 0;
 }
 
-static void dpu_wb_conn_cleanup_job(struct drm_writeback_connector 
*wb_connector,
+static void dpu_wb_conn_cleanup_job(struct drm_connector *connector,
                struct drm_writeback_job *job)
 {
-       struct drm_connector *connector =
-               container_of(wb_connector, struct drm_connector, writeback);
        struct dpu_wb_connector *dpu_wb_conn = to_dpu_wb_conn(connector);
 
        if (!job->fb)
diff --git a/drivers/gpu/drm/renesas/rcar-du/rcar_du_writeback.c 
b/drivers/gpu/drm/renesas/rcar-du/rcar_du_writeback.c
index 725981cc1d0c..e3aab132ded1 100644
--- a/drivers/gpu/drm/renesas/rcar-du/rcar_du_writeback.c
+++ b/drivers/gpu/drm/renesas/rcar-du/rcar_du_writeback.c
@@ -72,12 +72,10 @@ static int rcar_du_wb_prepare_job(struct drm_connector 
*connector,
        return 0;
 }
 
-static void rcar_du_wb_cleanup_job(struct drm_writeback_connector *connector,
+static void rcar_du_wb_cleanup_job(struct drm_connector *connector,
                                   struct drm_writeback_job *job)
 {
-       struct drm_connector *conn =
-               drm_writeback_to_connector(connector);
-       struct rcar_du_crtc *rcrtc = wb_to_rcar_crtc(conn);
+       struct rcar_du_crtc *rcrtc = wb_to_rcar_crtc(connector);
        struct rcar_du_wb_job *rjob = job->priv;
 
        if (!job->fb)
diff --git a/drivers/gpu/drm/vkms/vkms_writeback.c 
b/drivers/gpu/drm/vkms/vkms_writeback.c
index 032896fb5c5b..320d553f5f1f 100644
--- a/drivers/gpu/drm/vkms/vkms_writeback.c
+++ b/drivers/gpu/drm/vkms/vkms_writeback.c
@@ -102,13 +102,10 @@ static int vkms_wb_prepare_job(struct drm_connector 
*connector,
        return ret;
 }
 
-static void vkms_wb_cleanup_job(struct drm_writeback_connector *wb_conn,
+static void vkms_wb_cleanup_job(struct drm_connector *connector,
                                struct drm_writeback_job *job)
 {
        struct vkms_writeback_job *vkmsjob = job->priv;
-       struct drm_connector *connector = container_of(wb_conn,
-                                                      struct drm_connector,
-                                                      writeback);
        struct vkms_output *vkms_output = container_of(connector,
                                                       struct vkms_output,
                                                       wb_connector);
diff --git a/include/drm/drm_modeset_helper_vtables.h 
b/include/drm/drm_modeset_helper_vtables.h
index f6a14f396c40..483de9978e0c 100644
--- a/include/drm/drm_modeset_helper_vtables.h
+++ b/include/drm/drm_modeset_helper_vtables.h
@@ -1145,7 +1145,7 @@ struct drm_connector_helper_funcs {
         *
         * This callback is used by the atomic modeset helpers.
         */
-       void (*cleanup_writeback_job)(struct drm_writeback_connector *connector,
+       void (*cleanup_writeback_job)(struct drm_connector *connector,
                                      struct drm_writeback_job *job);
 
        /**
-- 
2.34.1

Reply via email to