LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* New Input PS/2 driver
@ 2001-10-03 13:34 Etienne Lorrain
  2001-10-03 19:28 ` James Simmons
  0 siblings, 1 reply; 6+ messages in thread
From: Etienne Lorrain @ 2001-10-03 13:34 UTC (permalink / raw)
  To: linux-kernel

James Simmons wrote:
> To use the keyboard driver you need to 
> 1) enable the input core and keyboard support in the input menu. 

  Would be nice to change the comment describing the Keyboard core
 support (CONFIG_INPUT_KEYBDEV) in this patch: people (as dumb me)
 may read the help in menuconfig and say:
 I have no USB HID or ADB keyboard...
 Recompile, reboot => no keyboard, no control-alt-del => Reset (fsck...).

  Else it is working here on a P133 with nothing special (std PS2 mouse).

  BTW you just undefine I8042_OVERRIDE_KEYLOCK but this define is
 never used.

  Etienne.

___________________________________________________________
Do You Yahoo!? -- Un e-mail gratuit @yahoo.fr !
Yahoo! Courrier : http://fr.mail.yahoo.com

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: New Input PS/2 driver
  2001-10-03 13:34 New Input PS/2 driver Etienne Lorrain
@ 2001-10-03 19:28 ` James Simmons
  2001-10-05 22:50   ` Vojtech Pavlik
  0 siblings, 1 reply; 6+ messages in thread
From: James Simmons @ 2001-10-03 19:28 UTC (permalink / raw)
  To: Etienne Lorrain; +Cc: Linux Kernel Mailing List, Alan Cox


>   Would be nice to change the comment describing the Keyboard core
>  support (CONFIG_INPUT_KEYBDEV) in this patch: people (as dumb me)
>  may read the help in menuconfig and say:
>  I have no USB HID or ADB keyboard...
>  Recompile, reboot => no keyboard, no control-alt-del => Reset (fsck...).

Yeah. That does need to be added. 

>   Else it is working here on a P133 with nothing special (std PS2 mouse).

>From the several reports I have had with the driver it looks pretty
stable. Alan what do you think about adding it to the ac tree?

>   BTW you just undefine I8042_OVERRIDE_KEYLOCK but this define is
>  never used.

Not implemented yet.


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: New Input PS/2 driver
  2001-10-03 19:28 ` James Simmons
@ 2001-10-05 22:50   ` Vojtech Pavlik
  2001-10-05 23:02     ` James Simmons
  0 siblings, 1 reply; 6+ messages in thread
From: Vojtech Pavlik @ 2001-10-05 22:50 UTC (permalink / raw)
  To: James Simmons; +Cc: Etienne Lorrain, Linux Kernel Mailing List

On Wed, Oct 03, 2001 at 12:28:34PM -0700, James Simmons wrote:
> 
> >   Would be nice to change the comment describing the Keyboard core
> >  support (CONFIG_INPUT_KEYBDEV) in this patch: people (as dumb me)
> >  may read the help in menuconfig and say:
> >  I have no USB HID or ADB keyboard...
> >  Recompile, reboot => no keyboard, no control-alt-del => Reset (fsck...).
> 
> Yeah. That does need to be added. 
> 
> >   Else it is working here on a P133 with nothing special (std PS2 mouse).
> 
> >From the several reports I have had with the driver it looks pretty
> stable. Alan what do you think about adding it to the ac tree?
> 
> >   BTW you just undefine I8042_OVERRIDE_KEYLOCK but this define is
> >  never used.
> 
> Not implemented yet.

Quite the opposite: #undef was forgotten in the .h file after the .c
file converted to a runtime option instead of a compiletime one. I
removed it in the CVS now.

-- 
Vojtech Pavlik
SuSE Labs

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: New Input PS/2 driver
  2001-10-05 22:50   ` Vojtech Pavlik
@ 2001-10-05 23:02     ` James Simmons
  2001-10-05 23:07       ` Vojtech Pavlik
  0 siblings, 1 reply; 6+ messages in thread
From: James Simmons @ 2001-10-05 23:02 UTC (permalink / raw)
  To: Vojtech Pavlik; +Cc: Etienne Lorrain, Linux Kernel Mailing List


> > Not implemented yet.
> 
> Quite the opposite: #undef was forgotten in the .h file after the .c
> file converted to a runtime option instead of a compiletime one. I
> removed it in the CVS now.

  Okay. Another thing is how to deal with IRQ's and the port regions. This
can vary from platform to platform. We could have this as a command line
option as well. In fact we might since it can be built as a module. Alot
of platforms added things to the command line inside the kernel code.
  Or we can do lots of #ifdef in i8042.h or using the asm/keyboard method
like now. Personally I don't like this method since even on mips the i8042
port range varies on different machines. So we still end up with a bunch
of messy #ifdef. 


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: New Input PS/2 driver
  2001-10-05 23:02     ` James Simmons
@ 2001-10-05 23:07       ` Vojtech Pavlik
  0 siblings, 0 replies; 6+ messages in thread
From: Vojtech Pavlik @ 2001-10-05 23:07 UTC (permalink / raw)
  To: James Simmons; +Cc: Etienne Lorrain, Linux Kernel Mailing List

On Fri, Oct 05, 2001 at 04:02:08PM -0700, James Simmons wrote:
> 
> > > Not implemented yet.
> > 
> > Quite the opposite: #undef was forgotten in the .h file after the .c
> > file converted to a runtime option instead of a compiletime one. I
> > removed it in the CVS now.
> 
>   Okay. Another thing is how to deal with IRQ's and the port regions. This
> can vary from platform to platform. We could have this as a command line
> option as well. In fact we might since it can be built as a module. Alot
> of platforms added things to the command line inside the kernel code.
>   Or we can do lots of #ifdef in i8042.h or using the asm/keyboard method
> like now. Personally I don't like this method since even on mips the i8042
> port range varies on different machines. So we still end up with a bunch
> of messy #ifdef. 

I'd prefer a bunch of #ifdefs in i8042.h with defaults and a command
line option to change that if needed. For some of the more differing
architectures (Sun Ultra AX) which have i8042-alike chips in them
complete separate drivers may make sense as well.


-- 
Vojtech Pavlik
SuSE Labs

^ permalink raw reply	[flat|nested] 6+ messages in thread

* New Input PS/2 driver 
@ 2001-10-02  4:47 James Simmons
  0 siblings, 0 replies; 6+ messages in thread
From: James Simmons @ 2001-10-02  4:47 UTC (permalink / raw)
  To: Linux Kernel Mailing List; +Cc: Linux console project


Okay. It is not that new since I have been using it for several months now
with now problems. I just ported from console project CVS to 2.4.10.
Basically this patch address the issues people have been having with the
current PS/2 driver. Plus it has the bonus of using the universal input
api. It even allows for unpluging the keyboard with no problem and you can
have two keybaord plugged in at the same time with no problems. NOTE:
don't have both keyboard plugged in at boot time. It confusses them. 
The olnly thing which I haven't had time to do is support PS/2 based
keybards that don't use the standard IRQ and port ranges. Several mips
devices have this issue. The patch is 60 K so I posted a link. To use the 
keyboard driver you need to 

1) enable the input core and keyboard support in the input menu. 

2) In the character menu select PS/2 port support and i8042 aux+kbd
   controller. Their are hooks for other types of PS/2 "clones" on other
   platforms.

3) You have 3 different devices to select from. I don't think I have to
   explain them.

   XT keyboard.
   AT and PS/2 keyboards
   PS/2 mouse
   
Here is the diff.

http://www.transvirtual.com/~jsimmons/input-ps2.diff

P.S
  For assumment I have a funny movie clip in the same area(dancemonkeyboy.mpeg).


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2001-10-05 23:07 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2001-10-03 13:34 New Input PS/2 driver Etienne Lorrain
2001-10-03 19:28 ` James Simmons
2001-10-05 22:50   ` Vojtech Pavlik
2001-10-05 23:02     ` James Simmons
2001-10-05 23:07       ` Vojtech Pavlik
  -- strict thread matches above, loose matches on Subject: below --
2001-10-02  4:47 James Simmons

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).