linux-input.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
To: Henrik Rydberg <rydberg@euromail.se>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>,
	aris@cathedrallabs.org, linux-input@vger.kernel.org
Subject: Re: [PATCH] uinput: allow events per packet to be specified
Date: Thu, 3 Mar 2011 00:06:08 -0800	[thread overview]
Message-ID: <20110303080608.GA29929@core.coreip.homeip.net> (raw)
In-Reply-To: <20110302171357.GB1553@polaris.bitmath.org>

On Wed, Mar 02, 2011 at 06:13:57PM +0100, Henrik Rydberg wrote:
> > > I agree that adding this ioctl is symmetrical from the a device
> > > emulation standpoint, but I was imagining eventually replacing the eep
> > > with an automatic scheme, or at least with a more realtime-oriented
> > > parameter. Fixating the interface as you propose will make such a
> > > transition much harder.
> > 
> > This is something we actually need rather than are having future
> > visions about, so imaginings for the future don't really help here
> > unfortunately. I'd obviously rather have a standard API than get into the
> > situation where more and more devices and distros simply won't run an
> > upstream kernel.
> > 
> > If the long term theoretical transition is a concern then we could simply
> > make the "set" behaviour
> > 
> > Set:	value is used as a hint to the uinput driver and input layer
> > 	that a given amount of buffering may be required. It may be
> > 	ignored by the input layer.
> > 
> > and if the long term imagining ever happens the interface cannot be an
> > implediment. It won't cause any problems because if the automatic
> > interfaces work then the hint isn't needed, and if  they never get
> > implemented or it turns out they never to work for all cases (most
> > likely I suspect) then you'll still need the hint.
> 
> Sure, we may need a hint, but if the better hint turns out to be
> events_per_millisecond rather than events_per_packet, we are still in
> a bad situation. Let's see what Dmitry thinks about this. I don't mind
> making the accompanying changes right away, if that helps you.
> 

Umm, before we do that could we get an idea what kind of device is that
that our current hints are not sufficient? Is the rate is so high
because the device is too noisy?

Thanks.

-- 
Dmitry

      reply	other threads:[~2011-03-03  8:06 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-01 15:47 [PATCH] uinput: allow events per packet to be specified Alan Cox
2011-03-01 16:35 ` Aristeu Rozanski
2011-03-02 14:13 ` Henrik Rydberg
2011-03-02 16:06   ` Alan Cox
2011-03-02 17:13     ` Henrik Rydberg
2011-03-03  8:06       ` Dmitry Torokhov [this message]

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=20110303080608.GA29929@core.coreip.homeip.net \
    --to=dmitry.torokhov@gmail.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=aris@cathedrallabs.org \
    --cc=linux-input@vger.kernel.org \
    --cc=rydberg@euromail.se \
    /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;
as well as URLs for NNTP newsgroup(s).