Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: "Maxin B. John" <maxin.john@intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 4/4] pointercal-xinput: add a dummy calibration file for qemu
Date: Thu, 16 Jun 2016 09:43:40 +0100	[thread overview]
Message-ID: <1466066620.13979.256.camel@linuxfoundation.org> (raw)
In-Reply-To: <20160616080546.GA12991@mbabyjoh-desk.ger.corp.intel.com>

On Thu, 2016-06-16 at 11:05 +0300, Maxin B. John wrote:
> Hi RP,
> 
> On Wed, Jun 15, 2016 at 05:58:18PM +0100, Richard Purdie wrote:
> > On Wed, 2016-06-15 at 16:13 +0300, Maxin B. John wrote:
> > > In qemu, the emulated PS/2 mouse reports itself as an "absolute
> > > coordinate"
> > > device and that makes xinput_calibrator think it could be
> > > calibrated.
> > > 
> > > Add a dummy calibration file as a work around to prevent
> > > xinput_calibrator from
> > > popping up on every boot in qemu.
> > > 
> > > [YOCTO #8380]
> > > 
> > > Signed-off-by: Maxin B. John <maxin.john@intel.com>
> > > Signed-off-by: Jussi Kukkonen <jussi.kukkonen@intel.com>
> > 
> > This doesn't sound right. In qemu, we add a usb touchpad which
> > allows
> > absolute coordinate entry, which means qemu doesn't have to grab
> > the
> > mouse.
> > 
> > Are there two devices? A mouse shouldn't be seen as an absolute
> > coordinate device.
> 
> Yes, both qemu monitor and xinput_calibrator in target shows two
> devices:
> 
> (qemu) info mice
>   Mouse #2: QEMU PS/2 Mouse
>   * Mouse #3: QEMU HID Tablet (absolute)
> 
> #  xinput_calibrator --list
>  Device "QEMU QEMU USB Tablet" id=8
>  Device "ImExPS/2 Generic Explorer Mouse" id=9

So with these patches, are we still calibrating the tablet input?

pointercal ships default calibrations for that under the current setup
but those likely aren't compatible with xinput and we'd need the same
thing but for xinput here?

Cheers,

Richard




  reply	other threads:[~2016-06-16  8:43 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-15 13:12 [PATCH 1/4] xtscal: remove recipe Maxin B. John
2016-06-15 13:12 ` [PATCH 2/4] x11-common: replace xtscal with xinput-calibrator Maxin B. John
2016-06-15 13:12 ` [PATCH 3/4] distro_alias.inc: remove xtscal Maxin B. John
2016-06-15 13:13 ` [PATCH 4/4] pointercal-xinput: add a dummy calibration file for qemu Maxin B. John
2016-06-15 16:58   ` Richard Purdie
2016-06-16  8:05     ` Maxin B. John
2016-06-16  8:43       ` Richard Purdie [this message]
2016-06-16 11:11         ` Maxin B. John

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=1466066620.13979.256.camel@linuxfoundation.org \
    --to=richard.purdie@linuxfoundation.org \
    --cc=maxin.john@intel.com \
    --cc=openembedded-core@lists.openembedded.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox