[PATCH] staging/lustre/ptlrpc: Removes potential null dereference

2016-05-12 Thread Lidza Louina
The lustre_msg_buf method could return NULL. Subsequent code didn't check if it's null before using it. This patch adds two checks. Signed-off-by: Lidza Louina diff --git a/drivers/staging/lustre/lustre/ptlrpc/sec.c b/drivers/staging/lustre/lustre/ptlrpc/sec.c index 187fd1d..e6fe

Re: [PATCH] staging/lustre/ptlrpc: Removes potential null dereference

2016-05-16 Thread Lidza Louina
On 05/13/2016 12:43 PM, Drokin, Oleg wrote: On May 12, 2016, at 6:33 PM, Lidza Louina wrote: The lustre_msg_buf method could return NULL. Subsequent code didn't check if it's null before using it. This patch adds two checks. Thank you for the patch, it looks good code-wise, but

[PATCH v2] staging/lustre/ptlrpc: Removes potential null dereference

2016-05-16 Thread Lidza Louina
The lustre_msg_buf method could return NULL. Subsequent code didn't check if it's null before using it. This patch adds two checks. Signed-off-by: Lidza Louina --- drivers/staging/lustre/lustre/ptlrpc/sec.c | 2 ++ drivers/staging/lustre/lustre/ptlrpc/sec_plain.c | 9 +++-

Re: [lustre-devel] [PATCH v2] staging/lustre/ptlrpc: Removes potential null dereference

2016-05-16 Thread Lidza Louina
_udesc)" block instead of at the top of the function, since it isn't used anywhere else. Cheers, Andreas Definitely, will do. I'll change desc to rc and update the if statement and send another patch now. Lidza - Patrick On 05/16/2016 09:17 AM, Lidza Louina wrote: The lustre_m

[PATCH v3] staging/lustre/ptlrpc: Removes potential null dereference

2016-05-16 Thread Lidza Louina
The lustre_msg_buf method could return NULL. Subsequent code didn't check if it's null before using it. This patch adds two checks. Signed-off-by: Lidza Louina --- drivers/staging/lustre/lustre/ptlrpc/sec.c | 3 +++ drivers/staging/lustre/lustre/ptlrpc/sec_plain.c | 8 ++-

Re: [lustre-devel] [PATCH v2] staging/lustre/ptlrpc: Removes potential null dereference

2016-05-17 Thread Lidza Louina
On 05/17/2016 02:53 AM, Dan Carpenter wrote: When I read the code, I just assumed desc was a pointer and it should have been: if (!desc) return NULL; For me, "if (rc) " is way more readable than "if (rc != 0) ". So readability could go either way depending on what you

[PATCH] staging/wilc1000: fixes kzalloc call

2016-05-27 Thread Lidza Louina
The wl pointer was initialized as a pointer to a struct wilc and assigned to a piece of memory the size of the pointer. It should be the size of struct wilc. Signed-off-by: Lidza Louina --- drivers/staging/wilc1000/linux_wlan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a

Re: [PATCH] staging/wilc1000: fixes kzalloc call

2016-05-27 Thread Lidza Louina
On 05/27/2016 02:07 PM, Joe Perches wrote: On Fri, 2016-05-27 at 13:51 -0400, Lidza Louina wrote: The wl pointer was initialized as a pointer to a struct wilc and assigned to a piece of memory the size of the pointer. It should be the size of struct wilc. This isn't necessary. The co

[PATCH] staging/lustre/lnet: correctly casts value for arithmetic

2016-06-23 Thread Lidza Louina
The code attempted to add an unsigned int to a an unsigned 64-bit integer. This patch casts the unsigned regular int to suppress smatch warnings. Signed-off-by: Lidza Louina --- drivers/staging/lustre/lnet/lnet/api-ni.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

Re: [lustre-devel] [PATCH] staging/lustre/lnet: correctly casts value for arithmetic

2016-06-23 Thread Lidza Louina
-devel] [PATCH] staging/lustre/lnet: correctly casts value for arithmetic NAK. On Jun 23, 2016, at 1:24 PM, Lidza Louina wrote: > The code attempted to add an unsigned int to a an unsigned 64-bit > integer. This patch casts the unsigned regular int to suppress > smatch warnings. > >

Re: [lustre-devel] [PATCH] staging/lustre/lnet: correctly casts value for arithmetic

2016-06-23 Thread Lidza Louina
-devel] [PATCH] staging/lustre/lnet: correctly casts value for arithmetic On Jun 23, 2016, at 2:07 PM, Lidza Louina wrote: > The smatch warning was "drivers/staging/lustre/lnet/lnet/api-ni.c:516 > lnet_res_lh_initialize() warn: should '1 << ibits' be a 64 bit type?"

[PATCH v2] staging/lustre/lnet: changes value to correct type for assignment

2016-06-23 Thread Lidza Louina
The code attempted to add an unsigned int to a an unsigned 64-bit integer. This patch makes the code use the correct type of int to suppress a smatch warning. Signed-off-by: Lidza Louina --- drivers/staging/lustre/lnet/lnet/api-ni.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [lustre-devel] [PATCH v2] staging/lustre/lnet: changes value to correct type for assignment

2016-06-23 Thread Lidza Louina
-devel] [PATCH v2] staging/lustre/lnet: changes value to correct type for assignment On Jun 23, 2016, at 2:56 PM, Lidza Louina wrote: > The code attempted to add an unsigned int to a an unsigned 64-bit > integer. This patch makes the code use the correct type of int to > suppress a smatc

Re: [lustre-devel] [PATCH v2] staging/lustre/lnet: changes value to correct type for assignment

2016-06-23 Thread Lidza Louina
/lustre/lnet: changes value to correct type for assignment On Jun 23, 2016, at 3:27 PM, Lidza Louina wrote: > > - Original Message - > From: oleg.dro...@intel.com > To: lidza.lou...@oracle.com > Cc: gre...@linuxfoundation.org, lustre-de...@lists.lustre.org, > de...@dr

[PATCH] staging/rtl8188eu/core/rtw_cmd: Fixes semaphore locking

2016-08-09 Thread Lidza Louina
The code did not lock the &pcmdpriv->terminate_cmdthread_sema before using it. The code had an up() where a down() should've been. This patch fixes that. Signed-off-by: Lidza Louina ---  drivers/staging/rtl8188eu/core/rtw_cmd.c | 2 +-  1 file changed, 1 insertion(+), 1 deletion(-)

Re: [PATCH v2 4/5] staging: dgap: tty.c: changes error handing to tty driver allocations

2013-09-26 Thread Lidza Louina
On Thu, Sep 26, 2013 at 6:19 AM, Dan Carpenter wrote: > This one is not right. > > On Wed, Sep 25, 2013 at 07:08:53PM -0400, Lidza Louina wrote: >> This patch changes error handling to the >> tty_driver allocations in dgap_tty_register. >> >> Before, it di

Re: [PATCH v2 5/5] staging: dgap: tty.c: removes smatch warning "unsigned '--un->un_open_count' is never less than zero"

2013-09-26 Thread Lidza Louina
On Thu, Sep 26, 2013 at 6:22 AM, Dan Carpenter wrote: > On Wed, Sep 25, 2013 at 07:08:54PM -0400, Lidza Louina wrote: >> This patch removes this smatch warning: >> unsigned '--un->un_open_count' is never less than zero >> >> The code decremented the un_ope

Re: [PATCH 1/1] staging: dgnc: Remove KERNEL_VERSION check

2013-09-27 Thread Lidza Louina
On Fri, Sep 27, 2013 at 5:50 AM, Sachin Kamat wrote: > This check is not required. > > Signed-off-by: Sachin Kamat > Cc: Lidza Louina > --- > drivers/staging/dgnc/dgnc_kcompat.h |5 - > 1 file changed, 5 deletions(-) > > diff --git a/drivers/staging/dgnc/

[PATCH v3 2/2] staging: dgap: tty.c: removes smatch warning "unsigned '--un->un_open_count' is never less than zero"

2013-10-01 Thread Lidza Louina
This patch removes this smatch warning: unsigned '--un->un_open_count' is never less than zero The code decremented the un_open_count variable and tested to see if it was less than zero. Because un_open_count is unsigned and can't be below zero, this test didn't work. Sign

[PATCH v3 1/2] staging: dgap: tty.c: adds error handing in tty driver allocations

2013-10-01 Thread Lidza Louina
This patch adds error handling to the tty_driver allocations in dgap_tty_register. Now the code handles the possibility of an alloc_tty_driver, a tty_register_driver, and a brd->SerialDriver->ttys or brd->PrintDriver->ttys allocation failure. Signed-off-by: Lidza Louina --- dri

Re: [PATCH v3 1/2] staging: dgap: tty.c: adds error handing in tty driver allocations

2013-10-01 Thread Lidza Louina
On Tue, Oct 1, 2013 at 12:09 PM, Dan Carpenter wrote: > On Tue, Oct 01, 2013 at 12:00:23PM -0400, Lidza Louina wrote: >> @@ -306,7 +316,16 @@ int dgap_tty_register(struct board_t *brd) >> >> DPR_INIT(("DGAP REGISTER TTY: MAJORS: %d %d\n", >> brd->S

[PATCH v4 2/2] staging: dgap: tty.c: removes smatch warning "unsigned '--un->un_open_count' is never less than zero"

2013-10-01 Thread Lidza Louina
This patch removes this smatch warning: unsigned '--un->un_open_count' is never less than zero The code decremented the un_open_count variable and tested to see if it was less than zero. Because un_open_count is unsigned and can't be below zero, this test didn't work. Sign

[PATCH v4 1/2] staging: dgap: tty.c: adds error handing in tty driver allocations

2013-10-01 Thread Lidza Louina
This patch adds error handling to the tty_driver allocations in dgap_tty_register. Now the code handles the possibility of an alloc_tty_driver, a tty_register_driver, and a brd->SerialDriver->ttys or brd->PrintDriver->ttys allocation failure. Signed-off-by: Lidza Louina --- dri

Re: [PATCH 1/1] staging: dgnc: Remove KERNEL_VERSION check

2013-10-01 Thread Lidza Louina
On Mon, Sep 30, 2013 at 9:46 PM, Greg KH wrote: > On Fri, Sep 27, 2013 at 07:11:47AM -0400, Lidza Louina wrote: >> On Fri, Sep 27, 2013 at 5:50 AM, Sachin Kamat >> wrote: >> > This check is not required. >> > >> > Signed-off-by: Sachin Kamat >

Re: [PATCH v3 1/2] staging: dgap: tty.c: adds error handing in tty driver allocations

2013-10-02 Thread Lidza Louina
On Tue, Oct 1, 2013 at 1:59 PM, Dan Carpenter wrote: > On Tue, Oct 01, 2013 at 12:32:06PM -0400, Lidza Louina wrote: >> On Tue, Oct 1, 2013 at 12:09 PM, Dan Carpenter >> wrote: >> > On Tue, Oct 01, 2013 at 12:00:23PM -0400, Lidza Louina wrote: >> >> @@ -30

Re: [PATCH v4 1/2] staging: dgap: tty.c: adds error handing in tty driver allocations

2013-10-03 Thread Lidza Louina
On Thu, Oct 3, 2013 at 1:23 AM, Greg KH wrote: > On Tue, Oct 01, 2013 at 12:54:20PM -0400, Lidza Louina wrote: >> + return 0; >> + >> +err_unregister_serial: >> +tty_unregister_driver(brd->SerialDriver); >> +err_free_print_ttys: >

[PATCH] staging: dgnc: changes arguments in sizeof

2013-10-04 Thread Lidza Louina
The arguments that were passed into sizeof were generic. This patch changes this by putting the actual item that we need a size of instead. For example: - kzalloc(sizeof(struct dgnc_board), GFP_KERNEL); + kzalloc(sizeof(*brd), GFP_KERNEL); Signed-off-by: Lidza Louina --- drivers/staging

[PATCH] staging: dgap: uses kzalloc for allocating memory

2013-12-19 Thread Lidza Louina
Originally, this driver created it's own allocating function. This patch removes that function and calls kzalloc directly. This patch affects: - driver.c - driver.h - fep5.c - tty.c Signed-off-by: Lidza Louina --- drivers/staging/dgap/dgap_driver.c | 18 ++ dr

Re: [PATCH 1/8] staging: dgap: Fix trailing whitespace in digi.h

2014-01-17 Thread Lidza Louina
Acked-by: Lidza Louina On Tue, Jan 14, 2014 at 10:40 AM, Masanari Iida wrote: > This patch fixed trailing whitespace error found by > checkpatch.pl in dgap/digi.h > > Signed-off-by: Masanari Iida > --- > drivers/staging/dgap/digi.h | 24 >

Re: [PATCH 1/2] staging: dgap: removes KERNEL_VERSION conditionals

2014-01-17 Thread Lidza Louina
I'll try this patchset again after Masanari Iida's patches are accepted after the merge window. Lidza On Fri, Jan 17, 2014 at 11:01 AM, Lidza Louina wrote: > This patch removes the KERNEL_VERSION conditionals. > The driver is built for this kernel version, so the > conditio

Re: [PATCH] Adds Digi driver to staging.

2013-07-18 Thread Lidza Louina
55AM -0400, Mark Hounschell wrote: >> > >> On 07/17/2013 05:49 PM, Greg KH wrote: >> > >>> On Wed, Jul 17, 2013 at 04:18:01PM -0400, Lidza Louina wrote: >> > >>>> This patch adds the Digi driver to staging. >> > >>>

[PATCH] Updates the list of maintainers for staging/comedi driver.

2013-07-24 Thread Lidza Louina
This patch updates the list of maintainers for the staging/comedi driver. Signed-off-by: Lidza Louina --- MAINTAINERS | 2 +- drivers/staging/comedi/TODO | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index bf61e04..6ddb33d

Re: [PATCH] staging: dgnc - adds dgnc digi driver

2013-08-01 Thread Lidza Louina
i International <http://www.digi.com>. >> >> Signed-off-by: Lidza Louina > > Looks good, but a few minor questions. > > This is the "raw" driver that Mark pointed you at, right? You haven't > changed anything for it yet? Yes, I haven't changed any of

Re: [PATCH v2] staging: dgnc - adds dgnc digi driver

2013-08-12 Thread Lidza Louina
On Mon, Aug 12, 2013 at 2:49 PM, Greg KH wrote: > On Mon, Aug 12, 2013 at 02:39:54PM -0400, Lidza Louina wrote: >> I've just realized I made a mistake by adding the Makefile.inc. > > Oops, I missed that. > >> Is it possible for me to make a fresh set of patches (add

Re: [PATCH v2] staging: dgnc - adds dgnc digi driver

2013-08-12 Thread Lidza Louina
On Mon, Aug 12, 2013 at 2:50 PM, Dan Carpenter wrote: > On Mon, Aug 12, 2013 at 02:39:54PM -0400, Lidza Louina wrote: >> I've just realized I made a mistake by adding the Makefile.inc. >> >> Is it possible for me to make a fresh set of patches (adding the files, >

[PATCH 0/9] staging: dgnc: builds this driver into the kernel

2013-08-12 Thread Lidza Louina
This series of patches builds the DGNC driver into the kernel and fixes small problems caused by the new tty layer's API. The DGNC driver is a TTY Serial Port Driver for the Digi International Neo and Classic PCI based product line by Digi International <http://www.digi.com>. Lidz

[PATCH 1/9] staging: dgnc: removes Makefile.inc

2013-08-12 Thread Lidza Louina
Signed-off-by: Lidza Louina --- drivers/staging/dgnc/Makefile.inc | 133 -- 1 file changed, 133 deletions(-) delete mode 100644 drivers/staging/dgnc/Makefile.inc diff --git a/drivers/staging/dgnc/Makefile.inc b/drivers/staging/dgnc/Makefile.inc deleted file

[PATCH 9/9] staging: dgnc: fixes warning: unused variable ‘buf’

2013-08-12 Thread Lidza Louina
Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_tty.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgnc/dgnc_tty.c index 299453f..f155c8f 100644 --- a/drivers/staging/dgnc/dgnc_tty.c +++ b/drivers/staging/dgnc/dgnc_tty.c @@ -683,7

[PATCH 8/9] staging: dgnc: removes read_cnt, real_raw and rawreadok

2013-08-12 Thread Lidza Louina
Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_tty.c | 154 +++- 1 file changed, 43 insertions(+), 111 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgnc/dgnc_tty.c index 1fbd09f..299453f 100644 --- a/drivers/staging

[PATCH 4/9] staging: dgnc: fixes error: invalid type argument of ‘->’ (have ‘struct ktermios’)

2013-08-12 Thread Lidza Louina
Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_tty.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgnc/dgnc_tty.c index 461e881..63b80e2 100644 --- a/drivers/staging/dgnc/dgnc_tty.c

[PATCH 7/9] staging: dgnc: fixes warning: initialization from incompatible pointer type

2013-08-12 Thread Lidza Louina
Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_tty.c | 37 ++--- 1 file changed, 2 insertions(+), 35 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgnc/dgnc_tty.c index 1e22b4b..1fbd09f 100644 --- a/drivers/staging/dgnc

[PATCH 2/9] staging: dgnc: builds into kernel

2013-08-12 Thread Lidza Louina
Signed-off-by: Lidza Louina --- drivers/staging/Kconfig | 1 + drivers/staging/Makefile | 1 + 2 files changed, 2 insertions(+) diff --git a/drivers/staging/Kconfig b/drivers/staging/Kconfig index 57d8b34..d461590 100644 --- a/drivers/staging/Kconfig +++ b/drivers/staging/Kconfig @@ -142,4

[PATCH 3/9] staging: dgnc: removes proc code

2013-08-12 Thread Lidza Louina
Signed-off-by: Lidza Louina --- drivers/staging/dgnc/Makefile |2 +- drivers/staging/dgnc/dgnc_driver.c |9 - drivers/staging/dgnc/dgnc_proc.c | 1551 3 files changed, 1 insertion(+), 1561 deletions(-) delete mode 100644 drivers/staging/dgnc

[PATCH 5/9] staging: dgnc: fixes error: expected ‘struct tty_port *’ but argument is of type ‘struct tty_struct *’

2013-08-12 Thread Lidza Louina
Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_tty.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgnc/dgnc_tty.c index 63b80e2..a00fe2b 100644 --- a/drivers/staging/dgnc/dgnc_tty.c +++ b/drivers

[PATCH 6/9] staging: dgnc: fixes error: incompatible types when assigning to type ‘struct ktermios *’ from type ‘struct ktermios’

2013-08-12 Thread Lidza Louina
Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_tty.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgnc/dgnc_tty.c index a00fe2b..1e22b4b 100644 --- a/drivers/staging/dgnc/dgnc_tty.c +++ b/drivers/staging/dgnc

Re: [PATCH 2/9] staging: dgnc: builds into kernel

2013-08-12 Thread Lidza Louina
On Mon, Aug 12, 2013 at 5:10 PM, Greg KH wrote: > On Mon, Aug 12, 2013 at 05:05:06PM -0400, Lidza Louina wrote: >> Signed-off-by: Lidza Louina >> --- >> drivers/staging/Kconfig | 1 + >> drivers/staging/Makefile | 1 + >> 2 files changed, 2 insertions(+) >

Re: [PATCH 3/9] staging: dgnc: removes proc code

2013-08-12 Thread Lidza Louina
On Mon, Aug 12, 2013 at 5:11 PM, Greg KH wrote: > On Mon, Aug 12, 2013 at 05:05:07PM -0400, Lidza Louina wrote: >> Signed-off-by: Lidza Louina > > Why? > > You need to supply a "why this patch is needed" type of a description > for something that is non-obvious

Re: [PATCH 1/9] staging: dgnc: removes Makefile.inc

2013-08-12 Thread Lidza Louina
On Mon, Aug 12, 2013 at 5:13 PM, Greg KH wrote: > On Mon, Aug 12, 2013 at 05:05:05PM -0400, Lidza Louina wrote: >> Signed-off-by: Lidza Louina >> --- >> drivers/staging/dgnc/Makefile.inc | 133 >> -- >> 1 file changed, 133 de

Re: [PATCH 6/9] staging: dgnc: fixes error: incompatible types when assigning to type ‘struct ktermios *’ from type ‘struct ktermios’

2013-08-13 Thread Lidza Louina
On Tue, Aug 13, 2013 at 3:30 AM, Dan Carpenter wrote: > > These parenthesis are not needed and look ugly. Alrighty. I'll fix that next version. > > regards, > dan carpenter > ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdri

Re: [PATCH 7/9] staging: dgnc: fixes warning: initialization from incompatible pointer type

2013-08-13 Thread Lidza Louina
On Tue, Aug 13, 2013 at 3:32 AM, Dan Carpenter wrote: > This does several things which are not described. It needs to be > broken into more than one patch as well. Ok, I'll break it up next version. One for changing the ioctl function params, and another for removing the 3 commands.

[PATCH 1/8] staging: dgnc: removes Makefile.inc

2013-08-13 Thread Lidza Louina
The Makefile contains the variables needed for the driver to build. The Makefile.inc isn't needed. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/Makefile.inc | 133 -- 1 file changed, 133 deletions(-) delete mode 100644 drivers/staging

[PATCH 7/8] staging: dgnc: fixes ioctl param list

2013-08-13 Thread Lidza Louina
The declaration for the ioctl function has changed. The previous version of this declaration took struct file *file as a parameter and the new one does not. This patch removes that parameter. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_tty.c | 4 ++-- 1 file changed, 2

[PATCH 5/8] staging: dgnc: dereferences ts var in dgnc_tty_close()

2013-08-13 Thread Lidza Louina
The value tty->termios needed to be dereferenced to be assigned to the variable ts. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_tty.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgnc/dgnc_tty.c in

[PATCH 3/8] staging: dgnc: fixes termios error

2013-08-13 Thread Lidza Louina
This patch fixes this error: invalid type argument of ‘->’ (have ‘struct ktermios’). There were changes in the tty layer's API. Access to the termios flags changed from tty->termios->*flag* to tty->termios.*flag*. Signed-off-by: Lidza Louina --- drivers/staging/dgn

[PATCH 8/8] staging: dgnc: removes read_cnt, real_raw, rawreadok and buf

2013-08-13 Thread Lidza Louina
This patch removes the use of read_cnt, real_raw, buf and rawreadok. The variable buf is never used in the code. The variables rawreadok read_cnt and real_raw don't exist in the new API. Reading the data raw is no longer supported by the tty layer. Signed-off-by: Lidza Louina --- dr

[PATCH 2/8] staging: dgnc: removes proc code

2013-08-13 Thread Lidza Louina
proc API. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/Makefile |2 +- drivers/staging/dgnc/dgnc_driver.c |6 +- drivers/staging/dgnc/dgnc_proc.c | 1551 3 files changed, 4 insertions(+), 1555 deletions(-) delete mode 100644 drivers/st

[PATCH 4/8] staging: dgnc: fixes tty_port and tty_struct params

2013-08-13 Thread Lidza Louina
The functions tty_flip_buffer_push, tty_insert_flip_string, tty_insert_flip_char and tty_buffer_request_room now require a struct of type tty_port instead of struct tty_struct. This patch makes those changes. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_tty.c | 14

[PATCH 6/8] staging: dgcn: removes unnecessary commands in ioctl

2013-08-13 Thread Lidza Louina
The commands TIOCGETP, TCGETS, and TCGETA are not supposed to be seen by the ioctl. This patch removes the switch cases for these commands. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_tty.c | 34 -- 1 file changed, 34 deletions(-) diff --git a

Re: [PATCH 2/8] staging: dgnc: removes proc code

2013-08-13 Thread Lidza Louina
Eek, for some reason, gmail has been sending html, sorry I'm spamming. =[ On Tue, Aug 13, 2013 at 7:20 PM, Dan Carpenter wrote: > On Tue, Aug 13, 2013 at 03:49:52PM -0400, Lidza Louina wrote: >> This patch removes the dgnc_proc.c file, all references to proc functions in >&g

Re: [PATCH 8/8] staging: dgnc: removes read_cnt, real_raw, rawreadok and buf

2013-08-13 Thread Lidza Louina
For some reason, gmail has been sending out html. Sorry for spamming. =[ On Tue, Aug 13, 2013 at 7:27 PM, Dan Carpenter wrote: > On Tue, Aug 13, 2013 at 03:49:58PM -0400, Lidza Louina wrote: >> This patch removes the use of read_cnt, real_raw, buf and rawreadok. The >> variab

Re: [PATCH 2/8] staging: dgnc: removes proc code

2013-08-14 Thread Lidza Louina
On Wed, Aug 14, 2013 at 5:04 PM, Greg KH wrote: > On Tue, Aug 13, 2013 at 03:49:52PM -0400, Lidza Louina wrote: >> This patch removes the dgnc_proc.c file, all references to proc functions in >> dgnc_driver.c and from the Makefile's build. >> >> There were a lot o

[PATCH v2] staging: dgnc: removes proc code

2013-08-14 Thread Lidza Louina
This patch removes the dgnc_proc.c file, all references to proc functions in dgnc_driver.c and from the Makefile's build. Drivers now use sysfs instead of proc. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/Makefile | 2 +- drivers/staging/dgnc/dgnc_driver.c | 9 - 2

Re: [PATCH v2] staging: dgnc: removes proc code

2013-08-14 Thread Lidza Louina
On Wed, Aug 14, 2013 at 7:04 PM, Greg KH wrote: > On Wed, Aug 14, 2013 at 07:01:49PM -0400, Lidza Louina wrote: >> This patch removes the dgnc_proc.c file, all references to proc >> functions in dgnc_driver.c and from the Makefile's build. >> >> Drivers now use sys

Re: [PATCH v3] staging: dgnc: removes proc code

2013-08-15 Thread Lidza Louina
On Wed, Aug 14, 2013 at 9:32 PM, Greg KH wrote: > Can you take this patch and apply it to a clean version of my > staging.git tree on the staging-next branch? > > Can you please rebase on this branch and try sending it out again? How > are you generating this patch? I found the problem, Sachin K

[PATCH v4] staging: dgnc: removes proc code

2013-08-15 Thread Lidza Louina
This patch removes the dgnc_proc.c and dgnc_proc.h files and all references to proc functions in dgnc_driver.c. This also removes proc.h from the include headers in driver.c, mgmt.c and sysfs.c and proc.o from the Makefile. Drivers now use sysfs instead of proc. Signed-off-by: Lidza Louina

Re: [PATCH v4] staging: dgnc: removes proc code

2013-08-15 Thread Lidza Louina
On Thu, Aug 15, 2013 at 1:14 PM, Greg KH wrote: > On Thu, Aug 15, 2013 at 10:28:22AM -0400, Lidza Louina wrote: >> This patch removes the dgnc_proc.c and dgnc_proc.h files and all >> references to proc functions in dgnc_driver.c. This also removes proc.h >> from the includ

[PATCH] staging: dgnc: driver.c: fixes warning about assigning pointer

2013-08-19 Thread Lidza Louina
This patch fixes a warning associated with assigining a pointer in the dgnc_mbuf function. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_driver.c b/drivers/staging/dgnc

[PATCH] staging: dgnc: mgmt.c: removes unused variable

2013-08-19 Thread Lidza Louina
This patch removes the inode variable from the dgnc_mgmt_ioctl function. It's never used in the function. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_mgmt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_mgmt.c b/drivers/staging/dgnc/dgnc_mgmt.c

Re: [PATCH] staging: dgnc: mgmt.c: removes unused variable

2013-08-20 Thread Lidza Louina
On Tue, Aug 20, 2013 at 4:48 AM, Dan Carpenter wrote: > The ifdef here is to support older kernels which don't have the > the ->unlocked_ioctl function pointer. We don't care about older > kernels so we could delete the #else side of this and remove the > #ifdef. > > You can do that in a later pa

[PATCH v2 2/2] staging: dgnc: removes ifdef HAVE_UNLOCKED_IOCTL conditionals

2013-08-20 Thread Lidza Louina
This patch removes the HAVE_UNLOCKED_IOCTL conditional statements from driver.c, mgmt.c and mgmt.h. This was used to support older kernels. It isn't needed now. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_driver.c | 4 drivers/staging/dgnc/dgnc_mgmt.c | 7 +-- dr

[PATCH v2 1/2] staging: dgnc: driver.c: fixes warning about assigning pointer

2013-08-20 Thread Lidza Louina
This patch fixes a warning associated with assigining a pointer in the dgnc_mbuf function. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_driver.c b/drivers/staging/dgnc

Re: [PATCH v2 1/2] staging: dgnc: driver.c: fixes warning about assigning pointer

2013-08-20 Thread Lidza Louina
Ok, I'm sending it now. On Tue, Aug 20, 2013 at 7:27 PM, Greg KH wrote: > On Tue, Aug 20, 2013 at 02:15:35PM -0400, Lidza Louina wrote: >> This patch fixes a warning associated with assigining >> a pointer in the dgnc_mbuf function. >> >> Signed-off-by: Lidza Lo

[PATCH] staging: dgnc: builds into kernel

2013-08-20 Thread Lidza Louina
This patch builds the dgnc driver into the kernel. Signed-off-by: Lidza Louina --- drivers/staging/Kconfig | 2 ++ drivers/staging/Makefile | 1 + 2 files changed, 3 insertions(+) diff --git a/drivers/staging/Kconfig b/drivers/staging/Kconfig index 1665705..0d9d92b 100644 --- a/drivers

[PATCH 03/14] staging: dgnc: driver.c: removes trailing whitespace

2013-08-21 Thread Lidza Louina
This patch removes trailing whitespace in the dgnc_driver.c file. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_driver.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_driver.c b/drivers/staging/dgnc

[PATCH 02/14] staging: dgnc: cls.h: removes trailing whitespace

2013-08-21 Thread Lidza Louina
This patch removes trailing whitespace in the dgnc_cls.h file. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_cls.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_cls.h b/drivers/staging/dgnc/dgnc_cls.h index dca5ea3..a273385

[PATCH 14/14] staging: dgnc: dpacompat.h: removes trailing whitespace

2013-08-21 Thread Lidza Louina
This patch removes trailing whitespace in the dpacompat.h file. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dpacompat.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/dgnc/dpacompat.h b/drivers/staging/dgnc/dpacompat.h index 44379eb..f96963b

[PATCH 01/14] staging: dgnc: cls.c: removes trailing whitespace

2013-08-21 Thread Lidza Louina
This patch removes trailing whitespace in the dgnc_cls.c file. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_cls.c | 48 - 1 file changed, 24 insertions(+), 24 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_cls.c b/drivers/staging/dgnc

[PATCH 12/14] staging: dgnc: tty.c: removes trailing whitespace

2013-08-21 Thread Lidza Louina
This patch removes trailing whitespace in the dgnc_tty.c file. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_tty.c | 178 1 file changed, 89 insertions(+), 89 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgnc

[PATCH 05/14] staging: dgnc: kcompat.h: removes trailing whitespace

2013-08-21 Thread Lidza Louina
This patch removes trailing whitespace in the kcompat.h file. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_kcompat.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_kcompat.h b/drivers/staging/dgnc/dgnc_kcompat.h index a0f5874

[PATCH 09/14] staging: dgnc: sysfs.c: removes trailing whitespace

2013-08-21 Thread Lidza Louina
This patch removes trailing whitespace in the sysfs.c file. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_sysfs.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_sysfs.c b/drivers/staging/dgnc/dgnc_sysfs.c index

[PATCH 13/14] staging: dgnc: digi.h: removes trailing whitespace

2013-08-21 Thread Lidza Louina
This patch removes trailing whitespace in the digi.h file. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/digi.h | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/staging/dgnc/digi.h b/drivers/staging/dgnc/digi.h index ab90382

[PATCH 07/14] staging: dgnc: neo.c: removes trailing whitespace

2013-08-21 Thread Lidza Louina
This patch removes trailing whitespace in the dgnc_neo.c file. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_neo.c | 46 - 1 file changed, 23 insertions(+), 23 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_neo.c b/drivers/staging/dgnc

[PATCH 10/14] staging: dgnc: sysfs.h: removes trailing whitespace

2013-08-21 Thread Lidza Louina
This patch removes trailing whitespace in the dgnc_sysfs.h file. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_sysfs.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_sysfs.h b/drivers/staging/dgnc/dgnc_sysfs.h index fe99110..4b87ce1

[PATCH 04/14] staging: dgnc: driver.h: removes trailing whitespace

2013-08-21 Thread Lidza Louina
This patch removes trailing whitespace in the driver.h file. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_driver.h | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_driver.h b/drivers/staging/dgnc/dgnc_driver.h index

[PATCH 06/14] staging: dgnc: mgmt.c: removes trailing whitespace

2013-08-21 Thread Lidza Louina
This patch removes trailing whitespace in the dgnc_mgmt.c file. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_mgmt.c | 38 +++--- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_mgmt.c b/drivers/staging/dgnc

[PATCH 11/14] staging: dgnc: trace.c: removes trailing whitespace

2013-08-21 Thread Lidza Louina
This patch removes trailing whitespace in the dgnc_trace.c file. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_trace.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_trace.c b/drivers/staging/dgnc/dgnc_trace.c index

[PATCH 08/14] staging: dgnc: neo.h: removes trailing whitespace

2013-08-21 Thread Lidza Louina
This patch removes trailing whitespace in the neo.h file. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_neo.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_neo.h b/drivers/staging/dgnc/dgnc_neo.h index ffb4209..f7f569b

[PATCH] staging: dgnc: tty.c: fixes pointer syntax

2013-08-21 Thread Lidza Louina
This patch fixes the error: "foo* bar" should be "foo *bar". Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_tty.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgnc/dgnc_tty.c index 593

[PATCH 2/5] staging: dgnc: driver.h: fixes code indent error

2013-08-21 Thread Lidza Louina
This patch fixes the error "code indent should use tabs where possible" in dgnc_driver.h. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_driver.h | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_d

[PATCH 4/5] staging: dgnc: sysfs.c: fixes code indent error

2013-08-21 Thread Lidza Louina
This patch fixes the error "code indent should use tabs where possible" in dgnc_sysfs.c. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_sysfs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_sysfs.c b/drivers/st

[PATCH 3/5] staging: dgnc: neo.c: fixes code indent error

2013-08-21 Thread Lidza Louina
This patch fixes the error "code indent should use tabs where possible" in dgnc_neo.c. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_neo.c | 46 - 1 file changed, 23 insertions(+), 23 deletions(-) diff --git a/drivers/staging/dgnc/

[PATCH 1/5] staging: dgnc: driver.c: fixes code indent error

2013-08-21 Thread Lidza Louina
This patch fixes the error "code indent should use tabs where possible" in dgnc_driver.c. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_driver.c | 70 +++--- 1 file changed, 35 insertions(+), 35 deletions(-) diff --git a/drivers/st

[PATCH 5/5] staging: dgnc: tty.c: fixes code indent error

2013-08-21 Thread Lidza Louina
This patch fixes the error "code indent should use tabs where possible" in dgnc_tty.c. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_tty.c | 156 1 file changed, 78 insertions(+), 78 deletions(-) diff --git a/drivers/staging/dgnc/

[PATCH] staging: dgnc: removes CVS code from files

2013-08-21 Thread Lidza Louina
This patch removes the code supporting CVS from its files. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_cls.c| 2 -- drivers/staging/dgnc/dgnc_driver.c | 2 -- drivers/staging/dgnc/dgnc_mgmt.c | 1 - drivers/staging/dgnc/dgnc_neo.c| 2 -- drivers/staging/dgnc/dgnc_pci.h

[PATCH] staging: dgnc: fixes struct declaration

2013-08-21 Thread Lidza Louina
This patch fixes the error: open brace '{' following struct go on the same line. Signed-off-by: Lidza Louina --- drivers/staging/dgnc/dgnc_driver.h | 3 +-- drivers/staging/dgnc/digi.h| 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/st

[PATCH 06/12] staging: dgap: tty.c: fixes termios error

2013-08-21 Thread Lidza Louina
This patch fixes this error: invalid type argument of ‘->’ (have ‘struct ktermios’). Signed-off-by: Lidza Louina --- drivers/staging/dgap/dgap_tty.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/drivers/staging/dgap/dgap_tty.c b/driv

[PATCH 08/12] staging: dgap: tty.c: fixes ioctl param list

2013-08-21 Thread Lidza Louina
The declaration for the ioctl function has changed. The previous version of this declaration took struct file *file as a parameter and the new one does not. This patch removes that parameter. It also removes cases for the commands TIOCGETP TCGETS and TCGETA. Signed-off-by: Lidza Louina

[PATCH 10/12] staging: dgap: mgmt.c: removes unused variable

2013-08-21 Thread Lidza Louina
This patch removes the unused variable page in mgmt.c. Signed-off-by: Lidza Louina --- drivers/staging/dgap/dgap_mgmt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/dgap/dgap_mgmt.c b/drivers/staging/dgap/dgap_mgmt.c index 1147f1d..dce33d4 100644 --- a/drivers/staging/dgap

[PATCH 02/12] staging: dgap: fixes incompatible type error

2013-08-21 Thread Lidza Louina
This patch fixes the error: incompatible types when assigning to type ‘struct ktermios *’ from type ‘struct ktermios’ Signed-off-by: Lidza Louina --- drivers/staging/dgap/dgap_fep5.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/dgap/dgap_fep5.c b/drivers

[PATCH 07/12] staging: dgap: tty.c: removes read_cnt, real_raw and rawreadok

2013-08-21 Thread Lidza Louina
This patch removes the use of read_cnt, real_raw and rawreadok. These variables don't exist in the new API. Reading the data raw is no longer supported by the tty layer. Signed-off-by: Lidza Louina --- drivers/staging/dgap/dgap_tty.c | 47 - 1

  1   2   3   >