Messages in this thread | | | Date | Wed, 28 Feb 2007 09:09:13 -0800 (PST) | From | Linus Torvalds <> | Subject | Re: [GIT PATCH] HID and USB HID update for 2.6.21-rc2 |
| |
On Wed, 28 Feb 2007, Jiri Kosina wrote: > > The diffstat looks larger because the usbhid code is moved from > USB-specific directory to HID-specific directory
No. The diffstat looks huge because you moved "hid_blacklist" into a header file, and that is a big enough change that git won't consider the rename to be just a rename any more (you basically moved the old usbdrivers/usb/input/hid-core.c file into *two* files: hid-core.c and usbhid.h).
Why do that? It now gets included INTO EVERY DAMN FILE that includes <usbhid.h>, since that one now has that
static const struct hid_blacklist
definition in it. Yet *nothing* wants it, except for the one C file that it used to be in.
No way in hell will I include this kind of crap without major explanations for why the "crap" is not crap. And quite frankly, I don't think such explanations exist.
I'm also not going to pull it if you just add a new commit to undo the idiocy. That thing needs to be totally re-done, as far as I'm concerned. I don't want to touch anything that has EVER even *seen* anything that stupid.
Yes, I'm grumpy. I don't like big changes at this stage, and if they are also STUPID big changes, as this seems to be, I refuse to pull them entirely.
Linus - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |