[E3-hacking] [PATCH v3 4/5(7)] input: serio: add support for Amstrad Delta serial keyboard port
Tony Lindgren
tony at atomide.com
Fri Apr 23 00:14:53 BST 2010
* Janusz Krzysztofik <jkrzyszt at tis.icnet.pl> [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 at mail.ru>
>
> 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
More information about the e3-hacking
mailing list