Removed the header files included twice.
Signed-off-by: Sachin Kamat
Cc: Larry Finger
---
drivers/staging/rtl8188eu/os_dep/ioctl_linux.c |2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/rtl8188eu/os_dep/ioctl_linux.c
b/drivers/staging/rtl8188eu/os_dep/ioctl_linux.c
index
Include version.h header file as detected by versioncheck.
Signed-off-by: Sachin Kamat
---
drivers/staging/rtl8188eu/include/usb_ops.h |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/rtl8188eu/include/usb_ops.h
b/drivers/staging/rtl8188eu/include/usb_ops.h
index c23de7b..
version.h header inclusion is not necessary as detected by
versioncheck.
Signed-off-by: Sachin Kamat
---
drivers/staging/rtl8188eu/include/osdep_service.h |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/rtl8188eu/include/osdep_service.h
b/drivers/staging/rtl8188eu/include/
Include version.h header file as detected by versioncheck.
Signed-off-by: Sachin Kamat
---
drivers/staging/rtl8188eu/os_dep/xmit_linux.c |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/rtl8188eu/os_dep/xmit_linux.c
b/drivers/staging/rtl8188eu/os_dep/xmit_linux.c
index 1e2
version.h header inclusion is not necessary as detected by
versioncheck.
Signed-off-by: Sachin Kamat
---
drivers/staging/gdm724x/netlink_k.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/gdm724x/netlink_k.c
b/drivers/staging/gdm724x/netlink_k.c
index 9e8be6a..77fc64e 100
version.h header inclusion is not necessary as detected by
versioncheck.
Signed-off-by: Sachin Kamat
---
drivers/staging/dgap/dgap_parse.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/dgap/dgap_parse.c
b/drivers/staging/dgap/dgap_parse.c
index d6d507b..5497e6d 100644
--
version.h header inclusion is not necessary as detected by
versioncheck.
Signed-off-by: Sachin Kamat
---
drivers/staging/dgap/dgap_trace.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/dgap/dgap_trace.c
b/drivers/staging/dgap/dgap_trace.c
index 1761002..0f9a956 100644
--
version.h header inclusion is not necessary as detected by
versioncheck.
Signed-off-by: Sachin Kamat
---
drivers/staging/dgap/dgap_trace.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/dgap/dgap_trace.c
b/drivers/staging/dgap/dgap_trace.c
index 1761002..0f9a956 100644
--
version.h header inclusion is not necessary as detected by
versioncheck.
Signed-off-by: Sachin Kamat
---
drivers/staging/dgap/dgap_sysfs.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/dgap/dgap_sysfs.c
b/drivers/staging/dgap/dgap_sysfs.c
index d4300ce..94da06f 100644
--
Include version.h header file as detected by versioncheck.
Signed-off-by: Sachin Kamat
---
drivers/staging/dgap/dgap_kcompat.h |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/dgap/dgap_kcompat.h
b/drivers/staging/dgap/dgap_kcompat.h
index 8ebf4b7..f21c8d7 100644
--- a/d
On 28 August 2013 10:40, Greg KH wrote:
> On Wed, Aug 28, 2013 at 10:16:06AM +0530, Sachin Kamat wrote:
>> Include version.h header file as detected by versioncheck.
>>
>> Signed-off-by: Sachin Kamat
>> ---
>> drivers/staging/dgap/dgap_kcompat.h |2 ++
>> 1 file changed, 2 insertions(+)
>
>
version.h header inclusion is not necessary as detected by
versioncheck.
Signed-off-by: Sachin Kamat
---
drivers/staging/dgap/dgap_sysfs.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/dgap/dgap_sysfs.c
b/drivers/staging/dgap/dgap_sysfs.c
index d4300ce..94da06f 100644
--
version.h header inclusion is not necessary as detected by
versioncheck.
Signed-off-by: Sachin Kamat
---
drivers/staging/dgap/dgap_parse.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/dgap/dgap_parse.c
b/drivers/staging/dgap/dgap_parse.c
index d6d507b..5497e6d 100644
--
On Wed, Aug 28, 2013 at 10:16:06AM +0530, Sachin Kamat wrote:
> Include version.h header file as detected by versioncheck.
>
> Signed-off-by: Sachin Kamat
> ---
> drivers/staging/dgap/dgap_kcompat.h |2 ++
> 1 file changed, 2 insertions(+)
Please use the scripts/get_maintainer.pl tool to de
Include version.h header file as detected by versioncheck.
Signed-off-by: Sachin Kamat
---
drivers/staging/dgap/dgap_kcompat.h |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/dgap/dgap_kcompat.h
b/drivers/staging/dgap/dgap_kcompat.h
index 8ebf4b7..f21c8d7 100644
--- a/d
On Tue, Aug 27, 2013 at 10:13:27PM -0400, Lidza Louina wrote:
> This patch replaces dgnc_driver_kzmalloc with kzalloc.
> A patch that follows removes the dgnc_driver_kzmalloc
> function.
>
> Signed-off-by: Lidza Louina
> ---
> drivers/staging/dgnc/dgnc_driver.c | 6 +++---
> drivers/staging/dgn
On Tue, Aug 27, 2013 at 10:13:28PM -0400, Lidza Louina wrote:
> This patch removes the dgnc_driver_kzmalloc function from
> driver.c and driver.h. A previous patch replaced all
> dgnc_driver_kzmalloc function calls with kzalloc.
>
> Signed-off-by: Lidza Louina
> ---
> drivers/staging/dgnc/dgnc_d
This patch edits the type casts neo_uart_struct and
cls_uart_struct. A previous patch added the marker __iomem
to these structs. This patch ensures that the change to
the marker is consistent. This also removes these
sparse warnings:
warning: incorrect type in assignment (different address spaces)
This patch removes the dgnc_driver_kzmalloc function from
driver.c and driver.h. A previous patch replaced all
dgnc_driver_kzmalloc function calls with kzalloc.
Signed-off-by: Lidza Louina
---
drivers/staging/dgnc/dgnc_driver.c | 15 ---
drivers/staging/dgnc/dgnc_driver.h | 1 -
2 f
This patch replaces dgnc_driver_kzmalloc with kzalloc.
A patch that follows removes the dgnc_driver_kzmalloc
function.
Signed-off-by: Lidza Louina
---
drivers/staging/dgnc/dgnc_driver.c | 6 +++---
drivers/staging/dgnc/dgnc_tty.c| 20 ++--
2 files changed, 13 insertions(+),
On Tue, Aug 27, 2013 at 7:48 PM, Greg KH wrote:
> On Tue, Aug 27, 2013 at 06:19:43PM -0400, Lidza Louina wrote:
>> This patch removes the dgnc_driver_kzmalloc function from
>> driver.c. The patch that follows replaces all
>> dgnc_driver_kzmalloc functions calls with kzalloc.
>
> Yes, but this patc
On Tue, Aug 27, 2013 at 7:29 PM, Dan Carpenter wrote:
> On Tue, Aug 27, 2013 at 06:19:43PM -0400, Lidza Louina wrote:
>> This patch removes the dgnc_driver_kzmalloc function from
>> driver.c. The patch that follows replaces all
>> dgnc_driver_kzmalloc functions calls with kzalloc.
>>
>
> You can't
On Tue, Aug 27, 2013 at 06:19:43PM -0400, Lidza Louina wrote:
> This patch removes the dgnc_driver_kzmalloc function from
> driver.c. The patch that follows replaces all
> dgnc_driver_kzmalloc functions calls with kzalloc.
Yes, but this patch just broke the build, right?
Do this in the opposite o
On Mon, Aug 26, 2013 at 05:10:04PM +0200, Iker Pedrosa wrote:
> Second of the patches that fixes the lines over 80 characters in
> phy_calibration.c
>
> Signed-off-by: Iker Pedrosa
This Subject: is identical to the previous patch, not good.
You also do that for 4 other patches in this series :
On Mon, Aug 26, 2013 at 05:10:19PM +0200, Iker Pedrosa wrote:
> According to the documentation it is not recommended to use msleep for 1ms -
> 20ms because it may sleep longer than 20ms. So, it is recommended to use
> usleep instead.
>
> Signed-off-by: Iker Pedrosa
> ---
> drivers/staging/winb
On Mon, Aug 26, 2013 at 05:10:07PM +0200, Iker Pedrosa wrote:
> According to the documentation it is not recommended to use msleep for 1ms -
> 20ms because it may sleep longer than 20ms. So, it is recommended to use
> usleep instead.
>
> Signed-off-by: Iker Pedrosa
> ---
> drivers/staging/winb
On Tue, Aug 27, 2013 at 06:19:43PM -0400, Lidza Louina wrote:
> This patch removes the dgnc_driver_kzmalloc function from
> driver.c. The patch that follows replaces all
> dgnc_driver_kzmalloc functions calls with kzalloc.
>
You can't do it in this order. The first patch will break the compile.
This patch removes the dgnc_driver_kzmalloc function from
driver.c. The patch that follows replaces all
dgnc_driver_kzmalloc functions calls with kzalloc.
Signed-off-by: Lidza Louina
---
drivers/staging/dgnc/dgnc_driver.c | 15 ---
1 file changed, 15 deletions(-)
diff --git a/driver
This patch replaces dgnc_driver_kzmalloc with kzalloc. A
previous patch removed that function from the code.
Signed-off-by: Lidza Louina
---
drivers/staging/dgnc/dgnc_driver.c | 6 +++---
drivers/staging/dgnc/dgnc_tty.c| 20 ++--
2 files changed, 13 insertions(+), 13 deletio
On Tue, Aug 27, 2013 at 2:46 PM, Dan Carpenter wrote:
> On Tue, Aug 27, 2013 at 11:20:06AM -0400, Lidza Louina wrote:
>> This patch changes the dgnc_driver_kzmalloc declaration.
>> The kernel's kmalloc function previously took an int
>> priority. That's been replaced with gfp_t flags.
>>
>> The wa
This patch edits the type casts neo_uart_struct and
cls_uart_struct. A previous patch added the marker __iomem
to these structs. This patch ensures that the change to
the marker is consistent. This also removes these
sparse warnings:
warning: incorrect type in assignment (different address spaces)
On Tue, Aug 27, 2013 at 2:39 PM, Dan Carpenter wrote:
> Could you redo this one?
>
> On Tue, Aug 27, 2013 at 11:12:16AM -0400, Lidza Louina wrote:
>> This patch edits the type casts neo_uart_struct and
>> cls_uart_struct. A previous patch added the marker __iomem
>> to these structs. This patch en
On Tue, Aug 27, 2013 at 04:53:40PM +0100, Rupesh Gujare wrote:
> Increase ISOC IN buffer depth to 100 units.
>
This change log is not useful. What are the user visible effects of
this bug?
regards,
dan carpenter
___
devel mailing list
de...@linuxdriv
On Tue, Aug 27, 2013 at 01:50:35PM -0400, Lidza Louina wrote:
> Ah, ok, I don't see anything unusual in that file either. I'll wait
> to see what he was talking about.
Yeah. You're both right. I was looking at the wrong file and there
is no locking problem. My bad.
regards,
dan carpenter
On Tue, Aug 27, 2013 at 11:20:06AM -0400, Lidza Louina wrote:
> This patch changes the dgnc_driver_kzmalloc declaration.
> The kernel's kmalloc function previously took an int
> priority. That's been replaced with gfp_t flags.
>
> The warning was caught by sparse.
>
Don't bother fixing these...
Could you redo this one?
On Tue, Aug 27, 2013 at 11:12:16AM -0400, Lidza Louina wrote:
> This patch edits the type casts neo_uart_struct and
> cls_uart_struct. A previous patch added the marker __iomem
> to these structs. This patch ensures that the change to
> the marker is consistent. This also
On Tue, Aug 27, 2013 at 1:25 PM, Mark Hounschell wrote:
> On 08/27/2013 01:20 PM, Lidza Louina wrote:
>>
>> On Fri, Aug 23, 2013 at 4:23 PM, Dan Carpenter
>> wrote:
>>>
>>> On Wed, Aug 21, 2013 at 09:48:38PM -0400, Lidza Louina wrote:
-static int dgap_tty_ioctl(struct tty_struct *tty, s
Hello.
On 08/27/2013 07:53 PM, Rupesh Gujare wrote:
Return error if we receive write(), while PD is not connected.
Signed-off-by: Rupesh Gujare
---
drivers/staging/ozwpan/ozcdev.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/ozwpan/ozcdev.c b/drivers/staging
On 08/27/2013 01:20 PM, Lidza Louina wrote:
On Fri, Aug 23, 2013 at 4:23 PM, Dan Carpenter wrote:
On Wed, Aug 21, 2013 at 09:48:38PM -0400, Lidza Louina wrote:
-static int dgap_tty_ioctl(struct tty_struct *tty, struct file *file, unsigned
int cmd,
+static int dgap_tty_ioctl(struct tty_struct
On Fri, Aug 23, 2013 at 4:23 PM, Dan Carpenter wrote:
> On Wed, Aug 21, 2013 at 09:48:38PM -0400, Lidza Louina wrote:
>> -static int dgap_tty_ioctl(struct tty_struct *tty, struct file *file,
>> unsigned int cmd,
>> +static int dgap_tty_ioctl(struct tty_struct *tty, unsigned int cmd,
>>
Return error if we receive write(), while PD is not connected.
Signed-off-by: Rupesh Gujare
---
drivers/staging/ozwpan/ozcdev.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/ozwpan/ozcdev.c b/drivers/staging/ozwpan/ozcdev.c
index 03b41ee..22cb2ae 100644
--- a/drivers/s
Reset PORT_ENABLE bit of port status on loosing PD.
Signed-off-by: Rupesh Gujare
---
drivers/staging/ozwpan/ozhcd.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/ozwpan/ozhcd.c b/drivers/staging/ozwpan/ozhcd.c
index b0398c5..39f5066 100644
--- a/drivers/s
> -Original Message-
> From: Jan Beulich [mailto:jbeul...@suse.com]
> Sent: Tuesday, August 27, 2013 12:26 AM
> To: KY Srinivasan
> Cc: o...@aepfle.de; a...@canonical.com; x...@kernel.org;
> de...@linuxdriverproject.org; gre...@linuxfoundation.org;
> jasow...@redhat.com; linux-ker...@vger.k
Fixes following error caused during unloading driver.
[ 1127.542888] usb 5-1: USB disconnect, device number 2
[ 1127.542909] ozwpan ozwpan: remove, state 1
[ 1127.542933] usb usb5: USB disconnect, device number 1
[ 1127.618634] hub 5-0:1.0: hub_port_status failed (err = -19)
[ 1127.618647] hub_por
Increase ISOC IN buffer depth to 100 units.
Signed-off-by: Rupesh Gujare
---
drivers/staging/ozwpan/ozhcd.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/ozwpan/ozhcd.c b/drivers/staging/ozwpan/ozhcd.c
index 3548860..b0398c5 100644
--- a/drivers/staging/o
This patch changes the dgnc_driver_kzmalloc declaration.
The kernel's kmalloc function previously took an int
priority. That's been replaced with gfp_t flags.
The warning was caught by sparse.
Signed-off-by: Lidza Louina
---
drivers/staging/dgnc/dgnc_driver.c | 4 ++--
drivers/staging/dgnc/dgnc
This patch edits the type casts neo_uart_struct and
cls_uart_struct. A previous patch added the marker __iomem
to these structs. This patch ensures that the change to
the marker is consistent. This also removes these
sparse warnings:
warning: incorrect type in assignment (different address spaces)
On Mon, 26 Aug 2013 19:43:06 +0300
Dan Carpenter wrote:
> Btw, all the white space nit-pick I pointed out were pretty minor. I
> don't like making people redo stuff so you can fix anything you want to
> in later patches.
>
> regards,
> dan carpenter
>
I write it down for the next series of pa
On 2013-08-27 11:16, Ian Abbott wrote:
(The
"ni_atmio16d" driver tried to be clever in allowing several channels to
be configured at once.)
Actually it wasn't, but what it was doing was pretty silly.
--
-=( Ian Abbott @ MEV Ltd.E-mail: )=-
-=( Tel: +44 (0)161 477 1898 FAX: +44 (0
On 2013-08-23 16:01, hartl...@visionengravers.com wrote:
Ian,
I have not had a chance to look this series over yet but I had an off topic
comment about kcomedilib.
Is it too late to rename the exports from that driver? It would be a lot
clearer if the "core" exports were prefixed with "comedi_"
On 2013-08-26 23:27, H Hartley Sweeten wrote:
These drivers all improperly implement the (*insn_config) for a DIO subdevice.
They
all treat the data[0] value as a flag to configure the channel in
insn->chanspec as
either an output or an input. The data[0] value is actually the 'instruction'
to
>>> On 27.08.13 at 01:42, "K. Y. Srinivasan" wrote:
> Hyper-V supports a mechanism for retrieving the local API frequency.
"APIC"?
> @@ -27,6 +27,13 @@
> #define HV_X64_MSR_VP_RUNTIME_AVAILABLE (1 << 0)
> /* Partition Reference Counter (HV_X64_MSR_TIME_REF_COUNT) available*/
> #d
52 matches
Mail list logo
