Subject: Re: kern/25960
To: None <kern-bug-people@netbsd.org, gnats-admin@netbsd.org,>
From: Hans Petter Selasky <hselasky@c2i.net>
List: netbsd-bugs
Date: 06/08/2005 15:55:02
The following reply was made to PR kern/25960; it has been noted by GNATS.
From: Hans Petter Selasky <hselasky@c2i.net>
To: gnats-bugs@NetBSD.org
Cc:
Subject: Re: kern/25960
Date: Wed, 8 Jun 2005 17:55:19 +0200
Hi,
sce->isorate must be initialized somewhere.
Also it must be range checked.
Is USBD_SET_RATE or USBD_GET_RATE documented somewhere.
I think it is not so smart to pass the total number of
bytes per second. What is important is the frame size,
to synchronize data. Therefore USBD_SET_FRAME_SIZE and
USBD_GET_FRAME_SIZE would be better. And more generic!
Can you change this?
Remember there is USB2.0 with 1000*8 frames per second.
So USBD_SET_RATE might require two different values for
the same application, which is not very good.
PS: EHCI ISOCHRONOUS support at:
http://home.c2i.net/hselasky/isdn4bsd/privat/usb/
--HPS