Re: [PATCH] Input: olpc_apsp - depend on ARM

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

 



On Thu 2019-04-11 07:04:19, Lubomir Rintel wrote:
> On Wed, 2019-04-10 at 19:55 +0200, Jean Delvare wrote:
> > On Wed, 10 Apr 2019 18:13:40 +0200, Lubomir Rintel wrote:
> > > On Tue, 2019-04-09 at 13:29 +0200, Jean Delvare wrote:
> > > > The OLPC XO-1.75 and XO-4 are ARM-based laptops, so don't present
> > > > this driver on other architectures except for build-testing purposes.
> > > > 
> > > > Signed-off-by: Jean Delvare <jdelvare@xxxxxxx>
> > > > Cc: Lubomir Rintel <lkundrak@xxxxx>
> > > > Cc: Pavel Machek <pavel@xxxxxx>
> > > > Cc: Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx>
> > > > ---
> > > > We could even go for a more specific dependency, if anyone knows
> > > > which would be appropriate.
> > > > 
> > > >  drivers/input/serio/Kconfig |    1 +
> > > >  1 file changed, 1 insertion(+)
> > > > 
> > > > --- linux-5.0.orig/drivers/input/serio/Kconfig	2019-03-04 00:21:29.000000000 +0100
> > > > +++ linux-5.0/drivers/input/serio/Kconfig	2019-04-09 13:20:09.822700623 +0200
> > > > @@ -254,6 +254,7 @@ config SERIO_APBPS2
> > > >  
> > > >  config SERIO_OLPC_APSP
> > > >  	tristate "OLPC AP-SP input support"
> > > > +	depends on ARM || COMPILE_TEST
> > > >  	help
> > > >  	  Say Y here if you want support for the keyboard and touchpad included
> > > >  	  in the OLPC XO-1.75 and XO-4 laptops.  
> > > 
> > > Looks good. Limit this to ARCH_MMP might make even more sense if you
> > > care about uncluttering make config.
> > 
> > Yes, that's pretty much my intention. Does ARCH_MMP cover both the
> > XO-1.75 (Armada 610 / Marvell Sheeva) and XO-4 (Marvell PXA2128)?
> 
> There's no PXA2128/MMP3 support in mainline.
> 
> If it's ever added, I'd guess ARCH_MMP would cover it. It does cover
> MMP and MMP2, and I think MMP3 is mostly the same, just with an extra
> PJ4 core and a newer GPU.

I guess depends on ARCH_MMP || COMPILE_TEST would be nicest solution,
then.

Anyway, this is ok, too.

Acked-by: Pavel Machek <pavel@xxxxxx>
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Attachment: signature.asc
Description: Digital signature


[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