Re: [PATCH] m68k: Fix atarimouse init

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

 



On Sun, 28 Dec 2008, Michael Schmitz wrote:
following up on Geert's report that atarimouse does fail to load as a module 
(with ENODEV), here's the fix. 

Brown paper bag, well and truely. Got the semantics of the atari_keyb_init 
return reversed (not that atari_keyb_init ever returns anything but success, but 
that's for another day).

I still get no mouse events in either /dev/input/mice or .../mouse0, though. I 
suspect the mouse open hook never gets called. Same bug might affect amimouse 
(though the open function does do the full hardware setup there).

Thanks, we're making progress :-)

I get events from /dev/input/mouse0 and /dev/input/mice (atari_defconfig).

The kernel says (ARAnyM, windowed mode):
| input: Atari mouse as /devices/virtual/input/input1
| Reseting weird mouse packet: 942, 379, 0
| Reseting weird mouse packet: 943, 379, 0
| Reseting weird mouse packet: 945, 378, 0
| Reseting weird mouse packet: 948, 378, 0
| Reseting weird mouse packet: 956, 376, 0

When running X under ARAnyM (depth 16), some shortcomings show up:
  - up and down are interchanged (works fine when rolling the mouse over the
    bottom surface of the table --- old arabic map style ;-)
  - the left and right mouse button are interchanged
  - the middle mouse button is dead

Please put a `---' here, so `git am' will strip the signature and the diffstat.

	Michael

 drivers/input/mouse/atarimouse.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/input/mouse/atarimouse.c b/drivers/input/mouse/atarimouse.c
index a57143c..1b5f4dd 100644
--- a/drivers/input/mouse/atarimouse.c
+++ b/drivers/input/mouse/atarimouse.c
@@ -118,7 +118,7 @@ static int __init atamouse_init(void)
 	if (!MACH_IS_ATARI || !ATARIHW_PRESENT(ST_MFP))
 		return -ENODEV;
 
-	if (!atari_keyb_init())
+	if (atari_keyb_init())
 		return -ENODEV;
 
 	atamouse_dev = input_allocate_device();

Signed-off-by: Michael Schmitz <schmitz@xxxxxxxxxx>

Please put the SoB just below the changelog, else `git am' will strip it off.

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Video for Linux]     [Yosemite News]     [Linux S/390]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux