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 <[email protected]>

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 -- [email protected]

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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to