Thread (5 messages) 5 messages, 2 authors, 2009-08-02

Re: [PATCH] [ARM] pxa/spitz: use GPIO matrix keypad driver

From: Pavel Machek <hidden>
Date: 2009-08-02 13:26:12

On Thu 2009-07-30 13:00:40, Eric Miao wrote:
Pavel Machek wrote:
quoted
On Mon 2009-07-20 19:41:54, Eric Miao wrote:
quoted
Signed-off-by: Eric Miao <redacted>
Does this make some other driver obsolete? Should we disable the other
one or even remove it completely?
Yes, specifically the following two:

  drivers/input/keyboard/corgikbd.c
  drivers/input/keyboard/spitzkbd.c

which I don't feel quite comfortable since they are referencing the PXA
GPIO registers directly.

However, marking obsolete might be more smooth for the moment.
Given that there are unused/unsuable after your patch is applied,
removing them from build system is probably good first step.
quoted
quoted
 static struct platform_device spitzkbd_device = {
-	.name		= "spitz-keyboard",
+	.name		= "matrix-keypad",
 	.id		= -1,
+	.dev		= {
+		.platform_data = &spitzkbd_pdata,
+	},
 };
								Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help