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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu 2009-07-30 13:00:40, Eric Miao wrote:
> Pavel Machek wrote:
> > On Mon 2009-07-20 19:41:54, Eric Miao wrote:
> >> Signed-off-by: Eric Miao <eric.y.miao@xxxxxxxxx>
> > 
> > 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.

> >>  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
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux