Re: [PATCH] Staging: rtl8192u: r8180_93cx6.c r8192U_wx.c fix checkpatch.pl errors and warnings

2014-05-27 Thread DaeSeok Youn
to zero, it doesn't need to set anymore. (I don't know what the 5 means :-) ) It is just my opinion. regards, Daeseok Youn Thanks, Chaitanya On Mon, May 26, 2014 at 11:56 PM, DaeSeok Youn daeseok.y...@gmail.com wrote: Hi, 2014-05-27 14:43 GMT+09:00 Chaitanya Hazarey c...@24.io: Fixed

Re: [PATCH 2/4] staging: dgap: implement proper error handling in dgap_firmware_load()

2014-05-26 Thread DaeSeok Youn
Hi, Dan 2014-05-26 19:50 GMT+09:00 Dan Carpenter : > On Mon, May 26, 2014 at 07:23:47PM +0900, Daeseok Youn wrote: >> When dgap_tty_init() and dgap_tty_register_ports() are failed, >> these are needed to free some memory properly. >> >> It can be handled

[PATCH 4/4] staging: dgap: move tty_port_init() for serial_ports.

2014-05-26 Thread Daeseok Youn
If printer_ports which is allocated after serial_ports is failed to allocate, tty_port_init for serial_ports doesn't need anymore. So move this after allocating memory for printer_ports. Signed-off-by: Daeseok Youn --- drivers/staging/dgap/dgap.c |6 +++--- 1 files changed, 3 insertions

[PATCH 3/4] staging: dgap: remove unneeded kfree() in dgap_tty_register_ports()

2014-05-26 Thread Daeseok Youn
When it failed to allocate for printer_ports, serial_ports can be freed in dgap_tty_uninit(). Signed-off-by: Daeseok Youn --- drivers/staging/dgap/dgap.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap/dgap.c index

[PATCH 2/4] staging: dgap: implement proper error handling in dgap_firmware_load()

2014-05-26 Thread Daeseok Youn
off-by: Daeseok Youn --- drivers/staging/dgap/dgap.c | 21 - 1 files changed, 16 insertions(+), 5 deletions(-) diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap/dgap.c index 60b7d70..db24f70 100644 --- a/drivers/staging/dgap/dgap.c +++ b/drivers/staging/dgap/dga

[PATCH 1/4] staging: dgap: remove unneeded kfree() for ttys in tty_driver

2014-05-26 Thread Daeseok Youn
In destruct_tty_driver() from put_tty_driver() will free the ttys in tty_driver. Signed-off-by: Daeseok Youn --- drivers/staging/dgap/dgap.c |4 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap/dgap.c index 1475532

[PATCH 1/4] staging: dgap: remove unneeded kfree() for ttys in tty_driver

2014-05-26 Thread Daeseok Youn
In destruct_tty_driver() from put_tty_driver() will free the ttys in tty_driver. Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/staging/dgap/dgap.c |4 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap

[PATCH 2/4] staging: dgap: implement proper error handling in dgap_firmware_load()

2014-05-26 Thread Daeseok Youn
-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/staging/dgap/dgap.c | 21 - 1 files changed, 16 insertions(+), 5 deletions(-) diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap/dgap.c index 60b7d70..db24f70 100644 --- a/drivers/staging/dgap/dgap.c +++ b/drivers

[PATCH 3/4] staging: dgap: remove unneeded kfree() in dgap_tty_register_ports()

2014-05-26 Thread Daeseok Youn
When it failed to allocate for printer_ports, serial_ports can be freed in dgap_tty_uninit(). Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/staging/dgap/dgap.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging/dgap/dgap.c b/drivers

[PATCH 4/4] staging: dgap: move tty_port_init() for serial_ports.

2014-05-26 Thread Daeseok Youn
If printer_ports which is allocated after serial_ports is failed to allocate, tty_port_init for serial_ports doesn't need anymore. So move this after allocating memory for printer_ports. Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/staging/dgap/dgap.c |6 +++--- 1 files

Re: [PATCH 2/4] staging: dgap: implement proper error handling in dgap_firmware_load()

2014-05-26 Thread DaeSeok Youn
Hi, Dan 2014-05-26 19:50 GMT+09:00 Dan Carpenter dan.carpen...@oracle.com: On Mon, May 26, 2014 at 07:23:47PM +0900, Daeseok Youn wrote: When dgap_tty_init() and dgap_tty_register_ports() are failed, these are needed to free some memory properly. It can be handled by calling dgap_tty_uninit

[PATCH] drivers: uio_dmem_genirq: Fix memory leak in uio_dmem_genirq_probe()

2014-05-21 Thread Daeseok Youn
be called when uio_register_device() is failed. So it is moved into handling error for uio_register_device(). Signed-off-by: Daeseok Youn --- drivers/uio/uio_dmem_genirq.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/uio/uio_dmem_genirq.c b/drivers/u

[PATCH] drivers: uio_dmem_genirq: Fix memory leak in uio_dmem_genirq_probe()

2014-05-21 Thread Daeseok Youn
into handling error for uio_register_device(). Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/uio/uio_dmem_genirq.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/uio/uio_dmem_genirq.c b/drivers/uio/uio_dmem_genirq.c index 1270f3b..8d0bba4

Re: [PATCH] staging: ozwpan: ozproto.c: Fix for possible null pointer dereference

2014-05-20 Thread DaeSeok Youn
I was already sent for this. :-) https://lkml.org/lkml/2014/5/16/114 Regards, Daeseok Youn. 2014-05-21 6:48 GMT+09:00 Dan Carpenter : > On Tue, May 20, 2014 at 11:28:59PM +0200, Rickard Strandqvist wrote: >> There is otherwise a risk of a possible null pointer dereference. >>

Re: [PATCH] staging: ozwpan: ozproto.c: Fix for possible null pointer dereference

2014-05-20 Thread DaeSeok Youn
I was already sent for this. :-) https://lkml.org/lkml/2014/5/16/114 Regards, Daeseok Youn. 2014-05-21 6:48 GMT+09:00 Dan Carpenter dan.carpen...@oracle.com: On Tue, May 20, 2014 at 11:28:59PM +0200, Rickard Strandqvist wrote: There is otherwise a risk of a possible null pointer dereference

[PATCH V4] staging: dgap: implement error handling in dgap_tty_register()

2014-05-19 Thread Daeseok Youn
are registered normally, and then set TRUE to dgap_major_serial{print}_registered. If one of drivers is failed to register, leave a default value as FALSE. Signed-off-by: Daeseok Youn --- V2: rebased on staging-next branch V3: removes unneeded information in e-mail V4: removes labels for kfree() calls

Re: [PATCH V3] staging: dgap: implement error handling in dgap_tty_register()

2014-05-19 Thread DaeSeok Youn
2014-05-19 18:39 GMT+09:00 DaeSeok Youn : > 2014-05-19 17:02 GMT+09:00 Dan Carpenter : >> Nice, but it needs a couple style improvements below. >> >> On Mon, May 19, 2014 at 11:10:30AM +0900, Daeseok Youn wrote: >>> + brd->dgap_major_serial_registered = TRUE; &

Re: [PATCH V3] staging: dgap: implement error handling in dgap_tty_register()

2014-05-19 Thread DaeSeok Youn
2014-05-19 17:02 GMT+09:00 Dan Carpenter : > Nice, but it needs a couple style improvements below. > > On Mon, May 19, 2014 at 11:10:30AM +0900, Daeseok Youn wrote: >> + brd->dgap_major_serial_registered = TRUE; >> + dgap_boards_by_major[brd->serial_driver-

Re: [PATCH V3] staging: dgap: implement error handling in dgap_tty_register()

2014-05-19 Thread DaeSeok Youn
2014-05-19 17:02 GMT+09:00 Dan Carpenter dan.carpen...@oracle.com: Nice, but it needs a couple style improvements below. On Mon, May 19, 2014 at 11:10:30AM +0900, Daeseok Youn wrote: + brd-dgap_major_serial_registered = TRUE; + dgap_boards_by_major[brd-serial_driver-major] = brd

Re: [PATCH V3] staging: dgap: implement error handling in dgap_tty_register()

2014-05-19 Thread DaeSeok Youn
2014-05-19 18:39 GMT+09:00 DaeSeok Youn daeseok.y...@gmail.com: 2014-05-19 17:02 GMT+09:00 Dan Carpenter dan.carpen...@oracle.com: Nice, but it needs a couple style improvements below. On Mon, May 19, 2014 at 11:10:30AM +0900, Daeseok Youn wrote: + brd-dgap_major_serial_registered = TRUE

[PATCH V4] staging: dgap: implement error handling in dgap_tty_register()

2014-05-19 Thread Daeseok Youn
are registered normally, and then set TRUE to dgap_major_serial{print}_registered. If one of drivers is failed to register, leave a default value as FALSE. Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- V2: rebased on staging-next branch V3: removes unneeded information in e-mail V4: removes

[PATCH V3] staging: dgap: implement error handling in dgap_tty_register()

2014-05-18 Thread Daeseok Youn
are registered normally, and then set TRUE to dgap_major_serial{print}_registered. If one of drivers is failed to register, leave a default value as FALSE. Signed-off-by: Daeseok Youn --- V2: rebased on staging-next branch V3: removes unneeded information in e-mail drivers/staging/dgap/dgap.c | 49

[PATCH V2] staging: dgap: implement error handling in dgap_tty_register()

2014-05-18 Thread Daeseok Youn
>From 3acdda98e73a153a5fb32c2cd401d2dde1653b0a Mon Sep 17 00:00:00 2001 From: Daeseok Youn Date: Fri, 25 Apr 2014 16:04:59 +0900 Subject: [PATCH] staging: dgap: implement error handling in dgap_tty_register() - alloc_tty_driver() is deprecated so it is changed to tty_alloc_driver() - Point

[PATCH V2] staging: dgap: implement error handling in dgap_tty_register()

2014-05-18 Thread Daeseok Youn
From 3acdda98e73a153a5fb32c2cd401d2dde1653b0a Mon Sep 17 00:00:00 2001 From: Daeseok Youn daeseok.y...@gmail.com Date: Fri, 25 Apr 2014 16:04:59 +0900 Subject: [PATCH] staging: dgap: implement error handling in dgap_tty_register() - alloc_tty_driver() is deprecated so it is changed

[PATCH V3] staging: dgap: implement error handling in dgap_tty_register()

2014-05-18 Thread Daeseok Youn
are registered normally, and then set TRUE to dgap_major_serial{print}_registered. If one of drivers is failed to register, leave a default value as FALSE. Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- V2: rebased on staging-next branch V3: removes unneeded information in e-mail drivers

Re: [PATCH] staging: dgap: implement error handling in dgap_tty_register()

2014-05-17 Thread DaeSeok Youn
2014-05-17 8:09 GMT+09:00, Greg KH : > On Fri, Apr 25, 2014 at 04:04:59PM +0900, Daeseok Youn wrote: >> - alloc_tty_driver() is deprecated so it is changed to >> tty_alloc_driver() >> - Pointers which are allocated by alloc_tty_driver() and kzalloc() >> can be NU

Re: [PATCH] staging: dgap: implement error handling in dgap_tty_register()

2014-05-17 Thread DaeSeok Youn
2014-05-17 8:09 GMT+09:00, Greg KH gre...@linuxfoundation.org: On Fri, Apr 25, 2014 at 04:04:59PM +0900, Daeseok Youn wrote: - alloc_tty_driver() is deprecated so it is changed to tty_alloc_driver() - Pointers which are allocated by alloc_tty_driver() and kzalloc() can be NULL so it need

Re: [PATCH] staging: dgap: implement error handling in dgap_tty_register()

2014-05-16 Thread DaeSeok Youn
2014-05-16 18:52 GMT+09:00 Dan Carpenter : > On Fri, May 16, 2014 at 06:40:28PM +0900, DaeSeok Youn wrote: >> Hi, >> >> This patch haven't been updated over the past 3 weeks. >> Will you check for me? > > Greg was trying to travel around the world in on

Re: [PATCH] staging: dgap: implement error handling in dgap_tty_register()

2014-05-16 Thread DaeSeok Youn
Hi, This patch haven't been updated over the past 3 weeks. Will you check for me? Regards, Daeseok Youn. 2014-04-28 8:21 GMT+09:00 DaeSeok Youn : > OK. I'll make my patch based on Mark's patch. > Thanks. > > Daeseok Youn. > > 2014-04-27 3:48 GMT+09:00 Dan Carpenter : >&

[PATCH V2] video : remove redundant error check

2014-05-16 Thread Daeseok Youn
It doesn't need to check "err" for printing info. And also use pr_info instead of printk. Signed-off-by: Daeseok Youn --- V2: removes unneeded lines for sending a patch drivers/video/fbdev/i810/i810_main.c |7 +++ 1 files changed, 3 insertions(+), 4 deletions(-) diff --git

[PATCH V2] staging: ozwpan: remove redundant NULL check for devs

2014-05-16 Thread Daeseok Youn
The "devs" is a pointer to g_net_dev in ozmain.c. g_net_dev has a default value as empty string. So "devs" cannot be NULL, removes NULL check for "devs". Signed-off-by: Daeseok Youn --- V2: remove unneeded lines for sending a patch. drivers/staging/ozwpan/ozprot

Re: [PATCH] staging: ozwpan: remove redundant NULL check for devs

2014-05-16 Thread DaeSeok Youn
Hi, Dan 2014-05-16 18:03 GMT+09:00 Dan Carpenter : > On Fri, May 16, 2014 at 05:08:10PM +0900, Daeseok Youn wrote: >> >From 1cebea7a40490d0d0b122eb444a105d424af42e6 Mon Sep 17 00:00:00 2001 >> From: Daeseok Youn >> Date: Fri, 16 May 2014 17:02:04 +0900 >> Subject: [

[PATCH] staging: ozwpan: remove redundant NULL check for devs

2014-05-16 Thread Daeseok Youn
>From 1cebea7a40490d0d0b122eb444a105d424af42e6 Mon Sep 17 00:00:00 2001 From: Daeseok Youn Date: Fri, 16 May 2014 17:02:04 +0900 Subject: [PATCH] staging: ozwpan: remove redundant NULL check for devs The "devs" is a pointer to g_net_dev in ozmain.c. g_net_dev has a default value as

[PATCH] staging: ozwpan: remove redundant NULL check for devs

2014-05-16 Thread Daeseok Youn
From 1cebea7a40490d0d0b122eb444a105d424af42e6 Mon Sep 17 00:00:00 2001 From: Daeseok Youn daeseok.y...@gmail.com Date: Fri, 16 May 2014 17:02:04 +0900 Subject: [PATCH] staging: ozwpan: remove redundant NULL check for devs The devs is a pointer to g_net_dev in ozmain.c. g_net_dev has a default

Re: [PATCH] staging: ozwpan: remove redundant NULL check for devs

2014-05-16 Thread DaeSeok Youn
Hi, Dan 2014-05-16 18:03 GMT+09:00 Dan Carpenter dan.carpen...@oracle.com: On Fri, May 16, 2014 at 05:08:10PM +0900, Daeseok Youn wrote: From 1cebea7a40490d0d0b122eb444a105d424af42e6 Mon Sep 17 00:00:00 2001 From: Daeseok Youn daeseok.y...@gmail.com Date: Fri, 16 May 2014 17:02:04 +0900

[PATCH V2] staging: ozwpan: remove redundant NULL check for devs

2014-05-16 Thread Daeseok Youn
The devs is a pointer to g_net_dev in ozmain.c. g_net_dev has a default value as empty string. So devs cannot be NULL, removes NULL check for devs. Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- V2: remove unneeded lines for sending a patch. drivers/staging/ozwpan/ozproto.c |2 +- 1

[PATCH V2] video : remove redundant error check

2014-05-16 Thread Daeseok Youn
It doesn't need to check err for printing info. And also use pr_info instead of printk. Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- V2: removes unneeded lines for sending a patch drivers/video/fbdev/i810/i810_main.c |7 +++ 1 files changed, 3 insertions(+), 4 deletions

Re: [PATCH] staging: dgap: implement error handling in dgap_tty_register()

2014-05-16 Thread DaeSeok Youn
Hi, This patch haven't been updated over the past 3 weeks. Will you check for me? Regards, Daeseok Youn. 2014-04-28 8:21 GMT+09:00 DaeSeok Youn daeseok.y...@gmail.com: OK. I'll make my patch based on Mark's patch. Thanks. Daeseok Youn. 2014-04-27 3:48 GMT+09:00 Dan Carpenter dan.carpen

Re: [PATCH] staging: dgap: implement error handling in dgap_tty_register()

2014-05-16 Thread DaeSeok Youn
2014-05-16 18:52 GMT+09:00 Dan Carpenter dan.carpen...@oracle.com: On Fri, May 16, 2014 at 06:40:28PM +0900, DaeSeok Youn wrote: Hi, This patch haven't been updated over the past 3 weeks. Will you check for me? Greg was trying to travel around the world in one month like Jackie Chan

[PATCH] video : remove redundant error check

2014-05-15 Thread Daeseok Youn
>From 060757f81f85babf393cc76714d49af25af7791d Mon Sep 17 00:00:00 2001 From: Daeseok Youn Date: Thu, 15 May 2014 16:51:35 +0900 Subject: [PATCH] video : remove redundant error check It doesn't need to check "err" for printing info. And also use pr_info instead of printk. Signed-of

[PATCH] video : remove redundant error check

2014-05-15 Thread Daeseok Youn
From 060757f81f85babf393cc76714d49af25af7791d Mon Sep 17 00:00:00 2001 From: Daeseok Youn daeseok.y...@gmail.com Date: Thu, 15 May 2014 16:51:35 +0900 Subject: [PATCH] video : remove redundant error check It doesn't need to check err for printing info. And also use pr_info instead of printk

[PATCH 2/5 V2] staging: cxt1e1: remove dead code in musycc.c

2014-05-09 Thread Daeseok Youn
Removes "#if 0" blocks. And the musycc_dump_rxbuffer_ring(ch, 0) which is commented out puts in RLD_DEBUG block and uncommented. Because this function may be used for debugging. Signed-off-by: Daeseok Youn --- V2 : fix the changelog, added a log that musycc_dump_rxbuffer_ring() is u

Re: [PATCH 2/5] staging: cxt1e1: remove dead code in musycc.c

2014-05-09 Thread DaeSeok Youn
2014-05-09 23:15 GMT+09:00, Dan Carpenter : > On Fri, May 09, 2014 at 11:09:35PM +0900, DaeSeok Youn wrote: >> 2014-05-09 19:51 GMT+09:00, Dan Carpenter : >> > On Fri, May 09, 2014 at 07:06:06PM +0900, Daeseok Youn wrote: >> >> Removes "#if 0" blocks.

Re: [PATCH 2/5] staging: cxt1e1: remove dead code in musycc.c

2014-05-09 Thread DaeSeok Youn
2014-05-09 19:51 GMT+09:00, Dan Carpenter : > On Fri, May 09, 2014 at 07:06:06PM +0900, Daeseok Youn wrote: >> Removes "#if 0" blocks. >> >> Signed-off-by: Daeseok Youn >> --- >> Dan, >> I decided to leave musycc_dump_rxbuffer_ring(ch, 0) which is

Re: [PATCH 1/5] staging: cxt1e1: Fix no spaces at the start of a line in musycc.c

2014-05-09 Thread DaeSeok Youn
2014-05-09 19:42 GMT+09:00, Dan Carpenter : > On Fri, May 09, 2014 at 07:02:20PM +0900, Daeseok Youn wrote: >> clean up checkpatch.pl warning: >> WARNING: please no spaces at the start of a line in >> >> Signed-off-by: Daeseok Youn >> --- > >&

[PATCH 3/5] staging: cxt1e1: remove redundant curly braces in musycc.c

2014-05-09 Thread Daeseok Youn
Signed-off-by: Daeseok Youn --- I'm not sure about removing "RLD_DUMP_BUFDATA" and "RLD_DEBUG" definitions. I think these can be defined temporarily by developer who want to debugging this module. drivers/staging/cxt1e1/musycc.c | 211 +++-

[PATCH 4/5] staging: cxt1e1: Fix line length over 80 characters in musycc.c

2014-05-09 Thread Daeseok Youn
clean up checkpatch.pl warning: WARNING: Line length over 80 characters Signed-off-by: Daeseok Youn --- drivers/staging/cxt1e1/musycc.c | 312 --- 1 files changed, 196 insertions(+), 116 deletions(-) diff --git a/drivers/staging/cxt1e1/musycc.c b/drivers

[PATCH 5/5] staging: cxt1e1: remove set a value to static variable

2014-05-09 Thread Daeseok Youn
cleanup checkpatch.pl error: ERROR: do not initialise statics to 0 or NULL Signed-off-by: Daeseok Youn --- drivers/staging/cxt1e1/musycc.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/cxt1e1/musycc.c b/drivers/staging/cxt1e1/musycc.c index

[PATCH 2/5] staging: cxt1e1: remove dead code in musycc.c

2014-05-09 Thread Daeseok Youn
Removes "#if 0" blocks. Signed-off-by: Daeseok Youn --- Dan, I decided to leave musycc_dump_rxbuffer_ring(ch, 0) which is commented out and make a block as "RLD_DEBUG". Because i think this block need to debug with define "RLD_DEBUG". If I am wrong, let me

Re: [PATCH 2/3] Staging: cxt1e1: Fix line length over 80 characters in musycc.c

2014-05-09 Thread DaeSeok Youn
2014-05-09 17:07 GMT+09:00 Dan Carpenter : > This patch is ok, but here are some additional hints. > > On Fri, May 09, 2014 at 10:39:48AM +0900, Daeseok Youn wrote: >> @@ -1100,7 +1102,9 @@ musycc_bh_rx_eom(mpi_t *pi, int gchan) >>*/ >>

Re: [PATCH 1/3] staging: cxt1e1: Fix no spaces at the start of a line in musycc.c

2014-05-09 Thread DaeSeok Youn
2014-05-09 16:59 GMT+09:00 Dan Carpenter : > On Fri, May 09, 2014 at 10:38:43AM +0900, Daeseok Youn wrote: >> clean up checkpatch.pl warning: >> WARNING: please no spaces at the start of a line in >> >> Signed-off-by: Daeseok Youn >> --- >> This patch

Re: [PATCH 1/3] staging: cxt1e1: Fix no spaces at the start of a line in musycc.c

2014-05-09 Thread DaeSeok Youn
2014-05-09 16:59 GMT+09:00 Dan Carpenter dan.carpen...@oracle.com: On Fri, May 09, 2014 at 10:38:43AM +0900, Daeseok Youn wrote: clean up checkpatch.pl warning: WARNING: please no spaces at the start of a line in Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- This patch includes

Re: [PATCH 2/3] Staging: cxt1e1: Fix line length over 80 characters in musycc.c

2014-05-09 Thread DaeSeok Youn
2014-05-09 17:07 GMT+09:00 Dan Carpenter dan.carpen...@oracle.com: This patch is ok, but here are some additional hints. On Fri, May 09, 2014 at 10:39:48AM +0900, Daeseok Youn wrote: @@ -1100,7 +1102,9 @@ musycc_bh_rx_eom(mpi_t *pi, int gchan

[PATCH 2/5] staging: cxt1e1: remove dead code in musycc.c

2014-05-09 Thread Daeseok Youn
Removes #if 0 blocks. Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- Dan, I decided to leave musycc_dump_rxbuffer_ring(ch, 0) which is commented out and make a block as RLD_DEBUG. Because i think this block need to debug with define RLD_DEBUG. If I am wrong, let me know. drivers

[PATCH 4/5] staging: cxt1e1: Fix line length over 80 characters in musycc.c

2014-05-09 Thread Daeseok Youn
clean up checkpatch.pl warning: WARNING: Line length over 80 characters Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/staging/cxt1e1/musycc.c | 312 --- 1 files changed, 196 insertions(+), 116 deletions(-) diff --git a/drivers/staging

[PATCH 5/5] staging: cxt1e1: remove set a value to static variable

2014-05-09 Thread Daeseok Youn
cleanup checkpatch.pl error: ERROR: do not initialise statics to 0 or NULL Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/staging/cxt1e1/musycc.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/cxt1e1/musycc.c b/drivers/staging

[PATCH 3/5] staging: cxt1e1: remove redundant curly braces in musycc.c

2014-05-09 Thread Daeseok Youn
Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- I'm not sure about removing RLD_DUMP_BUFDATA and RLD_DEBUG definitions. I think these can be defined temporarily by developer who want to debugging this module. drivers/staging/cxt1e1/musycc.c | 211

Re: [PATCH 1/5] staging: cxt1e1: Fix no spaces at the start of a line in musycc.c

2014-05-09 Thread DaeSeok Youn
2014-05-09 19:42 GMT+09:00, Dan Carpenter dan.carpen...@oracle.com: On Fri, May 09, 2014 at 07:02:20PM +0900, Daeseok Youn wrote: clean up checkpatch.pl warning: WARNING: please no spaces at the start of a line in Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- -#ifdef

Re: [PATCH 2/5] staging: cxt1e1: remove dead code in musycc.c

2014-05-09 Thread DaeSeok Youn
2014-05-09 19:51 GMT+09:00, Dan Carpenter dan.carpen...@oracle.com: On Fri, May 09, 2014 at 07:06:06PM +0900, Daeseok Youn wrote: Removes #if 0 blocks. Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- Dan, I decided to leave musycc_dump_rxbuffer_ring(ch, 0) which is commented out

Re: [PATCH 2/5] staging: cxt1e1: remove dead code in musycc.c

2014-05-09 Thread DaeSeok Youn
2014-05-09 23:15 GMT+09:00, Dan Carpenter dan.carpen...@oracle.com: On Fri, May 09, 2014 at 11:09:35PM +0900, DaeSeok Youn wrote: 2014-05-09 19:51 GMT+09:00, Dan Carpenter dan.carpen...@oracle.com: On Fri, May 09, 2014 at 07:06:06PM +0900, Daeseok Youn wrote: Removes #if 0 blocks. Signed

[PATCH 2/5 V2] staging: cxt1e1: remove dead code in musycc.c

2014-05-09 Thread Daeseok Youn
Removes #if 0 blocks. And the musycc_dump_rxbuffer_ring(ch, 0) which is commented out puts in RLD_DEBUG block and uncommented. Because this function may be used for debugging. Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- V2 : fix the changelog, added a log

[PATCH 3/3] staging: cxt1e1: remove set a value to static variable

2014-05-08 Thread Daeseok Youn
cleanup checkpatch.pl error: ERROR: do not initialise statics to 0 or NULL Signed-off-by: Daeseok Youn --- drivers/staging/cxt1e1/musycc.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/cxt1e1/musycc.c b/drivers/staging/cxt1e1/musycc.c index

[PATCH 2/3] Staging: cxt1e1: Fix line length over 80 characters in musycc.c

2014-05-08 Thread Daeseok Youn
clean up checkpatch.pl warning WARNING: Line length over 80 characters Signed-off-by: Daeseok Youn --- drivers/staging/cxt1e1/musycc.c | 29 +++-- 1 files changed, 19 insertions(+), 10 deletions(-) diff --git a/drivers/staging/cxt1e1/musycc.c b/drivers/staging/cxt1e1

[PATCH] [media] s2255drv: fix memory leak s2255_probe()

2014-05-08 Thread Daeseok Youn
smatch says: drivers/media/usb/s2255/s2255drv.c:2246 s2255_probe() warn: possible memory leak of 'dev' Signed-off-by: Daeseok Youn --- V2: use the same pattern for error handling. drivers/media/usb/s2255/s2255drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

[PATCH] [media] s2255drv: fix memory leak s2255_probe()

2014-05-08 Thread Daeseok Youn
smatch says: drivers/media/usb/s2255/s2255drv.c:2246 s2255_probe() warn: possible memory leak of 'dev' Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- V2: use the same pattern for error handling. drivers/media/usb/s2255/s2255drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions

[PATCH 2/3] Staging: cxt1e1: Fix line length over 80 characters in musycc.c

2014-05-08 Thread Daeseok Youn
clean up checkpatch.pl warning WARNING: Line length over 80 characters Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/staging/cxt1e1/musycc.c | 29 +++-- 1 files changed, 19 insertions(+), 10 deletions(-) diff --git a/drivers/staging/cxt1e1/musycc.c b

[PATCH 3/3] staging: cxt1e1: remove set a value to static variable

2014-05-08 Thread Daeseok Youn
cleanup checkpatch.pl error: ERROR: do not initialise statics to 0 or NULL Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/staging/cxt1e1/musycc.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/cxt1e1/musycc.c b/drivers/staging

[PATCH] staging: bcm: use kzalloc instead of kmalloc/memset

2014-05-07 Thread Daeseok Youn
Signed-off-by: Daeseok Youn --- drivers/staging/bcm/InterfaceDld.c |6 ++ 1 files changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/bcm/InterfaceDld.c b/drivers/staging/bcm/InterfaceDld.c index 005e460..e1925bd 100644 --- a/drivers/staging/bcm/InterfaceDld.c +++ b

[PATCH] staging: bcm: use kzalloc instead of kmalloc/memset

2014-05-07 Thread Daeseok Youn
Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/staging/bcm/InterfaceDld.c |6 ++ 1 files changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/bcm/InterfaceDld.c b/drivers/staging/bcm/InterfaceDld.c index 005e460..e1925bd 100644 --- a/drivers/staging/bcm

Re: [PATCH] [media] s2255drv: fix memory leak s2255_probe()

2014-05-06 Thread DaeSeok Youn
Hello, 2014-05-05 17:38 GMT+09:00 Sakari Ailus : > On Tue, Apr 15, 2014 at 07:54:43PM +0900, DaeSeok Youn wrote: >> Hi, Sakari >> >> 2014-04-15 18:33 GMT+09:00 Sakari Ailus : >> > Hi Daeseok, >> > >> > On Tue, Apr 15, 2014 at 01:49:34PM +0

Re: [PATCH] [media] s2255drv: fix memory leak s2255_probe()

2014-05-06 Thread DaeSeok Youn
Hello, 2014-05-05 17:38 GMT+09:00 Sakari Ailus sakari.ai...@iki.fi: On Tue, Apr 15, 2014 at 07:54:43PM +0900, DaeSeok Youn wrote: Hi, Sakari 2014-04-15 18:33 GMT+09:00 Sakari Ailus sakari.ai...@iki.fi: Hi Daeseok, On Tue, Apr 15, 2014 at 01:49:34PM +0900, Daeseok Youn wrote: smatch

Re: [PATCH] dmapool: remove redundant NULL check for dev in dma_pool_create()

2014-05-01 Thread DaeSeok Youn
2014-05-01 6:19 GMT+09:00, Andrew Morton : > On Tue, 29 Apr 2014 11:53:10 +0900 Daeseok Youn > wrote: > >> "dev" cannot be NULL because it is already checked before >> calling dma_pool_create(). >> >> Signed-off-by: Daeseok Youn >> --- &g

Re: [PATCH] dmapool: remove redundant NULL check for dev in dma_pool_create()

2014-05-01 Thread DaeSeok Youn
2014-05-01 6:19 GMT+09:00, Andrew Morton a...@linux-foundation.org: On Tue, 29 Apr 2014 11:53:10 +0900 Daeseok Youn daeseok.y...@gmail.com wrote: dev cannot be NULL because it is already checked before calling dma_pool_create(). Signed-off-by: Daeseok Youn daeseok.y...@gmail.com

[PATCH] firewire: fix NULL derefencing in fwnet_probe()

2014-04-29 Thread Daeseok Youn
"dev" and "net" are NULL when alloc_netdev() is failed. So just unlock and return an error. Signed-off-by: Daeseok Youn --- drivers/firewire/net.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/firewire/net.c b/drivers/firewire/net.c in

[PATCH] skd: remove redundant null check for kfree()

2014-04-29 Thread Daeseok Youn
smatch says: skd_acquire_msix() info: redundant null check on entries calling kfree() Signed-off-by: Daeseok Youn --- drivers/block/skd_main.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/block/skd_main.c b/drivers/block/skd_main.c index 608532d..fac4c08

[PATCH] skd: remove redundant null check for kfree()

2014-04-29 Thread Daeseok Youn
smatch says: skd_acquire_msix() info: redundant null check on entries calling kfree() Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/block/skd_main.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/block/skd_main.c b/drivers/block/skd_main.c

[PATCH] firewire: fix NULL derefencing in fwnet_probe()

2014-04-29 Thread Daeseok Youn
dev and net are NULL when alloc_netdev() is failed. So just unlock and return an error. Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/firewire/net.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/firewire/net.c b/drivers/firewire/net.c index

[PATCH] dmapool: remove redundant NULL check for dev in dma_pool_create()

2014-04-28 Thread Daeseok Youn
"dev" cannot be NULL because it is already checked before calling dma_pool_create(). Signed-off-by: Daeseok Youn --- If dev can be NULL, it has NULL deferencing when kmalloc_node() is called after enabling CONFIG_NUMA. mm/dmapool.c | 26 +- 1 files

[PATCH] dmapool: remove redundant NULL check for dev in dma_pool_create()

2014-04-28 Thread Daeseok Youn
dev cannot be NULL because it is already checked before calling dma_pool_create(). Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- If dev can be NULL, it has NULL deferencing when kmalloc_node() is called after enabling CONFIG_NUMA. mm/dmapool.c | 26 +- 1 files

Re: [PATCH next-20140424] staging/cxt1e1: sparse warning corrections

2014-04-27 Thread DaeSeok Youn
t; @@ -287,7 +287,7 @@ c4_hdw_init(struct pci_dev *pdev, int found) > return 1; > } > > -status_t __init > +static status_t __init > c4hw_attach_all(void) This function is used in "linux.c" > { > hdw_info_t *hi; And when you send a pa

Re: [PATCH] staging: dgap: implement error handling in dgap_tty_register()

2014-04-27 Thread DaeSeok Youn
OK. I'll make my patch based on Mark's patch. Thanks. Daeseok Youn. 2014-04-27 3:48 GMT+09:00 Dan Carpenter : > On Sat, Apr 26, 2014 at 11:39:38AM +0900, DaeSeok Youn wrote: >> Hi, >> >> please check below my comments. >> >> 2014-04-25 23:41 GMT+09:00 Mark Houn

Re: [PATCH] staging: dgap: implement error handling in dgap_tty_register()

2014-04-27 Thread DaeSeok Youn
OK. I'll make my patch based on Mark's patch. Thanks. Daeseok Youn. 2014-04-27 3:48 GMT+09:00 Dan Carpenter dan.carpen...@oracle.com: On Sat, Apr 26, 2014 at 11:39:38AM +0900, DaeSeok Youn wrote: Hi, please check below my comments. 2014-04-25 23:41 GMT+09:00 Mark Hounschell ma

Re: [PATCH next-20140424] staging/cxt1e1: sparse warning corrections

2014-04-27 Thread DaeSeok Youn
; And when you send a patch for this, please check maintainers with ./scripts/get_maintainer.pl. In case of this patch, my email is in CC list. :-) Regards, Daeseok Youn. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More

Re: [PATCH] staging: dgap: implement error handling in dgap_tty_register()

2014-04-25 Thread DaeSeok Youn
Hi, please check below my comments. 2014-04-25 23:41 GMT+09:00 Mark Hounschell : > On 04/25/2014 08:59 AM, Dan Carpenter wrote: >> On Fri, Apr 25, 2014 at 08:29:41AM -0400, Mark Hounschell wrote: >>> On 04/25/2014 07:02 AM, DaeSeok Youn wrote: >>>> Hi, Dan. >&g

Re: [PATCH] staging: silicom: Add appropriate ifdef conditions in bpctl_mod.c

2014-04-25 Thread DaeSeok Youn
2014-04-26 7:33 GMT+09:00, Greg KH : > On Mon, Apr 07, 2014 at 12:03:41PM +0900, Daeseok Youn wrote: >> >> The bp_proc_create() can be called if BP_PROC_SUPPORT is defined. >> >> Signed-off-by: Daeseok Youn >> --- >> drivers/staging/silicom/bpctl_mo

Re: [PATCH] staging: dgap: implement error handling in dgap_tty_register()

2014-04-25 Thread DaeSeok Youn
Hi, Dan. 2014-04-25 18:26 GMT+09:00 Dan Carpenter : > Mark, maybe you should add yourself to the MAINTAINERS entry for this > driver? > > On Fri, Apr 25, 2014 at 04:04:59PM +0900, Daeseok Youn wrote: >> @@ -1263,7 +1277,8 @@ static int dgap_tty_register(s

[PATCH] staging: dgap: implement error handling in dgap_tty_register()

2014-04-25 Thread Daeseok Youn
- alloc_tty_driver() is deprecated so it is changed to tty_alloc_driver() - Pointers which are allocated by alloc_tty_driver() and kzalloc() can be NULL so it need to check NULL for them. - If one of those is failed, it need to add proper handler for avoiding memory leak. Signed-off-by: Daeseok

[PATCH] staging: dgap: implement error handling in dgap_tty_register()

2014-04-25 Thread Daeseok Youn
- alloc_tty_driver() is deprecated so it is changed to tty_alloc_driver() - Pointers which are allocated by alloc_tty_driver() and kzalloc() can be NULL so it need to check NULL for them. - If one of those is failed, it need to add proper handler for avoiding memory leak. Signed-off-by: Daeseok

Re: [PATCH] staging: dgap: implement error handling in dgap_tty_register()

2014-04-25 Thread DaeSeok Youn
Hi, Dan. 2014-04-25 18:26 GMT+09:00 Dan Carpenter dan.carpen...@oracle.com: Mark, maybe you should add yourself to the MAINTAINERS entry for this driver? On Fri, Apr 25, 2014 at 04:04:59PM +0900, Daeseok Youn wrote: @@ -1263,7 +1277,8 @@ static int dgap_tty_register(struct board_t *brd

Re: [PATCH] staging: silicom: Add appropriate ifdef conditions in bpctl_mod.c

2014-04-25 Thread DaeSeok Youn
2014-04-26 7:33 GMT+09:00, Greg KH gre...@linuxfoundation.org: On Mon, Apr 07, 2014 at 12:03:41PM +0900, Daeseok Youn wrote: The bp_proc_create() can be called if BP_PROC_SUPPORT is defined. Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/staging/silicom/bpctl_mod.c |5

Re: [PATCH] staging: dgap: implement error handling in dgap_tty_register()

2014-04-25 Thread DaeSeok Youn
Hi, please check below my comments. 2014-04-25 23:41 GMT+09:00 Mark Hounschell ma...@compro.net: On 04/25/2014 08:59 AM, Dan Carpenter wrote: On Fri, Apr 25, 2014 at 08:29:41AM -0400, Mark Hounschell wrote: On 04/25/2014 07:02 AM, DaeSeok Youn wrote: Hi, Dan. 2014-04-25 18:26 GMT+09:00 Dan

Re: [PATCH] staging: silicom: remove redundant pci_get_drvdata() call

2014-04-23 Thread DaeSeok Youn
2014-04-23 17:35 GMT+09:00, Dan Carpenter : > On Wed, Apr 23, 2014 at 05:18:42PM +0900, Daeseok Youn wrote: >> The pci_get_drvdata() and checking NULL for dev are >> called twice in while loop in is_bypass_dev(). >> >> Signed-off-by: Daeseok Youn >> ---

[PATCH] staging: silicom: remove redundant pci_get_drvdata() call

2014-04-23 Thread Daeseok Youn
The pci_get_drvdata() and checking NULL for dev are called twice in while loop in is_bypass_dev(). Signed-off-by: Daeseok Youn --- This patch has an warning from checkpatch.pl. checkpatch.pl warning: WARNING: Too many leading tabs - consider code refactoring drivers/staging/silicom/bypasslib

[PATCH] staging: silicom: remove redundant pci_get_drvdata() call

2014-04-23 Thread Daeseok Youn
The pci_get_drvdata() and checking NULL for dev are called twice in while loop in is_bypass_dev(). Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- This patch has an warning from checkpatch.pl. checkpatch.pl warning: WARNING: Too many leading tabs - consider code refactoring drivers

Re: [PATCH] staging: silicom: remove redundant pci_get_drvdata() call

2014-04-23 Thread DaeSeok Youn
2014-04-23 17:35 GMT+09:00, Dan Carpenter dan.carpen...@oracle.com: On Wed, Apr 23, 2014 at 05:18:42PM +0900, Daeseok Youn wrote: The pci_get_drvdata() and checking NULL for dev are called twice in while loop in is_bypass_dev(). Signed-off-by: Daeseok Youn daeseok.y...@gmail.com

Re: [PATCH] fbdev: fix possible NULL pointer derefernce

2014-04-18 Thread DaeSeok Youn
Hello, 2014-04-17 9:00 GMT+09:00 DaeSeok Youn : > Hello, > > 2014-04-16 21:38 GMT+09:00 Jean-Christophe PLAGNIOL-VILLARD > : >> >> On Apr 16, 2014, at 5:40 PM, Daeseok Youn wrote: >> >>> >>> The spec->modedb can be NULL by fb_create_modedb(). &g

Re: [PATCH] fbdev: fix possible NULL pointer derefernce

2014-04-18 Thread DaeSeok Youn
Hello, 2014-04-17 9:00 GMT+09:00 DaeSeok Youn daeseok.y...@gmail.com: Hello, 2014-04-16 21:38 GMT+09:00 Jean-Christophe PLAGNIOL-VILLARD plagn...@jcrosoft.com: On Apr 16, 2014, at 5:40 PM, Daeseok Youn daeseok.y...@gmail.com wrote: The spec-modedb can be NULL by fb_create_modedb

[PATCH V4] staging: cxt1e1: replace OS_kmalloc/OS_kfree with kzalloc/kfree

2014-04-17 Thread Daeseok Youn
Replace OS_kmalloc/OS_kfree with kzalloc/kfree. And also some allocation doesn't need to use GFP_DMA so just use GFP_KERNEL. c4_new() function is never called, remove it. Signed-off-by: Daeseok Youn --- V4: update patch description, OS_kmalloc should be replaced with kzalloc for zeroed out

[PATCH V2] workqueue: jumps to use_dfl_pwq if the target cpumask is equal wq's

2014-04-17 Thread Daeseok Youn
by removing the conditional. This doesn't make any functional difference. Signed-off-by: Daeseok Youn --- V2: update patch description and comment of code kernel/workqueue.c |8 ++-- 1 files changed, 2 insertions(+), 6 deletions(-) diff --git a/kernel/workqueue.c b/kernel/workqueue.c index

[PATCH V2] workqueue: jumps to use_dfl_pwq if the target cpumask is equal wq's

2014-04-17 Thread Daeseok Youn
by removing the conditional. This doesn't make any functional difference. Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- V2: update patch description and comment of code kernel/workqueue.c |8 ++-- 1 files changed, 2 insertions(+), 6 deletions(-) diff --git a/kernel/workqueue.c b

<    3   4   5   6   7   8   9   10   11   12   >