Re: kern/25960

[Hans Petter Selasky]

The following reply was made to PR kern/25960; it has been noted by GNATS.

From: Hans Petter Selasky <hsel...@c2i.net>
To: gnats...@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