Re: [PATCH v3 4/5(7)] input: serio: add support for Amstrad Delta serial keyboard port

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

 



* Janusz Krzysztofik <jkrzyszt@xxxxxxxxxxxx> [100415 11:14]:
> Thursday 15 April 2010 18:14:51 Dmitry Torokhov napisał(a):
> > Hi Janusz,
> 
> Hi Dmitry,
> 
> > On Thu, Apr 15, 2010 at 04:31:03AM +0200, Janusz Krzysztofik wrote:
> > > +
> > > +config SERIO_AMS_DELTA
> > > +	tristate "Amstrad Delta (E3) mailboard support"
> > > +	depends on MACH_AMS_DELTA
> > > +	default y
> > > +	select AMS_DELTA_FIQ
> > > +	select KEYBOARD_ATKBD
> >
> > I'd rather not select atkbd here since it is not required for this
> > driver to work but simply adjust your defconfig.
> 
> I was just trying to address possible future support questions, as you can 
> already find them in the e3-hacking 2006 archives :).
> 
> OK, I'll drop it then and move the config back inside the "if SERIO"..."endif" 
> block.
> 
> > > +
> > > +	if (!(machine_is_ams_delta()))
> > > +		return -ENODEV;
> >
> > Extra parens.
> 
> Yes, it looks like I'm too much wary. I'll drop them.
> 
> > Otherwise:
> >
> > 	Acked-by: Dmitry Torokhov <dtor@xxxxxxx>
> 
> Thanks.
> 
> > I assume it is going to be merged with the rest of your patches through
> > a tree other than linux-input.
> 
> Probably yes, since it depends on most of them. Let's see first what Tony 
> says.

Sorry for the delay, quite a few patches to read through right now..

Looks pretty good to me, I have one comment on the FIQ though.

After the remaining fixes, could you please repost the whole series
one more time now with linux-arm-kernel also cc'd?

Then if no more comments there, I'll add add it to omap for-next.

Cheers,

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux