[PATCH 0/2] Staging: vt6656: fix two coding style issues

2013-10-23 Thread Johannes Löthberg
This patchset fixes two coding style issues reported by checkpatch.pl in drivers/staging/vt6656, one warning and one error Johannes Löthberg (2): Staging: vt6656: fix a brace coding style issue in power.c Staging: vt6656: fix code indenting error in power.c drivers/staging/vt6656/power.c | 5

[PATCH 1/2] Staging: vt6656: fix a brace coding style issue in power.c

2013-10-23 Thread Johannes Löthberg
This patch fixes a brace warning in power.c found by checkpatch.pl Signed-off-by: Johannes Löthberg --- drivers/staging/vt6656/power.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/vt6656/power.c b/drivers/staging/vt6656/power.c index edc8975..3002061 1006

[PATCH 2/2] Staging: vt6656: fix code indenting error in power.c

2013-10-23 Thread Johannes Löthberg
This patch fixes a code indentation error found by checkpatch.pl where a line was indented with spaces instead of tabs Signed-off-by: Johannes Löthberg --- drivers/staging/vt6656/power.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vt6656/power.c b/drivers/

[PATCH NEXT 2/5] staging: r8188eu: Fix sparse warnings in rtw_ieee80211.c

2013-10-23 Thread Larry Finger
Sparse reports the following: CHECK drivers/staging/rtl8188eu/core/rtw_ieee80211.c drivers/staging/rtl8188eu/core/rtw_ieee80211.c:1593:14: warning: cast to restricted __le16 Signed-off-by: Larry Finger --- drivers/staging/rtl8188eu/include/ieee80211.h | 16 1 file changed,

[PATCH NEXT 3/5] staging: r8188eu: Fix sparse warnings in rtl8188e.cmd.c

2013-10-23 Thread Larry Finger
Sparse displays the following: CHECK drivers/staging/rtl8188eu/hal/rtl8188e_cmd.c drivers/staging/rtl8188eu/hal/rtl8188e_cmd.c:285:15: warning: incorrect type in assignment (different base types) drivers/staging/rtl8188eu/hal/rtl8188e_cmd.c:285:15:expected unsigned short [usertype] *fctr

[PATCH 0/5 NEXT V2] staging: r8188eu: Fix sparse warnings.

2013-10-23 Thread Larry Finger
This set of patches resubmits the set of fixes for sparse warnings. The two that were criticized have been removed, and this set should not have white-space damage. Signed-off-by: Larry Finger Larry Finger (5): staging: r8188eu: Fix sparse warnings in ioctl_linux.c staging: r8188eu: Fix spa

[PATCH NEXT 4/5] staging: r8188eu: Fix sparse warnings in rtw_mlme_ext.c

2013-10-23 Thread Larry Finger
Sparse displays the following: CHECK drivers/staging/rtl8188eu/core/rtw_mlme_ext.c drivers/staging/rtl8188eu/core/rtw_mlme_ext.c:1874:15: warning: incorrect type in assignment (different base types) drivers/staging/rtl8188eu/core/rtw_mlme_ext.c:1874:15:expected unsigned short *fctrl driv

[PATCH NEXT 1/5] staging: r8188eu: Fix sparse warnings in ioctl_linux.c

2013-10-23 Thread Larry Finger
Sparse checking results in the following warnings: CHECK drivers/staging/rtl8188eu/os_dep/ioctl_linux.c drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3932:17: warning: cast removes address space of expression drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3933:31: warning: incorrect type in

[PATCH NEXT 5/5] staging: r8188eu: Fix sparse warnings in rtl_p2p.c

2013-10-23 Thread Larry Finger
Sparse displays the following: CHECK drivers/staging/rtl8188eu/core/rtw_p2p.c drivers/staging/rtl8188eu/core/rtw_p2p.c:162:15: warning: incorrect type in assignment (different base types) drivers/staging/rtl8188eu/core/rtw_p2p.c:162:15:expected unsigned short *fctrl drivers/staging/rtl81

[PATCH v2] drivers: staging: rtl8188eu: use 'ccflags-y' instead of EXTRA_CFLAGS in Makefile

2013-10-23 Thread Chen Gang
If command line use EXTRA_CFLAGS (e.g. "EXTRA_CFLAGS=-mmedium-calls" for arc architecture, with allmodconfig), it can not pass compiling, the related error: drivers/staging/rtl8188eu/core/rtw_ap.c:22:27: fatal error: osdep_service.h: No such file or directory Signed-off-by: Chen Gang Acked-b

[PATCH 0/8] treewide: Remove OOM message after input_alloc_device

2013-10-23 Thread Joe Perches
Joe Perches (8): Documentation: Remove OOM message after input_allocate_device cell: Remove OOM message after input_allocate_device hid: Remove OOM message after input_allocate_device input: Remove OOM message after input_allocate_device media: Remove OOM message after input_allocate_devi

Re: [PATCH] drivers: staging: rtl8188eu: use 'ccflags-y' instead of EXTRA_CFLAGS in Makefile

2013-10-23 Thread Chen Gang
On 10/24/2013 12:06 AM, Larry Finger wrote: > On 10/23/2013 03:52 AM, Chen Gang wrote: >> If command line use EXTRA_CFLAGS (e.g. "EXTRA_CFLAGS=-mmedium-calls" >> for arc architecture, with allmodconfig), it can not pass compiling, >> the related error: >> >>drivers/staging/rtl8188eu/core/rtw_ap

Re: [PATCH] drivers: staging: rtl8188eu: use 'ccflags-y' instead of EXTRA_CFLAGS in Makefile

2013-10-23 Thread Larry Finger
On 10/23/2013 08:10 PM, Chen Gang wrote: > On 10/24/2013 12:06 AM, Larry Finger wrote: >> On 10/23/2013 03:52 AM, Chen Gang wrote: >>> If command line use EXTRA_CFLAGS (e.g. "EXTRA_CFLAGS=-mmedium-calls" >>> for arc architecture, with allmodconfig), it can not pass compiling, >>> the related error:

Re: [PATCH] drivers: staging: bcm: Removed a developer debug statement.

2013-10-23 Thread Dan Carpenter
On Wed, Oct 23, 2013 at 03:11:20PM -0400, Chuong Ngo wrote: > Removed a developer debug statement per the TODO list. Additionally, removed > braces for the if-statement to match coding style. Line wrap the changelog at 72 characters. You don't really need to mention the removed braces because t

[PATCH 7/8] staging: Remove OOM message after input_allocate_device

2013-10-23 Thread Joe Perches
Emitting an OOM message isn't necessary after input_allocate_device as there's a generic OOM and a dump_stack already done. Signed-off-by: Joe Perches --- drivers/staging/cptm1217/clearpad_tm1217.c| 2 -- drivers/staging/iio/adc/mxs-lradc.c | 4 +--- drivers/staging/ste_rmi4/synapt

[PATCH] drivers: staging: bcm: Removed a developer debug statement.

2013-10-23 Thread Chuong Ngo
Removed a developer debug statement per the TODO list. Additionally, removed braces for the if-statement to match coding style. Signed-off-by: Chuong Ngo --- drivers/staging/bcm/Bcmchar.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/staging/bcm/Bcmchar.c b/dr

Re: [Patch v2][ 03/37] drm: Add the lacking DRM_MODE_FLAG_* for matching the DISPLAY_FLAGS_*

2013-10-23 Thread Denis Carikli
Hi, On 10/18/2013 09:46 AM, Ville Syrjälä wrote: +#define DRM_MODE_FLAG_PDATEN (1<<22) +#define DRM_MODE_FLAG_NDATEN (1<<23) +#define DRM_MODE_FLAG_PPIXDATEDGE (1<<24) +#define DRM_MODE_FLAG_NPIXDATEDGE (1<<25) Do we really need to

Re: [Patch v2][ 03/37] drm: Add the lacking DRM_MODE_FLAG_* for matching the DISPLAY_FLAGS_*

2013-10-23 Thread Matt Sealey
On Thu, Oct 17, 2013 at 10:02 AM, Denis Carikli wrote: > Without that fix, drivers using the drm_display_mode_from_videomode > function will not be able to get certain information because > some DISPLAY_FLAGS_* have no corresponding DRM_MODE_FLAG_*. > > Cc: Greg Kroah-Hartman > Cc: driverdev-

Re: [PATCH] drivers: staging: rtl8188eu: use 'ccflags-y' instead of EXTRA_CFLAGS in Makefile

2013-10-23 Thread Larry Finger
On 10/23/2013 03:52 AM, Chen Gang wrote: If command line use EXTRA_CFLAGS (e.g. "EXTRA_CFLAGS=-mmedium-calls" for arc architecture, with allmodconfig), it can not pass compiling, the related error: drivers/staging/rtl8188eu/core/rtw_ap.c:22:27: fatal error: osdep_service.h: No such file or d

Re: [Patch v2][ 03/37] drm: Add the lacking DRM_MODE_FLAG_* for matching the DISPLAY_FLAGS_*

2013-10-23 Thread Ville Syrjälä
On Wed, Oct 23, 2013 at 04:48:51PM +0200, Denis Carikli wrote: > Hi, > > On 10/18/2013 09:46 AM, Ville Syrjälä wrote: > >> +#define DRM_MODE_FLAG_PDATEN (1<<22) > >> +#define DRM_MODE_FLAG_NDATEN (1<<23) > >> +#define DRM_MODE_FLAG_PPIXDATEDGE (1<<

[PATCH] drivers: staging: rtl8188eu: use 'ccflags-y' instead of EXTRA_CFLAGS in Makefile

2013-10-23 Thread Chen Gang
If command line use EXTRA_CFLAGS (e.g. "EXTRA_CFLAGS=-mmedium-calls" for arc architecture, with allmodconfig), it can not pass compiling, the related error: drivers/staging/rtl8188eu/core/rtw_ap.c:22:27: fatal error: osdep_service.h: No such file or directory Signed-off-by: Chen Gang --- dr