The following checkpatch warning was fixed
WARNING: line over 80 characters
Signed-off-by: Athira Lekshmi C V andnlnb...@gmail.com
---
drivers/staging/wlan-ng/prism2fw.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wlan-ng/prism2fw.c
Em Fri, 21 Nov 2014 14:21:40 +0100
Abel Moyo abelmoyo...@gmail.com escreveu:
Replaced printk with pr_debug in dprintk
Signed-off-by: Abel Moyo abelmoyo...@gmail.com
---
drivers/staging/media/lirc/lirc_serial.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Fixed the checkpatch warning:
WARNING: Prefer ether_addr_copy() over memcpy() if the Ethernet addresses are
__aligned(2)
Signed-off-by: Athira Lekshmi C V andnlnb...@gmail.com
---
drivers/staging/vt6656/rxtx.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff
On Tue, Nov 25, 2014 at 05:45:40PM +0530, Athira Lekshmi C V wrote:
Fixed the checkpatch warning:
WARNING: Prefer ether_addr_copy() over memcpy() if the Ethernet addresses are
__aligned(2)
For these changes the change log must say how you determined both
source and destination are aligned
This patch fixes the checkpatch.pl warnings:
WARNING: line over 80 characters
+ int cores_in_use = core_state.baseline_cores -
atomic_read(core_state.available_cores);
WARNING: line over 80 characters
+ skb-data = skb-head + work-packet_ptr.s.addr -
Fixed the checkpatch warning:
WARNING: else is not generally useful after a break or return
Signed-off-by: Athira Lekshmi C V andnlnb...@gmail.com
---
drivers/staging/vt6656/rxtx.c | 35 ---
1 file changed, 16 insertions(+), 19 deletions(-)
diff --git
On Tue, Nov 25, 2014 at 07:05:51PM +0530, Athira Lekshmi C V wrote:
Fixed the checkpatch warning:
WARNING: else is not generally useful after a break or return
This patch adds a compile warning.
Signed-off-by: Athira Lekshmi C V andnlnb...@gmail.com
Btw, signing and from should be your
From: Markus Elfring elfr...@users.sourceforge.net
Date: Tue, 25 Nov 2014 16:06:19 +0100
The drm_fbdev_cma_restore_mode() function tests whether its argument is NULL
and then returns immediately. Thus the test around the call is not needed.
This issue was detected by using the Coccinelle
From: Markus Elfring elfr...@users.sourceforge.net
Date: Tue, 25 Nov 2014 16:51:08 +0100
The oz_free_urb_link() function tests whether its argument is NULL
and then returns immediately. Thus the test around the call is not needed.
This issue was detected by using the Coccinelle software.
Changing the following:
Adapater - Adapter
STATUS_SUCESS - STATUS_SUCCESS
Signed-off by: Siddharth Kannan kannan.siddhart...@gmail.com
---
drivers/staging/bcm/nvm.c | 26 +-
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/drivers/staging/bcm/nvm.c
On Tue, Nov 25, 2014 at 11:00:35PM +0530, icyflame wrote:
Changing the following:
Adapater - Adapter
STATUS_SUCESS - STATUS_SUCCESS
Signed-off by: Siddharth Kannan kannan.siddhart...@gmail.com
---
drivers/staging/bcm/nvm.c | 26 +-
1 file changed, 13
Your From email header should match your signed-off-by line. The BCM
driver was deleted. You should be working against linux-next.
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
Don't send private messages.
On Tue, Nov 25, 2014 at 06:18:55PM +, Siddharth Kannan wrote:
Due to some network limitations, I was unable to clone the repository using
the ssh URL given on the KernelBuild Linux Newbies page.
So, I cloned it from the github repo. (Linus Torvalds.
Changing the following:
Adapater - Adapter
STATUS_SUCESS - STATUS_SUCCESS
Signed-off-by: Siddharth Kannan kannan.siddhart...@gmail.com
---
drivers/staging/bcm/nvm.c | 26 +-
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/drivers/staging/bcm/nvm.c
On Tue, Nov 25, 2014 at 11:58:18PM +0530, Siddharth Kannan wrote:
Changing the following:
Adapater - Adapter
STATUS_SUCESS - STATUS_SUCCESS
The driver you are changing doesn't exist any more. Work against
linux-next.
regards,
dan carpenter
___
checkpatch makes an exception to the 80-colum rule for quotes strings, and
Documentation/CodingStyle recommends not splitting quotes strings across lines
because it breaks the ability to grep for the string. Fixing these.
WARNING: quoted string split across lines
Signed-off-by: Luis de
On Tue, 2014-11-25 at 20:19 +, Luis de Bethencourt wrote:
checkpatch makes an exception to the 80-colum rule for quotes strings, and
Documentation/CodingStyle recommends not splitting quotes strings across lines
because it breaks the ability to grep for the string. Fixing these.
[]
diff
checkpatch makes an exception to the 80-colum rule for quotes strings, and
Documentation/CodingStyle recommends not splitting quotes strings across lines
because it breaks the ability to grep for the string. Fixing these.
WARNING: quoted string split across lines
Signed-off-by: Luis de
On Tue, Nov 25, 2014 at 12:27:24PM -0800, Joe Perches wrote:
On Tue, 2014-11-25 at 20:19 +, Luis de Bethencourt wrote:
checkpatch makes an exception to the 80-colum rule for quotes strings, and
Documentation/CodingStyle recommends not splitting quotes strings across
lines
because it
This patch fixes pointer declarations from void * to void __user * in order
to remove some sparse warnings.
lib-lnet.h:798:48: warning: incorrect type in initializer (different address
spaces)
lib-lnet.h:798:48:expected void [noderef] asn:1*iov_base
lib-lnet.h:798:48:got void *dest
On Tue, Nov 25, 2014 at 01:00:07PM -0800, Joe Perches wrote:
On Tue, 2014-11-25 at 20:40 +, Luis de Bethencourt wrote:
On Tue, Nov 25, 2014 at 12:27:24PM -0800, Joe Perches wrote:
On Tue, 2014-11-25 at 20:19 +, Luis de Bethencourt wrote:
checkpatch makes an exception to the
From: Markus Elfring elfr...@users.sourceforge.net
Date: Tue, 25 Nov 2014 22:33:45 +0100
The vfree() function performs also input parameter validation.
Thus the test around the call is not needed.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
From: SF Markus Elfring elfr...@users.sourceforge.net
Date: Tue, 25 Nov 2014 22:55:34 +0100
From: Markus Elfring elfr...@users.sourceforge.net
Date: Tue, 25 Nov 2014 22:33:45 +0100
The vfree() function performs also input parameter validation.
Thus the test around the call is not needed.
On Tue, 2014-11-25 at 20:40 +, Luis de Bethencourt wrote:
On Tue, Nov 25, 2014 at 12:27:24PM -0800, Joe Perches wrote:
On Tue, 2014-11-25 at 20:19 +, Luis de Bethencourt wrote:
checkpatch makes an exception to the 80-colum rule for quotes strings, and
Documentation/CodingStyle
On 25 November 2014 at 01:33, Philipp Zabel p.za...@pengutronix.de wrote:
The imx-drm driver was put into staging mostly for the following reasons,
all of which have been addressed or superseded:
- convert the irq driver to use linear irq domains
- work out the device tree bindings, this
On Tue, 2014-11-25 at 21:44 +0100, Zahari Doychev wrote:
This patch fixes pointer declarations from void * to void __user * in order
to remove some sparse warnings.
This patch does more than that.
Please make sure to describe all of the changes
in a patch in the commit message.
diff --git
26 matches
Mail list logo