Peter Wu [mailto:pe...@lekensteyn.nl]
> Sent: Tuesday, December 08, 2015 10:33 PM
[...]
> I found another problem with runtime PM. When a device is suspended via
> autosuspend and a system suspend takes place, there is no network I/O
> after resume. Triggering a renegotiation (ethtool -r eth1) bri
On Tue, 2015-12-22 at 09:48 +, Hayes Wang wrote:
> Peter Wu [mailto:pe...@lekensteyn.nl]
> > Sent: Tuesday, December 08, 2015 10:33 PM
> [...]
> > I found another problem with runtime PM. When a device is suspended via
> > autosuspend and a system suspend takes place, there is no network I/O
>
Hello.
On 12/22/2015 4:13 AM, Chanwoo Choi wrote:
This patch depend on GPIOLIB configuration as following:
I modified it with following diff and applied it.
diff --git a/drivers/extcon/Kconfig b/drivers/extcon/Kconfig
index ba4db7d..3d89e60 100644
--- a/drivers/extcon/Kconfig
+++ b/drivers/ext
On 21.12.2015 19:39, Andor J Kiss wrote:
Hi Mathias,
Yes, native support for the graphics is in Kernel 4.3, not 4.2
(Intel South Lake HD520 chipset). I gather there's been a bit of
shoehorning to get 4.2 to work with this board. 4.3 should be in
Ubuntu 16.04 (next upgrade - few months)
Hi Mathias,
Ok, good to know. How do I add to the quirk list?
I contacted SP, they dn't understand how Linux works (told me to
update my drivers ). Anyway, maybe I'll look into the USB
analyzer. Will a virtual one work, or does one need a hardware device?
Regards,
Andor
On Tue, 2015
On 22.12.2015 14:16, Andor J Kiss wrote:
Hi Mathias,
Ok, good to know. How do I add to the quirk list?
Once we got code to workaround the issue we can name a quirk for it.
Then in drivers/usb/core/quirks.c we set the quirk based on device/vendor ID of
the device.
But we are not there yet
On Mon, Dec 14, 2015 at 11:17 PM, Ramajayam S
wrote:
> On Mon, Dec 14, 2015 at 9:50 PM, Felipe Balbi wrote:
>>
>> hi,
>>
>> Ramajayam S writes:
>>> Hi
>>>We are trying to enable g_serial in Intel platform and facing
>>> issue with RX path.
>>>
>>> stable kernel version: K3.14
>>
>> you need
On Tue, 22 Dec 2015, Oliver Neukum wrote:
> On Tue, 2015-12-22 at 01:12 +0100, Steinar H. Gunderson wrote:
> > On Tue, Dec 15, 2015 at 03:32:45PM +, David Laight wrote:
> > > That still isn't entirely correct.
> > >
> > > Someone with more knowledge than either of us has needs to sort out
> >
From: Markus Elfring
Date: Tue, 22 Dec 2015 16:43:12 +0100
Another update suggestion was taken into account after a patch was applied
from static source code analysis.
Markus Elfring (2):
Use a signed return type for fhci_create_ep()
Use a signed return type for endpoint_zero_init()
driver
On Tue, 22 Dec 2015, Peter Chen wrote:
> > I don't really understand this. However, you can always specify a USB
> > device by giving its port number on the parent hub, and the hub's port
> > number on _its_ parent hub, and so on back to the root hub and host
> > controller. That works even if y
On a custom mx28 board that has a USB hub we see the following
error:
hub 1-1:1.0: USB hub found
hub 1-1:1.0: 2 ports detected
usb 1-1: USB disconnect, device number 2
usb usb1-port1: cannot reset (err = -32)
usb usb1-port1: cannot reset (err = -32)
usb usb1-port1: cannot reset (err = -32)
usb usb
From: Markus Elfring
Date: Tue, 22 Dec 2015 16:10:14 +0100
The return type "u32" was used by the fhci_create_ep() function even though
it will eventually return a negative error code.
Improve this implementation detail by using the type "int" instead.
This issue was detected by using the Coccine
From: Markus Elfring
Date: Tue, 22 Dec 2015 16:24:46 +0100
The return type "u32" was used by the endpoint_zero_init() function
even though it can return a value which corresponds to a negative
error code from a call of the fhci_create_ep() function.
Improve this implementation detail by using the
This refactor includes the following:
* Cleaner state machine code;
* Reset state if MIDI message parsed is non-conformant;
* Fixed bug when a conformant MIDI message was followed by a non-conformant
causing the MIDI-USB message to use old temporary data (port->data[0..1]),
thus packing a
Since f_midi_transmit is called by both ALSA and USB frameworks, it can
potentially cause a race condition between both calls. This is bad because the
way f_midi_transmit is implemented can't handle concurrent calls. This is due
to the fact that the usb request fifo looks for the next element and o
Hi Dan,
On 21/12/15 13:20, Dan Carpenter wrote:
> We added a new error path to this function and we forgot to drop the
> lock.
>
> Fixes: e1e3d7ec5da3 ('usb: gadget: f_midi: pre-allocate IN requests')
> Signed-off-by: Dan Carpenter
>
> diff --git a/drivers/usb/gadget/function/f_midi.c
> b/driv
Felipe F. Tonello wrote:
> This refactor includes the following:
> * Cleaner state machine code;
It does not correctly handle system real time messages inserted between
the status and data bytes of other messages.
> * Reset state if MIDI message parsed is non-conformant;
Why? In a byte stream
Hi,
John Youn writes:
> On 12/3/2015 7:18 AM, Felipe Balbi wrote:
>> So far, dwc3 has always missed request->zero
>> handling for every endpoint. Let's implement
>> that so we can handle cases where transfer must
>> be finished with a ZLP.
>>
>> Note that dwc3 is a little special. Even though
>
Hi,
"Felipe F. Tonello" writes:
> This refactor includes the following:
> * Cleaner state machine code;
> * Reset state if MIDI message parsed is non-conformant;
> * Fixed bug when a conformant MIDI message was followed by a non-conformant
>causing the MIDI-USB message to use old temporar
Uwe Kleine-König writes:
> This is needed to let the omap3 otg controller make use of a generic
> usb-nop-xceiv phy.
missing SoB
--
balbi
signature.asc
Description: PGP signature
Hi,
Geliang Tang writes:
> Use list_for_each_entry_safe() instead of list_for_each_safe() to
> simplify the code.
>
> Signed-off-by: Geliang Tang
there are other cleanups in this patch which shouldn't be here. Please
split and I'll apply for v4.6 merge window.
--
balbi
signature.asc
Descri
Julia Lawall writes:
> The index variable of list_for_each_entry_safe is never NULL.
>
> Generated by: scripts/coccinelle/iterators/itnull.cocci
>
> CC: Geliang Tang
> Signed-off-by: Fengguang Wu
> Signed-off-by: Julia Lawall
doesn't apply. Does this depend on anything ?
--
balbi
signatur
On Tue, 22 Dec 2015, Felipe Balbi wrote:
> Julia Lawall writes:
>
> > The index variable of list_for_each_entry_safe is never NULL.
> >
> > Generated by: scripts/coccinelle/iterators/itnull.cocci
> >
> > CC: Geliang Tang
> > Signed-off-by: Fengguang Wu
> > Signed-off-by: Julia Lawall
>
> doe
Michael Reutman reports that an AMD/ATI EHCI host controller on one of
his computers does not stop transferring data when an active bulk QH
is unlinked from the async schedule. Apparently that host controller
fails to implement the IAA mechanism correctly when an active QH is
unlinked. This leads
This patch replaces the "exception" bitflag in the ehci_qh structure
with a more explicit "unlink_reason" bitmask. This is for use in the
following patch, where we will need to have a good idea of the
reason for unlinking a QH, not just "something exceptional happened".
Signed-off-by: Alan Stern
On Sun, Dec 20, 2015 at 6:07 AM, Kishon Vijay Abraham I wrote:
> Hi Rob,
>
> On Sunday 20 December 2015 09:09 AM, Rob Herring wrote:
>> On Tue, Dec 15, 2015 at 02:46:08PM +0530, Kishon Vijay Abraham I wrote:
>>> Deprecate using phy-omap-control driver to power on/off the PHY,
>>> and use *syscon*
Hi,
Ramneek Mehresh writes:
>> -Original Message-
>> From: Felipe Balbi [mailto:ba...@ti.com]
>> Sent: Saturday, October 10, 2015 3:04 AM
>> To: Mehresh Ramneek-B31383 ; linux-
>> ker...@vger.kernel.org
>> Cc: st...@rowland.harvard.edu; gre...@linuxfoundation.org; linux-
>> u...@vger.ker
> We added a new error path to this function and we forgot to drop the lock.
>
> Fixes: e1e3d7ec5da3 ('usb: gadget: f_midi: pre-allocate IN requests')
> Signed-off-by: Dan Carpenter
>
> diff --git a/drivers/usb/gadget/function/f_midi.c
> b/drivers/usb/gadget/function/f_midi.c
> index fb1fe96d..
Hi,
Julia Lawall writes:
> On Tue, 22 Dec 2015, Felipe Balbi wrote:
>
>> Julia Lawall writes:
>>
>> > The index variable of list_for_each_entry_safe is never NULL.
>> >
>> > Generated by: scripts/coccinelle/iterators/itnull.cocci
>> >
>> > CC: Geliang Tang
>> > Signed-off-by: Fengguang Wu
>>
[ +CC: linux-usb ]
On Tue, Dec 22, 2015 at 10:51:32AM +, John Ernberg wrote:
> From: John Ernberg
>
> In certain kernel configurations where the cdc_ether and option drivers
> are compiled as modules there can occur a race condition in enumeration.
> This causes the option driver to enumerat
On Tue, Dec 22, 2015 at 12:06:14PM -0600, Felipe Balbi wrote:
> Uwe Kleine-König writes:
>
> > This is needed to let the omap3 otg controller make use of a generic
> > usb-nop-xceiv phy.
>
> missing SoB
If this is your only concern I will happily resend the series with an
SoB and some dt-bindin
Hi,
Uwe Kleine-König writes:
> On Tue, Dec 22, 2015 at 12:06:14PM -0600, Felipe Balbi wrote:
>> Uwe Kleine-König writes:
>>
>> > This is needed to let the omap3 otg controller make use of a generic
>> > usb-nop-xceiv phy.
>>
>> missing SoB
>
> If this is your only concern I will happily resen
On 12/22/2015 9:31 AM, Felipe Balbi wrote:
>
> Hi,
>
> John Youn writes:
>> On 12/3/2015 7:18 AM, Felipe Balbi wrote:
>>> So far, dwc3 has always missed request->zero
>>> handling for every endpoint. Let's implement
>>> that so we can handle cases where transfer must
>>> be finished with a ZLP.
On Tue, 22 Dec 2015, John Youn wrote:
> >> The following fixes it:
> >>
> >> - if (ret == 0 && request->zero && (request->length % ep->maxpacket
> >> == 0))
> >> + if (ret == 0 && request->zero && (request->length % ep->maxpacket
> >> == 0) &&
> >> + (request->length != 0))
If the request->length is zero, a ZLP should already be sent due to that
and another ZLP is not needed to terminate the transfer.
Signed-off-by: John Youn
---
drivers/usb/dwc3/gadget.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/d
Hi,
John Youn writes:
> On 12/22/2015 9:31 AM, Felipe Balbi wrote:
>>
>> Hi,
>>
>> John Youn writes:
>>> On 12/3/2015 7:18 AM, Felipe Balbi wrote:
So far, dwc3 has always missed request->zero
handling for every endpoint. Let's implement
that so we can handle cases where transfe
Now that usb_bus_list has been removed and switched to idr
rename the related mutex accordingly.
Signed-off-by: Heiner Kallweit
---
drivers/usb/core/devices.c | 6 +++---
drivers/usb/core/hcd.c | 30 +++---
drivers/usb/core/hub.c | 4 ++--
drivers/usb/mon/mon_ma
Use mutex usb_bus_idr_lock to protect idr_find.
Signed-off-by: Heiner Kallweit
---
drivers/usb/host/r8a66597-hcd.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/usb/host/r8a66597-hcd.c b/drivers/usb/host/r8a66597-hcd.c
index 1ef8873..bfa7fa3 100644
--- a/drivers/usb/host/r8a66597
USB bus numbering is based on directly dealing with bitmaps and
defines a separate list of busses.
This can be simplified and unified by using existing idr functionality.
Signed-off-by: Heiner Kallweit
---
drivers/usb/core/devices.c | 10 ++
drivers/usb/core/hcd.c | 21
Hi,
On a mx28 custom board with a USB hub I can get a USB stick to get
detected after applying this change:
http://marc.info/?l=linux-usb&m=145079935324339&w=2
Things work fine if I boot without the USB stick connected. Then I can
insert/remove the USB device and it always gets detected.
However
Hello,
This is just a note to let you know that v0.1.0 of libusbg-neXt[1] has
been just released.
In the beginning the library was developed as a github fork[2] of
libusbg[3] with many patches sent to Matt and pull requests to mianline
repository. Unfortunately libusbg project seems to be (a
On Mon, Dec 21, 2015 at 06:40:04PM +0900, Yoshihiro Shimoda wrote:
> R-Car H3 has USB3.0 peripheral controllers. This controller's has the
> following features:
> - Supports super, high and full speed
> - Contains 30 pipes for bulk or interrupt transfer
> - Contains dedicated DMA controller
>
>
cp210x_get_config and cp210x_set_config are cumbersome to use. This change
switches large register access to use new block functions. The old
functions are removed because now they become unused.
Signed-off-by: Konstantin Shkolnyy
---
drivers/usb/serial/cp210x.c | 137 ---
cp210x_get_config and cp210x_set_config are cumbersome to use. This change
introduces new register access functions for 8 and 32-bit values, instead
of the above functions.
Signed-off-by: Konstantin Shkolnyy
---
drivers/usb/serial/cp210x.c | 92 ++---
1 fi
cp210x_get_config and cp210x_set_config are cumbersome to use. This change
introduces new register access functions for 16-bit values, instead of
the above functions.
Signed-off-by: Konstantin Shkolnyy
---
drivers/usb/serial/cp210x.c | 155 +++-
1 file cha
On 2015년 12월 22일 20:15, Sergei Shtylyov wrote:
> Hello.
>
> On 12/22/2015 4:13 AM, Chanwoo Choi wrote:
>
>> This patch depend on GPIOLIB configuration as following:
>> I modified it with following diff and applied it.
>>
>> diff --git a/drivers/extcon/Kconfig b/drivers/extcon/Kcon
Oliver Neukum [mailto:oneu...@suse.com]
[...]
> It is clear to me that you cannot get away with using the same operation
> for resume() and reset_resume() in your driver. It is fundamentally
> impossible. Firmware cannot fix it.
I would think how to fix it.
> Sorry for the length of the explanati
if we have a !PM kernel build, our runtime
suspend/resume callbacks will be left defined but
unused. Add a ifdef CONFIG_PM guard.
Signed-off-by: Felipe Balbi
---
drivers/usb/dwc3/dwc3-of-simple.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/usb/dwc3/dwc3-of-simple.c
b/drivers/u
Removed an unnecessary space between a function name and arguments.
Signed-off-by: Chase Metzger
---
drivers/usb/core/devio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/core/devio.c b/drivers/usb/core/devio.c
index 38ae877c..0bcd45e 100644
--- a/drivers/usb/c
Hi
> -Original Message-
> From: linux-usb-ow...@vger.kernel.org [mailto:linux-usb-
> ow...@vger.kernel.org] On Behalf Of Felipe Balbi
> Sent: Wednesday, December 23, 2015 2:21 AM
> To: Ramneek Mehresh ; linux-
> ker...@vger.kernel.org
> Cc: st...@rowland.harvard.edu; gre...@linuxfoundation
50 matches
Mail list logo
