Re: OOPS: musb_am335x: __device_attach

2015-02-07 Thread Matwey V. Kornilov
05.02.2015 19:30, Matwey V. Kornilov пишет: > > The issue is still there for 3.19.0-rc7 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-usb" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > I

Re: [PATCH] usb: gadget: nokia: Add mass storage driver to g_nokia

2015-02-07 Thread Ivaylo Dimitrov
On 2.02.2015 21:14, Felipe Balbi wrote: On Mon, Feb 02, 2015 at 08:07:51PM +0100, Pali Rohár wrote: On Monday 02 February 2015 20:01:11 Felipe Balbi wrote: Hi, On Mon, Feb 02, 2015 at 07:58:59PM +0100, Pali Rohár wrote: On Monday 02 February 2015 19:54:58 Felipe Balbi wrote: Hi, On Sat,

Re: [PATCH] usb: gadget: nokia: Add mass storage driver to g_nokia

2015-02-07 Thread Ivaylo Dimitrov
Hi, On 7.02.2015 20:01, Ivaylo Dimitrov wrote: On 2.02.2015 21:14, Felipe Balbi wrote: On Mon, Feb 02, 2015 at 08:07:51PM +0100, Pali Rohár wrote: On Monday 02 February 2015 20:01:11 Felipe Balbi wrote: Hi, On Mon, Feb 02, 2015 at 07:58:59PM +0100, Pali Rohár wrote: On Monday 02 Februar

Re: [PATCHv6 4/5] USB: gadget: atmel_usba_udc: Prepare for IRQ single edge support

2015-02-07 Thread Sylvain Rochet
Hello Nicolas, On Thu, Feb 05, 2015 at 06:19:55PM +0100, Nicolas Ferre wrote: > Le 22/01/2015 18:14, Boris Brezillon a écrit : > > On Thu, 22 Jan 2015 17:56:44 +0100 > > Sylvain Rochet wrote: > > > > > -static const struct usba_udc_errata at91sam9g45_errata = { > > > +static const struct usba_u

[PATCH] usb/core: Remove unneeded #ifdef and associated dead code

2015-02-07 Thread Andreas Ruprecht
In commit ceb6c9c862c8 ("USB / PM: Drop CONFIG_PM_RUNTIME from the USB core"), all occurrences of CONFIG_PM_RUNTIME in the USB core code were replaced by CONFIG_PM. This created the following structure of #ifdef blocks in drivers/usb/core/hub.c: [...] #ifdef CONFIG_PM #ifdef CONFIG_PM /* alway

[PATCH 2/2] drivers: isdn: icn: icn.h Clean up trivial checkpatch errors.

2015-02-07 Thread Bas Peters
Signed-off-by: Bas Peters --- drivers/isdn/icn/icn.h | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/isdn/icn/icn.h b/drivers/isdn/icn/icn.h index b713466..05daed2 100644 --- a/drivers/isdn/icn/icn.h +++ b/drivers/isdn/icn/icn.h @@ -54,7 +54,7 @@ typedef struct ic

[PATCH 0/3] Fix checkpatch errors in drivers/isdn/isdnloop

2015-02-07 Thread Bas Peters
This patchset adresses various checkpatch errors in the abovementioned driver. Bas Peters (3): drivers: isdn: isdnloop: isdnloop.c: remove assignment of variables in if conditions, in accordance with the CodingStyle. drivers: isdn: isdnloop: isdnloop.c: Fix brace positions according to

[PATCH 1/3] drivers: isdn: isdnloop: isdnloop.c: remove assignment of variables in if conditions, in accordance with the CodingStyle.

2015-02-07 Thread Bas Peters
Signed-off-by: Bas Peters --- drivers/isdn/isdnloop/isdnloop.c | 18 ++ 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/drivers/isdn/isdnloop/isdnloop.c b/drivers/isdn/isdnloop/isdnloop.c index 5a4da94..af96317 100644 --- a/drivers/isdn/isdnloop/isdnloop.c +++ b/dri

[PATCH 0/2] drivers: isdn: icn: fix checkpatch errors

2015-02-07 Thread Bas Peters
This patch cleans up all checkpatch errors in the icn directory. Bas Peters (2): drivers: isdn: icn: icn.c: clean up all checkpatch errors drivers: isdn: icn: icn.h Clean up trivial checkpatch errors. drivers/isdn/icn/icn.c | 52 ++ drivers/isd

Re: [PATCH 1/7] drivers: isdn: act2000: act2000_isa.c: Fix checkpatch errors

2015-02-07 Thread Bas Peters
Please discard this 2015-02-07 22:53 GMT+01:00 Bas Peters : > This patch adresses various checkpatch errors: > 3 assignments in if conditions > 1 return value enclosed in parenthesis > > Signed-off-by: Bas Peters > --- > drivers/isdn/act2000/act2000_isa.c | 11 +++ > 1 fi

[PATCH 1/2] drivers: isdn: icn: icn.c: clean up all checkpatch errors

2015-02-07 Thread Bas Peters
This patch cleans up various trivial checkpatch errors such as variable declarations in if statements, return values in parenthesis and a wrongly placed brace. Signed-off-by: Bas Peters --- drivers/isdn/icn/icn.c | 52 ++ 1 file changed, 31 inserti

Re: [PATCH V2 0/7] drivers: isdn: act2000: fix checkpatch errors.

2015-02-07 Thread Bas Peters
Please discard this 2015-02-07 22:53 GMT+01:00 Bas Peters : > This patchset adresses many checkpatch errors found in the act2000 driver. > > Bas Peters (7): > drivers: isdn: act2000: act2000_isa.c: Fix checkpatch errors > drivers: isdn: act2000: capi.c: fix checkpatch errors > drivers: isdn:

[PATCH 6/7] drivers: isdn: act2000: fix wrongly positioned brace.

2015-02-07 Thread Bas Peters
Trivial, but why not? :) Signed-off-by: Bas Peters --- drivers/isdn/act2000/module.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/isdn/act2000/module.c b/drivers/isdn/act2000/module.c index 889ffcb..9ba98ce 100644 --- a/drivers/isdn/act2000/module.c +++ b/drivers

[PATCH 2/6] drivers: usb: core: devio.c: fix whitespace errors thrown by checkpatch.pl

2015-02-07 Thread Bas Peters
This patch fixes errors generated by checkpatch.pl relating to whitespace issues. Signed-off-by: Bas Peters --- drivers/usb/core/devio.c | 61 1 file changed, 31 insertions(+), 30 deletions(-) diff --git a/drivers/usb/core/devio.c b/drivers/usb/c

[PATCH 5/6] drivers: usb: core: hub.c: remove assignment of variables in if conditions.

2015-02-07 Thread Bas Peters
As specified in the CodingStyle. Signed-off-by: Bas Peters --- drivers/usb/core/hub.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c index 82983d9..9afe8b0 100644 --- a/drivers/usb/core/hub.c +++ b/drivers/usb/core/hub.c

[PATCH 4/6] drivers: usb: core: hub.c: remove NULL initialization of static variables.

2015-02-07 Thread Bas Peters
NULL initialization of static variables is unnecessary as GCC kindly does this for us. Signed-off-by: Bas Peters --- drivers/usb/core/hub.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c index aeb50bb..82983d9 100644 --- a/

[PATCH 1/6] drivers: usb: core: devio.c: remove assignment of variables in if conditions.

2015-02-07 Thread Bas Peters
This patch removes assignment of variables in if conditions in accordance witht the CodingStyle. Signed-off-by: Bas Peters --- drivers/usb/core/devio.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/drivers/usb/core/devio.c b/drivers/usb/core/devio.c index 0

[PATCH 3/6] drivers: usb: core: hcd.c: remove assignment of variables in if conditions.

2015-02-07 Thread Bas Peters
This patch removes assignment of variables in if conditions, as specified in CodingStyle. Signed-off-by: Bas Peters --- drivers/usb/core/hcd.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/drivers/usb/core/hcd.c b/drivers/usb/core/hcd.c index 11cee55..37c40

[PATCH 6/6] drivers: usb: core: endpoint.c: fix trivial whitespace issue

2015-02-07 Thread Bas Peters
Changes space-based indentation to tab-based indentation. Signed-off-by: Bas Peters --- drivers/usb/core/endpoint.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/core/endpoint.c b/drivers/usb/core/endpoint.c index 39a2402..101983b 100644 --- a/drivers/usb/core/e

[PATCH 4/7] drivers: isdn: act2000: module.c: remove NULL-initialization of static variable.

2015-02-07 Thread Bas Peters
GCC takes care of this for us, thus it is not needed and theoretically only hoggs memory, allbeit only a bit. --- drivers/isdn/act2000/module.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/act2000/module.c b/drivers/isdn/act2000/module.c index 352916a..9359b36 1

[PATCH 7/7] drivers: isdn: act2000: capi.c: add macro \ and fix brace

2015-02-07 Thread Bas Peters
This patch adds the \ that was accidentally deleted in patch 2. It also adds a brace after the else statement, which is required due to the fact that the if statement has braces. --- drivers/isdn/act2000/capi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/isdn/a

[PATCH 1/7] drivers: isdn: act2000: act2000_isa.c: Fix checkpatch errors

2015-02-07 Thread Bas Peters
This patch adresses various checkpatch errors: 3 assignments in if conditions 1 return value enclosed in parenthesis Signed-off-by: Bas Peters --- drivers/isdn/act2000/act2000_isa.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/drivers/isdn/act20

[PATCH 5/7] drivers: isdn: act2000: module.c: remove parenthesres around return values.

2015-02-07 Thread Bas Peters
return is not a function, therefore parentheses are not needed. --- drivers/isdn/act2000/module.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/isdn/act2000/module.c b/drivers/isdn/act2000/module.c index 9359b36..889ffcb 100644 --- a/drivers/isdn/act2000/

Re: [PATCH 1/3] drivers: isdn: isdnloop: isdnloop.c: remove assignment of variables in if conditions, in accordance with the CodingStyle.

2015-02-07 Thread Bas Peters
Please discard all these e-mails, something went wrong and I sent the wrong directory of patches. 2015-02-07 22:54 GMT+01:00 Bas Peters : > Signed-off-by: Bas Peters > --- > drivers/isdn/isdnloop/isdnloop.c | 18 ++ > 1 file changed, 10 insertions(+), 8 deletions(-) > > diff --gi

[PATCH 3/7] drivers: isdn: act2000: remove assignments of variables in if conditions

2015-02-07 Thread Bas Peters
This patch removes all assignments of if conditions, which is not in accordance with the CodingStyle. --- drivers/isdn/act2000/module.c | 30 -- 1 file changed, 20 insertions(+), 10 deletions(-) diff --git a/drivers/isdn/act2000/module.c b/drivers/isdn/act2000/module.c

[PATCH 0/6] drivers: usb: core: fix various checkpatch errors.

2015-02-07 Thread Bas Peters
This patchset adresses various checkpatch errors found when running the checkpatch script on the directory. Bas Peters (6): drivers: usb: core: devio.c: remove assignment of variables in if conditions. drivers: usb: core: devio.c: fix whitespace errors thrown by checkpatch.pl

[PATCH 2/7] drivers: isdn: act2000: capi.c: fix checkpatch errors

2015-02-07 Thread Bas Peters
This patch fixes the following checkpatch errors: 1. trailing statement 1. assignment of variable in if condition 1. incorrectly placed brace after function definition Signed-off-by: Bas Peters --- drivers/isdn/act2000/capi.c | 9 ++--- 1 file changed, 6 insertions(+)

[PATCH V2 0/7] drivers: isdn: act2000: fix checkpatch errors.

2015-02-07 Thread Bas Peters
This patchset adresses many checkpatch errors found in the act2000 driver. Bas Peters (7): drivers: isdn: act2000: act2000_isa.c: Fix checkpatch errors drivers: isdn: act2000: capi.c: fix checkpatch errors drivers: isdn: act2000: remove assignments of variables in if conditions driver

[PATCH 1/3] drivers: usb: storage: alauda.c: properly place braces after function declarations

2015-02-07 Thread Bas Peters
This patch places braces on a new line following function declarations. Signed-off-by: Bas Peters --- drivers/usb/storage/alauda.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/drivers/usb/storage/alauda.c b/drivers/usb/storage/alauda.c index 62c2d9d..4b55a

[PATCH 3/3] drivers: usb: storage: datafab.c: clean up a variety of checkpatch errors.

2015-02-07 Thread Bas Peters
This patch cleans up a variety of checkpatch errors: Bunch of space issues. C99 comments converted to /* */ format. Some switch statement indentations. "foo * bar" -> "foo *bar" Signed-off-by: Bas Peters --- drivers/usb/storage/datafab.c | 183 +++

[PATCH 0/3] drivers: usb: storage: fix some checkpatch errors

2015-02-07 Thread Bas Peters
This patchset adresses checkpatch errors in a few of the files in usb storage. More to follow. Bas Peters (3): drivers: usb: storage: alauda.c: properly place braces after function declarations drivers: usb: storage: cypress_atacb.c: trivial checkpatch fixes drivers: usb: storage: da

[PATCH 2/3] drivers: usb: storage: cypress_atacb.c: trivial checkpatch fixes

2015-02-07 Thread Bas Peters
Fixes errors thrown by checkpatch over a space issue and the incorrect indentation of a switch statement. Signed-off-by: Bas Peters --- drivers/usb/storage/cypress_atacb.c | 17 - 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/drivers/usb/storage/cypress_atacb.c b

Re: [possible fix] HL-340 USB don't work correctly (ch340 based usb-rs232 adapter)

2015-02-07 Thread Karl Palsson
My work in Eddi De Pieri wrote: > Bus 003 Device 014: ID 1a86:7523 QinHeng Electronics HL-340 USB-Serial > adapter > > ch431 kernel driver don't work correctly with this device, > when URB_FUNCTION_BULK_OR_INTERRUPT_TRANSFER answer/send wrong character > > After some experiment by passing c

Re: [PATCH net-next v2 0/7] r8152: adjust the code

2015-02-07 Thread David Miller
From: Hayes Wang Date: Fri, 6 Feb 2015 11:30:44 +0800 > V2: > Correct the subject of patch #5. Replace "link feed" with "line feed". > > v1: > Code adjustment. Series applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@v