drm/radeon: remove radeon_fb_{add,remove}_connector functions
authorPankaj Bharadiya <pankaj.laxminarayan.bharadiya@intel.com>
Thu, 5 Mar 2020 12:04:29 +0000 (17:34 +0530)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 6 Mar 2020 13:19:57 +0000 (14:19 +0100)
drm_fb_helper_{add,remove}_one_connector() are dummy functions now
and serve no purpose. Hence remove their calls.

This is the preparatory step for removing the
drm_fb_helper_{add,remove}_one_connector() functions from
drm_fb_helper.h

Signed-off-by: Pankaj Bharadiya <pankaj.laxminarayan.bharadiya@intel.com>
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20200305120434.111091-3-pankaj.laxminarayan.bharadiya@intel.com
drivers/gpu/drm/radeon/radeon_dp_mst.c
drivers/gpu/drm/radeon/radeon_fb.c
drivers/gpu/drm/radeon/radeon_mode.h

index 28eef92..5a9fb0a 100644 (file)
@@ -303,23 +303,13 @@ static struct drm_connector *radeon_dp_add_mst_connector(struct drm_dp_mst_topol
 
 static void radeon_dp_register_mst_connector(struct drm_connector *connector)
 {
-       struct drm_device *dev = connector->dev;
-       struct radeon_device *rdev = dev->dev_private;
-
-       radeon_fb_add_connector(rdev, connector);
-
        drm_connector_register(connector);
 }
 
 static void radeon_dp_destroy_mst_connector(struct drm_dp_mst_topology_mgr *mgr,
                                            struct drm_connector *connector)
 {
-       struct radeon_connector *master = container_of(mgr, struct radeon_connector, mst_mgr);
-       struct drm_device *dev = master->base.dev;
-       struct radeon_device *rdev = dev->dev_private;
-
        drm_connector_unregister(connector);
-       radeon_fb_remove_connector(rdev, connector);
        drm_connector_cleanup(connector);
 
        kfree(connector);
index 85548cf..1c02cd7 100644 (file)
@@ -403,15 +403,3 @@ bool radeon_fbdev_robj_is_fb(struct radeon_device *rdev, struct radeon_bo *robj)
                return true;
        return false;
 }
-
-void radeon_fb_add_connector(struct radeon_device *rdev, struct drm_connector *connector)
-{
-       if (rdev->mode_info.rfbdev)
-               drm_fb_helper_add_one_connector(&rdev->mode_info.rfbdev->helper, connector);
-}
-
-void radeon_fb_remove_connector(struct radeon_device *rdev, struct drm_connector *connector)
-{
-       if (rdev->mode_info.rfbdev)
-               drm_fb_helper_remove_one_connector(&rdev->mode_info.rfbdev->helper, connector);
-}
index 629567d..c7f2237 100644 (file)
@@ -986,9 +986,6 @@ bool radeon_fbdev_robj_is_fb(struct radeon_device *rdev, struct radeon_bo *robj)
 
 void radeon_crtc_handle_vblank(struct radeon_device *rdev, int crtc_id);
 
-void radeon_fb_add_connector(struct radeon_device *rdev, struct drm_connector *connector);
-void radeon_fb_remove_connector(struct radeon_device *rdev, struct drm_connector *connector);
-
 void radeon_crtc_handle_flip(struct radeon_device *rdev, int crtc_id);
 
 int radeon_align_pitch(struct radeon_device *rdev, int width, int bpp, bool tiled);