Align #define macro.
Signed-off-by: Chaehyun Lim chaehyun@gmail.com
---
drivers/staging/wilc1000/host_interface.c | 93 +++
1 file changed, 46 insertions(+), 47 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
Remove unnecessary typecast in #define macro.
Signed-off-by: Chaehyun Lim chaehyun@gmail.com
---
drivers/staging/wilc1000/host_interface.c | 78 +++
1 file changed, 39 insertions(+), 39 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch silences the checkpatch warning 'min() should
probably be min_t'
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/lustre/lnet/selftest/rpc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
On Thu, Jun 18, 2015 at 1:54 PM, Guenter Roeck li...@roeck-us.net wrote:
On 06/17/2015 11:53 PM, Frans Klaver wrote:
On Thu, Jun 18, 2015 at 3:04 AM, Stephen Boyd sb...@codeaurora.org
wrote:
On 10/06/2014 10:28 PM, Guenter Roeck wrote:
Various drivers implement architecture and/or device
From: Hari Prasath Gujulan Elango hguju...@visteon.com
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 hguju...@visteon.com
---
drivers/staging/sm750fb/sm750.c | 6 +++---
1 file changed, 3
1) else statement after if is unnecessory, hence removed.
2) Fixed line over 80 characters warning.
Signed-off-by: Sunil Shahu shsh...@gmail.com
---
drivers/staging/rtl8712/rtl871x_security.c | 39 ++
1 file changed, 18 insertions(+), 21 deletions(-)
diff --git
From: Hari Prasath Gujulan Elango hguju...@visteon.com
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 hguju...@visteon.com
---
On 06/17/2015 11:53 PM, Frans Klaver wrote:
On Thu, Jun 18, 2015 at 3:04 AM, Stephen Boyd sb...@codeaurora.org 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
From: Abdul Hussain hab...@visteon.com
This patch removes check before freeing the memory
since kfree(NULL) is safe
Signed-off-by: Abdul Hussain hab...@visteon.com
---
drivers/staging/wilc1000/linux_wlan_spi.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch fixes a typo in the comments section as warned by
checkpatch.pl
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/dgnc/dgnc_driver.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
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
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 tell
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
Fix checkpatch warnings about lines over 80 characters.
Signed-off-by: Peter Karlsson pe...@zapto.se
---
drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 55 +++-
1 file changed, 35 insertions(+), 20 deletions(-)
diff --git
Fix checkpatch warnings about lines over 80 characters.
Signed-off-by: Peter Karlsson pe...@zapto.se
---
.../staging/ft1000/ft1000-usb/ft1000_download.c| 41 +++---
1 file changed, 21 insertions(+), 20 deletions(-)
diff --git
From: Hari Prasath Gujulan Elango hguju...@visteon.com
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 hguju...@visteon.com
---
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 local
Fix checkpatch warnings about lines over 80 characters.
Signed-off-by: Peter Karlsson pe...@zapto.se
---
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
Fix checkpatch warnings about CVS keyword marker.
Signed-off-by: Peter Karlsson pe...@zapto.se
---
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
Fix checkpatch warnings about lines over 80 characters.
Signed-off-by: Peter Karlsson pe...@zapto.se
---
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
On Thu, Jun 18, 2015 at 10:30 AM CEST, Sudip Mukherjee
sudipm.mukher...@gmail.com 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 jsitni...@gmail.com
---
snip
diff
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 Hartley
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
Fixes checkpatch error regarding the space before open square bracket '['
Signed-off-by: Jimmy Devine fxmul...@gmail.com
---
drivers/staging/rtl8188eu/include/odm.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8188eu/include/odm.h
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
[]
@@ -414,12
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 fxmul...@gmail.com
Can you resend this with a subject that gives us a hint as to what part
of the kernel it relates to?
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 int
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
___
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 isaac.a.trav...@gmail.com
---
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 geert+rene...@glider.be
Thanks Geert,
As Greg has now
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 hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/amplc_pci230.c | 44 +--
1 file
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
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 hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/das08_isa.c | 4 ++--
1 file changed, 2 insertions(+), 2
Make this function return a pointer to the boardinfo instead of an index.
Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/ni_at_a2150.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git
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 hswee...@visionengravers.com
---
v2: fix checkpatch issue
drivers/staging/comedi/drivers/ni_atmio.c | 27
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 hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/ni_at_a2150.c | 26 +-
1 file
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'
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 hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/adv_pci_dio.c | 48 ++--
1 file
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 hswee...@visionengravers.com
---
v2: new patch
drivers/staging/comedi/drivers/vmk80xx.c | 5 -
1 file changed, 4 insertions(+), 1
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 hswee...@visionengravers.com
---
v2: move sanity checking of 'context' to separate patch
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 hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/amplc_pc236_common.c | 12 ++--
1 file changed, 6
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 hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/cb_pcidas64.c | 218 +--
1 file
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 hswee...@visionengravers.com
---
v2: rename all 'thisboard' variables now
drivers/staging/comedi/drivers/das800.c | 36
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 hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/me4000.c | 30 +++---
1 file changed,
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 hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/das1800.c | 44
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 hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/cb_pcidda.c | 16
1 file changed, 8
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 hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/das1800.c | 50
1 file
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 hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/addi_apci_1516.c | 24
1 file
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 hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/amplc_pci224.c | 42 +--
1 file
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 hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/dt3000.c | 22 +++---
1 file changed, 11
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 hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/ni_670x.c | 12 ++--
1 file changed, 6 insertions(+),
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 hswee...@visionengravers.com
---
drivers/staging/comedi/drivers/addi_apci_3120.c | 14 +++---
1 file changed, 7
Fix checkpatch.pl 'foo * bar' should be 'foo *bar' errors
Signed-off-by: Greg Donald gdon...@gmail.com
---
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
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 bilel...@gmail.com
---
Fixes checkpatch error regarding the space before open square bracket '['
Signed-off-by: Jimmy Devine fxmul...@gmail.com
---
drivers/staging/rtl8188eu/include/odm.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8188eu/include/odm.h
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
On Tue, Jun 16, 2015 at 02:03:44PM +, Abdul, Hussain (H.) wrote:
From: Abdul Hussain hab...@visteon.com
This patch simplify a trivial if-return sequence. Possibly combine with
a preceding function call.
Signed-off-by: Abdul Hussain hab...@visteon.com
Always use
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 of
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
Patch 2:
On Thu, Jun 18, 2015 at 02:15:09PM +, Gujulan Elango, Hari Prasath (H.)
wrote:
From: Hari Prasath Gujulan Elango hguju...@visteon.com
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
Fix warning founded by checkpatch.pl.
WARNING: __func__ should be used instead of gcc specific __FUNCTION__
Signed-off-by: Chaehyun Lim chaehyun@gmail.com
---
drivers/staging/wilc1000/linux_wlan.c| 2 +-
drivers/staging/wilc1000/linux_wlan_common.h | 16
On Wed, Jun 17, 2015 at 09:40:42PM -0700, gre...@linuxfoundation.org wrote:
On Tue, Jun 16, 2015 at 05:09:35AM +, Gujulan Elango, Hari Prasath (H.)
wrote:
On Mon, Jun 15, 2015 at 09:55:15PM -0700, gre...@linuxfoundation.org wrote:
On Mon, Jun 15, 2015 at 11:44:24AM +, Gujulan
From: Hari Prasath Gujulan Elango hguju...@visteon.com
This patch removed a undefined macro and debug prints associated with
it.
Signed-off-by: Hari Prasath Gujulan Elango hguju...@visteon.com
---
drivers/staging/wlan-ng/p80211wep.c | 14 --
1 file changed, 14 deletions(-)
diff
On Thu, Jun 18, 2015 at 3:04 AM, Stephen Boyd sb...@codeaurora.org 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 variable pm_power_off to
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
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 hswee...@visionengravers.com
---
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 from
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
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 hswee...@visionengravers.com
---
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 jsitni...@gmail.com
---
snip
diff --git a/drivers/staging/rtl8188eu/hal/odm.c
b/drivers/staging/rtl8188eu/hal/odm.c
index
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()
Flag is never set. Remove it and the code that is dead because of it.
Signed-off-by: Jakub Sitnicki jsitni...@gmail.com
---
drivers/staging/rtl8188eu/hal/odm.c | 11 --
drivers/staging/rtl8188eu/hal/rtl8188e_cmd.c | 21 --
72 matches
Mail list logo