The debug output now contains the wrong variable, as seen from the compiler
warning:
drivers/usb/gadget/udc/atmel_usba_udc.c: In function 'usba_ep_enable':
drivers/usb/gadget/udc/atmel_usba_udc.c:632:550: error: 'ept_cfg' may be used
uninitialized in this function [-Werror=maybe-uninitialized]
Hi,
During the testing of the g_webcam on PXA27x (HTC Magician machine) I found a
race condition. In the situations when the UDC receives a SETUP DATA packet
sooner than it is "requested" by usb_ep_queue() it will block itself from the
reading the FIFO.
Below I suggest the fix, but it is very
On 03/08/2017 12:36 AM, Yuyang Du wrote:
Hi Krzysztof,
On Tue, Mar 07, 2017 at 10:25:31AM +0100, Krzysztof Opasiak wrote:
If it' not some "top secret" device maybe try to send us descriptors
(lsusb -vd VID:PID)?
Pasted at the end. Surprisingly, I don't see isochronous types. Is it
bulk str
Hi,
Ganesh Biradar writes:
> Hello experts,
>
> I'm working on a project which uses TI OMAP Plus platform. i'm trying
> to implement uvc gadget through configfs.
>
> kernel: 4.4
please try v4.10 or v4.11-rc1, or ask for support through TI's support
channel ;-)
--
balbi
signature.asc
Descrip
Felipe Balbi writes:
> Hi Thinh,
>
> Thinh Nguyen writes:
>>> drivers/usb/dwc3/gadget.c | 5 +
>>> 1 file changed, 5 insertions(+)
>>>
>>> diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c
>>> index 6faf484e5dfc..0a664d8eba3f 100644
>>> --- a/drivers/usb/dwc3/gadget.c
>>>
Hi Thinh,
Thinh Nguyen writes:
>> drivers/usb/dwc3/gadget.c | 5 +
>> 1 file changed, 5 insertions(+)
>>
>> diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c
>> index 6faf484e5dfc..0a664d8eba3f 100644
>> --- a/drivers/usb/dwc3/gadget.c
>> +++ b/drivers/usb/dwc3/gadget.c
>>
> On Mon, Mar 06, 2017 at 04:20:57PM +0200, Elena Reshetova wrote:
> > refcount_t type and corresponding API should be
> > used instead of atomic_t when the variable is used as
> > a reference counter. This allows to avoid accidental
> > refcounter overflows that might lead to use-after-free
> > si
> On Mon, Mar 06, 2017 at 04:20:55PM +0200, Elena Reshetova wrote:
> > refcount_t type and corresponding API should be
> > used instead of atomic_t when the variable is used as
> > a reference counter. This allows to avoid accidental
> > refcounter overflows that might lead to use-after-free
> > si
On Wed, Mar 08, 2017 at 09:42:09AM +, Reshetova, Elena wrote:
> > On Mon, Mar 06, 2017 at 04:20:55PM +0200, Elena Reshetova wrote:
> > > refcount_t type and corresponding API should be
> > > used instead of atomic_t when the variable is used as
> > > a reference counter. This allows to avoid ac
On 08.03.2017 02:51, Chunfeng Yun wrote:
On Tue, 2017-03-07 at 17:10 +0200, Mathias Nyman wrote:
On 07.03.2017 05:32, Chunfeng Yun wrote:
hcc_params is set in xhci_gen_setup() called from usb_add_hcd(),
so checks the Maximum Primary Stream Array Size in the hcc_params
register after adding prim
Hi Petr,
I saw your patch and at the same time I saw Laurent suggestion.
I have tried both patches both are working but I'm going with Laurent suggestion
UVC_ATTR(f_uvc_opts_, cname, cname).
I'm going with unofficial uvc-gadget.
Before I can test usb uvc gadget not enumerating.
Regards,
Gane
Hi,
Felipe Balbi writes:
>> Thinh Nguyen writes:
drivers/usb/dwc3/gadget.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c
index 6faf484e5dfc..0a664d8eba3f 100644
--- a/drivers/usb/dwc3/gadget.c
+
Hi list! I'm running into a USB3.0 regression on ipq806x hardware.
Consider one physical USB port as one linux-4.9 xHCI HC. It comes with
two USB busses with one logical USB port each. One high-speed port and
one superspeed port port.
] ehci_hcd: USB 2.0 'Enhanced' Host Controller (EHCI) Drive
Hi,
Felipe Balbi writes:
>>> I've been looking at this and based on sniffer and dwc3 tracepoints, it
>>> seems like dwc3 is behaving properly. The real issue seems to be that
>>> g_mass_storage isn't queueing a new request to IN endpoint.
>>>
>>> I'll continue debugging this and try to find a so
Hi,
(no top-posting, please)
Ganesh Biradar writes:
> Hi Felipe,
>
> I'm using TI latest vision sdk which has support for 4.4 kernel. i'm
> not getting much help from TI e2e that's why i am asking here. can't
> try v4.10 or any other kernel as it has dependencies with respect to
> SDK.
sorry,
Hi,
Arnd Bergmann writes:
> The debug output now contains the wrong variable, as seen from the compiler
> warning:
>
> drivers/usb/gadget/udc/atmel_usba_udc.c: In function 'usba_ep_enable':
> drivers/usb/gadget/udc/atmel_usba_udc.c:632:550: error: 'ept_cfg' may be used
> uninitialized in this f
Sometimes, we might get a completion for a TRB which is left with HWO
bit. Even in these cases, we should increment req->remaining to
properly report total transferred size. I noticed this while debuggin
a separate problem seen with MSC tests from USBCV. Sometimes we would
erroneously report a comp
On 07/03/2017 17:47, Mason wrote:
> As suggested by Marc, I'm trying to adapt
> drivers/pci/host/pcie-altera-msi.c
> to my platform.
For my own reference, I have enabled verbose XHCI debug logs.
I have highlighted suspicious output with *
[0.00] Booting Linux on physical CPU 0x0
[
On 02/03/17 14:22, Johan Hovold wrote:
On Tue, Feb 28, 2017 at 12:51:25PM +, Ian Abbott wrote:
Valid latency timer values are between 1 ms and 255 ms in 1 ms steps.
The store function for the "latency_timer" device attribute currently
allows any value, although only the lower 8-bits will be
On 02/03/17 14:29, Johan Hovold wrote:
On Tue, Feb 28, 2017 at 12:51:26PM +, Ian Abbott wrote:
The "event_char" device attribute value, when written, is interpreted as
an enable bit in bit 8, and an "event character" in bits 7 to 0. Return
an error for out-of-range values.
Signed-off-by: I
On 02/03/17 14:37, Johan Hovold wrote:
On Tue, Feb 28, 2017 at 12:51:28PM +, Ian Abbott wrote:
The 'store' function for the "event_char" device attribute currently
expects a base 10 value. The value is composed of an enable bit in bit
8 and an 8-bit "event character" code in bits 7 to 0. I
On Wed, Mar 08, 2017 at 01:59:50PM +0200, Felipe Balbi wrote:
>
> Hi,
>
> (no top-posting, please)
>
> Ganesh Biradar writes:
> > Hi Felipe,
> >
> > I'm using TI latest vision sdk which has support for 4.4 kernel. i'm
> > not getting much help from TI e2e that's why i am asking here. can't
I d
> On 03/06/2017 09:21 AM, Elena Reshetova wrote:
> > refcount_t type and corresponding API should be
> > used instead of atomic_t when the variable is used as
> > a reference counter. This allows to avoid accidental
> > refcounter overflows that might lead to use-after-free
> > situations.
> >
> >
> On 03/06/2017 03:21 PM, Elena Reshetova wrote:
> > refcount_t type and corresponding API should be
> > used instead of atomic_t when the variable is used as
> > a reference counter. This allows to avoid accidental
> > refcounter overflows that might lead to use-after-free
> > situations.
>
> The
It looks like a bunch of devices do not like to be polled
for their reports at init time. When you look into the details,
it seems that for those that are requiring the quirk
HID_QUIRK_NO_INIT_REPORTS, the driver fails to retrieve part
of the features/inputs while others (more generic) work.
IMO,
On Tue, Mar 07, 2017 at 11:30:54PM +0100, Mats Karrman wrote:
> > > So, both the DP controller and the USB PD phy are I2C devices, and now I
> > > have to make them both
> > > attach to the AM bus as well?
> > The DP controller would provide the driver and the USB PD phy
> > (actually, the typec c
On 02/03/17 14:17, Johan Hovold wrote:
On Tue, Feb 28, 2017 at 12:51:24PM +, Ian Abbott wrote:
If a BM type chip has iSerialNumber set to 0 in its EEPROM, an incorrect
value is read from the bcdDevice field of the USB descriptor, making it
look like an AM type chip. Attempt to correct this
On Wed, Mar 08, 2017 at 01:46:12PM +, Ian Abbott wrote:
> On 02/03/17 14:29, Johan Hovold wrote:
> > On Tue, Feb 28, 2017 at 12:51:26PM +, Ian Abbott wrote:
> >> The "event_char" device attribute value, when written, is interpreted as
> >> an enable bit in bit 8, and an "event character" in
The streaming_maxburst module parameter is 0 offset (0..15)
so we must add 1 while using it for wBytesPerInterval
calculation for the SuperSpeed companion descriptor.
Without this host uvcvideo driver will always see the wrong
wBytesPerInterval for SuperSpeed uvc gadget and may not find
a suitable
On 03/07/2017 10:50 PM, Peter Chen wrote:
You mean type-C trigger an ACPI event, and this ACPI event can notify
related USB controller driver doing role switch?
No (firmware programs the dual-role hw/registers), but never mind.
That could be the case.
If it is correct, there is a notifier b
On 03/08/2017 02:48 PM, Reshetova, Elena wrote:
On 03/06/2017 03:21 PM, Elena Reshetova wrote:
refcount_t type and corresponding API should be
used instead of atomic_t when the variable is used as
a reference counter. This allows to avoid accidental
refcounter overflows that might lead to use-af
On Wed, Mar 08, 2017 at 01:58:51PM +, Ian Abbott wrote:
> On 02/03/17 14:37, Johan Hovold wrote:
> > On Tue, Feb 28, 2017 at 12:51:28PM +, Ian Abbott wrote:
> >> The 'store' function for the "event_char" device attribute currently
> >> expects a base 10 value. The value is composed of an e
Hi Laurent & Felipe,
These are some fixes for SuperSpeed case.
--
cheers,
-roger
Roger Quadros (2):
usb: gadget: f_uvc: Fix SuperSpeed companion descriptor's
wBytesPerInterval
usb: gadget: f_uvc: Sanity check wMaxPacketSize for SuperSpeed
drivers/usb/gadget/function/f_uvc.c | 10 ++
Hi Roger,
Thank you for the patches.
On Wednesday 08 Mar 2017 16:05:42 Roger Quadros wrote:
> Hi Laurent & Felipe,
>
> These are some fixes for SuperSpeed case.
For both patches,
Reviewed-by: Laurent Pinchart
> --
> cheers,
> -roger
>
> Roger Quadros (2):
> usb: gadget: f_uvc: Fix SuperSp
As per USB3.0 Specification "Table 9-20. Standard Endpoint Descriptor",
for interrupt and isochronous endpoints, wMaxPacketSize must be set to
1024 if the endpoint defines bMaxBurst to be greater than zero.
Signed-off-by: Roger Quadros
---
drivers/usb/gadget/function/f_uvc.c | 8
1 file
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API.
Signed-off-by: Romain Perier
Acked-by: Peter Senna Tschudin
Tested-by: Peter Senna Tschudin
---
drivers/usb/gadget/udc/amd5536udc.c | 8
drivers/usb/gadget/udc/
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API.
Signed-off-by: Romain Perier
Reviewed-by: Peter Senna Tschudin
Acked-by: Felipe Balbi
---
drivers/usb/gadget/udc/pch_udc.c | 31 +++
1 file c
Now that all the drivers use dma pool API, we can remove the macro
functions for PCI pool.
Signed-off-by: Romain Perier
Reviewed-by: Peter Senna Tschudin
---
include/linux/pci.h | 9 -
1 file changed, 9 deletions(-)
diff --git a/include/linux/pci.h b/include/linux/pci.h
index eb3da1a..
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API.
Signed-off-by: Romain Perier
Acked-by: Peter Senna Tschudin
Tested-by: Peter Senna Tschudin
---
drivers/usb/gadget/udc/net2280.c | 12 ++--
drivers/usb/gadget/ud
This replaces remaining occurences of pci_pool by dma_pool, as
this is the new API that could be used for that purpose.
Signed-off-by: Romain Perier
Reviewed-by: Peter Senna Tschudin
---
drivers/usb/host/ehci-hcd.c | 2 +-
drivers/usb/host/fotg210-hcd.c | 2 +-
drivers/usb/host/oxu210hp-hc
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API.
Signed-off-by: Romain Perier
Acked-by: Peter Senna Tschudin
Tested-by: Peter Senna Tschudin
---
drivers/scsi/pmcraid.c | 10 +-
drivers/scsi/pmcraid.h | 2 +-
2
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API.
Signed-off-by: Romain Perier
Reviewed-by: Peter Senna Tschudin
Acked-by: Sumit Saxena
---
drivers/scsi/megaraid/megaraid_mbox.c | 33 +++
drivers/scsi/
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API.
Signed-off-by: Romain Perier
Reviewed-by: Peter Senna Tschudin
---
drivers/scsi/mpt3sas/mpt3sas_base.c | 73 +
1 file changed, 34 inse
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API.
Signed-off-by: Romain Perier
Reviewed-by: Peter Senna Tschudin
---
drivers/scsi/mvsas/mv_init.c | 6 +++---
drivers/scsi/mvsas/mv_sas.c | 6 +++---
2 files changed, 6 in
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API.
Signed-off-by: Romain Perier
Acked-by: Peter Senna Tschudin
Tested-by: Peter Senna Tschudin
---
drivers/scsi/be2iscsi/be_iscsi.c | 6 +++---
drivers/scsi/be2iscsi/be_mai
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API. It also updates
some comments, accordingly.
Signed-off-by: Romain Perier
Reviewed-by: Peter Senna Tschudin
---
drivers/scsi/lpfc/lpfc.h | 14 +++---
drivers/scsi/l
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API. It also updates
the name of some variables and the content of comments, accordingly.
Signed-off-by: Romain Perier
Reviewed-by: Peter Senna Tschudin
---
drivers/scsi/csios
Hi,
I'm not subscribed to linux-usb mailing list so I found your post just by a
pure luck. I've sent the patch for:
streaming_maxburst streaming_maxpacket streaming_interval
problem earlier this week. Try this patch [1]:
BTW please CC me if you will have another discusion in your threa
If a BM type chip has iSerialNumber set to 0 in its EEPROM, an incorrect
value is read from the bcdDevice field of the USB descriptor, making it
look like an AM type chip. Attempt to correct this in
ftdi_determine_type() by attempting to read the latency timer for an AM
type chip if it has iSerial
The latency timer was introduced with the FT232BM and FT245BM chips. Do
not bother attempting to read or write it for older chip versions.
Signed-off-by: Ian Abbott
---
v2:
- No change.
---
drivers/usb/serial/ftdi_sio.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/usb/seria
The 'store' function for the "event_char" device attribute currently
expects a base 10 value. The value is composed of an enable bit in bit
8 and an 8-bit "event character" code in bits 7 to 0. It seems
reasonable to allow hexadecimal and octal numbers to be written to the
device attribute in add
Valid latency timer values are between 1 ms and 255 ms in 1 ms steps.
The store function for the "latency_timer" device attribute currently
allows any value, although only the lower 16 bits will be sent to the
device, and the device only stores the lower 8 bits. The hardware
appears to accept the
The "event_char" device attribute value, when written, is interpreted as
an enable bit in bit 8, and an "event character" in bits 7 to 0.
Return an error -EINVAL for out-of-range values. Use kstrtouint() to
parse the integer instead of the obsolete simple_strtoul().
Signed-off-by: Ian Abbott
--
Some patches to skip accessing the FTDI latency timer on chips that
don't support it, detect "BM" chips with iSerialNumber bug, validate
written device attribute values, and allow hex and octal "event_char"
values.
This is v2 of the patch series, incorporating changes suggested by
Johan. The orig
From: jinghua
- Add a new compatoble string for the Armada 3700 SoCs
- add sbuscfg support for orion usb controller driver. For the SoCs
without hlock, need to program BAWR/BARD/AHBBRST fields in the sbuscfg
register to guarantee the AHB master's burst would not overrun or
underrun the FIF
The mvebu ARM64 SoCs no more select PLAT_ORION but some of them as the
Armada 37xx use the EHCI orion controller. This patch allow to build
the driver when ARCH_MVEBU is selected.
Signed-off-by: Gregory CLEMENT
---
drivers/usb/host/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
d
Armada 37xx SoC embedded an EHCI controller. This patch adds the device
tree node enabling its support.
Signed-off-by: Gregory CLEMENT
---
arch/arm64/boot/dts/marvell/armada-3720-db.dts | 6 ++
arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 7 +++
2 files changed, 13 insertions(+)
dif
Hi,
The EHCI controller in the Armada 37xx SoCs is the one used on many
other mvebu SoCs such as the orion5x, the kirkwood, or the
armada. However, for Armada 37xx an extra initialization step is
needed: this is the purpose of the first patch.
The second patch allows to build the driver for the A
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API.
Signed-off-by: Romain Perier
Reviewed-by: Peter Senna Tschudin
---
drivers/net/wireless/intel/ipw2x00/ipw2200.c | 13 +++--
1 file changed, 7 insertions(+), 6 del
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API.
Signed-off-by: Romain Perier
Acked-by: Peter Senna Tschudin
Tested-by: Peter Senna Tschudin
---
drivers/net/ethernet/mellanox/mlx4/cmd.c | 10 +-
drivers/net/et
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API.
Signed-off-by: Romain Perier
Reviewed-by: Peter Senna Tschudin
---
drivers/net/ethernet/mellanox/mlx5/core/cmd.c | 11 ++-
include/linux/mlx5/driver.h
by the dma pool API
and remove the defines.
Changes in v5:
- Re-worded the cover letter (remove sentence about checkpatch.pl)
- Rebased series onto next-20170308
- Fix typos in commit message
- Added Acked-by Tags
Changes in v4:
- Rebased series onto next-20170301
- Removed patch 20/20: checks done
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API.
Signed-off-by: Romain Perier
Acked-by: Peter Senna Tschudin
Tested-by: Peter Senna Tschudin
---
drivers/infiniband/hw/mthca/mthca_av.c | 10 +-
drivers/infiniba
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API.
Signed-off-by: Romain Perier
Acked-by: Peter Senna Tschudin
Tested-by: Peter Senna Tschudin
---
drivers/block/DAC960.c | 36 ++--
drivers
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API.
Signed-off-by: Romain Perier
Acked-by: Peter Senna Tschudin
Tested-by: Peter Senna Tschudin
---
drivers/dma/pch_dma.c | 12 ++--
1 file changed, 6 insertions(+),
The PCI pool API is deprecated. This commit replaces the PCI pool old
API by the appropriate function with the DMA pool API.
Signed-off-by: Romain Perier
Acked-by: Peter Senna Tschudin
Tested-by: Peter Senna Tschudin
---
drivers/net/ethernet/intel/e100.c | 12 ++--
1 file changed, 6 in
Dear all,
Just submitting a very minor patch to update my e-mail address in this file - I
sent
a patch years ago to update the listed firmware version to make newer versions
of the
Nokia 6288 mountable as mass storage devices. I hope my patch is in the right
format,
I've tried to pay attention
Hi Andrew,
On mer., mars 08 2017, Andrew Lunn wrote:
> Hi Gregory
[...]
Thanks for your comments I will fix the typos and the wording.
>> +#define USB_SBUSCFG 0x90
>> +#define USB_SBUSCFG_BAWR0x6
>> +#define USB_SBUSCFG_BARD0x3
>> +#define US
Hi Andrew,
On mer., mars 08 2017, Andrew Lunn wrote:
> On Wed, Mar 08, 2017 at 05:24:22PM +0100, Gregory CLEMENT wrote:
>> The mvebu ARM64 SoCs no more select PLAT_ORION but some of them as the
>> Armada 37xx use the EHCI orion controller. This patch allow to build
>> the driver when ARCH_MVEB
* Bin Liu [170308 08:26]:
> The DSPS glue calls del_timer_sync() in its musb_platform_disable()
> implementation, which requires the caller to not hold a lock. But
> musb_remove() calls musb_platform_disable() will musb->lock held. This
> could causes spinlock deadlock.
>
> So change musb_remove(
Hi Gregory
> - Add a new compatoble string for the Armada 3700 SoCs
compatible
>
> - add sbuscfg support for orion usb controller driver. For the SoCs
> without hlock, need to program BAWR/BARD/AHBBRST fields in the sbuscfg
> register to guarantee the AHB master's burst would not overrun or
On Wed, Mar 08, 2017 at 05:24:22PM +0100, Gregory CLEMENT wrote:
> The mvebu ARM64 SoCs no more select PLAT_ORION but some of them as the
> Armada 37xx use the EHCI orion controller. This patch allow to build
> the driver when ARCH_MVEBU is selected.
The mvebu ARM64 SoCs no longer selects PLAT_ORI
On 03/08/2017 08:49 AM, Reshetova, Elena wrote:
>> On 03/06/2017 09:21 AM, Elena Reshetova wrote:
>>> refcount_t type and corresponding API should be
>>> used instead of atomic_t when the variable is used as
>>> a reference counter. This allows to avoid accidental
>>> refcounter overflows that migh
On Tue, Mar 07, 2017 at 04:35:37AM +, Ajay Kaher wrote:
>
>
>
> > On Fri, 3 Mar 2017, Ajay Kaher wrote:
> >
> > > > usb_class->kref is not accessible outside the file.c
> > > > as usb_class is _static_ inside the file.c and
> > > > pointer of usb_class->kref is not passed anywhere.
> > >
Cleanly iounmap the pointer in error and exit paths.
Signed-off-by: Bin Liu
---
drivers/usb/musb/musb_dsps.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/usb/musb/musb_dsps.c b/drivers/usb/musb/musb_dsps.c
index 7c047c4a2565..c84f32d0dd4d 100644
--- a/drivers/u
On Mon, Mar 06, 2017 at 04:21:09PM +0200, Elena Reshetova wrote:
> refcount_t type and corresponding API should be
> used instead of atomic_t when the variable is used as
> a reference counter. This allows to avoid accidental
> refcounter overflows that might lead to use-after-free
> situations.
>
It hangs on resume from suspend if I have USB 3.0 enabled on the BIOS,
it works fine with ehci_hcd or USB 2.0.
The way I reproduce the problem is with this command:
$ i3lock && systemctl suspend
This is what I see on the screen when it hangs:
https://dl.dropboxusercontent.com/u/6005119/dell/IMG
The DSPS glue calls del_timer_sync() in its musb_platform_disable()
implementation, which requires the caller to not hold a lock. But
musb_remove() calls musb_platform_disable() will musb->lock held. This
could causes spinlock deadlock.
So change musb_remove() to call musb_platform_disable() witho
On Wed, Mar 08, 2017 at 10:24:31AM -0600, Bin Liu wrote:
> Cleanly iounmap the pointer in error and exit paths.
>
> Signed-off-by: Bin Liu
> ---
> drivers/usb/musb/musb_dsps.c | 5 -
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/usb/musb/musb_dsps.c b/drivers/usb
On Wed, Mar 08, 2017 at 08:02:59PM +0100, Johan Hovold wrote:
> On Wed, Mar 08, 2017 at 10:24:31AM -0600, Bin Liu wrote:
> > Cleanly iounmap the pointer in error and exit paths.
> >
> > Signed-off-by: Bin Liu
> > ---
> > drivers/usb/musb/musb_dsps.c | 5 -
> > 1 file changed, 4 insertions(+)
If usb_get_bos_descriptor() returns an error, usb->bos will be NULL.
Nevertheless, it is dereferenced unconditionally in
hub_set_initial_usb2_lpm_policy() if usb2_hw_lpm_capable is set.
This results in a crash.
usb 5-1: unable to get BOS descriptor
...
Unable to handle kernel NULL pointer derefere
Hi Bin,
Just wanted to clarify,
I have created already a thread(more then 2 weeks ago) which was based
on dual configuration support(mass storage and uvc device(g_webcam)). in
which i asked question based on 3.14 kernel but confgifs support for uvc
was added in 4.0 kernel. so i updated to lat
On Wed, Mar 08, 2017 at 11:24:16PM +0530, Ganesh Biradar wrote:
> Hi Bin,
>
> Just wanted to clarify,
>
> I have created already a thread(more then 2 weeks ago) which was
> based on dual configuration support(mass storage and uvc
> device(g_webcam)). in which i asked question based on 3.14 kernel
Hi Felipe,
I'm using TI latest vision sdk which has support for 4.4 kernel. i'm
not getting much help from TI e2e that's why i am asking here. can't
try v4.10 or any other kernel as it has dependencies with respect to
SDK.
regards,
grbesd1
On Wed, Mar 8, 2017 at 2:54 PM, Felipe Balbi
wrote:
>
>
On Wed, 8 Mar 2017, Phillip Potter wrote:
> Dear all,
>
> Just submitting a very minor patch to update my e-mail address in this file -
> I sent
> a patch years ago to update the listed firmware version to make newer
> versions of the
> Nokia 6288 mountable as mass storage devices. I hope my pa
On Wed, Mar 8, 2017 at 3:49 PM, Diego Viola wrote:
> It hangs on resume from suspend if I have USB 3.0 enabled on the BIOS,
> it works fine with ehci_hcd or USB 2.0.
>
> The way I reproduce the problem is with this command:
>
> $ i3lock && systemctl suspend
>
> This is what I see on the screen whe
Hi Greg,
On Wed, Mar 8, 2017 at 5:15 PM, Greg KH wrote:
> On Wed, Mar 08, 2017 at 03:49:19PM -0300, Diego Viola wrote:
>> It hangs on resume from suspend if I have USB 3.0 enabled on the BIOS,
>> it works fine with ehci_hcd or USB 2.0.
>>
>> The way I reproduce the problem is with this command:
>
On Wed, Mar 08, 2017 at 03:49:19PM -0300, Diego Viola wrote:
> It hangs on resume from suspend if I have USB 3.0 enabled on the BIOS,
> it works fine with ehci_hcd or USB 2.0.
>
> The way I reproduce the problem is with this command:
>
> $ i3lock && systemctl suspend
>
> This is what I see on th
Hello,
On Wed, 8 Mar 2017 17:24:21 +0100, Gregory CLEMENT wrote:
> Signed-off-by: jinghua
I think you need a full first name + last name for this Signed-off-by.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
--
T
Hi guys,
I’w working with a Quectel EC21 modem and i ran into an issue that the qcserial
driver is not getting installed for it.
Previously i was working with the Quectel EC20 which was working properly but
the cell carrier requested LTE Cat 1 modem so that’s why the switch to EC21.
Can you ad
Dear Alan,
Apologies. I will make the necessary changes and resubmit.
Regards,
Phil Potter
On 08/03/2017 20:04, Alan Stern wrote:
On Wed, 8 Mar 2017, Phillip Potter wrote:
Dear all,
Just submitting a very minor patch to update my e-mail address in this file - I
sent
a patch years ago to up
On Wed, 2017-03-08 at 17:19 +0100, Romain Perier wrote:
> The PCI pool API is deprecated. This commit replaces the PCI pool old
> API by the appropriate function with the DMA pool API.
>
> Signed-off-by: Romain Perier
> Acked-by: Peter Senna Tschudin
> Tested-by: Peter Senna Tschudin
> ---
> d
Updates the e-mail address of Phillip Potter, updater of the Nokia 6288 entry in
drivers/usb/storage/unusual_devs.h
Signed-off-by: Phillip Potter
---
--- a/drivers/usb/storage/unusual_devs.h2017-03-08 21:47:45.878007300
+
+++ b/drivers/usb/storage/unusual_devs.h2017-03-08 22
Hi,
> -Original Message-
> From: Baolin Wang [mailto:baolin.w...@linaro.org]
> Sent: Tuesday, March 07, 2017 5:39 PM
> To: NeilBrown
> Cc: Felipe Balbi ; Greg KH ;
> Sebastian Reichel ; Dmitry Eremin-Solenikov
> ; David Woodhouse ;
> r...@kernel.org; Jun Li ; Marek Szyprowski
> ; Ruslan B
On Wed, Mar 08, 2017 at 06:44:47AM -0800, Guenter Roeck wrote:
> On 03/07/2017 10:50 PM, Peter Chen wrote:
> >
> You mean type-C trigger an ACPI event, and this ACPI event can notify
> related USB controller driver doing role switch?
> >>>
> >>>No (firmware programs the dual-role hw/registe
While stress testing a usb controller using a bind/unbind looop, the
following error loop was observed.
usb 7-1.2: new low-speed USB device number 3 using xhci-hcd
usb 7-1.2: hub failed to enable device, error -108
usb 7-1-port2: cannot disable (err = -22)
usb 7-1-port2: couldn't allocate usb_devi
Dne 8.3.2017 v 11:46 Ganesh Biradar napsal(a):
> Hi Petr,
>
> I saw your patch and at the same time I saw Laurent suggestion.
>
> I have tried both patches both are working but I'm going with Laurent
> suggestion
>
> UVC_ATTR(f_uvc_opts_, cname, cname).
>
That's OK, they are equivalent (as I
On Mon, Feb 20, 2017 at 03:38:42PM +0100, Oliver Neukum wrote:
> There is a small window during which the an URB may
> remain active after disconnect has returned. If in that case
> already freed memory may be accessed and executed.
>
> The fix is to poison the URB befotre the work is flushed.
>
Hi,
On 9 March 2017 at 09:50, Jun Li wrote:
> Hi,
>
>> -Original Message-
>> From: Baolin Wang [mailto:baolin.w...@linaro.org]
>> Sent: Tuesday, March 07, 2017 5:39 PM
>> To: NeilBrown
>> Cc: Felipe Balbi ; Greg KH ;
>> Sebastian Reichel ; Dmitry Eremin-Solenikov
>> ; David Woodhouse ;
>
Upstream commit 98d74f9ceaef ("xhci: fix 10 second timeout on removal of
PCI hotpluggable xhci controllers") fixes a problem with hot pluggable PCI
xhci controllers which can result in excessive timeouts, to the point where
the system reports a deadlock.
The same problem is seen with hot pluggable
1 - 100 of 104 matches
Mail list logo