> >     - Shows high bandwidth endpoints with something like
> >       "1024*3" for maxpacket size; other endpoints just
> >       have two extra spaces there.
> 
> Why have the "*n" here?  Why not just spit out the whole number?  That
> would break less parsers :)

Because the "high bandwidth" mode doesn't affect
the maximum packet size ... it allows multiple packets
to be issued per microframe, each of up to maxpacket.
It's a new (bit)field in the endpoint descriptor.


> And remember to also modify Documentation/usb/proc_usb_info.txt if you
> modify the layout.

Good point, I'll submit that soon.


> > That means this patch could break some programs; I don't
> > know why any would bother parsing such lines.  The change
> > should be easily handled by detecting parsing errors.
> 
> Did you try running usbview or Randy's perl scripts?  I also think the
> KDE control panel parses that file.
>
> And I don't mind breaking backward compatiblilty to fix things, so don't
> be afraid of it :)

Don't have most of those on my development machine, so no.
(One of these days I should upgrade it.)

But since Randy posted his scripts I'll see how they act ... if they
break I probably can't fix, I hack Java not Perl.  :)

- Dave



_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to