Re: [PATCH v2 05/11] fblog: register one fblog object per framebuffer
From: Joe Perches <joe@perches.com>
Date: 2012-07-08 22:13:50
Also in:
linux-fbdev, lkml
On Sun, 2012-07-08 at 23:56 +0200, David Herrmann wrote:
Hi David. Trivial comments only:
quoted hunk ↗ jump to hunk
diff --git a/drivers/video/console/fblog.c b/drivers/video/console/fblog.c
[]
quoted hunk ↗ jump to hunk
@@ -23,15 +23,204 @@ * all fblog instances before running other graphics applications. */
Please add #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt before any #include and...
quoted hunk ↗ jump to hunk
static int __init fblog_init(void) { + int ret; + + ret = fb_register_client(&fblog_notifier); + if (ret) { + pr_err("fblog: cannot register framebuffer notifier");
Missing newline, pr_fmt would add module name.
pr_err("cannot register framebuffer notifier\n");