Re: [PATCH 1/6] Staging: comedi: dmm32at: Simplify a trivial if-return sequence

2015-06-18 Thread Abdul, Hussain (H.)
On Friday 19 June 2015 04:30 AM, gre...@linuxfoundation.org wrote: > On Tue, Jun 16, 2015 at 02:03:44PM +, Abdul, Hussain (H.) wrote: >> From: Abdul Hussain >> >> This patch simplify a trivial if-return sequence. Possibly combine with >> a preceding function call. >> >> Signed-off-by: Abdul Hu

[PATCH 5/6] Staging: comedi: ni_daq_dio24: Simplify a trivial if-return sequence

2015-06-18 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/ni_daq_dio24.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/

[PATCH 6/6] Staging: comedi: s626: Simplify a trivial if-return sequence

2015-06-18 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/s626.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drivers/

[PATCH 3/6] Staging: comedi: daqboard2000: Simplify a trivial if-return sequence

2015-06-18 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/daqboard2000.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/staging/comedi

[PATCH 4/6] Staging: comedi: dac02: Simplify a trivial if-return sequence

2015-06-18 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/dac02.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drivers

[PATCH 1/6] Staging: comedi: dmm32at: Simplify a trivial if-return sequence

2015-06-18 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/dmm32at.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drive

[PATCH 2/6] Staging: comedi: fl512: Simplify a trivial if-return sequence

2015-06-18 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/fl512.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drivers

[PATCH 0/6] Staging: comedi: Simplify a trivial if-return sequence

2015-06-18 Thread Abdul, Hussain (H.)
This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Abdul Hussain (6): staging: comedi: dmm32at: Simplify a trivial if-return sequence staging: comedi: fl512: Simplify a trivial if-return sequence staging: comedi: daqboard2000: Simplify

Re: [PATCH 9/9] Staging: sm750fb: ddk750_swi2c.c: Shorten lines to 80 characters

2015-06-18 Thread Greg KH
On Thu, Jun 18, 2015 at 09:48:56PM -0700, Isaac Assegai wrote: > Shorten lines to be under 80 characters in ddk750_swi2c.c > to rectify the following checkpatch warnings: > WARNING: line over 80 characters > > Signed-off-by: Isaac Assegai > --- > drivers/staging/sm750fb/ddk750_swi2c.c | 18 +

Re: [PATCH] drivers: staging: rtl8712: Resolved kernel coding style warnings

2015-06-18 Thread Greg KH
On Thu, Jun 18, 2015 at 06:57:34PM +0530, Sunil Shahu wrote: > 1) "else" statement after "if" is unnecessory, hence removed. > 2) Fixed line over 80 characters warning. That's two different things, this should be 2 different patches. thanks, greg k-h _

Re: [PATCH v2 7/7] ARM: shmobile: armadillo800eva dts: Add pinctrl and gpio-hog for lcdc0

2015-06-18 Thread Simon Horman
On Wed, Jun 17, 2015 at 10:38:56AM +0200, Geert Uytterhoeven wrote: > Configure pinctrl and a GPIO-controller board mux for LCD use. > This allows the armadillo800eva board staging code to enable lcdc0. > > Signed-off-by: Geert Uytterhoeven Thanks Geert, As Greg has now taken patches 1 - 6 I ha

Re: [PATCH] staging: wlan-ng: Fix memset warning found by sparse

2015-06-18 Thread Joe Perches
On Thu, 2015-06-18 at 16:04 -0700, Greg KH wrote: > On Thu, Jun 18, 2015 at 06:49:51PM +0100, Bilel DRIRA wrote: > > This patch fixes the following sparse warnings: [] > > diff --git a/drivers/staging/wlan-ng/prism2fw.c > > b/drivers/staging/wlan-ng/prism2fw.c [] > > @@ -245,10 +245,12 @@ static i

[PATCH] staging: wilc1000: fix checkpatch warning

2015-06-18 Thread Chaehyun Lim
Fix warning founded by checkpatch.pl. WARNING: __func__ should be used instead of gcc specific __FUNCTION__ Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/linux_wlan.c| 2 +- drivers/staging/wilc1000/linux_wlan_common.h | 16 drivers/staging/wilc1000/wilc_plat

Re: [PATCH] staging: wlan-ng: Fix memset warning found by sparse

2015-06-18 Thread Greg KH
On Thu, Jun 18, 2015 at 06:49:51PM +0100, Bilel DRIRA wrote: > This patch fixes the following sparse warnings: > > drivers/staging/wlan-ng/prism2fw.c:251:15: warning: memset with byte count of > 12 > drivers/staging/wlan-ng/prism2fw.c:470:15: warning: memset with byte count of > 12 > >

Re: [PATCH] staging: ste_rmi4: remove redundant __func__ from debug prints

2015-06-18 Thread gre...@linuxfoundation.org
On Thu, Jun 18, 2015 at 02:15:09PM +, Gujulan Elango, Hari Prasath (H.) wrote: > From: Hari Prasath Gujulan Elango > > This patch removes the redundant __func__ from dynamic debug prints as > the pr_xxx set of functions can be dynamically controlled to include > function name as well. But t

Re: [PATCH v4 0/7] staging: fsl-mc: New functionality to the MC bus driver

2015-06-18 Thread Greg KH
On Thu, Jun 18, 2015 at 03:01:22PM +, Jose Rivera wrote: > Greg, > > Thanks for your feedback. It is our fault that we did not articulate > clearly our intent with this patch series. First, for reference, let > us summarize here the patches in question: > > Patch 1: MC bus IRQ support > Patc

Re: [PATCH 1/6] Staging: comedi: dmm32at: Simplify a trivial if-return sequence

2015-06-18 Thread gre...@linuxfoundation.org
On Tue, Jun 16, 2015 at 02:03:44PM +, Abdul, Hussain (H.) wrote: > From: Abdul Hussain > > This patch simplify a trivial if-return sequence. Possibly combine with > a preceding function call. > > Signed-off-by: Abdul Hussain Always use scripts/get_maintainer.pl to properly cc the maintaine

Re: [PATCH 01/44] kernel: Add support for poweroff handler call chain

2015-06-18 Thread Stephen Boyd
On 06/18/2015 08:30 AM, Guenter Roeck wrote: > On Wed, Jun 17, 2015 at 06:04:54PM -0700, Stephen Boyd wrote: > [ ... ] >> What happened to this series? I want to add shutdown support to my >> platform and I need to write a register on the PMIC in one driver to >> configure it for shutdown instead o

Re: [PATCH v2 00/23] staging: comedi: cleanup boardinfo usage in drivers

2015-06-18 Thread Ian Abbott
On 18/06/15 18:54, H Hartley Sweeten wrote: Many of the comedi drivers support multiple boards with different features. The differences are usually handled by some type of const array that describes the board. A pointer this data is stored in the comedi_device as the 'dev->board_ptr'. Most of th

[PATCH] drivers: staging: sm750fb: Fix "'foo * bar' should be 'foo *bar'" errors

2015-06-18 Thread Greg Donald
Fix checkpatch.pl "'foo * bar' should be 'foo *bar'" errors Signed-off-by: Greg Donald --- drivers/staging/sm750fb/ddk750_chip.c | 2 +- drivers/staging/sm750fb/ddk750_help.c | 4 ++-- drivers/staging/sm750fb/ddk750_help.h | 2 +- drivers/staging/sm750fb/ddk750_mode.c | 4 ++-- drivers/s

[PATCH] staging: rtl8188eu: remove space before open square bracket '['

2015-06-18 Thread Jimmy Devine
Fixes checkpatch error regarding the space before open square bracket '[' Signed-off-by: Jimmy Devine --- drivers/staging/rtl8188eu/include/odm.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/include/odm.h b/drivers/staging/rtl8188eu/include/odm.h

[PATCH v2 17/23] staging: comedi: adv_pci_dio: rename 'this_board' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'this_board' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/adv_pci_dio.c | 48 ++-- 1 file changed, 24 insertions(+), 24 del

[PATCH v2 00/23] staging: comedi: cleanup boardinfo usage in drivers

2015-06-18 Thread H Hartley Sweeten
Many of the comedi drivers support multiple boards with different features. The differences are usually handled by some type of const array that describes the board. A pointer this data is stored in the comedi_device as the 'dev->board_ptr'. Most of the drivers use a local variable called 'board'

[PATCH v2 07/23] staging: comedi: ni_at_a2150: rename 'thisboard' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'thisboard' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/ni_at_a2150.c | 26 +- 1 file changed, 13 insertions(+), 13 deleti

[PATCH v2 04/23] staging: comedi: cb_pcidda: rename 'thisboard' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'thisboard' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/cb_pcidda.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff -

[PATCH v2 12/23] staging: comedi: das1800: cleanup das1800_probe()

2015-06-18 Thread H Hartley Sweeten
Make this function return a pointer to the boardinfo instead of an index. Since it's not typical, add a comment to clarify why this legacy driver does a probe. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/das1800.c | 44 +--- 1 file changed, 23

[PATCH v2 11/23] staging: comedi: das800: rename 'thisboard' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'thisboard' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- v2: rename all 'thisboard' variables now drivers/staging/comedi/drivers/das800.c | 36 -

[PATCH v2 13/23] staging: comedi: das1800: rename 'thisboard' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'thisboard' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/das1800.c | 50 1 file changed, 25 insertions(+), 25 dele

[PATCH v2 22/23] staging: comedi: addi_apci_1516: rename 'this_board' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'this_board' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/addi_apci_1516.c | 24 1 file changed, 12 insertions(+), 12 dele

[PATCH v2 01/23] staging: comedi: amplc_pc236_common: rename 'thisboard' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'thisboard' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/amplc_pc236_common.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) d

[PATCH v2 19/23] staging: comedi: vmk80xx: rename 'boardinfo' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'boardinfo' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- v2: move sanity checking of 'context' to separate patch drivers/staging/comedi/drivers/vmk80xx.c | 30 +++---

[PATCH v2 20/23] staging: comedi: vmk80xx: sanity check context used to get the boardinfo

2015-06-18 Thread H Hartley Sweeten
As done in all the comedi drivers that auto attach, sanity check the passed context that is used to get the boardinfo. Signed-off-by: H Hartley Sweeten --- v2: new patch drivers/staging/comedi/drivers/vmk80xx.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/stag

[PATCH v2 09/23] staging: comedi: das08: rename 'thisboard' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'thisboard' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/das08.c | 51 +- 1 file changed, 25 insertions(+), 26 dele

[PATCH v2 16/23] staging: comedi: cb_pcidas64: rename 'thisboard' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'thisboard' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/cb_pcidas64.c | 218 +-- 1 file changed, 108 insertions(+), 110 de

[PATCH v2 14/23] staging: comedi: me4000: rename 'thisboard' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'thisboard' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/me4000.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletio

[PATCH v2 15/23] staging: comedi: cb_pcidas: rename 'thisboard' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'thisboard' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/cb_pcidas.c | 64 +++--- 1 file changed, 32 insertions(+), 32 dele

[PATCH v2 05/23] staging: comedi: ni_670x: rename 'thisboard' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'thisboard' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/ni_670x.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a

[PATCH v2 23/23] staging: comedi: addi_apci_3120: rename 'this_board' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'this_board' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/addi_apci_3120.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) di

[PATCH v2 02/23] staging: comedi: amplc_pci224: rename 'thisboard' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'thisboard' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/amplc_pci224.c | 42 +-- 1 file changed, 21 insertions(+), 21 dele

[PATCH v2 18/23] staging: comedi: dt3000: rename 'this_board' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'this_board' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/dt3000.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-)

[PATCH v2 10/23] staging: comedi: das800: cleanup das800_probe()

2015-06-18 Thread H Hartley Sweeten
Make this function return a pointer to the boardinfo instead of an index. Since it's not typical, add a comment to clarify why this legacy driver does a probe. For aesthetics, rename the local variable 'board' to 'index. Consolidate the dev_dbg() messages when a probed boardinfo is used and remov

[PATCH v2 08/23] staging: comedi: das08_isa: rename 'thisboard' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'thisboard' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/das08_isa.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/driver

[PATCH v2 03/23] staging: comedi: amplc_pci230: rename 'thisboard' variables

2015-06-18 Thread H Hartley Sweeten
For aesthetics, rename the 'thisboard' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/amplc_pci230.c | 44 +-- 1 file changed, 22 insertions(+), 22 dele

[PATCH v2 06/23] staging: comedi: ni_at_a2150: cleanup a2150_probe()

2015-06-18 Thread H Hartley Sweeten
Make this function return a pointer to the boardinfo instead of an index. Signed-off-by: H Hartley Sweeten --- drivers/staging/comedi/drivers/ni_at_a2150.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/comedi/drivers/ni_at_a2150.c b/drive

[PATCH v2 21/23] staging: comedi: ni_atmio: cleanup ni_getboardtype()

2015-06-18 Thread H Hartley Sweeten
Make this function return a pointer to the boardinfo instead of an index. For aesthetics, rename the function to ni_atmio_probe(). Signed-off-by: H Hartley Sweeten --- v2: fix checkpatch issue drivers/staging/comedi/drivers/ni_atmio.c | 27 --- 1 file changed, 12 insert

[PATCH] staging: wlan-ng: Fix memset warning found by sparse

2015-06-18 Thread Bilel DRIRA
This patch fixes the following sparse warnings: drivers/staging/wlan-ng/prism2fw.c:251:15: warning: memset with byte count of 12 drivers/staging/wlan-ng/prism2fw.c:470:15: warning: memset with byte count of 12 Signed-off-by: Bilel DRIRA --- drivers/staging/wlan-ng/prism2fw.c |8 ++

RE: [PATCH 19/22] staging: comedi: vmk80xx: rename 'boardinfo' variables

2015-06-18 Thread Hartley Sweeten
On Thursday, June 18, 2015 1:03 AM, Sudip Mukherjee wrote: > On Wed, Jun 17, 2015 at 03:22:13PM -0700, H Hartley Sweeten wrote: >> For aesthetics, rename the 'boardinfo' variables to 'board'. That name >> is more commonly used for the boardinfo pointer in comedi drivers. >> >> Signed-off-by: H Har

Re: [PATCH] staging: rtl8188eu: kill unused hal_data_8188e::fw_ractrl flag

2015-06-18 Thread Jakub Sitnicki
On Thu, Jun 18, 2015 at 10:30 AM CEST, Sudip Mukherjee wrote: > On Thu, Jun 18, 2015 at 08:31:59AM +0200, Jakub Sitnicki wrote: >> Flag is never set. Remove it and the code that is dead because of it. >> >> Signed-off-by: Jakub Sitnicki >> --- > >> >> diff --git a/drivers/staging/rtl8188eu/ha

Re: [PATCH 1/5] staging: ft1000-usb: Fixed some issues with lines over 80 characters

2015-06-18 Thread Peter Karlsson
On 2015-06-18 08:51, Joe Perches wrote: > On Thu, 2015-06-18 at 15:50 +0200, Peter Karlsson wrote: > > Fix checkpatch warnings about lines over 80 characters. > [] > > diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_debug.c > > b/drivers/staging/ft1000/ft1000-usb/ft1000_debug.c > [] > > @@ -

Re: [PATCH] Remove space before open square bracket '['

2015-06-18 Thread Greg Kroah-Hartman
On Thu, Jun 18, 2015 at 09:51:00AM -0600, Jimmy Devine wrote: > Fixes checkpatch error regarding the space before open square bracket '[' > > Signed-off-by: Jimmy Devine Can you resend this with a subject that gives us a hint as to what part of the kernel it relates to? Something like:

[PATCH] Remove space before open square bracket '['

2015-06-18 Thread Jimmy Devine
Fixes checkpatch error regarding the space before open square bracket '[' Signed-off-by: Jimmy Devine --- drivers/staging/rtl8188eu/include/odm.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/include/odm.h b/drivers/staging/rtl8188eu/include/odm.h

Re: [PATCH 1/5] staging: ft1000-usb: Fixed some issues with lines over 80 characters

2015-06-18 Thread Joe Perches
On Thu, 2015-06-18 at 15:50 +0200, Peter Karlsson wrote: > Fix checkpatch warnings about lines over 80 characters. [] > diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_debug.c > b/drivers/staging/ft1000/ft1000-usb/ft1000_debug.c [] > @@ -414,12 +414,20 @@ static long ft1000_ioctl(struct file

Re: [PATCH 01/44] kernel: Add support for poweroff handler call chain

2015-06-18 Thread Guenter Roeck
On Wed, Jun 17, 2015 at 06:04:54PM -0700, Stephen Boyd wrote: [ ... ] > > What happened to this series? I want to add shutdown support to my > platform and I need to write a register on the PMIC in one driver to > configure it for shutdown instead of restart and then write an MMIO > register to te

RE: [PATCH v4 0/7] staging: fsl-mc: New functionality to the MC bus driver

2015-06-18 Thread Jose Rivera
Greg, Thanks for your feedback. It is our fault that we did not articulate clearly our intent with this patch series. First, for reference, let us summarize here the patches in question: Patch 1: MC bus IRQ support Patch 2: add device binding path 'driver_override' Patch 3: Propagate driver_over

Re: [PATCH 4/5] staging: wilc1000: remove prohibited spaces.

2015-06-18 Thread Greg KH
On Thu, Jun 18, 2015 at 01:51:15PM +0900, 조성훈 wrote: > Hi Greg, > I want to make sure one thing before going further. I cloned the > staging-testing and then made the first patch. There was no problem when I > sent > the first path to you. > Afterwards, I also pulled the repository to update my lo

[PATCH] staging: ste_rmi4: remove redundant __func__ from debug prints

2015-06-18 Thread Gujulan Elango, Hari Prasath (H.)
From: Hari Prasath Gujulan Elango This patch removes the redundant __func__ from dynamic debug prints as the pr_xxx set of functions can be dynamically controlled to include function name as well. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/ste_rmi4/synaptics_i2c_rmi4.c | 44

[PATCH 3/5] staging: ft1000-usb: Fixed some issues with lines over 80 characters

2015-06-18 Thread Peter Karlsson
Fix checkpatch warnings about lines over 80 characters. Signed-off-by: Peter Karlsson --- drivers/staging/ft1000/ft1000-usb/ft1000_hw.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_hw.c b/drivers/staging/ft1000/ft10

[PATCH 4/5] staging: ft1000-usb: Fixed some issues with lines over 80 characters

2015-06-18 Thread Peter Karlsson
Fix checkpatch warnings about lines over 80 characters. Signed-off-by: Peter Karlsson --- drivers/staging/ft1000/ft1000-usb/ft1000_ioctl.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_ioctl.h b/drivers/staging/ft1000/f

[PATCH 5/5] staging: ft1000-usb: Removed CVS keyword marker

2015-06-18 Thread Peter Karlsson
Fix checkpatch warnings about CVS keyword marker. Signed-off-by: Peter Karlsson --- drivers/staging/ft1000/ft1000-usb/ft1000_usb.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_usb.c b/drivers/staging/ft1000/ft1000-usb/ft1000_usb.c index 3b303b6..

[PATCH 2/5] staging: ft1000-usb: Fixed some issues with lines over 80 characters

2015-06-18 Thread Peter Karlsson
Fix checkpatch warnings about lines over 80 characters. Signed-off-by: Peter Karlsson --- .../staging/ft1000/ft1000-usb/ft1000_download.c| 41 +++--- 1 file changed, 21 insertions(+), 20 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_download.c b/drivers

[PATCH 1/5] staging: ft1000-usb: Fixed some issues with lines over 80 characters

2015-06-18 Thread Peter Karlsson
Fix checkpatch warnings about lines over 80 characters. Signed-off-by: Peter Karlsson --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 55 +++- 1 file changed, 35 insertions(+), 20 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_debug.c b/drivers/st

[PATCH 0/5] staging: ft1000-usb: checkpatch fixes

2015-06-18 Thread Peter Karlsson
Fixed checkpatch style issues. Peter Karlsson (5): staging: ft1000-usb: Fixed some issues with lines over 80 characters staging: ft1000-usb: Fixed some issues with lines over 80 characters staging: ft1000-usb: Fixed some issues with lines over 80 characters staging: ft1000-usb: Fixed some

[PATCH] staging: dgnc: fix typo in comments secion

2015-06-18 Thread Gujulan Elango, Hari Prasath (H.)
From: Hari Prasath Gujulan Elango This patch fixes a typo in the comments section as warned by checkpatch.pl Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/dgnc/dgnc_driver.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_driver.h

[PATCH] staging: dgnc: free memory allocated

2015-06-18 Thread Gujulan Elango, Hari Prasath (H.)
From: Hari Prasath Gujulan Elango The memory allocated in dgnc_tty_register() for two objects is not freed anywhere.This patch addresses this by freeing the memory in dgnc_tty_uninit. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/dgnc/dgnc_tty.c | 4 1 file changed, 4 ins

[PATCH] drivers: staging: rtl8712: Resolved kernel coding style warnings

2015-06-18 Thread Sunil Shahu
1) "else" statement after "if" is unnecessory, hence removed. 2) Fixed line over 80 characters warning. Signed-off-by: Sunil Shahu --- drivers/staging/rtl8712/rtl871x_security.c | 39 ++ 1 file changed, 18 insertions(+), 21 deletions(-) diff --git a/drivers/staging/r

[PATCH 1/2] staging: wilc1000: align #define macro

2015-06-18 Thread Chaehyun Lim
Align #define macro. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/host_interface.c | 93 +++ 1 file changed, 46 insertions(+), 47 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c index 322882

[PATCH 2/2] staging: wilc1000: remove unnecessary typecast in #define macro

2015-06-18 Thread Chaehyun Lim
Remove unnecessary typecast in #define macro. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/host_interface.c | 78 +++ 1 file changed, 39 insertions(+), 39 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host

[PATCH] staging: sm750fb: convert pr_err to pr_info

2015-06-18 Thread Gujulan Elango, Hari Prasath (H.)
From: Hari Prasath Gujulan Elango This patch modifies few debug prints from pr_err() to pr_info() as they fall under that category. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/sm750fb/sm750.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/s

[PATCH] Staging: wilc1000: NULL check before some freeing functions is not needed

2015-06-18 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch removes check before freeing the memory since kfree(NULL) is safe Signed-off-by: Abdul Hussain --- drivers/staging/wilc1000/linux_wlan_spi.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/wilc1000/linux_wlan_spi.c b/dr

Re: [PATCH 01/44] kernel: Add support for poweroff handler call chain

2015-06-18 Thread Frans Klaver
On Thu, Jun 18, 2015 at 1:54 PM, Guenter Roeck wrote: > On 06/17/2015 11:53 PM, Frans Klaver wrote: >> >> On Thu, Jun 18, 2015 at 3:04 AM, Stephen Boyd >> wrote: >>> >>> On 10/06/2014 10:28 PM, Guenter Roeck wrote: Various drivers implement architecture and/or device specific means to >

Re: [PATCH 01/44] kernel: Add support for poweroff handler call chain

2015-06-18 Thread Guenter Roeck
On 06/17/2015 11:53 PM, Frans Klaver wrote: On Thu, Jun 18, 2015 at 3:04 AM, Stephen Boyd wrote: On 10/06/2014 10:28 PM, Guenter Roeck wrote: Various drivers implement architecture and/or device specific means to remove power from the system. For the most part, those drivers set the global va

[PATCH] staging: lustre: lnet: selftest: use min_t in place of min

2015-06-18 Thread Gujulan Elango, Hari Prasath (H.)
From: Hari Prasath Gujulan Elango This patch silences the checkpatch warning 'min() should probably be min_t' Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/lustre/lnet/selftest/rpc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lne

Re: [PATCH 19/22] staging: comedi: vmk80xx: rename 'boardinfo' variables

2015-06-18 Thread Ian Abbott
On 18/06/15 09:02, Sudip Mukherjee wrote: On Wed, Jun 17, 2015 at 03:22:13PM -0700, H Hartley Sweeten wrote: For aesthetics, rename the 'boardinfo' variables to 'board'. That name is more commonly used for the boardinfo pointer in comedi drivers. Signed-off-by: H Hartley Sweeten --- drivers/

Re: [PATCH 10/22] staging: comedi: das800: cleanup das800_probe()

2015-06-18 Thread Ian Abbott
On 18/06/15 08:35, Sudip Mukherjee wrote: On Wed, Jun 17, 2015 at 03:22:04PM -0700, H Hartley Sweeten wrote: Make this function return a pointer to the boardinfo instead of an index. Since it's not typical, add a comment to clarify why this legacy driver does a probe. Consolidate the dev_dbg()

Re: [PATCH v2 0/9] fix sm7xxfb

2015-06-18 Thread Sudip Mukherjee
On Wed, Jun 17, 2015 at 09:45:40PM -0700, Greg Kroah-Hartman wrote: > On Wed, Jun 17, 2015 at 09:43:51PM -0700, Greg Kroah-Hartman wrote: > > On Wed, Jun 17, 2015 at 04:54:39PM +0530, Sudip Mukherjee wrote: > > > fixing the few remaining issues of sm7xxfb before sending the patch to > > > remove it

Re: [PATCH] staging: rtl8188eu: kill unused hal_data_8188e::fw_ractrl flag

2015-06-18 Thread Sudip Mukherjee
On Thu, Jun 18, 2015 at 08:31:59AM +0200, Jakub Sitnicki wrote: > Flag is never set. Remove it and the code that is dead because of it. > > Signed-off-by: Jakub Sitnicki > --- > > diff --git a/drivers/staging/rtl8188eu/hal/odm.c > b/drivers/staging/rtl8188eu/hal/odm.c > index 28b5e7b..710fdc3

Re: [PATCH 20/22] staging: comedi: ni_atmio: cleanup ni_getboardtype()

2015-06-18 Thread Sudip Mukherjee
On Wed, Jun 17, 2015 at 03:22:14PM -0700, H Hartley Sweeten wrote: > Make this function return a pointer to the boardinfo instead of an index. > > For aesthetics, rename the function to ni_atmio_probe(). > > Signed-off-by: H Hartley Sweeten > --- > drivers/staging/comedi/drivers/ni_atmio.c | 26

Re: [PATCH 19/22] staging: comedi: vmk80xx: rename 'boardinfo' variables

2015-06-18 Thread Sudip Mukherjee
On Wed, Jun 17, 2015 at 03:22:13PM -0700, H Hartley Sweeten wrote: > For aesthetics, rename the 'boardinfo' variables to 'board'. That name > is more commonly used for the boardinfo pointer in comedi drivers. > > Signed-off-by: H Hartley Sweeten > --- > drivers/staging/comedi/drivers/vmk80xx.c |

Re: [PATCH 10/22] staging: comedi: das800: cleanup das800_probe()

2015-06-18 Thread Sudip Mukherjee
On Wed, Jun 17, 2015 at 03:22:04PM -0700, H Hartley Sweeten wrote: > Make this function return a pointer to the boardinfo instead of an index. > Since it's not typical, add a comment to clarify why this legacy driver > does a probe. > > Consolidate the dev_dbg() messages when a probed boardinfo is