1. 30 Mar, 2004 10 commits
    • Dave Jones's avatar
      [PATCH] USB: kill off CONFIG_USB_BRLVGER detritus. · 38c2c0e8
      Dave Jones authored
      This died a while ago, but lingers on in defconfigs.
      38c2c0e8
    • Andries E. Brouwer's avatar
      [PATCH] USB Storage: datafab fix and unusual devices · 45fbc50e
      Andries E. Brouwer authored
      datafab.c has an often-seen bug: the SCSI READ_CAPACITY command
      does not need the number of sectors but the last sector.
      
      I just tried the CF and SM parts of a 5-in-1 card reader.
      The CF part works with US_PR_DATAFAB when the bug mentioned is fixed.
      The SM part works with US_PR_SDDR55.
      (Revision Number is 17.08 - that in case the 0000-ffff
      should prove to be too optimistic.)
      
      We still must discuss what setup to use for readers like this -
      I have several of them - that require different drivers for
      different LUNs. As it is now one has to compile usb-storage
      twice, once with CONFIG_USB_STORAGE_DATAFAB defined and once
      without, and remove one usb-storage.ko and insert the other
      to go from CF to SM. (And that hangs with 2.6.4 so a reboot
      is required..)
      45fbc50e
    • Paulo Marques's avatar
      [PATCH] USB: ftdi_sio.c: not unlinking urb on ftdi_close · c3e08bc4
      Paulo Marques authored
      This patch fixes a bug in ftdi_sio.c where the driver wouldn't unlink the
      read urb if the user application cleared the HUPCL flag on termios->c_cflag.
      c3e08bc4
    • Andrew Morton's avatar
      [PATCH] USB: drivers/usb/gadget/epautoconf.c gcc-3.5 build fix · cf2bbebe
      Andrew Morton authored
      drivers/usb/gadget/epautoconf.c: In function `ep_matches':
      drivers/usb/gadget/epautoconf.c:175: error: `typeof' applied to a bit-field
      cf2bbebe
    • David Brownell's avatar
      [PATCH] USB: set_configuration locking cleanups · 10766dde
      David Brownell authored
      I've posted all these before, the only notable change is
      treating that one gphoto2 case as warn-and-continue rather
      than return-with-failure.
      
      
      usb_set_configuration() cleanup
      
       * Remove it from the USB kernel driver API.  No drivers need it now,
         and the sysadmin can change bConfigurationValue using sysfs (say,
         when hotplugging an otherwise problematic device).
      
       * Simpler/cleaner locking:  caller must own dev->serialize.
      
       * Access from usbfs now uses usb_reset_configuration() sometimes,
         preventing sysfs thrash, and warns about some dangerous usage
         (which gphoto2 and other programs may be relying on).  (This is
         from Alan Stern, but I morphed an error return into a warning.)
      
       * Prevent a couple potential "no configuration" oopses. (Alan's?)
      
       * Remove one broken call from usbcore,  in the "device morphed" path
         of usb_reset_device().  This should be more polite now, hanging
         that one device rather than khubd.
      10766dde
    • David Brownell's avatar
      [PATCH] USB Gadget: RNDIS/Ethernet Gadget Driver (2/2) · 394d7256
      David Brownell authored
      This updates the existing Ethernet gadget driver to
      support an additional RNDIS configuration on all
      current USB controllers that can support one.
      
      It also includes a bit more work to address the complex
      autoconfiguration of this driver.  Needs testing on PXA.
      
      Patch (mostly) contributed by Robert Schwebel, and developed
      with support from Auerswald GmbH.
      394d7256
    • David Brownell's avatar
      [PATCH] USB Gadget: RNDIS/Ethernet Gadget Driver (1/2) · 166685d0
      David Brownell authored
      This patch adds the RNDIS message engine and kbuild/kconfig
      support for it.  This is currently labeled EXPERIMENTAL.
      
      Patch contributed by Robert Schwebel, and developed with
      support from Auerswald GmbH.
      166685d0
    • David Brownell's avatar
      [PATCH] USB: ohci unlink tweaks · 21d554cc
      David Brownell authored
      Minor unlink tweaks, including a case where SMP could oops
      if it were abused, as if from 'usbtest' or 'stir4200'.
      21d554cc
    • David Brownell's avatar
      [PATCH] USB: usb/core/config.c null pointers after kfree · 976942da
      David Brownell authored
      Prevents an oops with some other patchsets.
      
      
      Clear some pointers after the memory is kfreed, to avoid
      making some other patch combinations oops.
      976942da
    • David Brownell's avatar
      [PATCH] USB; minor usbfs locking updates · eea93a30
      David Brownell authored
      These are "obvious" locking fixes:  using the right lock
      to protect interface claim/release (should be the driver
      model bus lock, not BKL).
      eea93a30
  2. 26 Mar, 2004 20 commits
  3. 25 Mar, 2004 10 commits
    • Arjan van de Ven's avatar
      [PATCH] USB: usb hiddev stack usage patch · 0e80d432
      Arjan van de Ven authored
      Patch below fixes some obscenely high stack uage;
      struct hiddev_usage_ref_multi is well over 4Kb in size so really doesn't
      belong on the stack.
      0e80d432
    • Oliver Neukum's avatar
      [PATCH] USB: fix error paths in kobil_sct · 6f4388bc
      Oliver Neukum authored
      some error codes are incorrect and there's an URB leak in an error path.
      6f4388bc
    • Alan Stern's avatar
      [PATCH] USB: UHCI: Get rid of excessive spinlocks · 33d61f24
      Alan Stern authored
      This patch introduces a major simplification into the UHCI driver by
      replacing its multiple spinlocks with a single one.  The protected area of
      code is slightly larger and there's more possibilities for contention on
      an SMP system, but I think that shouldn't be a problem.  Stephen Hemminger
      has been kind enough to test this on his SMP computer and he hasn't
      encountered any difficulties.
      33d61f24
    • Alan Stern's avatar
      [PATCH] USB: UHCI: Improved handling of short control transfers · 2212df84
      Alan Stern authored
      This patch simplies the way the UHCI driver handles short control
      transfers.  When a transfer is short the HC will stop handling that
      endpoint, and it's necessary to get it going again so that the status
      stage of the control transfer can take place.  Currently the driver does
      this by allocating a new QH for the transfer and setting its element
      pointer to point at the final status TD.  The old QH is recycled.  But
      it's not necessary to go to all that trouble; the element pointer in the
      original QH can be updated directly.
      
      Normally the element pointer is supposed to be owned by the HC, and it's
      not safe to just change its value since the HC may overwrite it at any
      time.  But when a transfer is stopped because of a short packet, the
      current TD is marked inactive and the HC will not update the element
      pointer.  To write an unchanged pointer value back to memory would be a
      waste of PCI bus cycles.  Now the UHCI spec doesn't say explicitly that an
      HC _can't_ do this, but I've tested both Intel and VIA hardware and
      neither of them does.
      
      As a side effect of this change, some of the code for removing QHs can be
      eliminated.
      2212df84
    • Alan Stern's avatar
      [PATCH] USB: UHCI: Do short packet detection correctly · 6bfb2519
      Alan Stern authored
      This patch makes some simple changes to the way the UHCI driver does short
      packet detection.  The current implementation is incorrect in several
      ways:
      
      	The Short-Packet-Detect flag is set for OUT transfers, which
      	yields undefined behavior according to the UHCI spec.
      
      	It's not set for URBs with URB_SHORT_NOT_OK, which is just the
      	opposite of what we want!  Those are the ones where short packets
      	do matter.
      
      	It is set for the last packet in a transfer, which causes an
      	unnecessary pause in the data flow (except of course that the
      	pause _is_ necessary when URB_SHORT_NOT_OK is set).
      
      The patch also implements the URB_NO_INTERRUPT flag for bulk transfers,
      which can help improve system performance by reducing interrupt overhead.
      6bfb2519
    • Alan Stern's avatar
      [PATCH] USB: Don't trust raw descriptor length in devioc. · 59941877
      Alan Stern authored
      This is a very minor point, unlikely ever to come up.  But just in case...
      
      It's conceivable that a device might transmit different values for a
      configuration descriptor's wTotalLength the first time we ask for it (in
      order to get the length) and the second time (to get the entire
      descriptor).  Should that improbable event occur, the rawdescriptor buffer
      could be allocated using a size that's smaller than the length recorded in
      the rawdescriptor itself.  This patch protects devio.c against such a
      problem.
      
      If you feel this sequence of events is too unlikely to worry about, then
      don't bother to apply the patch.
      59941877
    • Oliver Neukum's avatar
      [PATCH] USB: race condition in open of w9968cf · af6f5064
      Oliver Neukum authored
      there's a race in how open handles multiple openers.
      You implement exclusive opening and wait for close
      in case of further openers. However if there are more than one
      waiter, only one of them must be allowed to proceed.
      af6f5064
    • Alan Stern's avatar
      [PATCH] USB: Unusual_devs.h update · c107b960
      Alan Stern authored
      On Thu, 18 Mar 2004, Urban Borstnik wrote:
      
      > The 2.6.4 and 2.6.3 (and possibly some earlier) kernels log the
      > following message when I plug in a Lexar CompactFlash Reader:
      >
      > usb-storage: This device (05dc,b002,0113 S 06 P 50) has unneeded
      > SubClass and Protocol entries in unusual_devs.h
      >    Please send a copy of this message to <linux-usb-devel@lists.sourceforge.net>
      >
      > Otherwise it has been working very well with the devepment kernels on at
      > least 4 machines ever since a trivial fix was introduced for this device
      > to unusual_devs.c over a year ago.
      >
      > Best regards,
      > Urban.
      
      Thank you for sending this in.  An update will appear soon.
      c107b960
    • Torrey Hoffman's avatar
      [PATCH] USB: fix race in ati_remote and small cleanup · befe2d08
      Torrey Hoffman authored
      On Thu, 2004-03-18 at 07:44, Oliver Neukum wrote:
      > Hi,
      >
      > you must use set_current_state() only after usb_submit_urb() with GFP_KERNEL
      > as second argument, because it may sleep to allocate memory and is woken up
      > resetting the state to TASK_RUNNING. In that case you had a busy polling loop.
      > Furthermore, always use wake_up unconditionally. It checkes anyway.
      
      Thanks for reviewing this code, I'm new to Linux driver development and
      more eyes on my work is a good thing.   I've actually been working on
      some more cleanups to the driver to fix the race between open and
      disconnect, and was just about to send it in...
      
      So, the attached patch against 2.6.5-rc1-mm1 includes a mutex to lock
      the open/disconnect paths, modelled after the usb-skeleton driver. It
      includes Oliver Neukum's fixes and other cleanups as well.
      befe2d08
    • Michael Still's avatar
      [PATCH] USB: kernel-doc comment tweak in vicam.c · df73571e
      Michael Still authored
      Correct kernel-doc comment with incorrect parameters documented
      df73571e