Ragner Magalhaes wrote:
> [...]
> +
> +#ifndef __LINUX_USB_VIDEO_H
> +#define __LINUX_USB_VIDEO_H
> +

>[...]

> +#define USB_VC_HEADER_SIZE(n)                (12+(n))
> +
> +/* As above, but more useful for defining your own descriptors: */
> +#define DECLARE_USB_VC_HEADER_DESCRIPTOR(n)                  \
> +struct usb_vc_header_descriptor_##n {                                \
> +     __u8  bLength;                                          \
> +     __u8  bDescriptorType;                                  \
> +     __u8  bDescriptorSubtype;                               \
> +     __le16 bcdUVC;                                          \
> +     __le16 wTotalLength;                                    \
> +     __le32 dwClockFrequency;                                \
> +     __u8  bInCollection;                                    \
> +     __u8  baInterfaceNr[n];                                 \
> +} __attribute__ ((packed))
> +
> +#endif
    ^^^^^^

I just noticed this: why is everything below this point declared outside 
"#ifndef __LINUX_USB_VIDEO_H" ?

That will produce double definitions if "linux/usb/video.h" is included 
more than once, no?

-- 
Paulo Marques - www.grupopie.com

"Prediction is hard. Especially of the future."
Niels Bohr

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to