Hi,
Sure - sorry, my description was a little .. basic.
So, I have a client who was having problems with machines hanging in
the field. Very rare, associated with a h/w change that introduced
more cores. Kernel dumps implied that the timer list was getting
corrupted.
This configuration of
On Thu, Jul 23, 2015 at 12:08:01PM -0700, Greg KH wrote:
Somewhere that people can refer to that describes this public-facing API
that must not ever be broken or changed. If you want to put it in a
documentation file, or a .h file, I don't care.
It is used e.g. in SUSE's xen kernel since
On Thu, Jul 23, 2015 at 03:08:08PM -0500, Felipe Balbi wrote:
I don't like fixes like this because no one now has any pressure to fix
it properly. Are you doing that work? If not, who is?
Heikki is author, I'd expect him to fix it up. We can also revert the
fix if you prefer, I'm totally
On 07/23/2015 03:00 AM, Peter Chen wrote:
Thanks, it is correct. But looking the code at afunc_set_alt:
the method of calculating uac2-p_pktsize seems incorrect, it
may need to change like below:
Ok, sorry. I just read the code again an figured you're right here, this
needs fixing. It doesn't
Please tell your email client to wrap lines of text after 72 columns or
so.
On Thu, 23 Jul 2015, Vasudevan, Krishna PrasathX K wrote:
Hi,
This mail is for RFC regarding the persist resume of some USB 2.0 devices,
Problem Summary: Problem has been observed for some USB 2.0 devices
while
23.07.2015 12:43, Oliver Neukum пишет:
On Mon, 2015-07-20 at 21:13 +0300, Eugene Shatokhin wrote:
[Race #5]
Race on dev-rx_urb_size. I reproduced it a similar way as the races
#2
and #3 (changing MTU while downloading files).
dev-rx_urb_size is written to here:
#0 usbnet_change_mtu
On Wed, Jul 15, 2015 at 01:35:58PM +0200, Michał Pecio wrote:
This commit fixes the following issues:
1. The 9th bit of buf was believed to be the LSB of divisor's
exponent, but the hardware interprets it as MSB (9th bit) of the
mantissa. The exponent is actually one bit shorter and
Dear Greg
Sorry for your inconvience.
This patch doesn't apply to my tree, can you please refresh the patch
agasint 4.2-rc3 and resend it so that I can apply it?
I will resend a new patch against 4.2-rc3 version.
Also, please try to fix your email client to not send base64 patches,
that made
From 0a8e0d63a9887735c6782d7b0c15c2c1fdf1952a Mon Sep 17 00:00:00 2001
From: Du, Changbin changbin...@intel.com
Date: Thu, 23 Jul 2015 20:08:04 +0800
Subject: [PATCH] usb/gadget: make composite gadget meet usb compliance for
vbus draw
USB-IF compliance requirement limits the vbus current
@@ -1176,15 +1188,16 @@ afunc_set_alt(struct usb_function *fn, unsigned
intf, unsigned alt)
factor = 1000;
} else {
ep_desc = hs_epin_desc;
- factor = 125;
+ factor = 8000;
Thanks, Pietrasiewicz.
From: Andrzej Pietrasiewicz [mailto:andrze...@samsung.com]
W dniu 23.07.2015 o 14:34, Du, Changbin pisze:
From 0a8e0d63a9887735c6782d7b0c15c2c1fdf1952a Mon Sep 17 00:00:00
void composite_disconnect(struct usb_gadget *gadget)
{
struct usb_composite_dev
Avoid the driver to enable WOL if the device doesn't support it.
Signed-off-by: Hayes Wang hayesw...@realtek.com
---
drivers/net/usb/r8152.c | 28 ++--
1 file changed, 22 insertions(+), 6 deletions(-)
diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c
index
v2:
Replace patch #2 with r8152: fix wakeup settings.
v1:
These patches are used to fix issues.
Hayes Wang (3):
r8152: fix the issue about U1/U2
- r8152: fix remote wakeup
+ r8152: fix wakeup settings
r8152: don't enable napi before rx ready
drivers/net/usb/r8152.c | 103
- Disable U1/U2 during initialization.
- Disable lpm when linking is on, and enable it when linking is off.
- Disable U1/U2 when enabling runtime suspend.
It is possible to let hw stop working, if the U1/U2 request occurs
during some situations. The patch is used to avoid it.
Signed-off-by:
Adjust napi_disable() and napi_enable() to avoid r8152_poll() start
working before rx ready. Otherwise, it may have race condition for
rx_agg.
Signed-off-by: Hayes Wang hayesw...@realtek.com
---
drivers/net/usb/r8152.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git
On 07/23/2015 09:08 PM, Greg KH wrote:
On Thu, Jul 23, 2015 at 08:46:17AM +0200, Juergen Gross wrote:
On 07/23/2015 06:36 AM, Greg KH wrote:
On Thu, Jul 23, 2015 at 06:04:39AM +0200, Juergen Gross wrote:
On 07/23/2015 01:46 AM, Greg KH wrote:
On Tue, Jun 23, 2015 at 08:53:23AM +0200, Juergen
On Thu, 23 Jul 2015, Petr Cvek wrote:
Hello,
Is this:
case USB_ENDPOINT_XFER_INT:
/* Bulk endpoints handle interrupt transfers,
* except the toggle-quirky iso-synch kind
*/
if (!ep-caps.type_int !ep-caps.type_bulk)
On Thu, Jul 23, 2015 at 03:23:26PM -0400, Alan Cooper wrote:
On Wed, Jul 22, 2015 at 10:03 PM, Felipe Balbi ba...@ti.com wrote:
Hi,
On Wed, Jul 22, 2015 at 06:27:29PM -0400, Alan Cooper wrote:
On Wed, Jul 22, 2015 at 5:29 PM, Felipe Balbi ba...@ti.com wrote:
On Wed, Jul 22, 2015 at
On Thu, 23 Jul 2015, Peter Chen wrote:
Assume the interval for each packet is 2ms, the rate is 192 kbytes
for both FS HS:
uac2-p_interval = 2000;
uac2-p_pktsize = 192 kbytes / 2000 = 96 bytes
It's important to keep track of the units. Intervals are measures of
time, and they use units like
Hi,
On Thu, Jul 23, 2015 at 11:00:29AM -0700, Greg Kroah-Hartman wrote:
On Thu, Jul 23, 2015 at 12:02:40AM -0500, Felipe Balbi wrote:
Hi,
On Wed, Jul 22, 2015 at 08:14:46PM -0700, Greg Kroah-Hartman wrote:
On Wed, Jul 22, 2015 at 09:04:40PM -0500, Felipe Balbi wrote:
On Wed, Jul
On Thu, 2015-07-23 at 18:00 +0200, Johan Hovold wrote:
The short answer is that we do not want custom driver ioctls. Any gpio
implementation needs to based on gpiolib, which provides a
standardised
interface.
I understand.
We will evaluate and decide if the work is worth it for us to do.
On Thu, 23 Jul 2015, Giulio Bernardi wrote:
Here is the output of usbmon - collected with usbmon program. The output
seems
the same of cat /sys/kernel/debug/usb/usbmon/0u with slightly more readable
timestamps and shortened addresses so I used that.
This trace shows the uas driver was
On Tue, May 19, 2015 at 04:06:53PM +0100, Richard Watts wrote:
Avoid usb reset crashes by making tty_io cdevs truly dynamic
What USB reset crashes are you referring to here?
Signed-off-by: Richard Watts r...@kynesim.co.uk
Reported-by: Duncan Mackintosh dmackint...@cbnl.com
---
Hi Changbin,
(I assume I address your name properly, if not please excuse)
W dniu 23.07.2015 o 14:34, Du, Changbin pisze:
From 0a8e0d63a9887735c6782d7b0c15c2c1fdf1952a Mon Sep 17 00:00:00 2001
snip
void composite_disconnect(struct usb_gadget *gadget)
{
struct
+ u32 baseline, mantissa, exponent;
Please keep these as unsigned int.
What's the reason for that? u32 is the exact width needed to perform
these computations, while unsigned int is something a bit unspecified.
+ /* Calculate and return the exact baud rate. */
+ baud = (baseline
Hi,
On Thu, Jul 23, 2015 at 06:40:40AM +0200, Petr Cvek wrote:
Hello,
Is this:
case USB_ENDPOINT_XFER_INT:
/* Bulk endpoints handle interrupt transfers,
* except the toggle-quirky iso-synch kind
*/
if (!ep-caps.type_int
Hi,
On Thu, Jul 23, 2015 at 03:41:35PM +0530, Nikhil Badola wrote:
Add adjust_frame_length_quirk for writing to fladj register
which adjusts (micro)frame length to value provided by
snps,configure-fladj property thus avoiding USB 2.0 devices
to time-out over a longer run
Signed-off-by:
Users should not be able to create more function instances than
minor numbers allocated for the functions. The problem is in
hid and printer functions. This short series fixes it by
enforcing the maximum number of function directories created.
Andrzej Pietrasiewicz (2):
usb: gadget: f_hid:
There is a predefined maximum number of hid instances, currently 4.
A chrdev region is allocated accordingly, but with configfs the user
can create as many hid function directories as they like. To make
the number of hid instances consistent with the number of allocated minors,
the limit is
There is a predefined maximum number of printer instances, currently 4.
A chrdev region is allocated accordingly, but with configfs the user
can create as many printer function directories as they like. To make the
number of printer instances consistent with the number of allocated
minors, the
Here is the output of usbmon - collected with usbmon program. The output seems
the same of cat /sys/kernel/debug/usb/usbmon/0u with slightly more readable
timestamps and shortened addresses so I used that.
Giulio
8---
343a9e40
On 07/23/2015 03:00 AM, Peter Chen wrote:
That detail is merely about completeness. The code that calculates the
value of wMaxPacketSize should take into account what is configured in
bInterval of the endpoint, so if users change one thing, they don't have
to tweak the other as well.
On Thu, Jul 23, 2015 at 08:39:41AM -0700, John D. Blair wrote:
Silicon Labs provides GPIOs on the CP210x USB serial hardware. Silicon
Labs also supplies a version of the cp210x driver which provides ioctl()
support allowing control of these GPIOs.
Support of the ioctl() requires adding some
On Thu, Jul 23, 2015 at 04:21:36PM +0200, Michał Pecio wrote:
+ u32 baseline, mantissa, exponent;
Please keep these as unsigned int.
What's the reason for that? u32 is the exact width needed to perform
these computations, while unsigned int is something a bit unspecified.
Because you
Hi again,
On Thu, Jul 23, 2015 at 09:55:32AM -0500, Felipe Balbi wrote:
diff --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h
index 0447788..b7a5119 100644
--- a/drivers/usb/dwc3/core.h
+++ b/drivers/usb/dwc3/core.h
@@ -124,6 +124,7 @@
#define DWC3_GEVNTCOUNT(n) (0xc40c + (n
Dear USB devs,
I have a question regarding GPIO support for the Silicon Labs cp210x USB
serial driver.
Silicon Labs provides GPIOs on the CP210x USB serial hardware. Silicon
Labs also supplies a version of the cp210x driver which provides ioctl()
support allowing control of these GPIOs.
Support
On Thu, Jul 23, 2015 at 08:39:41AM -0700, John D. Blair wrote:
Dear USB devs,
I have a question regarding GPIO support for the Silicon Labs cp210x USB
serial driver.
Silicon Labs provides GPIOs on the CP210x USB serial hardware. Silicon
Labs also supplies a version of the cp210x driver
Oliver Neukum [mailto:oneu...@suse.com]
[...]
If the device does not support remote wakeup and the driver enables it,
runtime
power management will be switched off.
That is the current state and it means that devices which don't support remote
wakeup cannot do runtime power management at
Trying this again as plain text... sorry about that.
Attached is a patch that adds device tree support to the da8xx musb driver.
The current driver expects a board file to setup the platform device and
perform the initialization. With this patch all of the setup is done through
the device
On Thu, Jul 23, 2015 at 02:58:03PM +0200, Dirk Behme wrote:
On 23.07.2015 12:36, Johan Hovold wrote:
On Thu, Jul 23, 2015 at 12:31:55PM +0200, Dirk Behme wrote:
Hi Johan,
On 20.07.2015 11:24, Johan Hovold wrote:
[ +CC: Bjørn and Dan ]
On Mon, Jul 20, 2015 at 08:14:19AM +0200, Dirk
There is a race condition between
finish_unlinks-finish_urb() function and
usb_kill_urb() in ohci controller case. The finish_urb
calls spin_unlock(ohci-lock) before
usb_hcd_giveback_urb() function call, then if during
this time, usb_kill_urb is called for another endpoint,
then new ed
On 23.07.2015 12:36, Johan Hovold wrote:
On Thu, Jul 23, 2015 at 12:31:55PM +0200, Dirk Behme wrote:
Hi Johan,
On 20.07.2015 11:24, Johan Hovold wrote:
[ +CC: Bjørn and Dan ]
On Mon, Jul 20, 2015 at 08:14:19AM +0200, Dirk Behme wrote:
Add support for the Sierra Wireless AR8550 device with
On Thu, Jul 23, 2015 at 08:46:17AM +0200, Juergen Gross wrote:
On 07/23/2015 06:36 AM, Greg KH wrote:
On Thu, Jul 23, 2015 at 06:04:39AM +0200, Juergen Gross wrote:
On 07/23/2015 01:46 AM, Greg KH wrote:
On Tue, Jun 23, 2015 at 08:53:23AM +0200, Juergen Gross wrote:
Add the definition of
Hello.
I'm the author of the da8xx.c file you're patching. :-)
On 07/23/2015 07:43 PM, Andrew Holcomb wrote:
Trying this again as plain text... sorry about that.
Please place such comments under the --- tear line (that goes after
signoff), else they end up in the patch change log
On Wed, Jul 22, 2015 at 10:03 PM, Felipe Balbi ba...@ti.com wrote:
Hi,
On Wed, Jul 22, 2015 at 06:27:29PM -0400, Alan Cooper wrote:
On Wed, Jul 22, 2015 at 5:29 PM, Felipe Balbi ba...@ti.com wrote:
On Wed, Jul 22, 2015 at 04:26:57PM -0500, Felipe Balbi wrote:
On Wed, Jul 22, 2015 at
Hello.
On 07/23/2015 05:11 PM, Andrzej Pietrasiewicz wrote:
There is a predefined maximum number of hid instances, currently 4.
A chrdev region is allocated accordingly, but with configfs the user
can create as many hid function directories as they like. To make
the number of hid instances
On Thu, Jul 23, 2015 at 10:17:38AM +, Vasudevan, Krishna PrasathX K wrote:
Hi,
This mail is for RFC regarding the persist resume of some USB 2.0 devices,
Problem Summary: Problem has been observed for some USB 2.0 devices while
resuming from sleep. When the USB “persist” feature is
This patch replace dynamicly allocated luns array with static one.
This simplifies the code of mass storage function and modules.
Signed-off-by: Krzysztof Opasiak k.opas...@samsung.com
Acked-by: Michal Nazarewicz min...@mina86.com
---
drivers/usb/gadget/function/f_mass_storage.c | 127
On Thu, Jul 23, 2015 at 12:02:40AM -0500, Felipe Balbi wrote:
Hi,
On Wed, Jul 22, 2015 at 08:14:46PM -0700, Greg Kroah-Hartman wrote:
On Wed, Jul 22, 2015 at 09:04:40PM -0500, Felipe Balbi wrote:
On Wed, Jul 22, 2015 at 02:39:34PM -0700, Greg Kroah-Hartman wrote:
On Tue, Jun 23, 2015
Hi,
On Thu, Jul 23, 2015 at 04:19:06PM +, Andrew Holcomb wrote:
Attached is a patch that adds device tree support to the da8xx musb driver.
The current driver expects a board file to setup the platform device and
perform the initialization. With this patch all of the setup is done
On Thu, Jul 23, 2015 at 04:43:55PM +, Andrew Holcomb wrote:
Trying this again as plain text... sorry about that.
Attached is a patch that adds device tree support to the da8xx musb driver.
The current driver expects a board file to setup the platform device and
perform the
On 07/23/2015 06:36 AM, Greg KH wrote:
On Thu, Jul 23, 2015 at 06:04:39AM +0200, Juergen Gross wrote:
On 07/23/2015 01:46 AM, Greg KH wrote:
On Tue, Jun 23, 2015 at 08:53:23AM +0200, Juergen Gross wrote:
Add the definition of pvUSB protocol used between the pvUSB frontend in
a Xen domU and
Adjust napi_disable() and napi_enable() to avoid r8152_poll() start
working before rx ready. Otherwise, it may have race condition for
rx_agg.
Signed-off-by: Hayes Wang hayesw...@realtek.com
---
drivers/net/usb/r8152.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git
- Disable U1/U2 during initialization.
- Disable lpm when linking is on, and enable it when linking is off.
- Disable U1/U2 when enabling runtime suspend.
It is possible to let hw stop working, if the U1/U2 request occurs
during some situations. The patch is used to avoid it.
Signed-off-by:
Set needs_remote_wakeup only when the device supports it.
Signed-off-by: Hayes Wang hayesw...@realtek.com
---
drivers/net/usb/r8152.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c
index e3a0110..eff1f25 100644
---
These patches are used to fix issues.
Hayes Wang (3):
r8152: fix the issue about U1/U2
r8152: fix remote wakeup
r8152: don't enable napi before rx ready
drivers/net/usb/r8152.c | 103
1 file changed, 60 insertions(+), 43 deletions(-)
--
On Thu, Jul 23, 2015 at 08:11:11AM +0200, Daniel Mack wrote:
On 07/23/2015 03:00 AM, Peter Chen wrote:
That detail is merely about completeness. The code that calculates the
value of wMaxPacketSize should take into account what is configured in
bInterval of the endpoint, so if users
On 23/06/15 07:53, Juergen Gross wrote:
Introduces the Xen pvUSB frontend. With pvUSB it is possible for a Xen
domU to communicate with a USB device assigned to that domU. The
communication is all done via the pvUSB backend in a driver domain
(usually Dom0) which is owner of the physical
Hi,
this is most likely wrong. Usbcore does check for a device's ability to
do remote wakeup and will block a runtime suspend if it detects that
a remote wakeup would be required but the device cannot deliver.
(static int autosuspend_check())
So by removing the flag in the probe() method
Add adjust_frame_length_quirk for writing to fladj register
which adjusts (micro)frame length to value provided by
snps,configure-fladj property thus avoiding USB 2.0 devices
to time-out over a longer run
Signed-off-by: Nikhil Badola nikhil.bad...@freescale.com
---
drivers/usb/dwc3/core.c | 12
On 07/23/2015 10:35 AM, Peter Chen wrote:
Assume the interval for each packet is 2ms, the rate is 192 kbytes
for both FS HS:
uac2-p_interval = 2000;
uac2-p_pktsize = 192 kbytes / 2000 = 96 bytes
And the uac2-p_pktsize is real usb packet length, it means hardware
would expect there are 96
Hi,
This mail is for RFC regarding the persist resume of some USB 2.0 devices,
Problem Summary: Problem has been observed for some USB 2.0 devices while
resuming from sleep. When the USB “persist” feature is enabled through sysfs
it is expected to retain its previous mount point across sleep
On Wed, 2015-07-22 at 21:33 +0300, Eugene Shatokhin wrote:
The following part is not necessary, I think. usbnet_bh() does not
touch
EVENT_NO_RUNTIME_PM bit explicitly and these bit operations are
atomic
w.r.t. each other.
+ mpn |= !test_and_clear_bit(EVENT_NO_RUNTIME_PM, dev-flags);
On 23/07/15 07:46, Juergen Gross wrote:
On 07/23/2015 06:36 AM, Greg KH wrote:
On Thu, Jul 23, 2015 at 06:04:39AM +0200, Juergen Gross wrote:
On 07/23/2015 01:46 AM, Greg KH wrote:
On Tue, Jun 23, 2015 at 08:53:23AM +0200, Juergen Gross wrote:
Add the definition of pvUSB protocol used between
On 23/06/15 07:53, Juergen Gross wrote:
Add the definition of pvUSB protocol used between the pvUSB frontend in
a Xen domU and the pvUSB backend in a Xen driver domain (usually Dom0).
This header was originally provided by Fujitsu for Xen based on Linux
2.6.18.
Changes are:
- adapt to
On Mon, 2015-07-20 at 21:13 +0300, Eugene Shatokhin wrote:
[Race #5]
Race on dev-rx_urb_size. I reproduced it a similar way as the races
#2
and #3 (changing MTU while downloading files).
dev-rx_urb_size is written to here:
#0 usbnet_change_mtu (usbnet.c:392)
dev-rx_urb_size =
On Thu, 2015-07-23 at 15:09 +0800, Hayes Wang wrote:
Set needs_remote_wakeup only when the device supports it.
Signed-off-by: Hayes Wang hayesw...@realtek.com
---
drivers/net/usb/r8152.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/net/usb/r8152.c
On Thu, 2015-07-23 at 09:55 +, Hayes Wang wrote:
Hi,
this is most likely wrong. Usbcore does check for a device's ability to
do remote wakeup and will block a runtime suspend if it detects that
a remote wakeup would be required but the device cannot deliver.
(static int
Hi Johan,
On 20.07.2015 11:24, Johan Hovold wrote:
[ +CC: Bjørn and Dan ]
On Mon, Jul 20, 2015 at 08:14:19AM +0200, Dirk Behme wrote:
Add support for the Sierra Wireless AR8550 device with
USB descriptor 0x1199, 0x68AB. For this, lsusb reports:
Thanks for the patch. This modem business is a
On Thu, Jul 23, 2015 at 12:31:55PM +0200, Dirk Behme wrote:
Hi Johan,
On 20.07.2015 11:24, Johan Hovold wrote:
[ +CC: Bjørn and Dan ]
On Mon, Jul 20, 2015 at 08:14:19AM +0200, Dirk Behme wrote:
Add support for the Sierra Wireless AR8550 device with
USB descriptor 0x1199, 0x68AB. For
Hello.
On 7/22/2015 9:56 PM, Peter E. Berger wrote:
From: Peter E. Berger pber...@brimson.com
Do what we can to verify that the driver's firmware image is valid
(before attempting to download it to the Edgeport) by adding a new
function, check_fw_sanity(), and a call to it in in
71 matches
Mail list logo