On 2022/05/11 07:54, Anton Lindqvist wrote:
> On Tue, May 10, 2022 at 02:51:07PM +0100, Stuart Henderson wrote:
> > On 2022/05/10 08:12, Anton Lindqvist wrote:
> > > On Mon, May 09, 2022 at 05:44:29PM +0100, Stuart Henderson wrote:
> > > > I have a USB combi keyboard/trackpad thing which is triggering "bad
> > > > repid 33" frequently while attached (between a couple of times a minute,
> > > > and once every few minutes). It does work but it's annoying.
> > > >
> > > > Presumably this is because it has non-contiguous report IDs?
> > > > Anyone have an idea how to handle it?
> > >
> > > Could you send me the raw report descriptors:
> > >
> > > $ (set -e; i=0; while :; do doas usbhidctl -f /dev/uhid$i -R
> > > >/tmp/uhid$i.raw; i=$((i + 1)); done)
> > >
>
> My guess here is that uhidev_maxrepid() does not observe all hid items
> as it passes hid_none as the kind to hid_start_parse(). The userspace
> equivalent of hid_start_parse() accepts a kind bitmask allowing
> hid_get_item() to return items of varying kinds. For the kernel, we
> could treat hid_none as a sentinel representing all possible hid kinds.
> Does the following diff get rid of the bad repid output?
Running it now and I haven't seen any yet, including after pressing the
strange keys and using the trackpad, so I think it works :) Thanks.
> diff --git sys/dev/hid/hid.c sys/dev/hid/hid.c
> index 1c4d5fa45e0..dd03d6d8943 100644
> --- sys/dev/hid/hid.c
> +++ sys/dev/hid/hid.c
> @@ -229,7 +229,7 @@ hid_get_item(struct hid_data *s, struct hid_item *h)
> * Only copy HID item, increment position and return
> * if correct kind!
> */
> - if (s->kind == c->kind) {
> + if (s->kind == hid_none || s->kind == c->kind) {
> *h = *c;
> DPRINTF("%u,%u,%u\n", h->loc.pos,
> h->loc.size, h->loc.count);
>