Re: Elo touch panel

Re: Elo touch panel

[Shaun Jackman]

On 2/7/06, Vojtech Pavlik <vojtech@suse.cz> wrote:
> On Mon, Feb 06, 2006 at 07:30:26PM -0700, Shaun Jackman wrote:
...
> > I then run...
> >       $ inputattach -elo /dev/ttyS3
> > .. at which point dmesg says...
> >       serio: Serial port ttyS3
> > but it never gives a message indicating that the Elo driver has been
> > attached to ttyS3. Is there anything else I must configure?
>
> Try to look at /proc/bus/input/devices. The elo_connect() function
> should get called at the point the port is created. Unfortunately, I
> never could test this driver, because I don't have any touchscreens
> anymore (ever had one and it broke when I was moving). So there may be
> problems in the driver. But I believe others had some success running
> it.
>
> Looking at the driver, one will probably need to set the panel type in
> inputattach ...

There's a bug in inputattach. The SERIO_RS232 constant is completely
mucking the bits that specify the protocol.

--- inputattach.c-      2006-02-07 14:37:04.000000000 -0700
+++ inputattach.c       2006-02-07 16:22:07.000000000 -0700
@@ -455,7 +455,7 @@
                return 1;
        }

-       devt = SERIO_RS232 | input_types[type].type | (id << 8) | (extra << 16);
+       devt = input_types[type].type | (id << 8) | (extra << 16);

        if(ioctl(fd, SPIOCSTYPE, &devt)) {
                fprintf(stderr, "inputattach: can't set device type\n");

Cheers,
Shaun

APPENDIX A

You probably know this better than I do, but the .type above is
misnamed. It should really be .proto. It's impossible, as far as I can
tell, to specify the type (like SERIO_RS232) with a SPIOCSTYPE call.

Re: Elo touch panel

[Vojtech Pavlik]

On Tue, Feb 07, 2006 at 04:41:16PM -0700, Shaun Jackman wrote:

> There's a bug in inputattach. The SERIO_RS232 constant is completely
> mucking the bits that specify the protocol.
> 
> --- inputattach.c-      2006-02-07 14:37:04.000000000 -0700
> +++ inputattach.c       2006-02-07 16:22:07.000000000 -0700
> @@ -455,7 +455,7 @@
>                 return 1;
>         }
> 
> -       devt = SERIO_RS232 | input_types[type].type | (id << 8) | (extra << 16);
> +       devt = input_types[type].type | (id << 8) | (extra << 16);
> 
>         if(ioctl(fd, SPIOCSTYPE, &devt)) {
>                 fprintf(stderr, "inputattach: can't set device type\n");
> 
> Cheers,
> Shaun
> 
> APPENDIX A
> 
> You probably know this better than I do, but the .type above is
> misnamed. It should really be .proto. It's impossible, as far as I can
> tell, to specify the type (like SERIO_RS232) with a SPIOCSTYPE call.
 
It's an incompatibility in a later rework of serio.h/serport.c.
SERIO_RS232 used to be 0x02000000, now the constant isn't shifted up
anymore, which messed up things. Also, the type used to be possible to
set, but I don't think that needs to be fixed, since there are so far no
ttys that'd need a different serio type. 

I'll apply your fix to inpuatttach. Thanks for spotting the bug.

-- 
Vojtech Pavlik
SuSE Labs, SuSE CR