On Thu, Oct 09, 2014 at 01:40:11PM +0900, Daeseok Youn wrote:
The dgap_init_module() need to unwind for cleanup variables properly.
Because dgap_init_module() calls dgap_cleanup_module() for freeing
variables but this function is possible to free variables
which are not allocated.
On Thu, Oct 09, 2014 at 05:03:07PM +0800, Jia He wrote:
Signed-off-by: Jia He hejia...@gmail.com
This fixes all coding style error problems at rtl8188eu which
makes checkpatch.pl unhappy
your commit message should be above your Signed-off line.
mention what type of error you are fixing.
you
On Thu, Oct 09, 2014 at 06:25:10PM +0800, WANG Chao wrote:
This fixes the following sparse error:
drivers/staging/lustre/lnet/klnds/socklnd/socklnd_lib-linux.c:393:9: error:
incompatible types in comparison expression (different address spaces)
Signed-off-by: WANG Chao
On Fri, Oct 10, 2014 at 11:09:20AM +0400, Igor Bogomazov wrote:
checkpatch.pl tiny fix
get rid of 2 warnings and 2 errors for hal/fw.c:
WARNING: braces {} are not necessary for single statement blocks
WARNING: line over 80 characters
ERROR: space required before the open brace '{'
ERROR:
we should be using DMA API rather than using bus specific DMA API.
converted the occurrence of pci_map_*() to dma_map_*(),
and at the same time used the dma_mapping_error().
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
drivers/staging/rtl8192e/rtl8192e/r8192E_dev.c | 12
On Sat, Oct 25, 2014 at 01:05:13AM +0100, Paul McQuade wrote:
ERROR: that open brace { should be on the previous line
Signed-off-by: Paul McQuade paulmcq...@gmail.com
---
drivers/staging/rtl8723au/core/rtw_ap.c | 130
++--
1 file changed, 40 insertions(+), 90
On Sat, Oct 25, 2014 at 12:22:32AM +0100, Paul McQuade wrote:
ERROR: that open brace { should be on the previous line
patch is corrupt.
thanks
sudip
Signed-off-by: Paul McQuade paulmcq...@gmail.com
---
drivers/staging/rtl8723au/core/rtw_ap.c | 137
++--
1
unused variables are removed. These variables were only assigned some
values and after that they were never being used. So they are safe to
be removed , and it has been build tested.
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
drivers/staging/rtl8712/hal_init.c| 9
On Mon, Oct 27, 2014 at 03:12:57PM +0300, Dan Carpenter wrote:
On Mon, Oct 27, 2014 at 05:22:21PM +0530, Sudip Mukherjee wrote:
On Mon, Oct 27, 2014 at 02:13:12PM +0300, Dan Carpenter wrote:
On Sun, Oct 26, 2014 at 06:03:33PM +0530, Sudip Mukherjee wrote:
@@ -262,7 +257,7 @@ static u8
On Tue, Oct 28, 2014 at 09:07:00PM +0100, Mikael Svantesson wrote:
Signed-off-by: Mikael Svantesson mik...@distopic.net
still the same problem.
ERROR: patch seems to be corrupt (line wrapped?)
---
drivers/staging/skein/skein_api.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
to be that Thunderbird thrashes empty lines when
double clicking the e-mail in Drafts or selecting Edit as new.
When forwarding the e-mail it doesn't seem to affect the empty lines,
very weird behaviour.
Regards,
Mikael
On 2014-10-29 06:25, Sudip Mukherjee wrote:
On Tue, Oct 28, 2014 at 09:07:00PM
On Wed, Oct 29, 2014 at 04:47:25PM +0800, Greg Kroah-Hartman wrote:
On Wed, Oct 22, 2014 at 02:24:46PM +0530, Sudip Mukherjee wrote:
we should be using DMA API rather than using bus specific DMA API.
Why?
as given in the DMA-API-HOWTO.txt :
the DMA API works with any bus independent
On Fri, Oct 31, 2014 at 10:06:46PM +0100, Mariusz Gorski wrote:
Remove Lindent and checkpatch.pl tasks from TODO file as the first one
is obsolete and the other one is already done.
when i checked panel.c with checkpatch and with --strict option, i got 20
checks.
thanks
sudip
On Fri, Oct 31, 2014 at 11:08:34PM +0100, Konrad Zapalowicz wrote:
On 10/31, Mariusz Gorski wrote:
Remove Lindent and checkpatch.pl tasks from TODO file as the first one
is obsolete and the other one is already done.
when i checked panel.c with checkpatch and with --strict option, i got
On Sat, Nov 01, 2014 at 10:47:22AM +0800, Chen Weixiang wrote:
Remove following code style errors from ft1000/ft1000-usb/ft1000_debug.c
and ft1000/ft1000-pcmcia/ft1000_hw.c:
ERROR: spaces required around that '=' (ctx:VxV)
ERROR: spaces required around that '' (ctx:VxV)
ERROR: spaces required
On Sat, Nov 01, 2014 at 10:47:24AM +0800, Chen Weixiang wrote:
Remove following code style error from ft1000/ft1000-usb/ft1000_debug.c:
ERROR: (foo*) should be (foo *)
again checkpatch gives total: 4 errors, 3 warnings, 16 lines checked.
this includes whitespace errors.
thanks
sudip
On Sat, Nov 01, 2014 at 10:47:23AM +0800, Chen Weixiang wrote:
Remove following code style errors from ft1000/ft1000-pcmcia/ft1000_hw.c:
ERROR: space prohibited after that open parenthesis '('
ERROR: space prohibited before that close parenthesis ')'
ERROR: space prohibited after that ''
On Sat, Nov 01, 2014 at 10:47:26AM +0800, Chen Weixiang wrote:
Remove code style error from ft1000/ft1000-pcmcia/ft1000_hw.c:
ERROR: else should follow close brace '}'
from checkpatch:
total: 1 errors, 2 warnings, 44 lines checked
NOTE: whitespace errors detected, you may wish to use
On Sat, Nov 01, 2014 at 10:47:27AM +0800, Chen Weixiang wrote:
Remove following code style error from ft1000/ft1000-usb/ft1000_debug.c:
ERROR: space required after that ',' (ctx:VxV)
from checkpatch:
total: 11 errors, 15 warnings, 46 lines checked
NOTE: whitespace errors detected, you may
On Fri, Oct 31, 2014 at 10:14:30PM -0700, Joe Perches wrote:
On Sat, 2014-11-01 at 10:35 +0530, Sudip Mukherjee wrote:
On Sat, Nov 01, 2014 at 10:47:22AM +0800, Chen Weixiang wrote:
Remove following code style errors from ft1000/ft1000-usb/ft1000_debug.c
and ft1000/ft1000-pcmcia
On Sat, Nov 01, 2014 at 10:55:01AM +0530, Sudip Mukherjee wrote:
On Fri, Oct 31, 2014 at 10:14:30PM -0700, Joe Perches wrote:
On Sat, 2014-11-01 at 10:35 +0530, Sudip Mukherjee wrote:
On Sat, Nov 01, 2014 at 10:47:22AM +0800, Chen Weixiang wrote:
Remove following code style errors from
On Sat, Nov 01, 2014 at 10:55:01AM +0530, Sudip Mukherjee wrote:
On Fri, Oct 31, 2014 at 10:14:30PM -0700, Joe Perches wrote:
On Sat, 2014-11-01 at 10:35 +0530, Sudip Mukherjee wrote:
On Sat, Nov 01, 2014 at 10:47:22AM +0800, Chen Weixiang wrote:
Remove following code style errors from
On Sat, Nov 01, 2014 at 10:09:39PM +0800, Chen Weixiang wrote:
On Sat, Nov 01, 2014 at 10:35:35AM +0530, Sudip Mukherjee wrote:
many whitespace errors in the patch.
checkpatch gives total: 8 errors, 23 warnings, 125 lines checked
thanks
sudip
Hi sudip,
Yes, I also run
On Tue, Nov 04, 2014 at 02:13:19AM +0200, Aya Mahfouz wrote:
This patch replaces dev_err by pr_err since the value
of ir is NULL when the message is displayed.
Signed-off-by: Aya Mahfouz mahfouz.saif.elya...@gmail.com
---
drivers/staging/media/lirc/lirc_zilog.c | 2 +-
1 file changed, 1
On Tue, Nov 04, 2014 at 11:48:26PM +0200, Aya Mahfouz wrote:
On Tue, Nov 04, 2014 at 03:06:53PM +0530, Sudip Mukherjee wrote:
On Tue, Nov 04, 2014 at 02:13:19AM +0200, Aya Mahfouz wrote:
This patch replaces dev_err by pr_err since the value
of ir is NULL when the message is displayed
On Tue, Nov 04, 2014 at 11:43:07PM +0200, Aya Mahfouz wrote:
This patches replaces one pr_debug call by dev_dbg and
changes the device used by one of the dev_err calls.
i think you should mention in the commit message why you are changing the
device.
and also for revised patch its better if
, then no
need to check it again.
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
hi,
struct mlme_priv *pmlmepriv and bLinked was moved to the beginning of
the function, otherwise it was giving warning about mixed code while
compiling.
drivers/staging/rtl8188eu/core/rtw_xmit.c | 4 +---
drivers
a variable is declared and initialized and then never updated in the
function.This default value is then used as the return from the
function.
So removed that unneeded return variable and returning that default
initial value directly.
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
the NULL check for memregion is not required as it has already been
checked for NULL after kzalloc. so we can reach this part of the code
only if memregion is not NULL.
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
drivers/staging/unisys/visorutil/memregion_direct.c | 6 ++
1 file
we can reach the label Away in total five situation and in four of
of them pDevInfo is NULL. so we were basically dereferencing a NULL
pointer.
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 3 ++-
1 file changed, 2 insertions
On Sat, Nov 22, 2014 at 11:34:29AM +0530, Anjana Sasindran wrote:
This patch fixes the checkpatch.pl error:
this patch is not applying to next-20141121.
ERROR: trailing whitespace
but your patch is adding a blank line in the code ?
thanks
sudip
Signed-off-by: Anjana Sasindran
On Sat, Nov 22, 2014 at 01:20:13PM +0530, anjana s wrote:
Just now I resend the same patch by making necessary changes.
Kindly please go through it
you replied only to me. You should always use reply to all unless there is some
specific reason not to send the reply to all.
thanks
sudip
On Sat, Nov 22, 2014 at 01:11:59PM +0530, Anjana Sasindran wrote:
This patch fixes the checkpatch.pl error:
it still does not apply to next-20141121
ERROR: trailing whitespace
and i am not seeing any trailing whitespace at line 191 of this file.
if you base your patch on
On Sun, Nov 23, 2014 at 03:26:54PM +0530, Lekshmi wrote:
The following chaeckpatch warning was fixed:
WARNING: line over 80 characters
Signed-off-by: Lekshmi andnlnb...@gmail.com
---
drivers/staging/wlan-ng/prism2fw.c |6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff
On Sat, Nov 22, 2014 at 09:50:20PM +0530, Sudip Mukherjee wrote:
On Sat, Nov 22, 2014 at 06:00:40AM -0800, Greg Kroah-Hartman wrote:
On Sat, Nov 22, 2014 at 12:32:02PM +0530, Sudip Mukherjee wrote:
2) most important - I do not have the hardware. So from the TODO list
dual head and 2D
these header files were included two times
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
drivers/staging/unisys/include/timskmod.h | 1 -
drivers/staging/unisys/uislib/uisutils.c | 1 -
2 files changed, 2 deletions(-)
diff --git a/drivers/staging/unisys/include/timskmod.h
b/drivers
On Tue, Dec 02, 2014 at 01:46:54AM -0800, Jeremiah Mahler wrote:
Sudip,
On Tue, Dec 02, 2014 at 02:34:58PM +0530, Sudip Mukherjee wrote:
these header files were included two times
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
drivers/staging/unisys/include/timskmod.h | 1
these header files were included multiple times
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
change in v2: missed seeing that linux/uuid.h is included 3 times
drivers/staging/unisys/include/timskmod.h | 1 -
drivers/staging/unisys/uislib/uisutils.c | 2 --
2 files changed, 3
On Tue, Dec 02, 2014 at 08:27:46AM -0600, Paul Hedman wrote:
From: Paul p...@mybb.com
This patch fixes various coding standards violations found by the
checkpatch.pl tool
Signed-off-by: Paul Hedman m...@paulhedman.com
---
1) you are doing multiple things in this patch. please do only
On Wed, Dec 03, 2014 at 10:38:27AM +0800, Fred Chou wrote:
From: Fred Chou fred.chou...@gmail.com
Fixed the following warnings in sparse:
drivers/staging/rtl8192u/r8192U_core.c:670:6: warning:
symbol 'dump_eprom' was not declared.
Should it be static?
On Tue, Dec 02, 2014 at 07:30:40PM -0600, Paul Hedman wrote:
Fixes spacing issues around operators, e.g. i=0 to i = 0, as
found by the checkpatch.pl tool.
no Signed-off-by ???
please run your patch through checkpatch before sending.
thanks
sudip
---
On Tue, Dec 02, 2014 at 11:45:00PM +, Sean Cleator wrote:
A patch to the slicoss.c file to fix some of the long line issues found by
the
checkpath.pl tool
Signed-off-by: Sean Cleator seanclea...@hotmail.co.uk
do not use whitespace before Signed-off-by
thanks
sudip
---
On Wed, Dec 03, 2014 at 02:30:29PM +0800, Fred Chou wrote:
From: Fred Chou fred.chou...@gmail.com
v2: alignment to match open parenthesis
this should not come in commit log, this should go as a comment below the ---
and this patch is not applying to next-20141201, maybe in staging-testing
On Thu, Dec 04, 2014 at 10:50:43AM +0800, Fred Chou wrote:
From: Fred Chou fred.chou...@gmail.com
Fixed the following warnings in sparse:
drivers/staging/rtl8192u/r8192U_core.c:670:6: warning:
symbol 'dump_eprom' was not declared.
Should it be static?
On Thu, Dec 04, 2014 at 04:12:47PM +0800, Fred Chou wrote:
On 4/12/2014 3:34 PM, Sudip Mukherjee wrote:
On Thu, Dec 04, 2014 at 10:50:43AM +0800, Fred Chou wrote:
From: Fred Chou fred.chou...@gmail.com
Fixed the following warnings in sparse:
drivers/staging/rtl8192u/r8192U_core.c
On Thu, Dec 04, 2014 at 05:34:11PM +0800, Fred Chou wrote:
On 4/12/2014 4:30 PM, Sudip Mukherjee wrote:
On Thu, Dec 04, 2014 at 04:12:47PM +0800, Fred Chou wrote:
On 4/12/2014 3:34 PM, Sudip Mukherjee wrote:
On Thu, Dec 04, 2014 at 10:50:43AM +0800, Fred Chou wrote:
From: Fred
On Fri, Dec 05, 2014 at 04:25:22PM +0530, Athira Lekshmi wrote:
Fixed the checkpatch warning:
Missing a blank line after declarations
this is not applying to next-20141204.
sudip
Signed-off-by: Athira Lekshmi andnlnb...@gmail.com
---
drivers/staging/octeon/ethernet-rx.c |1 +
1
On Fri, Dec 05, 2014 at 10:10:43PM +0100, Mariusz Gorski wrote:
Get rid of magic numbers in LCD commands and replace them with defined
values, so that it's more obvious that the commands are doing.
it is not applying to next-20141205. is it based on staging-testing ?
thanks
sudip
On Fri, Dec 05, 2014 at 10:28:19PM +0100, Mariusz Gorski wrote:
Get rid of magic numbers in LCD commands and replace them with defined
values, so that it's more obvious that the commands are doing.
Signed-off-by: Mariusz Gorski marius.gor...@gmail.com
---
v2: Found and got rid of two more
On Sun, Dec 07, 2014 at 11:02:44PM +0100, Rickard Strandqvist wrote:
Remove the function MACbCompareContext() that is not used anywhere.
This was partially found by using a static code analysis program called
cppcheck.
Signed-off-by: Rickard Strandqvist
On Mon, Dec 08, 2014 at 11:33:25PM +0530, Adithya K wrote:
From: Adithya Krishnamurthy linux.challe...@gmail.com
Fixed sparse warning fix.
you should mention what type of warning you have fixed.
sudip
Signed-off-by: Adithya Krishnamurthy linux.challen...@gmail.com
---
---
Signed-off-by: Sudip Mukherjee sudipm.mukher...@gmail.com
---
drivers/staging/bcm/Transmit.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/bcm/Transmit.c b/drivers/staging/bcm/Transmit.c
index 05fae92..622a482 100644
--- a/drivers/staging/bcm
---
drivers/staging/bcm/DDRInit.c | 59 ---
1 file changed, 39 insertions(+), 20 deletions(-)
diff --git a/drivers/staging/bcm/DDRInit.c b/drivers/staging/bcm/DDRInit.c
index 4564f40..4226c93 100644
--- a/drivers/staging/bcm/DDRInit.c
+++
Signed-off-by: Sudip Mukherjee sudipm.mukher...@gmail.com
---
drivers/staging/bcm/DDRInit.c | 59 ---
1 file changed, 39 insertions(+), 20 deletions(-)
diff --git a/drivers/staging/bcm/DDRInit.c b/drivers/staging/bcm/DDRInit.c
index 4564f40..4226c93
fixed sparse warning of following symbol not declared:
warning: symbol 'lnet_ptl_cleanup' was not declared. Should it be static?
warning: symbol 'lnet_ptl_setup' was not declared. Should it be static?
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
As of now these functions
fixed sparse warning of incorrect type in argument 1 and incorrect type in
argument 2
it was directly dereferencing a __iomem pointer , which will work in x86 but
will fail in other architectures.
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
hi,
can you please reveiew the patch
On Mon, Sep 01, 2014 at 12:35:14PM +0300, Dan Carpenter wrote:
On Sun, Aug 31, 2014 at 09:46:22PM +0530, Sudip Mukherjee wrote:
@@ -133,9 +134,15 @@ static ssize_t goldfish_audio_read(struct file *fp,
char __user *buf,
AUDIO_READ_BUFFER_AVAILABLE
fixed sparse warning : context imbalance in 'do_locked_client_insert'
different lock contexts for basic block
spin_unlock_irqrestore is called at a later stage before returning
from the function if locked is 1.
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
fixed sparse warning: incompatible types in comparison expression
(different address spaces)
wolw and reg both are being used only for the initialization of
the __iomem area.
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
drivers/staging/et131x/et131x.c | 4
On Tue, Sep 02, 2014 at 05:05:32PM +0300, Dan Carpenter wrote:
On Tue, Sep 02, 2014 at 04:39:47PM +0530, Sudip Mukherjee wrote:
fixed sparse warning : context imbalance in 'do_locked_client_insert'
different lock contexts for basic block
spin_unlock_irqrestore
On Tue, Sep 02, 2014 at 06:23:27PM +0300, Dan Carpenter wrote:
On Tue, Sep 02, 2014 at 08:44:27PM +0530, Sudip Mukherjee wrote:
On Tue, Sep 02, 2014 at 05:05:32PM +0300, Dan Carpenter wrote:
On Tue, Sep 02, 2014 at 04:39:47PM +0530, Sudip Mukherjee wrote:
fixed sparse warning : context
From: Sudip Mukherjee su...@vectorindia.org
removed unused variables
fixed sparse warning of context imbalance in 'do_locked_client_insert'
different lock contexts for basic block
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
v1 of the patch of the patch just
On Tue, Sep 02, 2014 at 11:24:02AM -0700, Greg Kroah-Hartman wrote:
On Tue, Sep 02, 2014 at 11:46:35PM +0530, Sudip Mukherjee wrote:
From: Sudip Mukherjee su...@vectorindia.org
removed unused variables
fixed sparse warning of context imbalance in 'do_locked_client_insert
From: Sudip Mukherjee sudipm.mukher...@gmail.com
removed unused variables
fixed sparse warning of context imbalance in 'do_locked_client_insert'
different lock contexts for basic block
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
v1 of the patch
On Wed, Sep 03, 2014 at 11:40:38AM +0300, Dan Carpenter wrote:
On Tue, Sep 02, 2014 at 11:46:35PM +0530, Sudip Mukherjee wrote:
From: Sudip Mukherjee su...@vectorindia.org
I really would prefer if you just figured out your email settings so
this isn't needed. The From: header is mostly
value
of the function is not beeing used, so it is safe to remove.
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
v1 of the patch of the patch just fixed the sparse warning.
On suggestion of Dan Carpenter v2 is the total rewrite of the function.
change from v2 to v3 : i messed up
hi,
please do not consider this patch. v4 has been sent with some changes as
suggested by Dan Carpenter.
thanks
sudip
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
On Wed, Sep 03, 2014 at 02:29:44PM +0530, Sudip Mukherjee wrote:
On Wed, Sep 03, 2014 at 11:40:38AM +0300, Dan Carpenter wrote:
On Tue, Sep 02, 2014 at 11:46:35PM +0530, Sudip Mukherjee wrote:
From: Sudip Mukherjee su...@vectorindia.org
I really would prefer if you just figured out your
On Wed, Sep 03, 2014 at 04:09:59PM +0200, Piotr Witoslawski wrote:
Done as Eudyptula task 10.
use proper commit message mentioning what changes you have done.
and you have not sent the patch to all the people who were
supposed to be sent.
thanks
sudip
Signed-off-by: Piotr Witoslawski
On Wed, Sep 03, 2014 at 06:41:30PM +0300, Dan Carpenter wrote:
On Wed, Sep 03, 2014 at 08:13:25PM +0530, Sudip Mukherjee wrote:
On Wed, Sep 03, 2014 at 02:29:44PM +0530, Sudip Mukherjee wrote:
On Wed, Sep 03, 2014 at 11:40:38AM +0300, Dan Carpenter wrote:
On Tue, Sep 02, 2014 at 11:46
On Mon, Sep 01, 2014 at 08:47:12AM -0400, Dipak Zope wrote:
you have not mentioned any commit message .
thanks
sudip
Signed-off-by: Dipak Zope dipakzope.kernel@gmail.com
---
drivers/staging/android/uapi/binder.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
On Wed, Sep 03, 2014 at 12:59:50PM +0300, Dan Carpenter wrote:
On Wed, Sep 03, 2014 at 02:59:21PM +0530, Sudip Mukherjee wrote:
1) removed unused variables
2) fixed sparse warning of context imbalance in 'do_locked_client_insert'
different lock contexts for basic
fixed sparse warning : incorrect type in argument 1
(different address spaces)
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
drivers/staging/iio/adc/mxs-lradc.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/iio/adc
please discard the patch. It generates additional sparse warning.
sorry for it.
thanks
sudip
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
On Thu, Sep 04, 2014 at 06:50:05PM +0200, Lars-Peter Clausen wrote:
On 09/04/2014 06:45 PM, Sudip Mukherjee wrote:
fixed sparse warning : incorrect type in argument 1
(different address spaces)
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
drivers/staging
On Thu, Sep 04, 2014 at 07:16:11PM +0200, Lars-Peter Clausen wrote:
On 09/04/2014 07:12 PM, Sudip Mukherjee wrote:
On Thu, Sep 04, 2014 at 06:50:05PM +0200, Lars-Peter Clausen wrote:
On 09/04/2014 06:45 PM, Sudip Mukherjee wrote:
fixed sparse warning : incorrect type in argument 1
On Fri, Sep 05, 2014 at 12:34:13PM +0300, Dan Carpenter wrote:
On Thu, Sep 04, 2014 at 10:15:42PM +0530, Sudip Mukherjee wrote:
fixed sparse warning : incorrect type in argument 1
(different address spaces)
The change log is not clear. It's the IS_ERR() which
On Mon, Sep 08, 2014 at 10:57:08AM -0500, Romer, Benjamin M wrote:
On Fri, 2014-09-05 at 14:52 +0530, Sudip Mukherjee wrote:
fixed sparse warning : context imbalance in 'pause_device'
unexpected unlock
this patch will generate warning from checkpatch for
lines over
On Mon, Sep 08, 2014 at 11:30:48AM -0500, Romer, Benjamin M wrote:
On Mon, 2014-09-08 at 21:57 +0530, Sudip Mukherjee wrote:
Hi Ben,
thanks. the same file is having two more similar warnings. if you want i can
resend a patch fixing all the three warnings , or i can send two separate
On Mon, Sep 08, 2014 at 11:48:44AM -0500, Romer, Benjamin M wrote:
On Mon, 2014-09-08 at 22:08 +0530, Sudip Mukherjee wrote:
Hi Ben,
sorry to disturb you again. i got confused , which one is perfect one
combined patch or
separate patches?
thanks
sudip
Two patches, as you
fixed sparse warning : context imbalance in 'resume_device'
unexpected unlock
this patch will generate warning from checkpatch for
lines over 80 character , but since those are user-visible strings
so it was not modified.
Signed-off-by: Sudip Mukherjee su
On Wed, Sep 10, 2014 at 08:29:26AM -0500, Romer, Benjamin M wrote:
On Tue, 2014-09-09 at 16:11 +0530, Sudip Mukherjee wrote:
fixed sparse warning : context imbalance in 'resume_device'
unexpected unlock
this patch will generate warning from checkpatch for
lines
fixed sparse warning : context imbalance in 'destroy_device'
unexpected unlock
this patch will generate warning from checkpatch for
lines over 80 character , but since those are user-visible strings
so it was not modified.
Signed-off-by: Sudip Mukherjee su
On Sat, Sep 20, 2014 at 12:09:11PM +0100, Grzegorz Swirski wrote:
Signed-off-by: Grzegorz Swirski grzeg...@swirski.name
you have not mentioned any commit log
---
drivers/staging/wlan-ng/prism2mib.c | 11 ---
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git
On Fri, Sep 26, 2014 at 12:11:56AM +0100, Mike Roocroft wrote:
you forgot to write the commit message and also mention there what kind of
warning you have fixed.
thanks
sudip
Signed-off-by: Mike Roocroft mike.li...@btinternet.com
---
drivers/staging/android/sw_sync.c | 2 ++
1 file
On Mon, Dec 22, 2014 at 02:23:42AM +0100, Helene Gsaenger wrote:
Signed-off-by: Helene Gsaenger helene.gsaen...@studium.fau.de
Signed-off-by: Simone Weiss simone.we...@fau.de
---
drivers/staging/iio/accel/adis16203.h | 41
++-
1 file changed, 31
On Thu, Dec 25, 2014 at 12:28:28PM -0800, David Decotigny wrote:
Before:
1 ERROR: code indent should use tabs where possible
1 WARNING: please, no spaces at the start of a line
After:
(none)
Signed-off-by: David Decotigny ddeco...@gmail.com
---
On Fri, Dec 26, 2014 at 10:00:00AM +, Jonathan Cameron wrote:
On 22/12/14 13:48, Sudip Mukherjee wrote:
On Mon, Dec 22, 2014 at 02:23:42AM +0100, Helene Gsaenger wrote:
Signed-off-by: Helene Gsaenger helene.gsaen...@studium.fau.de
Signed-off-by: Simone Weiss simone.we...@fau.de
On Sat, Dec 27, 2014 at 07:45:07PM +0100, Rickard Strandqvist wrote:
Avoid allocate memory if we will exit the function.
Was found by using a static code analysis program called cppcheck.
Signed-off-by: Rickard Strandqvist rickard_strandqv...@spectrumdigital.se
---
On Sun, Dec 28, 2014 at 09:30:44PM +0100, Jonas Lundqvist wrote:
Hi Jeremiah,
On 12/28/2014 04:44 PM, Jeremiah Mahler wrote:
Jonas,
On Sun, Dec 28, 2014 at 02:26:46PM +0100, Jonas Lundqvist wrote:
Fixed three errors in pcm.h found by checkpatch.pl.
What type of errors did you
On Mon, Dec 29, 2014 at 03:14:45AM +0530, jitendra kumar khasdev wrote:
This is a patch to the s626.c file that fixes up spcae and maximum
character limit warning found by the checkpatch.pl tool
you are doing two different kind of changes in this patch. it should be in two
different patches.
of current v2 indentation, I'll do it, please let me know.
well, if you check CodingStyle in Documentation, it clearly says:
Outside of comments, documentation and except in Kconfig, spaces are never
used for indentation.
Sudip
On Fri, Dec 26, 2014 at 11:00 PM, Sudip Mukherjee
sudipm.mukher
On Tue, Dec 30, 2014 at 07:04:35PM -0800, Serguey Parkhomovsky wrote:
On Tue, Dec 30, 2014 at 02:35:18PM -0800, Jeremiah Mahler wrote:
If you look at the source code just below these functions you will find:
EXPORT_SYMBOL(libcfs_arch_init);
EXPORT_SYMBOL(libcfs_arch_cleanup);
So
This reverts commit dc93c85235efa5201e9a3c116bc3fbd1afc1a182
Signed-off-by: Sudip Mukherjee su...@vectorindia.org
---
Hi Greg K-H
With reference to our discussion at https://lkml.org/lkml/2014/11/22/54 ,
now I have the hardware and I will be working on the code along with
Teddy Wang.
drivers
On Wed, Dec 31, 2014 at 09:33:43PM +, Lorenzo Stoakes wrote:
This patch fixes warnings/errors raised by checkpatch.pl relating to
whitespace
in r8192U_dm.c and additionally it removes inconsistent whitespace
throughout.
Additionally some 0x00... padding has been added for alignment to
On Wed, Dec 31, 2014 at 09:10:42PM +, Lorenzo Stoakes wrote:
This patch fixes warnings/errors raised by checkpatch.pl relating to redundant
code in r8192U_dm.c.
Signed-off-by: Lorenzo Stoakes lstoa...@gmail.com
---
drivers/staging/rtl8192u/r8192U_dm.c | 176
On Thu, Jan 01, 2015 at 10:19:56AM +0530, Sudip Mukherjee wrote:
On Wed, Dec 31, 2014 at 09:10:42PM +, Lorenzo Stoakes wrote:
This patch fixes warnings/errors raised by checkpatch.pl relating to
redundant
code in r8192U_dm.c.
Signed-off-by: Lorenzo Stoakes lstoa...@gmail.com
On Wed, Dec 31, 2014 at 09:40:13PM -0800, Joe Perches wrote:
On Thu, 2015-01-01 at 10:17 +0530, Sudip Mukherjee wrote:
On Wed, Dec 31, 2014 at 09:33:43PM +, Lorenzo Stoakes wrote:
This patch fixes warnings/errors raised by checkpatch.pl relating to
whitespace
in r8192U_dm.c
On Sat, Jan 03, 2015 at 02:06:23PM +0530, jitendra kumar khasdev wrote:
This is patch to file pcl812.c that fix up warning line
over 80 character which is found by checkpatch tool.
Singed-off-by: Jitendra Kumar Khasdev jkhas...@gmail.com
what is Singed-off ??? why you are doing Signed-off
On Sat, Jan 03, 2015 at 06:16:31PM +0530, jitendra kumar khasdev wrote:
This is patch to file pcl812.c that fix up warning line
over 80 character which is found by checkpatch tool.
Signed-off-by: Jitendra Kumar Khasdev jkhas...@gmail.com
I am assuming this one you wanted to make as v3.
but
1 - 100 of 962 matches
Mail list logo
