Skip to content

Commit

Permalink
drm/rockchip: use generic fbdev setup
Browse files Browse the repository at this point in the history
The Rockchip fbdev code does not add anything compared to
drm_fbdev_generic_setup(); the one custom function for .fb_mmap does the
same thing as gem_prime_mmap which is called by the helper.

Signed-off-by: John Keeping <john@metanate.com>
Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Link: https://patchwork.freedesktop.org/patch/msgid/20211029115014.264084-1-john@metanate.com
  • Loading branch information
John Keeping authored and Thomas Zimmermann committed Dec 7, 2021
1 parent 8697c41 commit 24af7c3
Show file tree
Hide file tree
Showing 5 changed files with 2 additions and 199 deletions.
1 change: 0 additions & 1 deletion drivers/gpu/drm/rockchip/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@

rockchipdrm-y := rockchip_drm_drv.o rockchip_drm_fb.o \
rockchip_drm_gem.o rockchip_drm_vop.o rockchip_vop_reg.o
rockchipdrm-$(CONFIG_DRM_FBDEV_EMULATION) += rockchip_drm_fbdev.o

rockchipdrm-$(CONFIG_ROCKCHIP_ANALOGIX_DP) += analogix_dp-rockchip.o
rockchipdrm-$(CONFIG_ROCKCHIP_CDN_DP) += cdn-dp-core.o cdn-dp-reg.o
Expand Down
10 changes: 2 additions & 8 deletions drivers/gpu/drm/rockchip/rockchip_drm_drv.c
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@

#include "rockchip_drm_drv.h"
#include "rockchip_drm_fb.h"
#include "rockchip_drm_fbdev.h"
#include "rockchip_drm_gem.h"

#define DRIVER_NAME "rockchip"
Expand Down Expand Up @@ -159,21 +158,18 @@ static int rockchip_drm_bind(struct device *dev)

drm_mode_config_reset(drm_dev);

ret = rockchip_drm_fbdev_init(drm_dev);
if (ret)
goto err_unbind_all;

/* init kms poll for handling hpd */
drm_kms_helper_poll_init(drm_dev);

ret = drm_dev_register(drm_dev, 0);
if (ret)
goto err_kms_helper_poll_fini;

drm_fbdev_generic_setup(drm_dev, 32);

return 0;
err_kms_helper_poll_fini:
drm_kms_helper_poll_fini(drm_dev);
rockchip_drm_fbdev_fini(drm_dev);
err_unbind_all:
component_unbind_all(dev, drm_dev);
err_iommu_cleanup:
Expand All @@ -189,7 +185,6 @@ static void rockchip_drm_unbind(struct device *dev)

drm_dev_unregister(drm_dev);

rockchip_drm_fbdev_fini(drm_dev);
drm_kms_helper_poll_fini(drm_dev);

drm_atomic_helper_shutdown(drm_dev);
Expand All @@ -203,7 +198,6 @@ DEFINE_DRM_GEM_FOPS(rockchip_drm_driver_fops);

static const struct drm_driver rockchip_drm_driver = {
.driver_features = DRIVER_MODESET | DRIVER_GEM | DRIVER_ATOMIC,
.lastclose = drm_fb_helper_lastclose,
.dumb_create = rockchip_gem_dumb_create,
.prime_handle_to_fd = drm_gem_prime_handle_to_fd,
.prime_fd_to_handle = drm_gem_prime_fd_to_handle,
Expand Down
2 changes: 0 additions & 2 deletions drivers/gpu/drm/rockchip/rockchip_drm_drv.h
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,6 @@ struct rockchip_crtc_state {
* @mm_lock: protect drm_mm on multi-threads.
*/
struct rockchip_drm_private {
struct drm_fb_helper fbdev_helper;
struct drm_gem_object *fbdev_bo;
struct iommu_domain *domain;
struct mutex mm_lock;
struct drm_mm mm;
Expand Down
164 changes: 0 additions & 164 deletions drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c

This file was deleted.

24 changes: 0 additions & 24 deletions drivers/gpu/drm/rockchip/rockchip_drm_fbdev.h

This file was deleted.

0 comments on commit 24af7c3

Please sign in to comment.