[haiku-bugs] Re: [Haiku] #5940: Header file for the USB video driver
- From: "mmlr" <trac@xxxxxxxxxxxx>
- Date: Sat, 29 Oct 2011 15:35:28 -0000
#5940: Header file for the USB video driver
---------------------------+-------------------------------
Reporter: Karvjorm | Owner: mmlr
Type: enhancement | Status: new
Priority: normal | Milestone: R1
Component: Drivers/USB | Version: R1/Development
Resolution: | Keywords: USB video drivers
Blocked By: | Blocking:
Has a Patch: 1 | Platform: All
---------------------------+-------------------------------
Comment (by mmlr):
Replying to [comment:7 Karvjorm]:
> My header file is not ready yet, but maybe I can add it already now and
update it later.
Hi there. The header looks basically fine by now, what do you mean by "not
ready yet"? The only thing I don't like is that the structs aren't
declared with the same name as they are typedef'ed to, meaning two names
polluting the namespace (i.e. typedef struct struct_name {}
usb_video_struct_name). Also the work on the UVC webcam add-on has sprung
a USB_video.h of its own in src/add-ons/media/media-add-
ons/usb_webcam/addons/uvc/USB_video.h which looks like it (obviously)
defines some of the same values. I'd prefer to use the header attached
here for its completeness and have the other one removed and dependent
code "ported" to this header. For now the attached header could be
commited as soon as the struct naming thing is clear.
--
Ticket URL: <http://dev.haiku-os.org/ticket/5940#comment:8>
Haiku <http://dev.haiku-os.org>
Haiku - the operating system.
Other related posts:
