All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anssi Hannula <anssi.hannula@gmail.com>
To: Jiri Kosina <jkosina@suse.cz>
Cc: linux-input@vger.kernel.org
Subject: Re: multi-input quirk and skip output reports quirk
Date: Sun, 06 Apr 2008 17:12:40 +0300	[thread overview]
Message-ID: <47F8DA58.10705@gmail.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0804061003490.26562@twin.jikos.cz>

Jiri Kosina wrote:
> On Tue, 1 Apr 2008, Anssi Hannula wrote:
> 
>> I've encountered another multi-input device that needs the skip output 
>> reports quirk (as it contains force feedback output reports), 0925:8866.
>> However, maybe we should use another approach, dropping
>> HID_QUIRK_SKIP_OUTPUT_REPORTS. Why not just *not* register the input
>> device for that report in case the report didn't contain anything that
>> was mapped by hidinput_configure_usage(), in hidinput_connect()?
> 
> Oh, and I seem to have completely forgotten -- is this quirk needed at 
> all? Since commit 82eb12198 we ignore the non-LED output reports in 
> hidinput_configure_usages() altogether.
> 
> Do you still actually see issues of devices being misconfigured due to 
> output reports confusion in hidinput_configure_usage() even in kernels 
> having this commit included?

Yes.

The point was that hidinput_connect() doesn't care whether all reports
were ignored by hidinput_configure_usage() or not.

-- 
Anssi Hannula

  reply	other threads:[~2008-04-06 14:12 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-01 11:25 multi-input quirk and skip output reports quirk Anssi Hannula
2008-04-01 14:12 ` Jiri Kosina
2008-04-01 14:15   ` Anssi Hannula
2008-04-04 14:37     ` Jiri Kosina
2008-04-04 14:40       ` Anssi Hannula
2008-04-04 14:51         ` Jiri Kosina
2008-04-06  8:06 ` Jiri Kosina
2008-04-06 14:12   ` Anssi Hannula [this message]
  -- strict thread matches above, loose matches on Subject: below --
2008-04-05  6:01 Sam Liddicott
2008-04-05  6:01 ` Jiri Kosina
2008-04-05  6:16 Sam Liddicott

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=47F8DA58.10705@gmail.com \
    --to=anssi.hannula@gmail.com \
    --cc=jkosina@suse.cz \
    --cc=linux-input@vger.kernel.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 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.