Thread (22 messages) 22 messages, 3 authors, 2012-07-17

Re: [PATCH v2 09/11] fblog: register console driver

From: Joe Perches <joe@perches.com>
Date: 2012-07-08 22:09:24
Also in: linux-serial, lkml

On Sun, 2012-07-08 at 23:56 +0200, David Herrmann wrote:
We want to print the kernel log to all FBs so we need a console driver.
This registers the driver on startup and writes all messages to all
registered fblog instances.
Hi David.  Trivia 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
@@ -47,6 +71,107 @@ static struct fblog_fb *fblog_fbs[FB_MAX];
 static bool active = 1;
 
 #define to_fblog_dev(_d) container_of(_d, struct fblog_fb, dev)
+#define FBLOG_STR(x) x, sizeof(x) - 1
That's a rather ugly macro.
+static void fblog_buf_resize(struct fblog_buf *buf, size_t width,
+			     size_t height)
+{
+	u16 **lines = NULL;
+	size_t i, j, minw, minh;
+
+	if (buf->height = height && buf->width = width)
+		return;
+
+	if (width && height) {
+		lines = kzalloc(height * sizeof(char*), GFP_KERNEL);
sizeof(char *) is a bit more kernel style like.

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help