lkml.org 
[lkml]   [2008]   [Oct]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
Subject[PATCH 2/2] HID: fix appletouch regression (was Re: several messages)

[ restored previous subject, sorry for the "several messagess" mess,
alpine didn't handle multi-message reply with different subjects nicely ]

On Mon, 20 Oct 2008, Jiri Slaby wrote:

> >> +enum hid_type {
> >> + HID_TYPE_UNKNOWN = 0,
> >> + HID_TYPE_MOUSE,
> >> + HID_TYPE_KEYBOARD
> > Do we really need the HID_TYPE_KEYBOARD at all? It's not used anywhere
> > in the code. I'd propose to add it when it is actually needed. I.e.
> > have the enum contain something like HID_TYPE_MOUSE HID_TYPE_OTHER for
> > now, and add whatever will become necessary in the future, what do you
> > think?
> I would use unknown rather than other, since on bluetooth mouse is unknown
> not other, if you don't mind?
> Or did you mean tristate unknown, mouse and other?

Well, basically the points I am trying to make:

1) the code doesn't currently need HID_TYPE_KEYBOARD at all

and

2) by definition, HID are not just mice and keyboards, so adding just
these two doesn't look particularly correct, if just one of them is
needed

For bluetooth we even don't have the possibility to distinguish these,
yes. So probably something like HID_TYPE_USBMOUSE/HID_TYPE_OTHER for now,
and we could extend the enum whenever the need to recognize a different
type comes up?

Thanks,

--
Jiri Kosina
SUSE Labs


\
 
 \ /
  Last update: 2008-10-20 00:27    [W:0.028 / U:0.296 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site