Thread (3 messages) 3 messages, 2 authors, 2014-04-01

Re: [PATCH] video: da8xx-fb: Add support for Densitron 84-0023-001T

From: Tomi Valkeinen <hidden>
Date: 2014-04-01 09:15:07
Also in: lkml

Hi,

On 05/03/14 20:05, jon@ringle.org wrote:
quoted hunk ↗ jump to hunk
From: Jon Ringle <redacted>

Signed-off-by: Jon Ringle <redacted>
---
 drivers/video/da8xx-fb.c | 14 ++++++++++++++
 1 file changed, 14 insertions(+)
diff --git a/drivers/video/da8xx-fb.c b/drivers/video/da8xx-fb.c
index e7f5937..83c43b2 100644
--- a/drivers/video/da8xx-fb.c
+++ b/drivers/video/da8xx-fb.c
@@ -243,6 +243,20 @@ static struct fb_videomode known_lcd_panels[] = {
 		.sync           = FB_SYNC_HOR_HIGH_ACT | FB_SYNC_VERT_HIGH_ACT,
 		.flag           = 0,
 	},
+	[3] = {
+		/* Densitron 84-0023-001T */
+		.name           = "Densitron_LCD",
I think the name should be a bit better. Why it is not something like
"Densitron_84-0023-001T", which is the style the other panels use?

 Tomi

Attachments

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