[PATCH] Several compat additions
- From: Trent Piepho <xyzzy (at) speakeasy.org>
- Date: Sun, 14 May 2006 08:40:03 -0700 (PDT)
On Sun, 14 May 2006 christophpfister (at) bluemail.ch wrote:
>
> Just a general consideration (I didn't follow this thread, so I may be wrong...):
>
> If only the order of includes matters, I'd suggest to include linux/usb.h
> and linux/input.h inside compat.h. (That shouldn't bother bother non-usb
> devices.)
My original patch said what is wrong with that. linux/usb.h defines a number
of symbols that conflict with other code in non-usb drivers. You can't add
linux/usb.h to compat.h for drivers that didn't have it already. I've come up
with two options:
1. Require that linux/usb.h and linux/input.h are included before compat.h
to turn on linux/usb_input.h compatibility.
2. Define V4L_COMPAT_USB_INPUT before including compat.h to turn on
linux/usb_input.h compatibility.
The first one is nicer looking and cleaner, but the second would be easier to
strip out be gentree.pl. Of course the first doesn't NEED to be stripped,
there is nothing wrong with putting an include in another spot, it's just
a cosmetic change.
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request (at) redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list