[haiku-bugs] Re: [Haiku] #5940: Header file for the USB video driver
- From: Jorma Karvonen <karvonen.jorma@xxxxxxxxx>
- To: haiku-bugs@xxxxxxxxxxxxx
- Date: Sat, 29 Oct 2011 19:06:37 +0300
Hi,
I'm sorry to reply via this way, but my login to Trac does not work
and even password reset failed.
If I can recall it correct, there were couple of tables in the USB
Video specification that I did not quite understand and therefore
specification is not 100 percent completed. I planned to return to
these tables later, but forgot. But you can well use the present
header file now, if you want. Also, you can change typedefs so that
there is only one struct name in the namespace.
Best regards,
Karvjorm
2011/10/29 mmlr <trac@xxxxxxxxxxxx>:
> #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:
