summaryrefslogtreecommitdiffstats
path: root/drivers/gpu/drm/omapdrm
diff options
context:
space:
mode:
authorThomas Zimmermann <tzimmermann@suse.de>2023-04-03 12:40:32 +0200
committerTomi Valkeinen <tomi.valkeinen@ideasonboard.com>2023-04-06 14:49:58 +0200
commit38129bc9260d3394b2e5d4e5eebc6baaeac02e8f (patch)
treed980cb934041d66e0f0b3991dae7eaf498f1ade5 /drivers/gpu/drm/omapdrm
parentdrm/omapdrm: Include <linux/of.h> (diff)
downloadlinux-38129bc9260d3394b2e5d4e5eebc6baaeac02e8f.tar.xz
linux-38129bc9260d3394b2e5d4e5eebc6baaeac02e8f.zip
drm/omapdrm: Remove fb from struct omap_fbdev
Fbdev's struct fb_helper stores a pointer to the framebuffer. Remove struct omap_fbdev.fb, which contains the same value. No functional changes. Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de> Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> Link: https://patchwork.freedesktop.org/patch/msgid/20230403104035.15288-3-tzimmermann@suse.de
Diffstat (limited to 'drivers/gpu/drm/omapdrm')
-rw-r--r--drivers/gpu/drm/omapdrm/omap_fbdev.c11
1 files changed, 6 insertions, 5 deletions
diff --git a/drivers/gpu/drm/omapdrm/omap_fbdev.c b/drivers/gpu/drm/omapdrm/omap_fbdev.c
index a6c8542087ec..b3d57fe4e6ac 100644
--- a/drivers/gpu/drm/omapdrm/omap_fbdev.c
+++ b/drivers/gpu/drm/omapdrm/omap_fbdev.c
@@ -25,7 +25,6 @@ module_param_named(ywrap, ywrap_enabled, bool, 0644);
struct omap_fbdev {
struct drm_fb_helper base;
- struct drm_framebuffer *fb;
struct drm_gem_object *bo;
bool ywrap_enabled;
@@ -170,7 +169,6 @@ static int omap_fbdev_create(struct drm_fb_helper *helper,
DBG("fbi=%p, dev=%p", fbi, dev);
- fbdev->fb = fb;
helper->fb = fb;
fbi->fbops = &omap_fb_ops;
@@ -193,7 +191,7 @@ static int omap_fbdev_create(struct drm_fb_helper *helper,
DBG("par=%p, %dx%d", fbi->par, fbi->var.xres, fbi->var.yres);
- DBG("allocated %dx%d fb", fbdev->fb->width, fbdev->fb->height);
+ DBG("allocated %dx%d fb", fb->width, fb->height);
return 0;
@@ -266,6 +264,7 @@ void omap_fbdev_fini(struct drm_device *dev)
{
struct omap_drm_private *priv = dev->dev_private;
struct drm_fb_helper *helper = priv->fbdev;
+ struct drm_framebuffer *fb;
struct omap_fbdev *fbdev;
DBG();
@@ -273,6 +272,8 @@ void omap_fbdev_fini(struct drm_device *dev)
if (!helper)
return;
+ fb = helper->fb;
+
drm_fb_helper_unregister_info(helper);
drm_fb_helper_fini(helper);
@@ -284,8 +285,8 @@ void omap_fbdev_fini(struct drm_device *dev)
omap_gem_unpin(fbdev->bo);
/* this will free the backing object */
- if (fbdev->fb)
- drm_framebuffer_remove(fbdev->fb);
+ if (fb)
+ drm_framebuffer_remove(fb);
drm_fb_helper_unprepare(helper);
kfree(fbdev);