[patch] add usb hub (0x2514 / 0x0424) to usbdevs

Alexander Best alexbestms at wwu.de
Thu Mar 11 20:09:13 UTC 2010


Hans Petter Selasky schrieb am 2010-03-11:
> On Thursday 11 March 2010 13:15:13 Alexander Best wrote:
> > Hans Petter Selasky schrieb am 2010-03-11:
> > > On Thursday 11 March 2010 12:25:46 Alexander Best wrote:
> > > > just a product addition to usbdevs.

> > > > cheers.
> > > > alex

> > > > oh...and one question:

> > > Hi,

> > > See USB P4 ID 175574.

> > thanks. :)

> > > > why does `usbconfig dump_device_desc` say

> > > I think you need to compile the kernel with:

> > > option USB_VERBOSE

> > > Before it works like you expect.

> > options USB_VERBOSE seems broken on r204988:


> Try this patch:

> http://www.mail-archive.com/svn-src-all@freebsd.org/msg02230.html

thanks. now i can build a kernel with USB_VERBOSE. it seems the changes made
by r185998 were lost/reverted during a later commit. my usbdevs is r204632 and
still had the "/* Microdia products */" section in it.

> > Stop in /usr/src.

> > > > although the vendor id is already in usbdevs?

> > > --HPS

> > i'd also like to ask what the purpose of this code in
> > storage/umass.c is:

> > #define UMASS_EXT_BUFFER
> > #ifdef UMASS_EXT_BUFFER
> > /* this enables loading of virtual buffers into DMA */
> > #define UMASS_USB_FLAGS .ext_buffer=1,
> > #else
> > #define UMASS_USB_FLAGS
> > #endif

> When ext_buffer is set, no data is copied. Else data is copied into
> an
> intermediate buffer.


> > isn't that just doing the same as:

> Probably we should expand those macros.

> Send me a patch!

hehe...ok. i'll see what i can do. ;)


> > #define UMASS_EXT_BUFFER
> > /* this enables loading of virtual buffers into DMA */
> > #define UMASS_USB_FLAGS .ext_buffer=1,


> --HPS


More information about the freebsd-usb mailing list