RE: randconfig build error with 3.13-rc6, in drivers/staging/lustre/lustre/lmv/lproc_lmv.c

2013-12-30 Thread Peng, Tao
>-Original Message- >From: Jim Davis [mailto:jim.ep...@gmail.com] >Sent: Tuesday, December 31, 2013 1:02 AM >To: linux-ker...@vger.kernel.org; Greg Kroah-Hartman; >andreas.dil...@intel.com; Peng, Tao; sachin.ka...@linaro.org; >de...@driverdev.osuosl.org >Subject: randconfig build error wi

Re: [PATCH] Staging: rtl8188eu: Fixed coding style issues

2013-12-30 Thread Larry Finger
On 12/30/2013 08:53 PM, Tim Jester-Pfadt wrote: Fixed required and prohibited spaces to make rtw_io.h checkpatch.pl clean Signed-off-by: Tim Jester-Pfadt --- drivers/staging/rtl8188eu/include/rtw_io.h | 36 +++--- 1 file changed, 18 insertions(+), 18 deletions(-) Ac

[PATCH] Staging: rtl8188eu: Fixed coding style issues

2013-12-30 Thread Tim Jester-Pfadt
Fixed required and prohibited spaces to make rtw_io.h checkpatch.pl clean Signed-off-by: Tim Jester-Pfadt --- drivers/staging/rtl8188eu/include/rtw_io.h | 36 +++--- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_io.h

Re: [PATCH] comedi: Humusoft MF634 and MF624 DAQ cards driver

2013-12-30 Thread Dan Carpenter
These days comedi drivers are looking really nice. :) You will need to resend because it's missing a change log and a Signed-off-by line. Wait for Ian or Hartley to comment before resending. I had a few minor style nits mentioned inline below. Run the patch through `scripts/checkpatch.pl --str

[PATCH] comedi: Humusoft MF634 and MF624 DAQ cards driver

2013-12-30 Thread Rostislav Lisovy
create mode 100644 drivers/staging/comedi/drivers/mf6x4.c diff --git a/drivers/staging/comedi/Kconfig b/drivers/staging/comedi/Kconfig index bfa27e7..89e25b4 100644 --- a/drivers/staging/comedi/Kconfig +++ b/drivers/staging/comedi/Kconfig @@ -884,6 +884,12 @@ config COMEDI_GSC_HPDI To

[PATCH] comedi: Humusoft MF634 and MF624 DAQ cards driver

2013-12-30 Thread Rostislav Lisovy
This patch adds Comedi driver for Humusoft MF634 (PCIe) and MF624 (PCI) data acquisition cards. The legacy card Humusoft MF614 is not supported. More info about the cards may be found at http://humusoft.cz/produkty/datacq/ The driver was tested with both cards. Everything seems to work properly. Ju

[PATCH 3/3] staging: xillybus: Open Firmware driver supporting coherent DMA

2013-12-30 Thread Eli Billauer
If the "dma-coherent" property is present in the device tree, the driver will not perform cache invalidations. This feature significantly improves data throughput and reduces CPU load. Signed-off-by: Eli Billauer --- drivers/staging/xillybus/xillybus_of.c | 21 - 1 files ch

[PATCH 1/3] staging: xillybus: Added documentation on device tree bindings

2013-12-30 Thread Eli Billauer
Signed-off-by: Eli Billauer --- .../devicetree/bindings/staging/xillybus.txt | 20 1 files changed, 20 insertions(+), 0 deletions(-) create mode 100644 Documentation/devicetree/bindings/staging/xillybus.txt diff --git a/Documentation/devicetree/bindings/staging/xil

[PATCH 2/3] staging: xillybus: Changed Open Firmware "compatible" property

2013-12-30 Thread Eli Billauer
The previous "compatible" string was poorly chosen, but remains in the match list to support existing DTBs. There is no risk for a naming clash. Signed-off-by: Eli Billauer --- drivers/staging/xillybus/xillybus_of.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/drive

[PATCH 3/7] vt6655: Fixed most of the checkpatch warnings in wpactl

2013-12-30 Thread Michael Gunselmann
wpactl.h: Checkpatch does no longer complain about anything wpactl.c: The following errors and warnings remain: ERROR: need consistens spacing around '&' in line 364, 372 and 411 This is okay, 'cause we need pointers to

[PATCH 1/7] vt6655: Remove unused macros in 80211hdr.h

2013-12-30 Thread Michael Gunselmann
From: Martin Hofmann The file 80211hdr.h contained 4 macros (in both little and big endain fashion) that were not used in the driver. Thus, this patch removes them. Signed-off-by: Martin Hofmann Signed-off-by: Michael Gunselmann --- drivers/staging/vt6655/80211hdr.h | 8 1 file chang

[PATCH 0/7] vt6655: Cleanup of checkpatch errors

2013-12-30 Thread Michael Gunselmann
This patch series cleans up all checkpatch errors and some warnings in some files of the vt6655 driver in staging. The reference git tree is linux-next. Some checkpatch warnings were not fixed, they are commented in the particular patches. Not all files of the driver were cleaned up in this patc

[PATCH 4/7] vt6655: Fixed most of the checkpatch warnings in wpa2

2013-12-30 Thread Michael Gunselmann
wpa2.h: Checkpatch does no longer complain about anything wpa2.c: Checkpatch complains about some lines that are longer than 80 characters. Breaking them would deteriorate the readability so these lines were not touched. vntwifi.c: Fixing style problems in wpa2.h made it necessary

[PATCH 5/7] vt6655: Fixed most of the checkpatch warnings in wpa

2013-12-30 Thread Michael Gunselmann
wpa.h: Checkpatch does no longer complain about anything. wpactl.c: Some long-line-warnings still remain but fixing them would deteriorate code readability. Signed-off-by: Michael Gunselmann Signed-off-by: Martin Hofmann --- drivers/staging/vt6655/wpa.c | 156

[PATCH 6/7] vt6655: Remove typedefs in 80211hdr.h

2013-12-30 Thread Michael Gunselmann
From: Martin Hofmann The file 80211hdr.h contained typedefs for 5 types. To satisfy checkpatch, this commit removes them. In 10 other files, every occurence of a now deleted type has been substituted with the correct struct ... syntax. Signed-off-by: Martin Hofmann Signed-off-by: Michael Gunsel

[PATCH 2/7] vt6655: Fix most of checkpatch.pl errors in wroute

2013-12-30 Thread Michael Gunselmann
wroute.h: Fixed all line-over-80-character errors. No errors remain. wroute.c: Fixed line-over-80-character errors, bracing errors and C99-comments. Three warnings remain, fixing them would deteriorate readability. One warning on a memory barrier without comment in lin

[PATCH] Staging: bcm: Fixed excessive line lengths cleaned up some code

2013-12-30 Thread Arthur Schwalbenberg
This is a patch to the Adapter.h file that fixes up excesssive line length warnings found by the checkpatch.pl tool Signed off by: Arthur Schwalbenberg --- drivers/staging/bcm/Adapter.h | 228 -- 1 file changed, 133 insertions(+), 9

[PATCH 0/4] beeceem: Several checkpatch cleanups in InterfaceTx.c

2013-12-30 Thread Ralph Mueck
This patch series fixes some checkpatch issues in InterfaceTx.c which is part of the beeceem driver. The following errors/warnings have been fixed: - prohibited spaces before a (semi)colon - missing space before a variable assignment - missing spaces between function argume

[PATCH 3/4] beeceem: Fix position of braces in conditional statements in InterfaceTx.c

2013-12-30 Thread Ralph Mueck
This corrects the position of (opening) braces in if-conditionals to make checkpatch shut up. Signed-off-by: Ralph Mueck Signed-off-by: Matthias Oefelein --- drivers/staging/bcm/InterfaceTx.c | 63 - 1 file changed, 20 insertions(+), 43 deletions(-) diff

[PATCH 4/4] beeceem: Replace C99-style comments with C89 pendants in InterfaceTx.c

2013-12-30 Thread Ralph Mueck
Checkpatch complains about the use of //-comments, thus they are replaced by C89-style comments. Signed-off-by: Ralph Mueck Signed-off-by: Matthias Oefelein --- drivers/staging/bcm/InterfaceTx.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drive

[PATCH 2/4] beeceem: Add spaces before opening parentheses in if-conditionals in InterfaceTx.c

2013-12-30 Thread Ralph Mueck
This patch adds spaces before the opening parentheses of if-conditionals to comply with offical code styling guidelines. Signed-off-by: Ralph Mueck Signed-off-by: Matthias Oefelein --- drivers/staging/bcm/InterfaceTx.c | 30 +++--- 1 file changed, 15 insertions(+), 1

[PATCH 1/4] beeceem: Fix several whitespace issues in InterfaceTx.c

2013-12-30 Thread Ralph Mueck
This patch takes care of the following issues: - missing spaces at variable assignments - missing spaces between function arguments - spaces at the beginning of a line where tabs should be - prohibited spaces before a (semi)colon Signed-off-by: Ralph Mueck Signed-