video: fbdev: simplefb: Fix info message during probe
authorPeter Robinson <pbrobinson@gmail.com>
Mon, 28 Dec 2020 18:39:34 +0000 (18:39 +0000)
committerHans de Goede <hdegoede@redhat.com>
Wed, 20 Jan 2021 11:06:32 +0000 (12:06 +0100)
The info message was showing the mapped address for the framebuffer. To avoid
security problems, all virtual addresses are converted to __ptrval__, so
the message has pointless information:

simple-framebuffer 3ea9b000.framebuffer: framebuffer at 0x3ea9b000, 0x12c000 bytes, mapped to 0x(____ptrval____)

Drop the extraneous bits to clean up the message:

simple-framebuffer 3ea9b000.framebuffer: framebuffer at 0x3ea9b000, 0x12c000 bytes

Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20201228183934.1117012-1-pbrobinson@gmail.com
drivers/video/fbdev/simplefb.c

index 533a047..62f0ded 100644 (file)
@@ -477,9 +477,8 @@ static int simplefb_probe(struct platform_device *pdev)
        simplefb_clocks_enable(par, pdev);
        simplefb_regulators_enable(par, pdev);
 
-       dev_info(&pdev->dev, "framebuffer at 0x%lx, 0x%x bytes, mapped to 0x%p\n",
-                            info->fix.smem_start, info->fix.smem_len,
-                            info->screen_base);
+       dev_info(&pdev->dev, "framebuffer at 0x%lx, 0x%x bytes\n",
+                            info->fix.smem_start, info->fix.smem_len);
        dev_info(&pdev->dev, "format=%s, mode=%dx%dx%d, linelength=%d\n",
                             params.format->name,
                             info->var.xres, info->var.yres,