While trying to copy some file(>400 MB) from PC to device in mass
storage mode,an I/O error is observed.
>From the bus trace, it is observed that vfs_sync() is taking a lot of
time when handing ALLOW_MEDIUM_REMOVAL.There is a similar problem
already discussed earlier,although with no concrete conc
Make sure sysfs attributes are created at port probe.
A recent patch ("USB: iuu_phoenix: fix port-data memory leak") removed
the sysfs-attribute creation by mistake.
Reported-by: Yuanhan Liu
Cc:
Signed-off-by: Johan Hovold
---
Hi Yuanhan,
Thanks for catching this.
Johan
drivers/usb/seria
Make sure port data is initialised before creating sysfs attributes to
avoid a race.
A recent patch ("USB: io_ti: fix port-data memory leak") got the
sysfs-attribute creation and port-data initialisation ordering wrong.
Cc:
Signed-off-by: Johan Hovold
---
drivers/usb/serial/io_ti.c | 14 ++
On Wed, Oct 17, 2012 at 10:07 PM, Sarah Sharp
wrote:
> On Wed, Oct 17, 2012 at 06:38:31AM +0200, Felipe Contreras wrote:
>> Hi,
>>
>> On Tue, Oct 16, 2012 at 7:24 PM, Sarah Sharp
>> wrote:
>> > The 3.5 kernel added USB 3.0 Link Power Management for Intel Panther
>> > Point xHCI hosts. We've had
On Thu, Oct 18, 2012 at 11:58 AM, Felipe Contreras
wrote:
> On Wed, Oct 17, 2012 at 10:07 PM, Sarah Sharp
> wrote:
>> On Wed, Oct 17, 2012 at 06:38:31AM +0200, Felipe Contreras wrote:
>>> Hi,
>>>
>>> On Tue, Oct 16, 2012 at 7:24 PM, Sarah Sharp
>>> wrote:
>>> > The 3.5 kernel added USB 3.0 Link
Hello Sebastian,
Thank you for the patches. By no means am I questioning your work;
I have a question, though. Please see inline.
On Wednesday, October 17, 2012 9:07 PM Sebastian Andrzej Siewior wrote:
> +static struct usb_function *try_get_usb_function(const char *name)
> +{
> + struct us
Hello Sebastian,
Which Linux version does this patch apply to?
On Wednesday, October 17, 2012 9:07 PM Sebastian Andrzej Siewior wrote:
>
> Signed-off-by: Sebastian Andrzej Siewior
> ---
> drivers/usb/gadget/f_sourcesink.c | 70
> -
> 1 file changed, 45
Am 16.10.2012 15:22, schrieb Constantine Shulyupin:
> Questions:
> - have same issues with PDF documentation and datasheets?
No.
> - do you think documentation and datasheets in HTML format on a site
> can be more useful than PDF?
I would prefer if the source for the (PDF-)documentation would b
On 2012-10-18, Jon Loeliger wrote:
>>
>> I would prefer if the source for the (PDF-)documentation would be
>> available in some open, free and widely used meta-language, e.g. LaTeX
>> (or XML or txt2tags or ..., but I'm oldschool and prefer LaTeX ;) ). So
>> everyone could build it's copy in what
>
> I would prefer if the source for the (PDF-)documentation would be
> available in some open, free and widely used meta-language, e.g. LaTeX
> (or XML or txt2tags or ..., but I'm oldschool and prefer LaTeX ;) ). So
> everyone could build it's copy in whatever format he prefers or needs.
Perhaps
On Thu, 18 Oct 2012, Martin Vysny wrote:
> Good day,
>thank you for your response, please see the answers below.
>
>
> On 10/17/2012 08:05 PM, Alan Stern wrote:
> > On Wed, 17 Oct 2012, Martin Vysny wrote:
> >
> >> Good day,
> >> thank you for your mail. I was finally able to reproduce t
On Tue, Oct 16, 2012 at 12:59:33PM +0300, Felipe Balbi wrote:
> Hi,
>
> > +#ifdef CONFIG_PM_SLEEP
> > +static int xhci_plat_suspend(struct device *dev)
> > +{
> > + struct usb_hcd *hcd= dev_get_drvdata(dev);
> > + struct xhci_hcd *xhci = hcd_to_xhci(hcd);
> > +
> > + /* Make sure that
On Tue, Oct 16, 2012 at 04:53:28PM +0300, Felipe Balbi wrote:
> Hi,
>
> On Tue, Oct 16, 2012 at 05:10:39PM +0530, Vikas Sajjan wrote:
> > Hi Felipe,
...
> did you have a transfer going through when you suspended ? If you
> didn't, then you haven't stressed well enough.
>
> > 2) We tested by runn
Analyzed a few Windows driver description files, supporting
this long list of devices:
%ztewwan.DeviceDesc0002%= ztewwan.ndi, USB\VID_19D2&PID_0002&MI_01
%ztewwan.DeviceDesc0012%= ztewwan.ndi, USB\VID_19D2&PID_0012&MI_01
%ztewwan.DeviceDesc0017%= ztewwan.ndi, USB\VID_19D2&PID_0017&MI_0
Hello Felipe,
On Wednesday, October 17, 2012 3:56 PM Felipe Balbi wrote:
> On Wed, Oct 17, 2012 at 11:43:11AM +0200, Andrzej Pietrasiewicz wrote:
> > Demonstrate a USB gadget configured entirely through configfs.
> > This is a work in progress.
> >
>
> this is wrong. we don't want *another*
Based on information from the ZTE Windows drivers.
Cc:
Signed-off-by: Bjørn Mork
---
drivers/usb/serial/option.c | 68 ++-
1 file changed, 48 insertions(+), 20 deletions(-)
diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c
index 30
Cc:
Signed-off-by: Bjørn Mork
---
drivers/usb/serial/option.c | 16
1 file changed, 16 insertions(+)
diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c
index 231a5c1..72b7961 100644
--- a/drivers/usb/serial/option.c
+++ b/drivers/usb/serial/option.c
@@ -89
On Thu, Oct 18, 2012 at 08:48:20AM +0300, Felipe Balbi wrote:
> Hi Greg,
>
> Here's my first set of fixes for v3.7-rc cycle. I still have pending fixes
> in the mailing list, but I wouldn't have enough time to verify those in time
> for -rc2, so I'm postponing to -rc3.
>
> Anyway, let me know if
Bjørn Mork writes:
> +static const struct option_blacklist_info net_intf6_blacklist = {
> + .reserved = BIT(5),
> +};
Yuck. Why do I see such stupid errors only *after* sending?
Forget this patch. I'll send a fixed version.
--
To unsubscribe from this list: send the line "unsubscribe lin
Based on information from the ZTE Windows drivers.
Cc:
Signed-off-by: Bjørn Mork
---
v2: fixed mindless typo on the net_intf6_blacklist
drivers/usb/serial/option.c | 68 ++-
1 file changed, 48 insertions(+), 20 deletions(-)
diff --git a/drivers/usb/se
Takashi Iwai a écrit :
> At Tue, 16 Oct 2012 18:01:13 +0200,
> Matthieu CASTET wrote:
>> Takashi Iwai a écrit :
>>> At Mon, 15 Oct 2012 19:41:40 +0200,
>>> Matthieu CASTET wrote:
Hi Takashi,
But I believe I found other races in the alsa char device handling. With
the
attac
On Wed, 17 Oct 2012, vikram pandita wrote:
>
> Alan
> I was looking at your implementation [1] on Periodic schedule
> enable/disable using hrtimer.
> Looks really neat.
Thank you.
> Current implementation is:
> On Periodic Enable, Sate is: PSE =1 , PSS dont care - will be set
> automatica
On Thu, 18 Oct 2012, Sarah Sharp wrote:
> For system suspend while the DW3 hardware is in host mode, doesn't the
> USB core prevent drivers from submitting URBs just before the
> PCI/platform suspend is called? Alan?
Sure it does.
Alan Stern
--
To unsubscribe from this list: send the line "uns
On Thu, 18 Oct 2012, Pavan Kondeti wrote:
> Hi Alan,
>
> I have a theoretical question in EHCI driver. We ensure that there is
> 5 msec window between suspend and next resume by keeping time_before()
> check in ehci_bus_resume function. If the next_statechange update in
> suspend occur just befor
On 18/10/12 01:11 PM, Arvid Brodin wrote:
On 2012-10-17 00:32, Richard Retanubun wrote:
Hi Arvid,
I am working on isp1763-hcd (a smaller, kind-of-sort-of variant of isp1760)
The driver was adapted from the original isp1760 before you worked on it a
bunch.
I am looking at my adapted driver ag
On 2012-10-17 00:32, Richard Retanubun wrote:
> Hi Arvid,
>
> I am working on isp1763-hcd (a smaller, kind-of-sort-of variant of isp1760)
>
> The driver was adapted from the original isp1760 before you worked on it a
> bunch.
>
> I am looking at my adapted driver again because we have a case wh
On Thu, 18 Oct 2012, megha dey wrote:
> While trying to copy some file(>400 MB) from PC to device in mass
> storage mode,an I/O error is observed.
>
> From the bus trace, it is observed that vfs_sync() is taking a lot of
> time when handing ALLOW_MEDIUM_REMOVAL.There is a similar problem
> alread
On Thu, 18 Oct 2012, Felipe Contreras wrote:
> Nope, still fails.
>
> http://felipec.org/dmesg.txt.xz
>
> I had some issues booting, probably not related to your patches, so I
> rebased them on top of 3.6.2.
A usbmon trace (starting from before the non-working device is plugged
in) might shed s
Hi,
On Thu, Oct 18, 2012 at 12:44:50PM -0400, Alan Stern wrote:
> On Thu, 18 Oct 2012, Sarah Sharp wrote:
>
> > For system suspend while the DW3 hardware is in host mode, doesn't the
> > USB core prevent drivers from submitting URBs just before the
> > PCI/platform suspend is called? Alan?
>
>
Hi,
On Thu, Oct 18, 2012 at 07:59:56AM -0700, Sarah Sharp wrote:
> On Tue, Oct 16, 2012 at 12:59:33PM +0300, Felipe Balbi wrote:
> > Hi,
> >
> > > +#ifdef CONFIG_PM_SLEEP
> > > +static int xhci_plat_suspend(struct device *dev)
> > > +{
> > > + struct usb_hcd *hcd= dev_get_drvdata(dev);
> > >
On Thu, 18 Oct 2012, Peter Chen wrote:
> > I decided to use a different approach. The core should know which
> > ports are suspended without asking the HCD. Can you test this patch in
> > place of the other one?
> It shows below oops, you may not consider there is no device at port
> condition
From: Bjørn Mork
Date: Thu, 18 Oct 2012 17:11:29 +0200
> Analyzed a few Windows driver description files, supporting
> this long list of devices:
...
> Adding the ones we were missing.
>
> Signed-off-by: Bjørn Mork
Applied.
--
To unsubscribe from this list: send the line "unsubscribe linux-us
On Thu, 18 Oct 2012, Felipe Contreras wrote:
> On Thu, Oct 18, 2012 at 8:03 PM, Felipe Contreras
> wrote:
> > On Thu, Oct 18, 2012 at 7:26 PM, Alan Stern
> > wrote:
> >> On Thu, 18 Oct 2012, Felipe Contreras wrote:
> >>
> >>> Nope, still fails.
> >>>
> >>> http://felipec.org/dmesg.txt.xz
> >>>
On Tue, 16 Oct 2012, Don Zickus wrote:
> > > I did not get a chance to test the patch yet.
> >
> > Let me know what happens.
>
> It might take a while as our customer's customers have trouble reproducing
> this which means it is difficult to tell if the problem is fixed or just
> harder to repro
From: Greg Suarez
NCM and MBIM can share most of the bind function. Split
out the shareable part and add MBIM functional descriptor
parsing.
Signed-off-by: Greg Suarez
Signed-off-by: Bjørn Mork
---
drivers/net/usb/cdc_ncm.c | 47 +++--
1 file changed
The MBIM specification allows a MBIM device to disguise
itself as NCM for backwards compatibility, using additional
altsettings with different subclass (control) or protocol
(data):
C:* #Ifs= 2 Cfg#= 1 Atr=80 MxPwr=100mA
I: If#= 0 Alt= 0 #EPs= 1 Cls=02(comm.) Sub=0d Prot=00 Driver=cdc_mbim
E: Ad
From: Greg Suarez
The CDC Mobile Broadband Interface Model (MBIM) specification
extends CDC NCM by
- removing the redundant ethernet header from the point-to-point
USB channel
- adding support for multiple IP (v4 and/or v6) sessions multiplexed
on the same USB channel
- adding a MBIM con
If the device refuses our updated value, then we must be prepared
to receive URBs as big as the device wants to send. Set rx_max
to the device provided value on error.
Signed-off-by: Bjørn Mork
---
drivers/net/usb/cdc_ncm.c |7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff -
Signed-off-by: Bjørn Mork
---
drivers/net/usb/Kconfig | 18 ++
drivers/net/usb/Makefile |1 +
2 files changed, 19 insertions(+)
diff --git a/drivers/net/usb/Kconfig b/drivers/net/usb/Kconfig
index c1ae769..2ab8043 100644
--- a/drivers/net/usb/Kconfig
+++ b/drivers/net/usb/
From: Greg Suarez
MBIM and NCM are very similar, so we can reuse most of the
setup and bind logic in cdc_ncm for CDC MBIM devices. Handle
a few minor differences in ncm_setup.
Signed-off-by: Greg Suarez
Signed-off-by: Bjørn Mork
---
drivers/net/usb/cdc_ncm.c | 51 ++
Verifying and handling received MBIM and NCM frames will need
to be different in three areas:
- verifying the NDP signature
- checking valid datagram length
- datagram header manipulation
This makes it inconvenient to share rx_fixup in whole. But
some verification parts are common. Split thes
MBIM devices can support up to 256 independent IP Streams.
The main network device will only handle SessionID 0. Mapping
SessionIDs 1 to 255 to VLANs using the SessionID as VLAN ID
allow userspace to use these streams with traditional tools
like vconfig.
Signed-off-by: Bjørn Mork
---
drivers/net
Adding multiplexed NDP support to cdc_ncm_fill_tx_frame, allowing
transmissions of multiple independent sessions within the same NTB.
Refactoring the code quite a bit to avoid having to store copies
of multiple NDPs being prepared for tx. The old code would still
reserve enough room for a maximum
The NCM 1.0 spefication makes provisions for linking more than
one NDP into a single NTB. This is important for MBIM support,
where these NDPs might be of different types.
Following the chain of NDPs is also correct for NCM, and will
not change anything in the common case where there is only
one N
From: Greg Suarez
Based on revision 1.0 of "Universal Serial Bus Communications
Class Subclass Specification for Mobile Broadband Interface
Model" available from www.usb.org
Signed-off-by: Greg Suarez
[bmork: added DSS defines]
Signed-off-by: Bjørn Mork
---
include/linux/usb/cdc.h | 23
Move symbols and definitons which can be shared with a
MBIM driver in a new header.
Signed-off-by: Bjørn Mork
---
drivers/net/usb/cdc_ncm.c | 97 -
include/linux/usb/cdc_ncm.h | 124 +++
2 files changed, 135 insertions(
The USB Communications Device Class "Mobile Broadband Interface Model"
(MBIM) is the USB-IFs alternative to the current chipset/vendor
specific solutions to "Mobile Broadband" device management. The
specification, including the management protocol description, can be
downloaded from http://www.usb.
Some usbnet based devices may want to use a rx_urb_size
greater than RX_MAX_QUEUE_MEMORY. Prevent this from
creating a zero length rx queue.
Signed-off-by: Bjørn Mork
---
drivers/net/usb/usbnet.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/usb/usbnet.c
Alan
On 10/18/2012 09:02 AM, Alan Stern wrote:
On Wed, 17 Oct 2012, vikram pandita wrote:
Alan
I was looking at your implementation [1] on Periodic schedule
enable/disable using hrtimer.
Looks really neat.
Thank you.
Current implementation is:
On Periodic Enable, Sate is: PSE =1 , PS
I didn't see patch #14, where is it?
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
David Miller writes:
> I didn't see patch #14, where is it?
It took an extra round in my mail queue due to a problem with one of the
receiving servers. Sorry about that. It will show up as soon as my mail
server decides to retry.
Bjørn
--
To unsubscribe from this list: send the line "unsubscri
On Thu, Oct 18, 2012 at 10:40:56PM +0200, Bjørn Mork wrote:
> From: Greg Suarez
>
> Based on revision 1.0 of "Universal Serial Bus Communications
> Class Subclass Specification for Mobile Broadband Interface
> Model" available from www.usb.org
>
> Signed-off-by: Greg Suarez
> [bmork: added DSS
On Thu, Oct 18, 2012 at 10:40:53PM +0200, Bjørn Mork wrote:
> The USB Communications Device Class "Mobile Broadband Interface Model"
> (MBIM) is the USB-IFs alternative to the current chipset/vendor
> specific solutions to "Mobile Broadband" device management. The
> specification, including the man
On Thu, 18 Oct 2012, vikram pandita wrote:
> >> Why on enable/disable would we not want to ensure PSS state is same as PSE?
> >
> > Because it would be a waste of time. Eventually the controller will
> > set PSS equal to PSE; why should the driver wait for it? The driver
> > can go on to do othe
MBIM devices can support up to 256 generic streams called
Device Service Streams (DSS). The MBIM spec says
The format of the Device Service Stream payload depends
on the device service (as identified by the corresponding
UUID) that is used when opening the data stream.
Example use cases
On Thu, Oct 18, 2012 at 08:15:57AM +0330, A Sh wrote:
> You told you will send me directions to compile a custom kernel.
Ok, here's some instructions for how to compile a custom kernel:
http://kernelnewbies.org/KernelBuild
Please download the latest stable kernel (currently v3.6.2), patch it
wit
On Thursday 18 October 2012 22:40:55 Bjørn Mork wrote:
> If the device refuses our updated value, then we must be prepared
> to receive URBs as big as the device wants to send. Set rx_max
> to the device provided value on error.
Problematic in principle. How do you allocate a buffer of arbitrary
Since you both seem to be asking the same question:
On Wed, Oct 17, 2012 at 10:34:39AM +0530, ankit patel wrote:
> Thanks Sarah,
>I just need to know that driver is doing reset when device is not
> attached by writing HCRST to 1 in USB Command Register of xHC. Does
> the host controller needs
Oliver Neukum writes:
> On Thursday 18 October 2012 22:40:55 Bjørn Mork wrote:
>> If the device refuses our updated value, then we must be prepared
>> to receive URBs as big as the device wants to send. Set rx_max
>> to the device provided value on error.
>
> Problematic in principle. How do you
On Wednesday 17 of October 2012 19:07:25 Ming Lei wrote:
> On Wed, Oct 17, 2012 at 1:43 PM, Rafael J. Wysocki wrote:
> >
> > This appears to be a bit too heavy handed. First of all, it seems to affect
> > all memory allocations going in parallel with the resume callback. Second,
>
> No, the fla
On Fri, Oct 19, 2012 at 12:09 AM, Bjørn Mork wrote:
> Oliver Neukum writes:
>> On Thursday 18 October 2012 22:40:55 Bjørn Mork wrote:
>>> If the device refuses our updated value, then we must be prepared
>>> to receive URBs as big as the device wants to send. Set rx_max
>>> to the device provide
On Fri, Oct 19, 2012 at 7:16 AM, Rafael J. Wysocki wrote:
>
> Well, the question is how many runtime resume callbacks actually allocate
> memory. If they are not too many, we can just flag all of them. Otherwise,
At least, almost all pci devices driver in some platform(acpi) and all usb
devices
Let's move what we can from plat/usb.h to the local usb.h
for ARM common zImage support.
This is needed so we can remove plat/usb.h for ARM common
zImage support.
Cc: Felipe Balbi
Cc: Samuel Ortiz
Cc: Alan Stern
Cc: Greg Kroah-Hartman
Cc: Partha Basak
Cc: Keshava Munegowda
Cc: linux-usb@vge
From: Felipe Balbi
In order to make single zImage work for ARM architecture,
we need to make sure we don't depend on private headers.
Move USB platform_data to
and add a minimal drivers/mfd/usb-omap.h.
Cc: Samuel Ortiz
Cc: Alan Stern
Cc: Greg Kroah-Hartman
Cc: Partha Basak
Cc: Keshava Mune
On Thu, Oct 18, 2012 at 7:54 AM, Andrew Morton
wrote:
>
> local_irq_save() and local_irq_restore() were mistakes :( It's silly to
> write what appears to be a C function and then have it operate like
> Pascal (warning: I last wrote some Pascal in 66 B.C.).
Considered that spin_lock_irqsave/spin_u
ethernet, ipv4, and ipv6 address testing uses 3 different api naming styles.
ethernet uses: is__ether_addr
ipv4 uses: ipv4_is_
ipv6 uses: ipv6_addr_
Standardize on the ipv6 style of _addr_ to reduce
the number of styles to remember.
The new consistent styles are:
eth_addr_(const u8 *
Hi Felipe
> > This patch enabled dma mapping, and used dma transfer handler
> > on mod_host
> >
> > Signed-off-by: Kuninori Morimoto
> > ---
> > Felipe
> >
> > I sent some patches before, and I guess it will go to v3.8 branch.
> > please let me know if you want me to resend these.
> >
> > dr
On Thu, Oct 18, 2012 at 03:16:04PM -0400, Alan Stern wrote:
> On Thu, 18 Oct 2012, Peter Chen wrote:
>
> > > I decided to use a different approach. The core should know which
> > > ports are suspended without asking the HCD. Can you test this patch in
> > > place of the other one?
> > It shows
Alexey Orishko writes:
> On Fri, Oct 19, 2012 at 12:09 AM, Bjørn Mork wrote:
>> Oliver Neukum writes:
>>> On Thursday 18 October 2012 22:40:55 Bjørn Mork wrote:
If the device refuses our updated value, then we must be prepared
to receive URBs as big as the device wants to send. Set rx
69 matches
Mail list logo
