Printing the framebuffer base is unnecessary because it's already contained in the dev_printf prefix. Make the message shorter and only print it on success. Signed-off-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> --- drivers/video/simplefb-client.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/video/simplefb-client.c b/drivers/video/simplefb-client.c index e1f241d4d874..2367713ba4ae 100644 --- a/drivers/video/simplefb-client.c +++ b/drivers/video/simplefb-client.c @@ -122,9 +122,6 @@ static int simplefb_probe(struct device_d *dev) info->fbops = &simplefb_ops; - dev_info(dev, "framebuffer at 0x%p, 0x%lx bytes\n", - info->screen_base, info->screen_size); - info->dev.parent = dev; ret = register_framebuffer(info); if (ret < 0) { @@ -132,7 +129,7 @@ static int simplefb_probe(struct device_d *dev) return ret; } - dev_info(dev, "simplefb registered!\n"); + dev_info(dev, "size %s registered\n", size_human_readable(info->screen_size)); return 0; } -- 2.30.2 _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox