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