All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Ville Syrjälä" <syrjala@sci.fi>
To: Jiri Kosina <jkosina@suse.cz>
Cc: Peter Stokes <linux@dadeos.co.uk>,
	linux-input@vger.kernel.org, dmitry.torokhov@gmail.com
Subject: Re: [PATCH] ati_remote2 autorepeat and loadable keymap support
Date: Tue, 4 Mar 2008 16:07:07 +0200	[thread overview]
Message-ID: <20080304140707.GB531@sci.fi> (raw)
In-Reply-To: <Pine.LNX.4.64.0803041457200.15209@jikos.suse.cz>

On Tue, Mar 04, 2008 at 03:01:44PM +0100, Jiri Kosina wrote:
> On Tue, 4 Mar 2008, Ville Syrjälä wrote:
> 
> > > I don't seem to see you in MAINTAINERS entry, so that's probably why you 
> > > didn't receive confirmation when Andrew has put this patch into -mm.
> > I guess MODULE_AUTHOR() isn't enough then. I'll send a patch for
> > MAINTAINERS.
> 
> I don't know the exact algorithm Andrew uses to send out CCs when patch is 
> merged, but MAINTAINERS entry would definitely help.
> 
> > Hmm. IIRC I had one USB mouse which also sent repeat events and the 
> > driver didn't seem to filter them out which is why I added checks to 
> > DirectFB's input driver to filter them in user space. Was that a bug in 
> > the usbhid driver or does it depend on the actual hardware? I still have 
> > that mouse so I can re-test the behavior if needed. 
> 
> Button events from mice definitely shouldn't be auto-repeated, if they are 
> reported probably by the hardware (i.e. HID_UP_BUTTON usage page, 
> MOUSE/POINTER application).
> 
> If you could recompile your kernel with CONFIG_HID_DEBUG, modprobe the 
> 'hid' module with 'debug=1' parameter, and send me the output that appears 
> in kernel log when you press (and hold) the button, I will look into it.

I'll do that tonight.

-- 
Ville Syrjälä
syrjala@sci.fi
http://www.sci.fi/~syrjala/
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2008-03-04 14:07 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-16 16:22 [PATCH] ati_remote2 autorepeat and loadable keymap support Peter Stokes
2008-03-03 22:53 ` Jiri Kosina
2008-03-04 12:47 ` Ville Syrjälä
2008-03-04 13:20   ` Jiri Kosina
2008-03-04 13:47     ` Ville Syrjälä
2008-03-04 14:01       ` Jiri Kosina
2008-03-04 14:07         ` Ville Syrjälä [this message]
2008-03-04 18:40         ` Ville Syrjälä
2008-03-04 18:50           ` Jiri Kosina
2008-03-04 19:53             ` Ville Syrjälä
2008-03-04 20:28               ` Jiri Kosina
2008-03-04 20:32                 ` Ville Syrjälä
2008-03-04 20:42                   ` Jiri Kosina
2008-03-04 21:21                     ` Ville Syrjälä
2008-03-04 22:02                       ` Jiri Kosina
2008-03-04 18:55   ` Peter Stokes
2008-03-04 20:38     ` Ville Syrjälä
2008-03-04 21:34       ` Peter Stokes
2008-03-04 22:17         ` Ville Syrjälä
2008-03-04 22:25           ` Jiri Kosina
2008-03-25  8:23             ` Jiri Kosina
2008-04-07 20:39               ` Dmitry Torokhov

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=20080304140707.GB531@sci.fi \
    --to=syrjala@sci.fi \
    --cc=dmitry.torokhov@gmail.com \
    --cc=jkosina@suse.cz \
    --cc=linux-input@vger.kernel.org \
    --cc=linux@dadeos.co.uk \
    /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.