fbdev/clps711x-fb: Do not set struct fb_info.apertures
authorThomas Zimmermann <tzimmermann@suse.de>
Mon, 19 Dec 2022 16:05:05 +0000 (17:05 +0100)
committerThomas Zimmermann <tzimmermann@suse.de>
Fri, 13 Jan 2023 11:54:25 +0000 (12:54 +0100)
Generic fbdev drivers use the apertures field in struct fb_info to
control ownership of the framebuffer memory and graphics device. Do
not set the values in clps711x-fb.

Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20221219160516.23436-8-tzimmermann@suse.de
drivers/video/fbdev/clps711x-fb.c

index a1061c2..45c75ff 100644 (file)
@@ -251,16 +251,8 @@ static int clps711x_fb_probe(struct platform_device *pdev)
                goto out_fb_release;
        }
 
-       info->apertures = alloc_apertures(1);
-       if (!info->apertures) {
-               ret = -ENOMEM;
-               goto out_fb_release;
-       }
-
        cfb->buffsize = resource_size(res);
        info->fix.smem_start = res->start;
-       info->apertures->ranges[0].base = info->fix.smem_start;
-       info->apertures->ranges[0].size = cfb->buffsize;
 
        cfb->clk = devm_clk_get(dev, NULL);
        if (IS_ERR(cfb->clk)) {
@@ -345,7 +337,7 @@ static int clps711x_fb_probe(struct platform_device *pdev)
                                       &clps711x_lcd_ops);
        if (!IS_ERR(lcd))
                return 0;
-       
+
        ret = PTR_ERR(lcd);
        unregister_framebuffer(info);