Fixed few coding style errors in header files
base.h:58: WARNING: please, no space before tabs
base.h:81: ERROR: (foo*) should be (foo *)
pci.h:338: ERROR: foo * bar should be foo *bar
pci.h:151: ERROR: do not use C99 // comments
Signed-off-by: Surendra Patil surendra@gmail.com
---
ip.h is included in four files but not being used, so
remove include/ip.h header file and inclusion of this header file.
Signed-off-by: navin patidar navin.pati...@gmail.com
---
drivers/staging/rtl8188eu/core/rtw_recv.c |1 -
drivers/staging/rtl8188eu/core/rtw_xmit.c |1 -
if_ether.h is included in three files but not being used, so
remove include/if_ether.h header file and inclusion of this header file.
Signed-off-by: navin patidar navin.pati...@gmail.com
---
drivers/staging/rtl8188eu/core/rtw_recv.c |1 -
drivers/staging/rtl8188eu/hal/rtl8188eu_recv.c |
ethernet.h is included in three files but only rtw_recv.c using two macros
defined in ethernet.h, so move used macros in rtw_recv.c and
remove include/ethernet.h header file and inclusion of this header file.
Signed-off-by: navin patidar navin.pati...@gmail.com
---
On Fri, 7 Feb 2014 20:23:51 +
Russell King - ARM Linux li...@arm.linux.org.uk wrote:
Here's my changes to the TDA998x driver to add support for the component
helper. The TDA998x driver retains support for the old way so that
drivers can be transitioned. For any one DRM card the
On Sun, Feb 09, 2014 at 10:22:19AM +0100, Jean-Francois Moine wrote:
On Fri, 7 Feb 2014 20:23:51 +
Russell King - ARM Linux li...@arm.linux.org.uk wrote:
Here's my changes to the TDA998x driver to add support for the component
helper. The TDA998x driver retains support for the old way
There are references to four undefined Kconfig macros in the code.
Remove these as the checks for them will always evaluate to false.
There are additional cleanups possible now, but I'll gladly leave those
to people that are actually familiar with the code.
Signed-off-by: Paul Bolle
Commit d632dfefd36f ([media] v4l: omap4iss: Add support for OMAP4
camera interface - Build system) added a Kconfig entry for
VIDEO_OMAP4_DEBUG. But nothing uses that symbol.
This entry was apparently copied from a similar entry for OMAP 3
Camera debug messages. But a corresponding Makefile line
On 02/09/2014 01:30 AM, Surendra Patil wrote:
Fixed multiple coding style errors and warnings
wifi.h:1077: WARNING: please, no space before tabs
wifi.h:762: WARNING: missing space after struct definition
wifi.h:972: WARNING: please, no spaces at the start of a line
wifi.h:1825: WARNING:
On 02/09/2014 01:30 AM, Surendra Patil wrote:
Fixed multiple coding style errors and warnings
wifi.h:1077: WARNING: please, no space before tabs
wifi.h:762: WARNING: missing space after struct definition
wifi.h:972: WARNING: please, no spaces at the start of a line
wifi.h:1825: WARNING:
On 02/09/2014 02:29 AM, Surendra Patil wrote:
Fixed few coding style errors in header files
base.h:58: WARNING: please, no space before tabs
base.h:81: ERROR: (foo*) should be (foo *)
pci.h:338: ERROR: foo * bar should be foo *bar
pci.h:151: ERROR: do not use C99 // comments
Signed-off-by:
The symbol is an orphan, get rid of it.
Signed-off-by: Richard Weinberger rich...@nod.at
---
drivers/staging/tidspbridge/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/tidspbridge/Kconfig
b/drivers/staging/tidspbridge/Kconfig
index 1b6d581..b5e74e9
On Sun, 2014-02-09 at 19:47 +0100, Richard Weinberger wrote:
The symbol is an orphan, get rid of it.
Signed-off-by: Richard Weinberger rich...@nod.at
---
drivers/staging/tidspbridge/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
As has been noted, this driver is very rough. This set of patches is
intended to smooth some of the rough edges.
Signed-off-by: Larry Finger larry.fin...@lwfinger.net
Larry Finger (6):
staging: r8188eu: Replace wrapper around _rtw_memcmp()
staging: r8188eu: Remove dead file
staging:
The headers for this driver contain a number of unused structs and macros
that are removed. File include/ioctl_cfg80211.h is now empty and was
deleted.
Signed-off-by: Larry Finger larry.fin...@lwfinger.net
---
drivers/staging/rtl8188eu/include/ioctl_cfg80211.h | 107 -
From 16140b1ec1b1e1060f74707e4a6661aface81a14 Mon Sep 17 00:00:00 2001
From: Daeseok Youn daeseok.y...@gmail.com
Date: Mon, 10 Feb 2014 14:36:48 +0900
Subject: [PATCH] staging : android : sync : fix a checkpatch warning
- WARNING: missing space after return type
Signed-off-by: Daeseok Youn
From 866c5eb6cb88b59702802876a724dfd0144c447e Mon Sep 17 00:00:00 2001
From: Daeseok Youn daeseok.y...@gmail.com
Date: Mon, 10 Feb 2014 14:38:05 +0900
Subject: [PATCH] staging: android: timed_output: fix a checkpatch warning
- WARNING: Multiple spaces after return type
Signed-off-by: Daeseok
I think It looks better than reported by checkpatch.pl.
But I have a qeustion, if your patch is applied to checkpatch.pl file,
a return type of void * line will be changed.
for example,
void * (*map_kernel)(struct ion_heap *heap, struct ion_buffer *buffer);
= void *(*map_kernel)(struct ion_heap
CC+ Joe Perches.
2014-02-10 14:58 GMT+09:00 Daeseok Youn daeseok.y...@gmail.com:
From 16140b1ec1b1e1060f74707e4a6661aface81a14 Mon Sep 17 00:00:00 2001
From: Daeseok Youn daeseok.y...@gmail.com
Date: Mon, 10 Feb 2014 14:36:48 +0900
Subject: [PATCH] staging : android : sync : fix a checkpatch
19 matches
Mail list logo
