[PATCH synaptics] Adding support for OpenBSD

Alexandr Shadchin alexandr.shadchin at gmail.com
Tue Aug 23 21:35:12 PDT 2011


On Wed, Aug 24, 2011 at 5:57 AM, Peter Hutterer
<peter.hutterer at who-t.net> wrote:
> On Tue, Aug 23, 2011 at 09:34:58PM +0600, Alexandr Shadchin wrote:
>> On Tue, Aug 23, 2011 at 01:20:10PM +1000, Peter Hutterer wrote:
>> > > +static void
>> > > +WSConsDeviceOnHook(InputInfoPtr pInfo, SynapticsParameters *para)
>> > > +{
>> > > +    int wsmouse_mode = WSMOUSE_NATIVE;
>> > > +
>> > > +    if (ioctl(pInfo->fd, WSMOUSEIO_SETMODE, &wsmouse_mode) == -1)
>> > > +        xf86IDrvMsg(pInfo, X_ERROR, "cannot set native mode\n");
>> > > +}
>> > > +
>> > > +static void
>> > > +WSConsDeviceOffHook(InputInfoPtr pInfo)
>> > > +{
>> > > +    int wsmouse_mode = WSMOUSE_COMPAT;
>> > > +
>> > > +    if (ioctl(pInfo->fd, WSMOUSEIO_SETMODE, &wsmouse_mode) == -1)
>> > > +        xf86IDrvMsg(pInfo, X_ERROR, "cannot set compat mode\n");
>> > > +}
>> >
>> > if these two are an error, we should change the On/Off hooks to return
>> > boolean so we can bail out of the caller. that'd be a separate patch, at
>> > least eventcomm.c needs it too.
>> >
>>
>> Hmm, in http://cgit.freedesktop.org/xorg/driver/xf86-input-synaptics/
>> On/Off hooks is 'static void'. Maybe it's in your local repository?
>> Could not you show this patch?
>
> sorry, I meant this is an unrelated patch that should be done too. Quietly
> failing in the device backends is not a good thing imo.
>
> Cheers,
>  Peter
>

Now I understand you. I'll do it.

-- 
Alexandr Shadchin


More information about the xorg-devel mailing list