Re: [GIT PATCH] HID and USB HID update for 2.6.21-rc2

[Jiri Kosina]

On Wed, 28 Feb 2007, Linus Torvalds wrote:

> > But OK, I will leave it in there.
> No. You need to realize just WHY it was wrong. Not just an "But OK".

Yep, I totally agree that with the usbhid.h thing I really had a bad day, 
it was braindamage without excuse, sorry.

I still think that creating a separate header file solely for purpose of 
having the large hid blacklist and all related defines separate from the 
actual implementation is needed. The pages and pages of blacklist just 
pollute the hid-core.c needlessly. 

Of course hid-core.c must be the only user of this header. But seems like 
this solution oposes your taste ("The point of a header file is to 
*declare* things, not to have big data structures in"), so I would 
probably not go this way.

Thanks,

-- 
Jiri Kosina
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/