- 21 Sep, 2010 40 commits
-
-
Greg Kroah-Hartman authored
As there are symbols that conflict with the in-kernel usb-storage module, we can't build this into the kernel. Reported-by:
Randy Dunlap <randy.dunlap@oracle.com> Cc: Al Cho <acho@novell.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@suse.de>
-
Max Vozeler authored
The way the event handler works can cause it to delay events until eventual wakeup for another event. For example, on device detach (vhci): - Write to sysfs detach file -> usbip_event_add(VDEV_EVENT_DOWN) -> wakeup() #define VDEV_EVENT_DOWN (USBIP_EH_SHUTDOWN | USBIP_EH_RESET). - Event thread wakes up and passes the event to event_handler() to process. - It processes and clears the USBIP_EH_SHUTDOWN flag then returns. - The outer event loop (event_handler_loop()) calls wait_event_interruptible(). The processing of the second flag which is part of VDEV_EVENT_DOWN (USBIP_EH_RESET) did not happen yet. It is delayed until the next event. This means the ->reset callback may not happen for a long time (if ever), leaving the usbip port in a weird state which prevents its reuse. This patch changes the handler to process all flags before waiting for another wakeup. I have verified this change to fix a problem which prevented reattach of a usbip device. It also helps for socket errors which missed the RESET as well. The delayed event processing also affects the stub side of usbip and the error handling there. Signed-off-by:Max Vozeler <mvz@vozeler.com> Reported-by:
Marco Lancione <marco@optikam.com> Tested-by:
Luc Jalbert <ljalbert@optikam.com> Cc: stable <stable@kernel.org> Signed-off-by:
-
Max Vozeler authored
This patch changes vhci to behave like dummy and other hcds when disconnecting a device. Previously detaching a device from the root hub did not notify the usb core of the disconnect and left the device visible. Signed-off-by:
-
Mike McCormack authored
Signed-off-by:
Mike McCormack <mikem@ring3k.org> Signed-off-by:
-
Mike McCormack authored
Signed-off-by:
-
Mike McCormack authored
Signed-off-by:
-
Mike McCormack authored
Signed-off-by:
-
Mike McCormack authored
Signed-off-by:
-
Mike McCormack authored
Signed-off-by:
-
Mike McCormack authored
Signed-off-by:
-
Mike McCormack authored
Signed-off-by:
-
Mike McCormack authored
Signed-off-by:
-
Mike McCormack authored
Signed-off-by:
-
Tracey Dent authored
Fixed numerous coding style issues using checkpatch.pl Signed-off-by:
Tracey Dent <tdent48227@gmail.com> Signed-off-by:
-
Tracey Dent authored
Fixed numerous coding style issues using checkpatch.pl Signed-off-by:
-
Tracey Dent authored
Fixed numerous coding style issues using checkpatch.pl Signed-off-by:
-
Tracey Dent authored
Fixed numerous coding style issues using checkpatch.pl Signed-off-by:
-
Tracey Dent authored
Fixed numerous coding style issues Signed-off-by:
-
Victor Rosales authored
Just fixed all checkpatch's errors but not the warinings. Signed-off-by:
Victor Rosales <victorhrosales@gmail.com> Signed-off-by:
-
Pekka Enberg authored
The w35und uses atomics such as "ThreadCount" and "FireCount" to emulate locking in the TX paths, for example. Document this bug in the TODO list. Cc: Lars Lindley <lindley@coyote.org> Cc: Pavel Machek <pavel@ucw.cz> Cc: Ruslan Pisarev <ruslan@rpisarev.org.ua> Signed-off-by:
Pekka Enberg <penberg@kernel.org> Signed-off-by:
-
Pekka Enberg authored
The wbsoft_tx() function is a simple wrapper on top of MLMESendFrame() so inline the latter to the former. Cc: Lars Lindley <lindley@coyote.org> Cc: Pavel Machek <pavel@ucw.cz> Cc: Ruslan Pisarev <ruslan@rpisarev.org.ua> Signed-off-by:
-
Pekka Enberg authored
This patch changes MLMESendFrame to return NETDEV_TX_BUSY if MLME frame is in use so that wbsoft_tx() doesn't blindly return NETDEV_TX_OK in that case. Cc: Sandro Bonazzola <sandro.bonazzola@gmail.com> Cc: Lars Lindley <lindley@coyote.org> Cc: Pavel Machek <pavel@ucw.cz> Cc: Ruslan Pisarev <ruslan@rpisarev.org.ua> Signed-off-by:
-
Pekka Enberg authored
This patch removes unused fields from "struct wbsoft_priv". Cc: Lars Lindley <lindley@coyote.org> Cc: Pavel Machek <pavel@ucw.cz> Cc: Ruslan Pisarev <ruslan@rpisarev.org.ua> Signed-off-by:
-
Pekka Enberg authored
This patch removes unused spinlocks from "struct mlme_frame" and "struct wbsoft_priv". Cc: Lars Lindley <lindley@coyote.org> Cc: Pavel Machek <pavel@ucw.cz> Cc: Ruslan Pisarev <ruslan@rpisarev.org.ua> Signed-off-by:
-
Pekka Enberg authored
The mlme_s.h header is included in one place. As the header is very small, just inline it to core.h. Cc: Lars Lindley <lindley@coyote.org> Cc: Pavel Machek <pavel@ucw.cz> Cc: Ruslan Pisarev <ruslan@rpisarev.org.ua> Signed-off-by:
-
Pekka Enberg authored
There's bunch of macros in mlme_s.h that aren't used for anything. Kill them off. Cc: Lars Lindley <lindley@coyote.org> Cc: Pavel Machek <pavel@ucw.cz> Cc: Ruslan Pisarev <ruslan@rpisarev.org.ua> Signed-off-by:
-
Pekka Enberg authored
The scan_s.h header is not actually used for anything so just kill it off. Cc: Lars Lindley <lindley@coyote.org> Cc: Pavel Machek <pavel@ucw.cz> Cc: Ruslan Pisarev <ruslan@rpisarev.org.ua> Signed-off-by:
-
Pekka Enberg authored
This patch removes remaining typedef declarations from the w35und driver. Most of them were unused so I just killed them off completely. Cc: Lars Lindley <lindley@coyote.org> Acked-by:
Pavel Machek <pavel@ucw.cz> Cc: Ruslan Pisarev <ruslan@rpisarev.org.ua> Signed-off-by:
-
Pekka Enberg authored
Fixes an uninitialized variable access in _rx_iq_calibration_loop_winbond(): CC [M] drivers/staging/winbond/phy_calibration.o drivers/staging/winbond/phy_calibration.c: In function ‘_rx_iq_calibration_loop_winbond’: drivers/staging/winbond/phy_calibration.c:1138: warning: ‘val’ is used uninitialized in this function Cc: Lars Lindley <lindley@coyote.org> Acked-by: -
Jonathan Cameron authored
Also remove no unused precision attr. Signed-off-by:
Jonathan Cameron <jic23@cam.ac.uk> Signed-off-by:
Manuel Stahl <manuel.stahl@iis.fraunhofer.de> Signed-off-by:
-
Jonathan Cameron authored
This has been replaced by a separate _index attribute Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
Jonathan Cameron authored
Also fixes a missing NULL to terminate one of the attribute arrays. Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
