Re: [PATCH] USB enclosures seem to require read(16) with 2TB drives

2012-11-12 Thread Stefan Richter
On Nov 11 Stefan Richter wrote: On Nov 09 Elliott, Robert (Server Storage) wrote: I recommend broadening this patch. T10 is discussing making READ (10), WRITE (10), etc. obsolete in SBC-4 in favor of their 16-byte CDB counterparts. The algorithm should be: 1. During discovery,

[PATCH] usb: misc: usbtest: send ISO packets for g_zero

2012-11-12 Thread Felipe Balbi
From: Boyan Nedeltchev bo...@ti.com since commit b4036cc (usb: gadget: add isochronous support to gadget zero), g_zero has learned about isochronous transfers, which allows us to use usbtest.ko to exercise isochronous pipes. All we need to do to enable that functionality on usbtest.ko, is set

Re: [PATCH] USB enclosures seem to require read(16) with 2TB drives

2012-11-12 Thread James Bottomley
On Fri, 2012-11-09 at 16:33 +, Elliott, Robert (Server Storage) wrote: I recommend broadening this patch. T10 is discussing making READ (10), WRITE (10), etc. obsolete in SBC-4 in favor of their 16-byte CDB counterparts. The algorithm should be: 1. During discovery, determine if

Re: [PATCH] USB enclosures seem to require read(16) with 2TB drives

2012-11-12 Thread James Bottomley
On Fri, 2012-11-09 at 11:08 -0500, Jason J. Herne wrote: diff --git a/drivers/usb/storage/scsiglue.c b/drivers/usb/storage/scsiglue.c index 13b8bcd..6ff785e 100644 --- a/drivers/usb/storage/scsiglue.c +++ b/drivers/usb/storage/scsiglue.c @@ -251,6 +251,11 @@ static int slave_configure(struct

Re: autodetect USB MTP device characteristics without libusb_open()

2012-11-12 Thread Alessio Treglia
On Wed, Nov 7, 2012 at 10:22 PM, Linus Walleij linus.wall...@linaro.org wrote: I'll see what I can dig up. Mostly this has come from upstream, Alessio do you have something at hand for these color calibration crashes you've been seeing? Sorry but no, I don't. Actually, I've never had to deal

Re: [PATCH] USB enclosures seem to require read(16) with 2TB drives

2012-11-12 Thread Paolo Bonzini
Il 12/11/2012 12:33, James Bottomley ha scritto: On Fri, 2012-11-09 at 11:08 -0500, Jason J. Herne wrote: diff --git a/drivers/usb/storage/scsiglue.c b/drivers/usb/storage/scsiglue.c index 13b8bcd..6ff785e 100644 --- a/drivers/usb/storage/scsiglue.c +++ b/drivers/usb/storage/scsiglue.c @@

Re: [PATCH] USB enclosures seem to require read(16) with 2TB drives

2012-11-12 Thread Jason J. Herne
On Mon, Nov 12, 2012 at 9:31 AM, Paolo Bonzini pbonz...@redhat.com wrote: Il 12/11/2012 12:33, James Bottomley ha scritto: On Fri, 2012-11-09 at 11:08 -0500, Jason J. Herne wrote: diff --git a/drivers/usb/storage/scsiglue.c b/drivers/usb/storage/scsiglue.c index 13b8bcd..6ff785e 100644 ---

Re: [PATCH] USB enclosures seem to require read(16) with 2TB drives

2012-11-12 Thread James Bottomley
On Mon, 2012-11-12 at 15:31 +0100, Paolo Bonzini wrote: Il 12/11/2012 12:33, James Bottomley ha scritto: On Fri, 2012-11-09 at 11:08 -0500, Jason J. Herne wrote: diff --git a/drivers/usb/storage/scsiglue.c b/drivers/usb/storage/scsiglue.c index 13b8bcd..6ff785e 100644 ---

Re: [PATCH] USB enclosures seem to require read(16) with 2TB drives

2012-11-12 Thread Jason J. Herne
On Mon, Nov 12, 2012 at 10:10 AM, James Bottomley james.bottom...@hansenpartnership.com wrote: On Mon, 2012-11-12 at 15:31 +0100, Paolo Bonzini wrote: Il 12/11/2012 12:33, James Bottomley ha scritto: On Fri, 2012-11-09 at 11:08 -0500, Jason J. Herne wrote: diff --git

Re: [PATCH] USB enclosures seem to require read(16) with 2TB drives

2012-11-12 Thread James Bottomley
On Mon, 2012-11-12 at 10:01 -0500, Jason J. Herne wrote: Any reason not to do this always on 2TB drives, which basically means changing this: - } else if (block 0x) { + } else if (sdkp-capacity 0x) { and nothing else? This was the intent of my patch,

Re: [PATCH] USB enclosures seem to require read(16) with 2TB drives

2012-11-12 Thread Paolo Bonzini
Il 12/11/2012 16:10, James Bottomley ha scritto: Actually it only turns it on for large capacity drives, as said in the comment. sdp-force_read_16 only matters for 2TB drives: If you follow the discussion, we'll need to turn it on for some drives regardless of size. Even if the two

Re: [PATCH] USB enclosures seem to require read(16) with 2TB drives

2012-11-12 Thread Alan Stern
On Mon, 12 Nov 2012, Paolo Bonzini wrote: Il 12/11/2012 16:10, James Bottomley ha scritto: Actually it only turns it on for large capacity drives, as said in the comment. sdp-force_read_16 only matters for 2TB drives: If you follow the discussion, we'll need to turn it on for some

Re: [PATCH v1] usb: musb: Make dma_controller_create __devinit

2012-11-12 Thread Sergei Shtylyov
Hello. On 08/09/2012 06:38 PM, Shubhrajyoti D wrote: dma_controller_create is called only from musb_init_controller which is __devint so annotate dma_controller_create also with __devint. fixes the warn WARNING: vmlinux.o(.devinit.text+0x6fa8): Section mismatch in reference from the

[PATCH] cppi_dma: mark cppi_controller_create() and cppi_pool_init() as '__devinit'

2012-11-12 Thread Sergei Shtylyov
This patch fixes the following: WARNING: vmlinux.o(.devinit.text+0x24ac): Section mismatch in reference from the function dma_controller_create() to the function .init.text:cppi_controller_start() The function __devinit dma_controller_create() references a function __init

Re: [PATCH 02/10] arm: at91: move platfarm_data to include/linux/platform_data/atmel.h

2012-11-12 Thread Jean-Christophe PLAGNIOL-VILLARD
On 11:54 Sat 10 Nov , Joachim Eastwood wrote: Hi Jean-Christophe, On 7 November 2012 12:22, Jean-Christophe PLAGNIOL-VILLARD plagn...@jcrosoft.com wrote: Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD plagn...@jcrosoft.com Cc: Nicolas Ferre nicolas.fe...@atmel.com Cc:

[PATCH] xhci: Add Lynx Point LP to list of Intel switchable hosts

2012-11-12 Thread Russell Webb
Like Lynx Point, Lynx Point LP is also switchable. See 1c12443ab8eba71a658fae4572147e56d1f84f66 for more details. This patch should be backported to stable kernels as old as 3.0, that contain commit 69e848c2090aebba5698a1620604c7dccb448684 Intel xhci: Support EHCI/xHCI port switching.

Re: Fresco Logic Problem

2012-11-12 Thread Sarah Sharp
On Sat, Nov 10, 2012 at 02:30:36PM +0330, A Sh wrote: Ah, then you probably have the xHCI driver built-in, not built as a module. What does it mean? Wow, I googled around, and even though I know what search terms to look for, there's really no good documentation on this question. I'll add

Re: [PATCH] xhci: fix null-pointer dereference when destroying half-built segment rings

2012-11-12 Thread Sarah Sharp
On Thu, Nov 01, 2012 at 01:28:46PM -0700, Julius Werner wrote: Is it just for (prev = *first; prev; prev = prev-next) xhci_segment_free(xhci, prev); ? Yeah, that seems cleaner. Sarah Sharp I can submit it that way if you want, but I would advise against it. This way

[Pull Request] xHCI patches for 3.8

2012-11-12 Thread Sarah Sharp
The following changes since commit 2f02bc8af3abb846823811af65ec6cc46a4d525d: USB: report submission of active URBs (2012-11-11 18:10:46 -0800) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/sarah/xhci.git for-usb-next-2012-11-12 for you to fetch

[PATCH 2/7] xhci: Fix conditional check in bandwidth calculation.

2012-11-12 Thread Sarah Sharp
David reports that at drivers/usb/host/xhci.c:2257: static bool xhci_is_sync_in_ep(unsigned int ep_type) { return (ep_type == ISOC_IN_EP || ep_type != INT_IN_EP); } The static analyser cppcheck says [linux-3.7-rc2/drivers/usb/host/xhci.c:2257]: (style) Redundant condition: If ep_type == 5,

[PATCH 1/7] xhci: Avoid global symbol pollution with handshake.

2012-11-12 Thread Sarah Sharp
Non-static xHCI driver symbols should start with the xhci_ prefix, in order to avoid namespace pollution. Rename the handshake function to xhci_handshake. Signed-off-by: Sarah Sharp sarah.a.sh...@linux.intel.com Reported-by: Ben Hutchings b...@decadent.org.uk --- drivers/usb/host/xhci-ring.c |

[PATCH 5/7] xhci: Extend Fresco Logic MSI quirk.

2012-11-12 Thread Sarah Sharp
Ali reports that plugging a device into the Fresco Logic xHCI host with PCI device ID 1400 produces an IRQ error: do_IRQ: 3.176 No irq handler for vector (irq -1) Other early Fresco Logic host revisions don't support MSI, even though their PCI config space claims they do. Extend the quirk to

[PATCH 6/7] usb: host: xhci: Stricter conditional for Z1 system models for Compliance Mode Patch

2012-11-12 Thread Sarah Sharp
From: Alexis R. Cortes alexis.cor...@ti.com This minor patch creates a more stricter conditional for the Z1 sytems for applying the Compliance Mode Patch, this to avoid the quirk to be applied to models that contain a Z1 in their dmi product string but are different from Z1 systems. This patch

Re: autodetect USB MTP device characteristics without libusb_open()

2012-11-12 Thread Peter Stuge
Bjørn Mork wrote: The problem appear when you ask a device which is not MTP for that descriptor, some of them just die, so I cannot do that. Really? You ask for a string descriptor and the device dies? Won't those devices also die if they are connected to a Windows system? Yes, but

Re: autodetect USB MTP device characteristics without libusb_open()

2012-11-12 Thread Peter Stuge
Linus Walleij wrote: - some devices bug out on libusb_open() Please send me a debug log from when this happens. Exact steps are at http://libusb.org/wiki/debug I'll see what I can dig up. Mostly this has come from upstream, Alessio do you have something at hand for these color

Re: 3.7-rc5 modpost/link failure: ezusb

2012-11-12 Thread Greg Kroah-Hartman
On Mon, Nov 12, 2012 at 03:44:35PM +0200, Meelis Roos wrote: MODPOST 490 modules ERROR: ezusb_fx1_ihex_firmware_download [drivers/usb/serial/whiteheat.ko] undefined! ERROR: ezusb_fx1_ihex_firmware_download [drivers/usb/serial/keyspan_pda.ko] undefined! ERROR: ezusb_fx1_set_reset

[PATCH v2] usb: dwc3: exynos: use devm_ functions

2012-11-12 Thread Jingoo Han
The devm_ functions allocate memory that is released when a driver detaches. This makes the code smaller and a bit simpler. Signed-off-by: Jingoo Han jg1@samsung.com --- Changes since v1: - Rebased on usb-next branch drivers/usb/dwc3/dwc3-exynos.c | 51

Re: [RFC PATCH 4/5] usb: add runtime pm support for usb port device

2012-11-12 Thread Lan Tianyu
On 2012年11月12日 10:43, Alan Stern wrote: On Mon, 12 Nov 2012, Lan Tianyu wrote: This will consume more power than suspend it agian. No it won't, because the device will suspend itself after 3 ms. But the premise is they see a constant Idle state on their upstream facing bus lines for

RE: [PATCH] usb: ohci-exynos: use clk_prepare_enable and clk_disable_unprepare

2012-11-12 Thread Kukjin Kim
Thomas Abraham wrote: Convert clk_enable/clk_disable to clk_prepare_enable/clk_disable_unprepare calls as required by common clock framework. Signed-off-by: Thomas Abraham thomas.abra...@linaro.org (+ Alan Stern) Acked-by: Kukjin Kim kgene@samsung.com Thanks. Best regards, Kgene. --

Re: [PATCH v2] usb: dwc3: exynos: use devm_ functions

2012-11-12 Thread Felipe Balbi
Hi, On Tue, Nov 13, 2012 at 11:20:49AM +0900, Jingoo Han wrote: The devm_ functions allocate memory that is released when a driver detaches. This makes the code smaller and a bit simpler. Signed-off-by: Jingoo Han jg1@samsung.com Next time you want to use the output of