2003-03-26 00:43:56

by Greg KH

[permalink] [raw]
Subject: [BK PATCH] USB changes for 2.5.66

Hi,

Here are some small USB changes. Basically all little cleanups and
bugfixes, nothing major.

Please pull from: bk://linuxusb.bkbits.net/linus-2.5

thanks,

greg k-h

CREDITS | 4
MAINTAINERS | 5
drivers/usb/Makefile | 1
drivers/usb/class/audio.c | 1
drivers/usb/class/cdc-acm.c | 2
drivers/usb/core/buffer.c | 1
drivers/usb/core/hcd.c | 9
drivers/usb/core/hub.c | 56 +--
drivers/usb/core/message.c | 1
drivers/usb/core/usb-debug.c | 1
drivers/usb/core/usb.c | 13
drivers/usb/host/ohci-hcd.c | 1
drivers/usb/image/mdc800.c | 1
drivers/usb/media/ov511.c | 1
drivers/usb/media/stv680.c | 1
drivers/usb/misc/emi26.c | 37 +-
drivers/usb/misc/speedtch.c | 646 ++++++++++++++++------------------------
drivers/usb/misc/usbtest.c | 7
drivers/usb/net/cdc-ether.c | 19 -
drivers/usb/serial/kobil_sct.c | 17 -
drivers/usb/storage/isd200.c | 5
drivers/usb/storage/scsiglue.c | 54 +--
drivers/usb/storage/transport.c | 3
drivers/usb/storage/usb.c | 6
drivers/usb/usb-skeleton.c | 6
25 files changed, 385 insertions(+), 513 deletions(-)
-----

[email protected], 2003-03-25 15:16:26-08:00, [email protected]
USB: remove unneeded #include <linux/version.h>

drivers/usb/class/audio.c | 1 -
drivers/usb/core/buffer.c | 1 -
drivers/usb/core/usb-debug.c | 1 -
drivers/usb/host/ohci-hcd.c | 1 -
drivers/usb/image/mdc800.c | 1 -
drivers/usb/media/ov511.c | 1 -
drivers/usb/media/stv680.c | 1 -
7 files changed, 7 deletions(-)
------

[email protected], 2003-03-25 14:13:46-08:00, [email protected]
[PATCH] USB: CDC Ethernet maintainer transfer

CREDITS | 4 ++++
MAINTAINERS | 5 +++--
2 files changed, 7 insertions(+), 2 deletions(-)
------

[email protected], 2003-03-25 14:11:56-08:00, [email protected]
[PATCH] USB: CDC Ethernet zero packet fix

drivers/usb/net/cdc-ether.c | 19 ++++---------------
1 files changed, 4 insertions(+), 15 deletions(-)
------

[email protected], 2003-03-25 13:59:05-08:00, [email protected]
[PATCH] USB: fix compiler warning in usb-storage

drivers/usb/storage/scsiglue.c | 1 -
1 files changed, 1 deletion(-)
------

[email protected], 2003-03-25 13:49:07-08:00, [email protected]
[PATCH] usb-storage: cleanup

This patch changes some debugging output to be a bit more clear, and
removes some un-needed code -- it's no longer possible for us to have
active URBs in the disconnect path.

drivers/usb/storage/usb.c | 6 +-----
1 files changed, 1 insertion(+), 5 deletions(-)
------

[email protected], 2003-03-25 13:45:59-08:00, [email protected]
[PATCH] usb-storage: initialize urb status

This patch initializes the URB status before it's used. While not
technically required, it's good programming practice (and a similar bug
just bit us on 2.4 with UHCI).

drivers/usb/storage/transport.c | 1 +
1 files changed, 1 insertion(+)
------

[email protected], 2003-03-25 13:45:34-08:00, [email protected]
[PATCH] usb-storage: LUN and isd200

This patch (developed with assistance from Jan Harkes
<[email protected]>) makes the LUN field of a bulk-only transport come
from a known-good source, rather than the likely-good command-byte. It
also updates the ISD200 driver to work with this change.

drivers/usb/storage/isd200.c | 5 +++++
drivers/usb/storage/transport.c | 2 +-
2 files changed, 6 insertions(+), 1 deletion(-)
------

[email protected], 2003-03-25 13:30:08-08:00, [email protected]
[PATCH] USB: storage: add logging to reset

- add logging to reset

drivers/usb/storage/scsiglue.c | 13 ++++---------
1 files changed, 4 insertions(+), 9 deletions(-)
------

[email protected], 2003-03-25 13:29:44-08:00, [email protected]
[PATCH] USB: storage device reset cleanup

> In the absence of far-reaching changes to the API, my suggestion is to
> have the emulated SCSI bus reset code in usb-storage do nothing but log an
> error message and return an error code. For the time being, considering
> how infrequently these resets occur, we can simply rely on the user
> unplugging the USB cable and putting it back in or cycling the power to
> the drive. (Yes, there are situations where these resets crop up
> regularly -- but they are the result of some other incompatibility that a
> device reset won't fix anyway.)

OK, as the consensus seems to be that in the short run changing things
for a full reset implementation is not worth it, here's an implementation
that does the best we can do without.
It issues a reset only if we can be sure that there are no other users
of the device in question.
As the version currently in the storage driver is broken anyway,
this is a definite improvement. And it addresses the need of exporting
the probe/remove functions for storage's sake.

drivers/usb/storage/scsiglue.c | 40 ++++++++++++++--------------------------
1 files changed, 14 insertions(+), 26 deletions(-)
------

[email protected], 2003-03-25 13:23:01-08:00, [email protected]
[PATCH] USB: Another memory allocation in block IO error handling path

- memory allocation in block io error code path with GFP_KERNEL

drivers/usb/core/hub.c | 2 +-
1 files changed, 1 insertion(+), 1 deletion(-)
------

[email protected], 2003-03-25 13:07:05-08:00, [email protected]
[PATCH] USB: fix Makefile to allow usb midi driver to be built if it's the only class driver selected.

drivers/usb/Makefile | 1 +
1 files changed, 1 insertion(+)
------

[email protected], 2003-03-25 12:41:14-08:00, [email protected]
[PATCH] USB: usb/misc/emi26.c stack reduction

Reduces stack usage in emi26_load_firmware().

drivers/usb/misc/emi26.c | 37 +++++++++++++++++++++++++------------
1 files changed, 25 insertions(+), 12 deletions(-)
------

[email protected], 2003-03-25 12:40:48-08:00, [email protected]
[PATCH] USB: Compiler error in cdc-acm when DEBUG defined

drivers/usb/class/cdc-acm.c | 2 +-
1 files changed, 1 insertion(+), 1 deletion(-)
------

[email protected], 2003-03-25 11:54:38-08:00, [email protected]
[PATCH] USB: usb_skeleton.c trivial fix

Remove redundant __FILE__.

drivers/usb/usb-skeleton.c | 2 +-
1 files changed, 1 insertion(+), 1 deletion(-)
------

[email protected], 2003-03-25 11:54:15-08:00, [email protected]
[PATCH] USB: usb-skeleton, usbtest use "real" device ids

I'll be switching "gadget zero" to use real product IDs
(donated by NetChip), and these are the two drivers that
will need to recognize them.

drivers/usb/misc/usbtest.c | 5 +++++
drivers/usb/usb-skeleton.c | 4 +++-
2 files changed, 8 insertions(+), 1 deletion(-)
------

[email protected], 2003-03-25 11:53:49-08:00, [email protected]
[PATCH] USB: missing include

at least Alpha needs mm.h for "page_address".

drivers/usb/core/message.c | 1 +
1 files changed, 1 insertion(+)
------

[email protected], 2003-03-25 11:34:31-08:00, [email protected]
[PATCH] USB speedtouch: eliminate ATM open/close races

The list of open vccs is modified by open/close, and traversed by the
receive tasklet. This is the last race I know of in this driver.

drivers/usb/misc/speedtch.c | 20 ++++++++++++++++++--
1 files changed, 18 insertions(+), 2 deletions(-)
------

[email protected], 2003-03-25 11:34:04-08:00, [email protected]
[PATCH] USB speedtouch: per vcc data cleanups

Use struct list_head rather than a singly linked list in udsl_vcc_data. Reject
attempts to open multiple vccs with the same vpi/vci pair. Some cleanups too.

drivers/usb/misc/speedtch.c | 204 +++++++++++++++++++++-----------------------
1 files changed, 98 insertions(+), 106 deletions(-)
------

[email protected], 2003-03-25 11:33:39-08:00, [email protected]
[PATCH] USB speedtouch: trivial cleanups

drivers/usb/misc/speedtch.c | 110 ++++++++++++++++++++------------------------
1 files changed, 51 insertions(+), 59 deletions(-)
------

[email protected], 2003-03-25 11:33:14-08:00, [email protected]
[PATCH] USB speedtouch: code reorganization

Remove dead code from sarlib, reorganize live sarlib code (trivial transformations).

drivers/usb/misc/speedtch.c | 312 ++++++++++++++------------------------------
1 files changed, 100 insertions(+), 212 deletions(-)
------

[email protected], 2003-03-21 17:01:26-08:00, [email protected]
[PATCH] add missing usb_put_urb() after error

This is a multi-part message in MIME format.

drivers/usb/core/hcd.c | 9 +++++----
1 files changed, 5 insertions(+), 4 deletions(-)
------

[email protected], 2003-03-21 17:00:25-08:00, [email protected]
[PATCH] USB: potential dereference of user pointer errors in kobil_sct.c

drivers/usb/serial/kobil_sct.c | 17 ++---------------
1 files changed, 2 insertions(+), 15 deletions(-)
------

[email protected], 2003-03-21 12:54:21-08:00, [email protected]
[PATCH] usbtest catch -ENOMEM

Smatch seems to be returning mostly false positives,
but not this time.

drivers/usb/misc/usbtest.c | 2 ++
1 files changed, 2 insertions(+)
------

[email protected], 2003-03-21 12:53:59-08:00, [email protected]
[PATCH] usb hub diagnostics

More uniformity-in-diagnostics work, and don't emit
a error message in one known non-error case.

drivers/usb/core/hub.c | 54 ++++++++++++++++++++++++++-----------------------
1 files changed, 29 insertions(+), 25 deletions(-)
------

[email protected], 2003-03-21 12:53:36-08:00, [email protected]
[PATCH] usb_connect() kerneldoc

previous text was wrong/confusing, all it does
is pick a number. this routine should be merged
with usb_new_device() someday, but doing that
would mean changing the hcds (as well as hub.c)
so it's not worth it yet.

Push file://home/greg/linux/BK/gregkh-2.5 -> file://home/greg/linux/BK/bleed-2.5
drivers/usb/core/usb.c | 13 +++++--------
1 files changed, 5 insertions(+), 8 deletions(-)
------


2003-03-26 18:14:17

by Bill Davidsen

[permalink] [raw]
Subject: Re: [BK PATCH] USB changes for 2.5.66

On Tue, 25 Mar 2003, Greg KH wrote:

> Hi,
>
> Here are some small USB changes. Basically all little cleanups and
> bugfixes, nothing major.
>
> Please pull from: bk://linuxusb.bkbits.net/linus-2.5

Another "bk-only" patch. Guess I'd better look at the free (as in license,
not cost) clone again.

--
bill davidsen <[email protected]>
CTO, TMR Associates, Inc
Doing interesting things with little computers since 1979.

2003-03-26 18:24:54

by Greg KH

[permalink] [raw]
Subject: Re: [BK PATCH] USB changes for 2.5.66

On Wed, Mar 26, 2003 at 01:17:56PM -0500, Bill Davidsen wrote:
> On Tue, 25 Mar 2003, Greg KH wrote:
>
> > Hi,
> >
> > Here are some small USB changes. Basically all little cleanups and
> > bugfixes, nothing major.
> >
> > Please pull from: bk://linuxusb.bkbits.net/linus-2.5
>
> Another "bk-only" patch. Guess I'd better look at the free (as in license,
> not cost) clone again.

Not true at all (as I've stated many times in the past...)

The USB patches that I send to Linus through BK are all sent to the
linux-usb-devel mailing list as followups to this message.

They are also placed at kernel.org in:
/pub/linux/kernel/people/gregkh/usb/2.5/*.2.5.66*

So no, I always provide patches, I just don't flood lkml with USB
specific ones.

thanks,

greg k-h

2003-03-26 18:38:23

by Linus Torvalds

[permalink] [raw]
Subject: Re: [BK PATCH] USB changes for 2.5.66


On Wed, 26 Mar 2003, Bill Davidsen wrote:
>
> Another "bk-only" patch. Guess I'd better look at the free (as in license,
> not cost) clone again.

Well, since BK has made it so trivial for me to merge with Greg, the thing
is already integrated into my tree, and as a result the patches should
already have been sent out on the patch lists by the robots, and the
snapshots will follow shortly as the automation decides to kick in.

In short, give BK the credit it deserves. You get all the information you
want, and the fact that you depend on and force yourself to use inferior
tools is not the fault of BK.

In other words: _despite_ your luddite ways you actually have more
information available to you than you would have had without BK.

So stop whining about BK. Put up or shut up - you get timely non-BK
snapshots, and the fact that others see the value of their tools in the
things they do for them shouldn't be an issue for _you_.

Stay in the stone age if you wish, but don't expect your stone-age
muscle-propellered log car to go as fast as the rocket of the future. And
don't complain to us who don't want to expend energy on stuff that
shouldn't need it. We've got better tools.

Linus

2003-03-26 18:38:39

by Alan

[permalink] [raw]
Subject: Re: [BK PATCH] USB changes for 2.5.66

On Wed, 2003-03-26 at 18:17, Bill Davidsen wrote:
> On Tue, 25 Mar 2003, Greg KH wrote:
>
> > Hi,
> >
> > Here are some small USB changes. Basically all little cleanups and
> > bugfixes, nothing major.
> >
> > Please pull from: bk://linuxusb.bkbits.net/linus-2.5
>
> Another "bk-only" patch. Guess I'd better look at the free (as in license,
> not cost) clone again.

Greg posts the patches to the usb-devel list, which is how come they
tend to end up in -ac at times when Linus is being slack 8)

2003-03-26 18:58:28

by Alan

[permalink] [raw]
Subject: Re: [linux-usb-devel] Re: [BK PATCH] USB changes for 2.5.66

On Wed, 2003-03-26 at 18:34, Linus Torvalds wrote:
> Stay in the stone age if you wish, but don't expect your stone-age
> muscle-propellered log car to go as fast as the rocket of the future. And

Rocket propelled cars cause lots of flames unfortunately 8)

2003-03-26 20:32:08

by Thomas Molina

[permalink] [raw]
Subject: Re: [linux-usb-devel] Re: [BK PATCH] USB changes for 2.5.66

On 26 Mar 2003, Alan Cox wrote:

> On Wed, 2003-03-26 at 18:34, Linus Torvalds wrote:
> > Stay in the stone age if you wish, but don't expect your stone-age
> > muscle-propellered log car to go as fast as the rocket of the future. And
>
> Rocket propelled cars cause lots of flames unfortunately 8)

Can we hold the bk flamewars to one per week please ;)

2003-03-27 22:45:26

by Bill Davidsen

[permalink] [raw]
Subject: Re: [BK PATCH] USB changes for 2.5.66

On Wed, 26 Mar 2003, Linus Torvalds wrote:

>
> On Wed, 26 Mar 2003, Bill Davidsen wrote:
> >
> > Another "bk-only" patch. Guess I'd better look at the free (as in license,
> > not cost) clone again.
>
> Well, since BK has made it so trivial for me to merge with Greg, the thing
> is already integrated into my tree, and as a result the patches should
> already have been sent out on the patch lists by the robots, and the
> snapshots will follow shortly as the automation decides to kick in.
>
> In short, give BK the credit it deserves. You get all the information you
> want, and the fact that you depend on and force yourself to use inferior
> tools is not the fault of BK.
>
> In other words: _despite_ your luddite ways you actually have more
> information available to you than you would have had without BK.
>
> So stop whining about BK. Put up or shut up - you get timely non-BK
> snapshots, and the fact that others see the value of their tools in the
> things they do for them shouldn't be an issue for _you_.

The issue is not *value* it's *price*.

> Stay in the stone age if you wish, but don't expect your stone-age
> muscle-propellered log car to go as fast as the rocket of the future. And
> don't complain to us who don't want to expend energy on stuff that
> shouldn't need it. We've got better tools.

We have no funky licenses here. If it's free we use it, if we buy it we
use it. Anything which has a license which is free for one thing and not
another just isn't used. It's too easy for someone to use the software in
violation and we're not comfortable risking that. Price of painful
honesty and legal paranoia.

If you think I wouldn't use bk other than the license, I bet you think I
don't drive a Bentley because I don't notice it's nicer than my old Ford.
This isn't a technical issue.

--
bill davidsen <[email protected]>
CTO, TMR Associates, Inc
Doing interesting things with little computers since 1979.

2003-03-29 14:23:44

by John Goerzen

[permalink] [raw]
Subject: Re: [BK PATCH] USB changes for 2.5.66

Linus Torvalds <[email protected]> writes:

> In other words: _despite_ your luddite ways you actually have more
> information available to you than you would have had without BK.

Not everyone that doesn't use BitKeeper is a luddite. For instance,
Ben Collins has been explicitly prevented from using it simply because
he happens to work on Subversion. His only option is to shell out
thousands for the commercial version. It would serve everyone well to
remember that a minority of contributors to this list have the
financial resources to work around the onerous licensing terms of BK.