Am 09.08.2013 21:32, schrieb Alan Stern:
On Fri, 9 Aug 2013, Oleksij Rempel wrote:
Is there any way to prevent the device from losing its firmware during
a USB reset or suspend?
For suspend - yes. It is possible to ignore suspend command or put the
SoC in low power mode - but is it probably n
> From: linux-usb-ow...@vger.kernel.org
> [mailto:linux-usb-ow...@vger.kernel.org] On Behalf Of Alexandra Yates
> Sent: Friday, August 09, 2013 12:21 PM
Hi Alexandra,
>
> +/* USB 2.0 BOS descriptor and a capability descriptor, combined */
> +static u8 usb2_bos_descriptor[] = {
> + USB_DT_
Modified the xHCI roothub descriptor to return USB2.0 extension
descriptor Best Effort Service Latency (BESL) and Deep Best Effort
Service Latency (DBESL) values when set on the xHCI host.
On link power management the BESL and DBESL values are used to
estimate L1 exit latency for USB2.0 host and d
From: David Daney
When CONFIG_USB_SUPPORT is not selected we get things like:
scripts/kconfig/mconf Kconfig
warning: (MIPS_SEAD3 && PMC_MSP && CPU_CAVIUM_OCTEON) selects
USB_EHCI_BIG_ENDIAN_MMIO which has unmet direct dependencies (USB_SUPPORT &&
USB)
It is much cleaner to make the various sy
On Fri, 9 Aug 2013, David Daney wrote:
> From: David Daney
>
> When CONFIG_USB_SUPPORT is not selected we get things like:
>
> scripts/kconfig/mconf Kconfig
> warning: (MIPS_SEAD3 && PMC_MSP && CPU_CAVIUM_OCTEON) selects
> USB_EHCI_BIG_ENDIAN_MMIO which has unmet direct dependencies (USB_SUPPO
From: David Daney
When CONFIG_USB_SUPPORT is not selected we get things like:
scripts/kconfig/mconf Kconfig
warning: (MIPS_SEAD3 && PMC_MSP && CPU_CAVIUM_OCTEON) selects
USB_EHCI_BIG_ENDIAN_MMIO which has unmet direct dependencies (USB_SUPPORT &&
USB)
It is much cleaner to make the various sy
Oh! Bummer, I rant the tool but somehow I missed it. I will add the tool,
change and resend patch.
Thank you,
Alexandra.
-Original Message-
From: Sarah Sharp [mailto:sarah.a.sh...@linux.intel.com]
Sent: Friday, August 09, 2013 4:43 PM
To: Yates, Alexandra
Cc: Alexandra Yates; gre.
The following changes since commit c095ba7224d8edc71dcef0d655911399a8bd4a3f:
Linux 3.11-rc4 (2013-08-04 13:46:46 -0700)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git/
tags/usb-3.11-rc5
for you to fetch changes up to 444ce9d44d00969479a
Hi Alex,
Can you make sure to add the checkpatch.pl to your git pre-commit hooks?
This patch fails checkpatch:
sarah@xanatos:~/git/kernels/xhci$ git am -s ~/Maildir.fetchmail/.to-apply
Applying: xhci-hub: Roothub USB2.0 descriptor for BESL DBESL
ERROR: spaces prohibited around that '->' (ctx:WxV)
On Fri, 9 Aug 2013, Greg KH wrote:
> On Thu, Aug 08, 2013 at 10:07:19PM +0200, Martin MOKREJŠ wrote:
> > Hi,
> > I get plenty of these in /sys/kernel/debug/kmemleak:
> >
> > unreferenced object 0x88019f675268 (size 32):
> > comm "usb-storage", pid 11411, jiffies 4310515592 (age 1538.100s)
On Fri, Aug 09, 2013 at 02:12:17PM -0700, David Miller wrote:
> From: Ming Lei
> Date: Thu, 8 Aug 2013 21:48:21 +0800
>
> > This patchset allows drivers to pass sg buffers which size can't be divided
> > by max packet size of endpoint if the host controllers(such ax xHCI) support
> > this kind o
From: Ming Lei
Date: Thu, 8 Aug 2013 21:48:21 +0800
> This patchset allows drivers to pass sg buffers which size can't be divided
> by max packet size of endpoint if the host controllers(such ax xHCI) support
> this kind of sg buffers.
>
> Previously we added check[1] on the situation and don't
From: Jussi Kivilinna
Date: Wed, 07 Aug 2013 16:26:29 +0300
> URB setup packet must not be allocated as part of larger structure
> because DMA coherence issues.
>
> Patch changes catc to allocate ctrl_dr member as separate buffer.
>
> Patch is only compile tested.
>
> Cc:
> Signed-off-by: Jus
On 09/08/13 19:08, Alan Stern wrote:
On Fri, 9 Aug 2013, Lionel Landwerlin wrote:
Hi there,
I'm playing with 2 small usb webcams, showing up the stream of the
camera with it's plugged in and hiding it when it's unplugged. I'm
basically using 2 external camera + the integrated one.
I've notice
On Fri, Aug 09, 2013 at 04:31:25PM +0300, Felipe Balbi wrote:
> On Thu, Aug 08, 2013 at 07:25:47AM -0700, Sarah Sharp wrote:
> > I've heard there's little to do at the software level, but I haven't
>
> you've heard ? IIRC your name is in the spec :-)
/me checks. Huh, yes, you're right it is in t
Will do! Thank you Sarah and Greg for the great feedback to make this patch
better :)
Thank you,
Alexandra.
-Original Message-
From: linux-usb-ow...@vger.kernel.org [mailto:linux-usb-ow...@vger.kernel.org]
On Behalf Of Sarah Sharp
Sent: Friday, August 09, 2013 1:03 PM
To: Alexandr
Hello.
On 08/09/2013 08:03 PM, Sebastian Andrzej Siewior wrote:
This is what I observe:
On the first connect, the musb starts with DEVCTL.Session set. On
disconnect, musb_core calls try_idle. That functions removes the Session
bit signalize that the session is over (something that only in OTG i
Hi Alex,
For just one patch, you don't need to send a cover letter, especially
when it contains the contents of your patch description. Also, please
use PATCH instead of RFC in the future.
I'll test this patch out, and try to come up with a way to get the USB
core to use bcdUSB 2.10 for xHCI USB
On Fri, 9 Aug 2013, Greg KH wrote:
> On Fri, Aug 09, 2013 at 09:52:13AM -0500, Thomas Pugliese wrote:
> > This patch fixes a kernel panic that can occur when disconnecting a
> > wireless USB->serial device. When the serial device disconnects, the
> > device cleanup procedure ends up calling u
On Thu, Aug 08, 2013 at 10:07:19PM +0200, Martin MOKREJŠ wrote:
> Hi,
> I get plenty of these in /sys/kernel/debug/kmemleak:
>
> unreferenced object 0x88019f675268 (size 32):
> comm "usb-storage", pid 11411, jiffies 4310515592 (age 1538.100s)
> hex dump (first 32 bytes):
> 05 0f 16 0
On Fri, 9 Aug 2013, Oleksij Rempel wrote:
> > Is there any way to prevent the device from losing its firmware during
> > a USB reset or suspend?
>
> For suspend - yes. It is possible to ignore suspend command or put the
> SoC in low power mode - but is it probably not so easy to bring it back.
Modified the xHCI roothub descriptor to return USB2.0 extension
descriptor Best Effort Service Latency (BESL) and Deep Best Effort
Service Latency (DBESL) values when set on the xHCI host.
On link power management the BESL and DBESL values are used to
estimate L1 exit latency for USB2.0 host and d
Modified the xHCI roothub descriptor to return USB2.0 extension
descriptor Best Effort Service Latency (BESL) and Deep Best Effort
Service Latency (DBESL) values when set on the xHCI host.
On link power management the BESL and DBESL values are used to
estimate L1 exit latency for USB2.0 host and d
On Fri, Aug 09, 2013 at 09:52:13AM -0500, Thomas Pugliese wrote:
> This patch fixes a kernel panic that can occur when disconnecting a
> wireless USB->serial device. When the serial device disconnects, the
> device cleanup procedure ends up calling usb_hcd_disable_endpoint on the
> serial devic
On Fri, 9 Aug 2013, Alberto Gonzalez wrote:
> On Wednesday 31 July 2013 14:38:34 Alan Stern wrote:
> > On Wed, 31 Jul 2013, Alberto Gonzalez wrote:
> > >
> > > There seems to be a regression in Kernel 3.10.3 (from 3.10.2) that
> > > prevents me being able to mount my external USB hard drive.
> >
Am 09.08.2013 19:16, schrieb Alan Stern:
On Fri, 9 Aug 2013, Oleksij Rempel wrote:
Am 09.08.2013 16:52, schrieb Alan Stern:
On Fri, 9 Aug 2013, Oleksij Rempel wrote:
What about a "get firmware version" sort of thing? There really should
be a way for the driver to tell whether the firmware h
On Thu, 8 Aug 2013, Jussi Kivilinna wrote:
> On 08.08.2013 18:14, Petko Manolov wrote:
> > On Wed, 7 Aug 2013, Jussi Kivilinna wrote:
> >
> >> rtl8150 allocates URB transfer_buffer and setup_packet as part of same
> >> structure 'struct async_req'. This can cause same cacheline to be
> >> DMA-m
On Aug 9, 2013, at 11:28 AM, Mark Rutland wrote:
> On Fri, Aug 09, 2013 at 04:40:32PM +0100, Kumar Gala wrote:
>> The binding spec wasn't clear that the order of the phandles in the
>> usb-phy array has meaning. Clarify this point in the binding that
>> it should be .
>>
>> Signed-off-by: Kumar
Some bad gadget drivers do not check the return status of usb_add_config.
Thus they get a not correctly initialized config and when this gadget gets
deactivated the whole kernel crashes. Since on initialization failure cdev
is set to NULL it can be used to detect this problem situation. It can be
a
On Fri, Aug 09, 2013 at 08:51:30AM -0700, Sarah Sharp wrote:
> On Thu, Aug 08, 2013 at 04:11:46PM -0700, Greg KH wrote:
> > Note, I don't apply "RFC" patches, and rarely review them. Why are you
> > claiming that is what this is, when it is in the 3rd version already?
>
> This is a new approach f
On Fri, 9 Aug 2013, Lionel Landwerlin wrote:
> Hi there,
>
> I'm playing with 2 small usb webcams, showing up the stream of the
> camera with it's plugged in and hiding it when it's unplugged. I'm
> basically using 2 external camera + the integrated one.
>
> I've noticed the usb port just beco
On Wednesday 31 July 2013 14:38:34 Alan Stern wrote:
> On Wed, 31 Jul 2013, Alberto Gonzalez wrote:
> >
> > There seems to be a regression in Kernel 3.10.3 (from 3.10.2) that
> > prevents me being able to mount my external USB hard drive.
>
> See
>
> http://marc.info/?l=linux-usb&m=137523
From: Eric Dumazet
Date: Fri, 09 Aug 2013 10:50:36 -0700
> Some compiler folk decided 'extern' were not mandatory for code, but its
> really adding confusion and endless discussions.
I certainly stopped talking about it, you could too :-)
--
To unsubscribe from this list: send the line "unsubscr
On Fri, Aug 9, 2013 at 2:40 PM, Stephen Hemminger
wrote:
>> > +extern const struct driver_info ax88172a_info;
>>
>> You could drop the 'extern' here.
>>
>> All other function prototypes in this header file do not use 'extern'.
>
> That is data, not function prototype, so yes extern is needed.
Yo
On Fri, 2013-08-09 at 10:40 -0700, Stephen Hemminger wrote:
> On Fri, 9 Aug 2013 14:39:06 -0300
> Fabio Estevam wrote:
>
> > On Fri, Aug 9, 2013 at 2:31 PM, Mark Brown wrote:
> > > From: Mark Brown
> > >
> > > Ensure that the definition of ax88172a_info matches the declaration seen
> > > by use
On Fri, 9 Aug 2013 14:39:06 -0300
Fabio Estevam wrote:
> On Fri, Aug 9, 2013 at 2:31 PM, Mark Brown wrote:
> > From: Mark Brown
> >
> > Ensure that the definition of ax88172a_info matches the declaration seen
> > by users and silence sparse warnings about symbols without declarations
> > in the
On Fri, Aug 9, 2013 at 2:31 PM, Mark Brown wrote:
> From: Mark Brown
>
> Ensure that the definition of ax88172a_info matches the declaration seen
> by users and silence sparse warnings about symbols without declarations
> in the global namespace by moving the declaration into the shared header
>
From: Mark Brown
Make functions that are only referenced from ops structures static, they
do not need to be in the global namespace and sparse complains about this.
Signed-off-by: Mark Brown
---
drivers/net/usb/ax88172a.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --gi
From: Mark Brown
Ensure that the definition of ax88172a_info matches the declaration seen
by users and silence sparse warnings about symbols without declarations
in the global namespace by moving the declaration into the shared header
asix.h.
Signed-off-by: Mark Brown
---
drivers/net/usb/asix.
Hi Shawn,
I noticed that the ChromeOS kernel tree is still using this particular
patch, and thought it was probably time to revisit it.
On Sat, May 25, 2013 at 09:57:57AM -0700, Shawn Nematbakhsh wrote:
> Hi Sarah and Alan,
>
> Thanks for the comments. I will make the following revisions:
>
> 1
On Fri, 9 Aug 2013, Ismail, Abdul R wrote:
> Alan,
>
> Just wondering why we don't require the HCDs to validate the length
> instead of blindly copying the descriptor to the buffer passed in. In
> the pseudo host controller we are developing we only copy the number
> of bytes requested to the buf
On Fri, 9 Aug 2013, Oleksij Rempel wrote:
> Am 09.08.2013 16:52, schrieb Alan Stern:
> > On Fri, 9 Aug 2013, Oleksij Rempel wrote:
> >
> >>> What about a "get firmware version" sort of thing? There really should
> >>> be a way for the driver to tell whether the firmware has already been
> >>> upd
You are right that comment should read USB2.0! Will resend.
Thank you.
From: linux-usb-ow...@vger.kernel.org [linux-usb-ow...@vger.kernel.org] on
behalf of Greg KH [gre...@linuxfoundation.org]
Sent: Thursday, August 08, 2013 5:36 PM
To: Alexandra Yates
On Fri, Aug 09, 2013 at 04:40:32PM +0100, Kumar Gala wrote:
> The binding spec wasn't clear that the order of the phandles in the
> usb-phy array has meaning. Clarify this point in the binding that
> it should be .
>
> Signed-off-by: Kumar Gala
> ---
> Documentation/devicetree/bindings/usb/dwc3
Hi,
On Fri, 2013-08-09 at 16:23 +0300, Felipe Balbi wrote:
> Hi,
>
> On Tue, Aug 06, 2013 at 02:53:11PM +0300, Ivan T. Ivanov wrote:
> > diff --git a/drivers/usb/dwc3/dwc3-msm.c b/drivers/usb/dwc3/dwc3-msm.c
> > new file mode 100644
> > index 000..e509abc
> > --- /dev/null
> > +++ b/drivers/
This is what I observe:
On the first connect, the musb starts with DEVCTL.Session set. On
disconnect, musb_core calls try_idle. That functions removes the Session
bit signalize that the session is over (something that only in OTG is
required). A new device, that is plugged, is no longer recognized.
Alan,
Just wondering why we don't require the HCDs to validate the length instead of
blindly copying the descriptor to the buffer passed in. In the pseudo host
controller we are developing we only copy the number of bytes requested to the
buffer passed in.
Thx!
Rahman
-Original Message---
On Thu, Aug 08, 2013 at 04:11:46PM -0700, Greg KH wrote:
> Note, I don't apply "RFC" patches, and rarely review them. Why are you
> claiming that is what this is, when it is in the 3rd version already?
This is a new approach from the last patch (dynamic allocation vs.
making the static tbuf bigge
Am 09.08.2013 16:52, schrieb Alan Stern:
On Fri, 9 Aug 2013, Oleksij Rempel wrote:
What about a "get firmware version" sort of thing? There really should
be a way for the driver to tell whether the firmware has already been
updated.
I was not able to find good direct way to check firmware ve
The binding spec wasn't clear that the order of the phandles in the
usb-phy array has meaning. Clarify this point in the binding that
it should be .
Signed-off-by: Kumar Gala
---
Documentation/devicetree/bindings/usb/dwc3.txt | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git
Hi,
On Fri, 2013-08-09 at 16:32 +0300, Felipe Balbi wrote:
> On Fri, Aug 09, 2013 at 12:53:47PM +0300, Ivan T. Ivanov wrote:
> > From: "Ivan T. Ivanov"
> >
> > DWC3 glue layer is hardware layer around Synopsys DesignWare
> > USB3 core. Its purpose is to supply Synopsys IP with required
> > clo
> On Fri, Aug 09, 2013 at 01:34:09PM +, Wang, Yu Y wrote:
> > > On Wed, Aug 07, 2013 at 12:03:34PM +, Wang, Yu Y wrote:
> > > > Hi Balbi,
> > > >
> > > > Because dwc3 driver request_threaded_irq with flags IRQF_ONESHOT
> > > > and IRQF_SHARED.
> > > > But xHCI driver will not set IRQF_ONESH
On Fri, 9 Aug 2013, Felipe Balbi wrote:
> > > heh, it doesn't need to be entirely in the core. Core could have the
> > > generic calls and HCDs could implement some callbacks, but I think quite
> > > a bit of the code will be similar if we implement the same thing on all
> > > HCDs.
> >
> > What
On Fri, Aug 9, 2013 at 10:26 AM, Felipe Balbi wrote:
> looks good to me too, please send it as a real patch (if you already
> have, sorry about that, catching up with my inbox now).
Yes, I have already sent it as a formal patch.
--
To unsubscribe from this list: send the line "unsubscribe linux-
On Fri, Aug 09, 2013 at 05:38:57PM +0300, Felipe Balbi wrote:
> On Thu, Aug 08, 2013 at 12:35:04PM +0100, Mark Brown wrote:
> > Systems with the common clock API need clk_prepare() as well as the enable
> > step.
> clk_prepare() is done on probe()... -ECONFUSED
Ah, so it is, missed that - though
On Fri, 9 Aug 2013, Alexander Shishkin wrote:
> Alan Stern writes:
>
> > On Fri, 9 Aug 2013, Alexander Shishkin wrote:
> >
> >> Peter Chen writes:
> >>
> >> > Now, chipidea host has already depended on USB_EHCI_HCD
> >> >
> >> > Signed-off-by: Peter Chen
> >>
> >> Alan, what do you think?
>
This patch fixes a kernel panic that can occur when disconnecting a
wireless USB->serial device. When the serial device disconnects, the
device cleanup procedure ends up calling usb_hcd_disable_endpoint on the
serial device's endpoints. The wusbcore uses the ABORT_RPIPE command to
abort all t
On Fri, 9 Aug 2013, Oleksij Rempel wrote:
> > What about a "get firmware version" sort of thing? There really should
> > be a way for the driver to tell whether the firmware has already been
> > updated.
>
> I was not able to find good direct way to check firmware version. If i
> would add some
Hi,
On Fri, Aug 09, 2013 at 10:37:50AM -0400, Alan Stern wrote:
> > > > Wait a minute, didn't we discuss a while back that these test features
> > > > should be built into usbcore so that we could have a usbcv clone for
> > > > linux ?
> > >
> > > There's no way this can be built into the core.
On Fri, Aug 09, 2013 at 03:50:10PM +0200, Laurent Pinchart wrote:
> Hi Felipe,
>
> Here's a small fix for the UVC gadget driver, could you please take it in
> your
> tree for v3.12 ?
>
> The following changes since commit 941ea3616c747545d0278fc432fb7919b6d0d8f0:
>
> usb: dwc3: use dev_get_p
Alan Stern writes:
> On Fri, 9 Aug 2013, Alexander Shishkin wrote:
>
>> Peter Chen writes:
>>
>> > Now, chipidea host has already depended on USB_EHCI_HCD
>> >
>> > Signed-off-by: Peter Chen
>>
>> Alan, what do you think?
>>
>> > ---
>> > drivers/usb/host/Kconfig |4 ++--
>> > 1 files c
On Thu, Aug 08, 2013 at 12:35:04PM +0100, Mark Brown wrote:
> From: Mark Brown
>
> Systems with the common clock API need clk_prepare() as well as the enable
> step.
>
> Signed-off-by: Mark Brown
clk_prepare() is done on probe()... -ECONFUSED
--
balbi
signature.asc
Description: Digital sig
On Fri, 9 Aug 2013, Felipe Balbi wrote:
> > > Wait a minute, didn't we discuss a while back that these test features
> > > should be built into usbcore so that we could have a usbcv clone for
> > > linux ?
> >
> > There's no way this can be built into the core. This test requires the
> > behavio
Am 09.08.2013 16:13, schrieb Alan Stern:
On Fri, 9 Aug 2013, Christian Lamparter wrote:
After loading firmware, a reset generally is necessary. Some devices
will do it themselves; others require you to call usb_reset_device().
This makes things complicated. Because, as far as I remember,
usb
On Fri, 9 Aug 2013, Alexander Shishkin wrote:
> Peter Chen writes:
>
> > Now, chipidea host has already depended on USB_EHCI_HCD
> >
> > Signed-off-by: Peter Chen
>
> Alan, what do you think?
>
> > ---
> > drivers/usb/host/Kconfig |4 ++--
> > 1 files changed, 2 insertions(+), 2 deletion
On Fri, 9 Aug 2013, Sujith Manoharan wrote:
> Christian Lamparter wrote:
> > So, if we call usb_reset_device there and the driver is unbound
> > and later rebound. the next ath9k_htc .probe will start again and
> > again and again not knowing that it is already initialized
> > (and we have a loop
On Fri, Aug 02, 2013 at 09:24:10PM +0200, Sebastian Andrzej Siewior wrote:
> This driver is currently used by musb' cppi41 couter part. I may merge
> both dma engine user of musb at some point but not just yet.
>
> The driver seems to work in RX/TX mode in host mode, tested on mass
> storage. I in
This patch fixes a kernel panic that can occur when unplugging the HWA
dongle while a downstream device is in the process of disconnecting.
This involved 2 changes. First, call usb_lock_device_for_reset before
usb_reset_device to synchronize the HWA's post_rest and disconnect
routines. Secon
Hi Tony,
here's a pull request of one patch to avoid conflicts during the merge
window.
Please consider applying to your tree and I'll take this same patch
upstream.
The following changes since commit 5ae90d8e467e625e447000cb4335c4db973b1095:
Linux 3.11-rc3 (2013-07-28 20:53:33 -0700)
are av
On Thu, 8 Aug 2013, Sean O. Stalley wrote:
> rh_call_control() contains a buffer, tbuf, which it uses to hold
> USB descriptors. These discriptors are eventually copied into the
> transfer_buffer in the URB. The buffer in the URB is dynamically
> defined and is always large enough to hold the amou
On Fri, 9 Aug 2013, Christian Lamparter wrote:
> > After loading firmware, a reset generally is necessary. Some devices
> > will do it themselves; others require you to call usb_reset_device().
>
> This makes things complicated. Because, as far as I remember,
> usb_reset_device() will cause the
On Fri, Aug 09, 2013 at 07:46:00PM +0800, Peter Chen wrote:
> On Fri, Aug 09, 2013 at 04:23:11PM +0300, Alexander Shishkin wrote:
> > Peter Chen writes:
> >
> > > CI_HDRC_REGS_SHARED stands for the controller registers is shared
> > > with other USB drivers, if all USB drivers are at chipidea/, i
Hi Felipe,
Here's a small fix for the UVC gadget driver, could you please take it in your
tree for v3.12 ?
The following changes since commit 941ea3616c747545d0278fc432fb7919b6d0d8f0:
usb: dwc3: use dev_get_platdata() (2013-07-31 16:52:02 +0300)
are available in the git repository at:
git
On Fri, Aug 09, 2013 at 01:34:09PM +, Wang, Yu Y wrote:
> > On Wed, Aug 07, 2013 at 12:03:34PM +, Wang, Yu Y wrote:
> > > Hi Balbi,
> > >
> > > Because dwc3 driver request_threaded_irq with flags IRQF_ONESHOT and
> > > IRQF_SHARED.
> > > But xHCI driver will not set IRQF_ONESHOT. Then will
Hi,
On Thu, Jul 25, 2013 at 05:33:48PM -0400, Alan Stern wrote:
> > On Thu, Jul 25, 2013 at 03:44:20PM -0400, Alan Stern wrote:
> > > On Thu, 25 Jul 2013, Greg KH wrote:
> > >
> > > > On Tue, Jul 02, 2013 at 08:13:52PM -0700, Jack Pham wrote:
> > > > > From: Manu Gautam
> > > > >
> > > > > The
On Fri, Aug 09, 2013 at 04:23:11PM +0300, Alexander Shishkin wrote:
> Peter Chen writes:
>
> > CI_HDRC_REGS_SHARED stands for the controller registers is shared
> > with other USB drivers, if all USB drivers are at chipidea/, it doesn't
> > needed to set.
>
> We still have the msm driver that us
On Tue, Aug 06, 2013 at 11:27:11PM +0400, Sergei Shtylyov wrote:
> Hello.
>
> On 08/06/2013 11:16 PM, Tuomas Tynkkynen wrote:
>
> >[...]
>
> >>>+ match = of_match_device(tegra_ehci_of_match, &pdev->dev);
> >>>+ if (!match) {
> >>>+ dev_err(&pdev->dev, "Error: No device match found\n")
> On Wed, Aug 07, 2013 at 12:03:34PM +, Wang, Yu Y wrote:
> > Hi Balbi,
> >
> > Because dwc3 driver request_threaded_irq with flags IRQF_ONESHOT and
> > IRQF_SHARED.
> > But xHCI driver will not set IRQF_ONESHOT. Then will met IRQ request
> > failed if use same IRQ number.
> >
> >
> > <4>[1
On Fri, Aug 09, 2013 at 12:53:47PM +0300, Ivan T. Ivanov wrote:
> From: "Ivan T. Ivanov"
>
> DWC3 glue layer is hardware layer around Synopsys DesignWare
> USB3 core. Its purpose is to supply Synopsys IP with required
> clocks, voltages and interface it with the rest of the SoC.
>
> Signed-off-b
On Thu, Aug 08, 2013 at 07:25:47AM -0700, Sarah Sharp wrote:
> I've heard there's little to do at the software level, but I haven't
you've heard ? IIRC your name is in the spec :-)
--
balbi
signature.asc
Description: Digital signature
Hi,
On Wed, Aug 07, 2013 at 12:12:59PM +, Wang, Yu Y wrote:
>
> Hi Balbi,
>
> Because dwc3 driver request_threaded_irq with flags
> IRQF_ONESHOT and IRQF_SHARED.
> But xHCI driver will not set IRQF_ONESHOT. Then will met
> IRQ request failed if use same IRQ number.
can you please fix your e
On Wed, Aug 07, 2013 at 12:03:34PM +, Wang, Yu Y wrote:
> Hi Balbi,
>
> Because dwc3 driver request_threaded_irq with flags
> IRQF_ONESHOT and IRQF_SHARED.
> But xHCI driver will not set IRQF_ONESHOT. Then will met
> IRQ request failed if use same IRQ number.
>
>
> <4>[1.019248] Call Tra
On Thu, Aug 08, 2013 at 10:25:36AM -0700, Sarah Sharp wrote:
> Hi Yu,
>
> Please test this patch, and make sure that interrupts aren't registered
> twice. I think this approach is better, since it creates a new quirk
> specifically for xhci platform devices, so we can tell them apart from
> PCI d
On Wed, Aug 07, 2013 at 06:01:00PM +0800, Andy Green wrote:
> On 7 August 2013 17:34, Mark Brown wrote:
> > On Tue, Aug 06, 2013 at 10:35:52PM -0300, Fabio Estevam wrote:
> >> On Tue, Aug 6, 2013 at 12:49 PM, Mark Brown wrote:
> >> > From: Andy Green
> >> >
> >> > You might have CONFIG_PM, but y
Hi,
On Tue, Aug 06, 2013 at 01:21:38PM +0100, Pawel Moll wrote:
> > @@ -47,3 +64,25 @@ Example device nodes:
> > vddcx-supply = <&supply>;
> > v1p8-supply = <&supply>;
> > };
> > +
> > + usb@fd4ab000 {
> > + compatible = "qcom,dwc-usb3-msm";
> > +
Hi,
On Tue, Aug 06, 2013 at 02:53:11PM +0300, Ivan T. Ivanov wrote:
> diff --git a/drivers/usb/dwc3/dwc3-msm.c b/drivers/usb/dwc3/dwc3-msm.c
> new file mode 100644
> index 000..e509abc
> --- /dev/null
> +++ b/drivers/usb/dwc3/dwc3-msm.c
> @@ -0,0 +1,175 @@
> +#undef CONFIG_REGULATOR
why ?
Peter Chen writes:
> CI_HDRC_REGS_SHARED stands for the controller registers is shared
> with other USB drivers, if all USB drivers are at chipidea/, it doesn't
> needed to set.
We still have the msm driver that uses REGS_SHARED.
> CI_HDRC_PULLUP_ON_VBUS stands for pullup dp when the vbus is on
Peter Chen writes:
> On Fri, Aug 09, 2013 at 03:55:56PM +0300, Alexander Shishkin wrote:
>> > + } else {
>> > + /*
>> > + * If the controller is not OTG capable, but support
>> > + * role switch, the defalt role is gadget, and the
>> >
On Mon, Aug 05, 2013 at 03:08:43PM -0700, Andi Kleen wrote:
> From: Andi Kleen
>
> Fix (randconfig) build problem with DEBUG_FS on
>
> Cc: ba...@ti.com
> Cc: linux-usb@vger.kernel.org
> Signed-off-by: Andi Kleen
care to show me the defconfig which makes the build fail ?
--
balbi
signature.
On Sun, Aug 04, 2013 at 08:45:24PM +0400, Sergei Shtylyov wrote:
> Hello.
>
>I have basically two questions on this change:
>
> 1) why it wasn't published before committing?
look deeper
> 2) why you omitted am35x.c from this commit?
mistake
--
balbi
signature.asc
Description: Digital s
On Fri, Aug 09, 2013 at 04:00:13PM +0300, Alexander Shishkin wrote:
> Peter Chen writes:
>
> > This file is mainly used to access otgsc currently, it may
> > add otg related things in the future.
> >
> > Tested-by: Marek Vasut
> > Signed-off-by: Peter Chen
> > ---
> > drivers/usb/chipidea/Make
Peter Chen writes:
A few typos:
> Move otg relate things to otg file.
"related"
>
> Tested-by: Marek Vasut
> Signed-off-by: Peter Chen
> ---
> drivers/usb/chipidea/core.c | 63
> +--
> drivers/usb/chipidea/otg.c | 57
On Fri, Aug 09, 2013 at 03:55:56PM +0300, Alexander Shishkin wrote:
> Peter Chen writes:
>
> > Since we need otgsc to know vbus's status at some chipidea
> > controllers even it is peripheral-only mode. Besides, some
> > SoCs (eg, AR9331 SoC) don't have otgsc register even
> > the DCCPARAMS_DC an
Peter Chen writes:
> This file is mainly used to access otgsc currently, it may
> add otg related things in the future.
>
> Tested-by: Marek Vasut
> Signed-off-by: Peter Chen
> ---
> drivers/usb/chipidea/Makefile |2 +-
> drivers/usb/chipidea/bits.h | 10
> drivers/usb/chipide
Peter Chen writes:
> Since we need otgsc to know vbus's status at some chipidea
> controllers even it is peripheral-only mode. Besides, some
> SoCs (eg, AR9331 SoC) don't have otgsc register even
> the DCCPARAMS_DC and DCCPARAMS_HC are both 1 at CAP_DCCPARAMS.
>
> We inroduce flag CI_HDRC_DUAL_RO
On Fri, Aug 09, 2013 at 06:25:29PM +0800, Peter Chen wrote:
> > -void usbmisc_unset_ops(const struct usbmisc_ops *ops)
> > -{
> > - usbmisc_ops = NULL;
> > -}
> > -EXPORT_SYMBOL_GPL(usbmisc_unset_ops);
> > -
> > -int usbmisc_get_init_data(struct device *dev, struct usbmisc_usb_device
> > *usbdev
On Fri, Aug 09, 2013 at 08:56:56AM +0200, Sascha Hauer wrote:
> The chipidea i.MX driver is split into two drivers. The ci_hdrc_imx driver
> handles the chipidea cores and the usbmisc_imx driver handles the noncore
> registers common to all chipidea cores (but SoC specific). Current flow is:
>
> -
On Fri, Aug 09, 2013 at 08:20:45AM +0200, Sascha Hauer wrote:
> On Thu, Aug 08, 2013 at 06:19:43PM +0800, Peter Chen wrote:
> > > > - compatible = "fsl,imx25-usbmisc";
> > > > - clocks = <&clks 9>, <&clks 70>, <&clks
> > > > 8>;
> > > > -
Fabio Estevam writes:
> From: Fabio Estevam
>
> After the rename to ci_hdrc we ended up with two MODULE_ALIAS entries, so
> remove the old one.
>
> Signed-off-by: Fabio Estevam
Applied and pushed, thanks!
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of
Lothar Waßmann writes:
> This patch provides a cleaner solution to the problem described in
> commit 20a677fd.
>
> The goal to be achieved is to force USB_CHIPIDEA=m if either
> USB_EHCI_HCD=m or USB_GADGET=m.
> If both are 'y' USB_CHIPIDEA may be selected to be 'm' or 'y'.
>
> The old patch had
1 - 100 of 127 matches
Mail list logo
