From: "Daniel P. Berrange" <berrange@redhat.com>
To: Markus Armbruster <armbru@redhat.com>
Cc: Gerd Hoffmann <kraxel@suse.de>,
Xen devel list <xen-devel@lists.xensource.com>
Subject: Re: [patch] pvfb: Split mouse and keyboard into separate devices.
Date: Wed, 7 Feb 2007 14:36:48 +0000 [thread overview]
Message-ID: <20070207143648.GA26258@redhat.com> (raw)
In-Reply-To: <877iuu2g5f.fsf@pike.pond.sub.org>
On Wed, Feb 07, 2007 at 03:31:24PM +0100, Markus Armbruster wrote:
> Gerd Hoffmann <kraxel@suse.de> writes:
>
> > Hi,
> >
> > This patch creates two separate input devices for keyboard and mouse
> > events. Also includes some key bitmap fixes (allow all keyboard keys,
> > allow eight mouse buttons).
> >
> > I hope everyone is happy with that now after the lengthy discussion ;)
> >
> > please apply,
> > Gerd
> >
> > --
> > Gerd Hoffmann <kraxel@suse.de>
> > pvfb: Split mouse and keyboard into separate devices.
> >
> > This patch creates two separate input devices for keyboard and mouse
> > events. The reason for this is to separate them in the linux input
> > layer and allow them being routed different ways.
> >
> > Use case: Configure the X-Server like this to get the mouse
> > events directly from the linux input layer, which has the major
> > advantage that absolute coordinates work correctly:
> >
> > Section "InputDevice"
> > Driver "evdev"
> > Identifier "Mouse"
> > Option "Device" "/dev/input/event<nr>"
> > EndSection
> >
> > This makes the keyboard stop working though in case mouse and
> > keyboard events are coming through the same input device, at least
> > with older Xorg (6.9) versions.
> >
> > Signed-off-by: Gerd Hoffmann <kraxel@suse.de>
>
> New, not mentioned in the changelog:
>
> * Initialization of struct input_dev members phys, id.bustype,
> id.vendor, id.product.
>
> * Take care of the FIXME regarding initialization struct input_dev
> member keybit (thanks!).
>
> * Take care of the TODO to enable all pointer buttons. Perhaps should
> better go in together with the fix to tools/xenfb/vncfb.c posted by
> Daniel.
Yep, the kernel portion of my patches are obsoleted by Gerd's updated
patch. The userspace portion still needs applying.
I've no objections to Gerd's patch being applied.
Regards,
Dan.
--
|=- Red Hat, Engineering, Emerging Technologies, Boston. +1 978 392 2496 -=|
|=- Perl modules: http://search.cpan.org/~danberr/ -=|
|=- Projects: http://freshmeat.net/~danielpb/ -=|
|=- GnuPG: 7D3B9505 F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 -=|
next prev parent reply other threads:[~2007-02-07 14:36 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-02-07 11:31 [patch] pvfb: Split mouse and keyboard into separate devices Gerd Hoffmann
2007-02-07 14:31 ` Markus Armbruster
2007-02-07 14:36 ` Daniel P. Berrange [this message]
-- strict thread matches above, loose matches on Subject: below --
2007-02-01 10:59 Gerd Hoffmann
2007-02-01 13:15 ` Markus Armbruster
2007-02-01 13:47 ` Gerd Hoffmann
2007-02-01 17:37 ` Markus Armbruster
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20070207143648.GA26258@redhat.com \
--to=berrange@redhat.com \
--cc=armbru@redhat.com \
--cc=kraxel@suse.de \
--cc=xen-devel@lists.xensource.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.