GitHub/exynos8895/android_kernel_samsung_universal8895.git
12 years agostaging: comedi: rtd520: remove RtdInterruptClearMask macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:54:15 +0000 (16:54 -0700)]
staging: comedi: rtd520: remove RtdInterruptClearMask macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'writew'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdInterruptClear macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:53:44 +0000 (16:53 -0700)]
staging: comedi: rtd520: remove RtdInterruptClear macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'readw'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdInterruptMask macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:53:15 +0000 (16:53 -0700)]
staging: comedi: rtd520: remove RtdInterruptMask macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'writew'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdInterruptStatus macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:52:45 +0000 (16:52 -0700)]
staging: comedi: rtd520: remove RtdInterruptStatus macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'readw'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdPacerStop macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:52:14 +0000 (16:52 -0700)]
staging: comedi: rtd520: remove RtdPacerStop macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'writel'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdPacerStart macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:51:46 +0000 (16:51 -0700)]
staging: comedi: rtd520: remove RtdPacerStart macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'readl'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdFifoStatus macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:51:16 +0000 (16:51 -0700)]
staging: comedi: rtd520: remove RtdFifoStatus macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'readl'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdAdcFifoGet2 macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:50:45 +0000 (16:50 -0700)]
staging: comedi: rtd520: remove RtdAdcFifoGet2 macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. It's also not used and the comment says it
does not work. Remove it.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdAdcFifoGet macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:50:12 +0000 (16:50 -0700)]
staging: comedi: rtd520: remove RtdAdcFifoGet macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'readw'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdAdcStart macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:49:28 +0000 (16:49 -0700)]
staging: comedi: rtd520: remove RtdAdcStart macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'writel'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdTriggerPolarity macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:48:58 +0000 (16:48 -0700)]
staging: comedi: rtd520: remove RtdTriggerPolarity macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. It's also not used. Remove it.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdAboutStopEnable macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:48:22 +0000 (16:48 -0700)]
staging: comedi: rtd520: remove RtdAboutStopEnable macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'writel'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdPacerTriggerMode macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:47:46 +0000 (16:47 -0700)]
staging: comedi: rtd520: remove RtdPacerTriggerMode macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. It's also not used. Remove it.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdAdcSampleCounterSource macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:47:12 +0000 (16:47 -0700)]
staging: comedi: rtd520: remove RtdAdcSampleCounterSource macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. It's also not used. Remove it.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdPacerClockSource macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:46:37 +0000 (16:46 -0700)]
staging: comedi: rtd520: remove RtdPacerClockSource macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'writel'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdPacerStopSource macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:45:57 +0000 (16:45 -0700)]
staging: comedi: rtd520: remove RtdPacerStopSource macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'writel'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdPacerStartSource macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:45:22 +0000 (16:45 -0700)]
staging: comedi: rtd520: remove RtdPacerStartSource macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'writel'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdBurstStartSource macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:44:49 +0000 (16:44 -0700)]
staging: comedi: rtd520: remove RtdBurstStartSource macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'writel'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdAdcConversionSource macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:44:15 +0000 (16:44 -0700)]
staging: comedi: rtd520: remove RtdAdcConversionSource macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'writel'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdAdcClearFifo macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:43:32 +0000 (16:43 -0700)]
staging: comedi: rtd520: remove RtdAdcClearFifo macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'writel'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdWriteCGLatch macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:42:48 +0000 (16:42 -0700)]
staging: comedi: rtd520: remove RtdWriteCGLatch macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'writel'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdWriteCGTable macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:42:08 +0000 (16:42 -0700)]
staging: comedi: rtd520: remove RtdWriteCGTable macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'writel'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdEnableCGT macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:41:29 +0000 (16:41 -0700)]
staging: comedi: rtd520: remove RtdEnableCGT macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'writel'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdClearCGT macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:40:55 +0000 (16:40 -0700)]
staging: comedi: rtd520: remove RtdClearCGT macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'writel'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdResetCGT macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:40:11 +0000 (16:40 -0700)]
staging: comedi: rtd520: remove RtdResetCGT macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. It's also not used. Remove it.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: rtd520: remove RtdResetBoard macro
H Hartley Sweeten [Tue, 10 Jul 2012 23:39:25 +0000 (16:39 -0700)]
staging: comedi: rtd520: remove RtdResetBoard macro

This macro uses the 'devpriv' macro which relies on a local variable
having a specific name. Plus it's just a wrapper around a simple
'writel'. Remove the macro.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoMerge tag 'togreg-3.6a' of git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio...
Greg Kroah-Hartman [Thu, 12 Jul 2012 19:29:44 +0000 (12:29 -0700)]
Merge tag 'togreg-3.6a' of git://git./linux/kernel/git/jic23/iio into staging-next

IIO: One new driver and a couple of nice cleanups.

12 years agoMerge tag 'fixes-togreg-3.6a' of git://git.kernel.org/pub/scm/linux/kernel/git/jic23...
Greg Kroah-Hartman [Thu, 12 Jul 2012 19:26:33 +0000 (12:26 -0700)]
Merge tag 'fixes-togreg-3.6a' of git://git./linux/kernel/git/jic23/iio into work-next

Various minor IIO fixes for staging-next.

12 years agoiio: double unlock on error path
Dan Carpenter [Wed, 11 Jul 2012 06:34:00 +0000 (07:34 +0100)]
iio: double unlock on error path

We should be holding the mutex when we goto error_free_chans.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
12 years agoiio: dac: ad5064: fix section mismatch in ad5064_init() in linux-next
Gerard Snitselaar [Wed, 11 Jul 2012 04:05:00 +0000 (05:05 +0100)]
iio: dac: ad5064: fix section mismatch in ad5064_init() in linux-next

ad5064_init() calls ad5064_spi_unregister_driver() which is annotated
__exit.

Signed-off-by: Gerard Snitselaar <dev@snitselaar.org>
Acked-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
12 years agoiio: ad5064: Move bus write callbacks to #if protected sections
Lars-Peter Clausen [Wed, 11 Jul 2012 08:01:00 +0000 (09:01 +0100)]
iio: ad5064: Move bus write callbacks to #if protected sections

Move the SPI and I2C specific write callbacks to the respective
"#if IS_ENABLED(CONFIG_SPI_MASTER)" and "#if IS_ENABLED(CONFIG_I2C)"
protected sections of the code.

This fixes the following warning which occurs if CONFIG_I2C is not set:
drivers/iio/dac/ad5064.c: In function ‘ad5064_i2c_write’:
drivers/iio/dac/ad5064.c:132: error: implicit declaration of function ‘i2c_master_send’

And the follwing warning which occurs when CONFIG_SPI_MASTER is not set:
drivers/iio/dac/ad5064.c:137: warning: ‘ad5064_spi_write’ defined but not used

Reported-by: Randy Dunlap <rdunlap@xenotime.net>
Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
12 years agostaging:iio:max1363: Don't free uninitialized variable
Lars-Peter Clausen [Wed, 11 Jul 2012 09:28:00 +0000 (10:28 +0100)]
staging:iio:max1363: Don't free uninitialized variable

It is possible that on one of the error paths we are going to try to free
'rxbuf', even though it has not been allocated yet, which cause the following
warning:
drivers/staging/iio/adc/max1363_ring.c: In function 'max1363_trigger_handler':
drivers/staging/iio/adc/max1363_ring.c:87:7: warning: 'rxbuf' may be used
uninitialized in this function

Reported-by: Fengguang Wu <wfg@linux.intel.com>
Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
12 years agoiio: add adjd_s311 I2C digital color sensor driver
Peter Meerwald [Tue, 10 Jul 2012 21:32:00 +0000 (22:32 +0100)]
iio: add adjd_s311 I2C digital color sensor driver

sensor has 4 channels (10-bit each, R/G/B and clear), sensitivity
and gain is controlled in the driver by ext_info integration_time
and CHAN_INFO_HARDWAREGAIN

driver supports triggered buffer and IIO_CHAN_INFO_RAW to get the
sensor data

v5: address comments by Jonathan Cameron
* use macro for channel declaration
* get timestamp right before measurement
* cleanups

v4: address comments by Lars-Peter Clausen
* make sure trigger handler is exited with iio_trigger_notify_done()
  and IRQ_HANDLED
* kfree()/kalloc() -> krealloc()

v3:
* fix warnings

v2: address comments by Lars-Peter Clausen
* buffer allocation now in update_scan_mode instead of in trigger
  handler
* simplify trigger code (assume active_scan_mask is not empty, use
  for_each_set_bit, use iio_push_to_buffer)
* reorder entry in Makefile and Kconfig
* fix remove

Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
12 years agostaging/slicoss: disable pci device at remove
Devendra Naga [Wed, 11 Jul 2012 06:09:15 +0000 (11:54 +0545)]
staging/slicoss: disable pci device at remove

at probe we enabled the device, and we should disable it at remove.

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/phison: use module_pci_driver macro
Devendra Naga [Tue, 10 Jul 2012 06:46:30 +0000 (12:16 +0530)]
staging/phison: use module_pci_driver macro

remove the duplication of module_pci_driver and use this macro instead

module_pci_driver macro does the same things as the code below does

static int __init pci_test_dev_init(void)
{
       return pci_register_driver(&pci_test_driver_ops);
}

static void __exit pci_test_dev_exit(void)
{
       pci_unregister_driver(&pci_test_driver_ops);
}

module_init(pci_test_dev_init);
module_exit(pci_test_dev_exit);

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/sbe-2t3e3: use module_pci_driver macro
Devendra Naga [Tue, 10 Jul 2012 06:40:04 +0000 (12:10 +0530)]
staging/sbe-2t3e3: use module_pci_driver macro

the _init and _exit functions do nothing but pci_register_driver,
and pci_unregister_driver,

so replace these and also the module _init and _exit
macros with the module_pci_driver macro.

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/et131x: use module_pci_driver macro
Devendra Naga [Tue, 10 Jul 2012 06:41:03 +0000 (12:11 +0530)]
staging/et131x: use module_pci_driver macro

remove code duplicating module_pci_driver
and also the obvious comments about the _init and _exit points.

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/sep: use module_pci_driver macro
Devendra Naga [Tue, 10 Jul 2012 06:41:48 +0000 (12:11 +0530)]
staging/sep: use module_pci_driver macro

this driver duplicates the module_pci_driver code, so remove the
duplicated code and use module_pci_driver and also remove the
obvious comments about _init and _exit.

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/sm7xxfb: return a proper err for smtc_alloc_fb_info failure
Devendra Naga [Tue, 10 Jul 2012 05:49:15 +0000 (11:19 +0530)]
staging/sm7xxfb: return a proper err for smtc_alloc_fb_info failure

as smtc_alloc_fb_info can fail, but we are returning the 0,
how? because the pci_enable_device succeeded, which makes the probe
return 0, and may cause panics or some strange problems at remove
when driver unloaded by modprobe -r.

so return err properly as smtc_alloc_fb_info is doing kzallocs its
good to do -ENOMEM

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/sm7xxfb: use module_pci_driver macro
Devendra Naga [Tue, 10 Jul 2012 05:49:14 +0000 (11:19 +0530)]
staging/sm7xxfb: use module_pci_driver macro

as the manual of module_pci_driver says that
it can be used when the init and exit functions of
the module does nothing but the pci_register_driver
and pci_unregister_driver.

use it for the sm7xxfb driver, as the driver does nothing in
its _init and _exit functions but the register and unregister.

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: ft1000: Fix typo in ft1000_usb.c
Masanari Iida [Wed, 11 Jul 2012 15:41:44 +0000 (00:41 +0900)]
staging: ft1000: Fix typo in ft1000_usb.c

Correct spelling typo within debug message of ft1000_usb.c

Signed-off-by: Masanari Iida <standby24x7@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/ft1000: remove all spaces, tabs issue in while loop of ft1000_interrupt
Devendra Naga [Tue, 10 Jul 2012 05:06:05 +0000 (10:36 +0530)]
staging/ft1000: remove all spaces, tabs issue in while loop of ft1000_interrupt

all spaces in the main while loop of ft1000_interrupt are converted
to tabs.

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: csr: Remove unused macros
Marcos Paulo de Souza [Tue, 10 Jul 2012 02:43:13 +0000 (23:43 -0300)]
staging: csr: Remove unused macros

These macros are not used, so remove it.

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: csr: unifi_sme.c: Remove unused macros
Marcos Paulo de Souza [Tue, 10 Jul 2012 02:43:12 +0000 (23:43 -0300)]
staging: csr: unifi_sme.c: Remove unused macros

These macros are not used, so remove it.

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: csr: netdev.c: Remove unused macros
Marcos Paulo de Souza [Tue, 10 Jul 2012 02:43:11 +0000 (23:43 -0300)]
staging: csr: netdev.c: Remove unused macros

These macros are not used, so we can remove it.

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: csr: csr_util.h: Remove commented macros
Marcos Paulo de Souza [Tue, 10 Jul 2012 02:43:10 +0000 (23:43 -0300)]
staging: csr: csr_util.h: Remove commented macros

All these macros are commented, so we can removed it.

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: csr: Remove dead code
Marcos Paulo de Souza [Tue, 10 Jul 2012 02:43:09 +0000 (23:43 -0300)]
staging: csr: Remove dead code

All this code is dead, so remove it.

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: bcm: Misc.c: Remove unused macros
Marcos Paulo de Souza [Tue, 10 Jul 2012 01:32:40 +0000 (22:32 -0300)]
staging: bcm: Misc.c: Remove unused macros

These macros were reported by forgotten-macros tool
(https://github.com/marcosps/forgotten_macros).

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: bcm: Remove all commented macros
Marcos Paulo de Souza [Tue, 10 Jul 2012 01:32:39 +0000 (22:32 -0300)]
staging: bcm: Remove all commented macros

All these macros are commented, and can be removed.

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: bcm: Remove code that will never execute
Marcos Paulo de Souza [Tue, 10 Jul 2012 01:32:38 +0000 (22:32 -0300)]
staging: bcm: Remove code that will never execute

This patch removes all references of "if 0" blocks in the bcm drivers.

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Remove useless condition.
Miguel Gómez [Fri, 6 Jul 2012 10:40:54 +0000 (12:40 +0200)]
Staging: xgifb: Remove useless condition.

Remove last if branch in XGIfb_do_set_var(), as it has no action assigned.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Fix coding style.
Miguel Gómez [Fri, 6 Jul 2012 10:40:53 +0000 (12:40 +0200)]
Staging: xgifb: Fix coding style.

Fix small coding style warnings in several files.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Rework some conditions in XGI_SetCRT2ModeRegs().
Miguel Gómez [Fri, 6 Jul 2012 10:40:52 +0000 (12:40 +0200)]
Staging: xgifb: Rework some conditions in XGI_SetCRT2ModeRegs().

Rework some conditions to reduce indentation and fix style warnings.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Rework conditions in XGI_EnableBridge().
Miguel Gómez [Fri, 6 Jul 2012 10:40:51 +0000 (12:40 +0200)]
Staging: xgifb: Rework conditions in XGI_EnableBridge().

Rework some conditions to reduce indentation and fix style warnings.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Rework conditions in XGI_SetLockRegs().
Miguel Gómez [Fri, 6 Jul 2012 10:40:50 +0000 (12:40 +0200)]
Staging: xgifb: Rework conditions in XGI_SetLockRegs().

Rework some conditions to reduce indentation and fix style warnings.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Refactor XGI_GetCRT2ResInfo().
Miguel Gómez [Fri, 6 Jul 2012 10:40:49 +0000 (12:40 +0200)]
Staging: xgifb: Refactor XGI_GetCRT2ResInfo().

Use a goto to remove one level of indentation, fixing style warnings.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Rework conditions in XGI_GetVBInfo().
Miguel Gómez [Fri, 6 Jul 2012 10:40:48 +0000 (12:40 +0200)]
Staging: xgifb: Rework conditions in XGI_GetVBInfo().

Rework some conditions to reduce indentation and fix style warnings.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Refactor XGI_GetVBType().
Miguel Gómez [Fri, 6 Jul 2012 10:40:47 +0000 (12:40 +0200)]
Staging: xgifb: Refactor XGI_GetVBType().

Refactor XGI_GetVBType() to reduce indentation and fix style warnings.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Refactor XGI_AjustCRT2Rate().
Miguel Gómez [Fri, 6 Jul 2012 10:40:46 +0000 (12:40 +0200)]
Staging: xgifb: Refactor XGI_AjustCRT2Rate().

Refactor some conditions to reduce indentation and fix style warnings.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Rework conditions in XGI_SetATTRegs().
Miguel Gómez [Fri, 6 Jul 2012 10:40:45 +0000 (12:40 +0200)]
Staging: xgifb: Rework conditions in XGI_SetATTRegs().

Rework some conditions to reduce indentation and fix style warnings.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Remove useless always true condition.
Miguel Gómez [Fri, 6 Jul 2012 10:40:44 +0000 (12:40 +0200)]
Staging: xgifb: Remove useless always true condition.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Remove unneeded var and condition in XGIfb_search_vesamode().
Miguel Gómez [Fri, 6 Jul 2012 10:40:43 +0000 (12:40 +0200)]
Staging: xgifb: Remove unneeded var and condition in XGIfb_search_vesamode().

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Simplify XGIfb_GetXG21DefaultLVDSModeIdx().
Miguel Gómez [Fri, 6 Jul 2012 10:40:42 +0000 (12:40 +0200)]
Staging: xgifb: Simplify XGIfb_GetXG21DefaultLVDSModeIdx().

Simplify the code and make it more readable.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Remove useless switch in XGIfb_detect_VB().
Miguel Gómez [Fri, 6 Jul 2012 10:40:41 +0000 (12:40 +0200)]
Staging: xgifb: Remove useless switch in XGIfb_detect_VB().

The switch performs no action so it can be removed.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Improve debug strings.
Miguel Gómez [Fri, 6 Jul 2012 10:40:40 +0000 (12:40 +0200)]
Staging: xgifb: Improve debug strings.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Remove unused struct _chswtable.
Miguel Gómez [Fri, 6 Jul 2012 10:40:39 +0000 (12:40 +0200)]
Staging: xgifb: Remove unused struct _chswtable.

Struct _chswtable and the variable mychswtable are not used anywhere, so remove
them.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Remove LCDA detection in xgifb_probe().
Miguel Gómez [Fri, 6 Jul 2012 10:40:38 +0000 (12:40 +0200)]
Staging: xgifb: Remove LCDA detection in xgifb_probe().

The piece of code that checks for LCDA in xgifb_probe() just checks for some
register values but doesn't really do anything in response to them (the actions
that should be executed are commented).
As nothing is really being done, the code can be safely removed.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Remove unneeded XGI_LCD_TYPE enumeration.
Miguel Gómez [Fri, 6 Jul 2012 10:40:37 +0000 (12:40 +0200)]
Staging: xgifb: Remove unneeded XGI_LCD_TYPE enumeration.

LCD_TYPEs are taken from drivers/video/sis/sis.h, so this enumeration is not
needed.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: xgifb: Comment cleaning.
Miguel Gómez [Fri, 6 Jul 2012 10:40:36 +0000 (12:40 +0200)]
Staging: xgifb: Comment cleaning.

Remove commented code and useless comments. Leave only those with relevant
hints to the code.

Signed-off-by: Miguel Gómez <magomez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoiio: Trivial documentation fix to correct kernel version info in
Jonathan Cameron [Tue, 10 Jul 2012 19:30:55 +0000 (20:30 +0100)]
iio: Trivial documentation fix to correct kernel version info in
21cd1fab058671313f7c178b640999fcd0d8de21

Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Reported-by: Peter Meerwald <pmeerw@pmeerw.net>
12 years agoIIO channel type and modifiers for CCT and RGBC data
Jon Brenner [Wed, 16 May 2012 15:46:42 +0000 (10:46 -0500)]
IIO channel type and modifiers for CCT and RGBC data

Add iio channel type and modifiers for Correlated Color Temperature (CCT)
and RGBC (red/green/blue/clear) data.
Add CCT and RGBC descriptions to documentation.

Changes:
Revised/condensed RGBC descriptions.

Merge and trivial fix done by Jonathan Cameron.

Signed-off-by: Jon Brenner <jbrenner@taosinc.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
12 years agoMerge tag 'togreg_8_7_2012' of git://git.kernel.org/pub/scm/linux/kernel/git/jic23...
Greg Kroah-Hartman [Mon, 9 Jul 2012 20:39:08 +0000 (13:39 -0700)]
Merge tag 'togreg_8_7_2012' of git://git./linux/kernel/git/jic23/iio into staging-next

Mixed back of updates and trivial examples fixes.
Nothing terribly interesting in this one.

8 July 2012

12 years agoMerge tag 'fixes-togreg_8_7_2012' of git://git.kernel.org/pub/scm/linux/kernel/git...
Greg Kroah-Hartman [Mon, 9 Jul 2012 20:35:32 +0000 (13:35 -0700)]
Merge tag 'fixes-togreg_8_7_2012' of git://git./linux/kernel/git/jic23/iio into staging-next

Mixed bag of IIO fixes.

Quite a few cases of interrupt handlers returning
errors and a few more specific bits and bobs.

Most of these only effect fairly obscure error
paths. The IRQF_ONESHOT one may cause trouble
given the requests will now be rejected.

Jonathan

8_7_2012

12 years agostaging/slicoss: return -ENODEV if no devid matches
Devendra Naga [Mon, 9 Jul 2012 17:34:20 +0000 (23:04 +0530)]
staging/slicoss: return -ENODEV if no devid matches

if no case matches we are simply asserting and doing break.
and i think we may need to return that -ENODEV , no device is
present, rather assert'ing.

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/slicoss: remove not-needed ASSERT
Devendra Naga [Mon, 9 Jul 2012 17:34:18 +0000 (23:04 +0530)]
staging/slicoss: remove not-needed ASSERT

As the private pointer is valid at the remove of driver, and remove wont' be called if probe fails, so no point for checking of ASSERT

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging "speakup" Fix typos.
Justin P. Mattock [Mon, 9 Jul 2012 14:30:27 +0000 (07:30 -0700)]
staging "speakup" Fix typos.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6655: hostap.c: Remove all unused macros
Marcos Paulo de Souza [Mon, 9 Jul 2012 02:51:26 +0000 (23:51 -0300)]
staging: vt6655: hostap.c: Remove all unused macros

All these macros were reported by forgotten-macros tool
(https://github.com/marcosps/forgotten_macros).

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6655: Remove all commented macros
Marcos Paulo de Souza [Mon, 9 Jul 2012 02:51:25 +0000 (23:51 -0300)]
staging: vt6655: Remove all commented macros

These macros were reported by forgotten-macros tool
(https://github.com/marcosps/forgotten_macros).

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6655: baseband.h: Remove all unused macros
Marcos Paulo de Souza [Mon, 9 Jul 2012 02:51:24 +0000 (23:51 -0300)]
staging: vt6655: baseband.h: Remove all unused macros

These macros were reported by forgotten-macros tool
(https://github.com/marcosps/forgotten_macros).

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6655: rf.c: Remove all unused macros
Marcos Paulo de Souza [Mon, 9 Jul 2012 02:51:23 +0000 (23:51 -0300)]
staging: vt6655: rf.c: Remove all unused macros

These macros were reported by forgotte-macros tool
(https://github.com/marcosps/forgotten_macros).

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6655: mac.h: Remove all commented macros
Marcos Paulo de Souza [Mon, 9 Jul 2012 02:51:22 +0000 (23:51 -0300)]
staging: vt6655: mac.h: Remove all commented macros

These macros were reported by forgotten-macros tool
(https://github.com/marcosps/forgotten_macros).

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6655: Remove all "if 0" blocks from driver
Marcos Paulo de Souza [Mon, 9 Jul 2012 02:51:21 +0000 (23:51 -0300)]
staging: vt6655: Remove all "if 0" blocks from driver

This commit removes code that will never be executed by vt6655 driver.

Was the forgotten-macros tool(https://github.com/marcosps/forgotten_macros)
who reported these blocks for us.

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6656: Remove unsed macros
Marcos Paulo de Souza [Mon, 9 Jul 2012 01:55:50 +0000 (22:55 -0300)]
staging: vt6656: Remove unsed macros

These macros were reported by forgotten-macros tool
(https://github.com/marcosps/forgotten_macros).

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6656: Remove all commented macros
Marcos Paulo de Souza [Mon, 9 Jul 2012 01:55:49 +0000 (22:55 -0300)]
staging: vt6656: Remove all commented macros

These macros were reported by forgotten-macros tool
(https://github.com/marcosps/forgotten_macros).

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6656: hostap.c: Remove commented code
Marcos Paulo de Souza [Mon, 9 Jul 2012 01:55:48 +0000 (22:55 -0300)]
staging: vt6656: hostap.c: Remove commented code

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6656: hostap.c: Remove unused macros and some useless comments
Marcos Paulo de Souza [Mon, 9 Jul 2012 01:55:47 +0000 (22:55 -0300)]
staging: vt6656: hostap.c: Remove unused macros and some useless comments

The unused macros were reporteds by forgotten-macros
tool(https://github.com/marcosps/forgotten_macros).

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6656: desc.h: Remove commented macros and useless comments
Marcos Paulo de Souza [Mon, 9 Jul 2012 01:55:46 +0000 (22:55 -0300)]
staging: vt6656: desc.h: Remove commented macros and useless comments

The commented macros are reported by forgotten-macros
tool(https://github.com/marcosps/forgotten_macros).

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: Fixes some checkpatch warnings in asus_oled/asus_oled.c
Cruz Julian Bishop [Sun, 8 Jul 2012 07:00:56 +0000 (17:00 +1000)]
staging: Fixes some checkpatch warnings in asus_oled/asus_oled.c

This fixes the following warnings:

1: Changes a printk(KERN_ERR) call to a pr_err call in line 785
2: Changes a printk(KERN_ERR) call to a pr_err call in line 791
3: Changes a printk(KERN_ERR) call to a pr_err call in line 798
4: Reduces line length below 80 at line 785 (Not intended)
5: Reduces line length below 80 at line 798 (Not intended)

Signed-off-by: Cruz Julian Bishop <cruz@massive-dynamics.biz>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/ft1000: remove unnecessary assignment of ret with STATUS_SUCCESS
Devendra Naga [Sat, 7 Jul 2012 12:05:19 +0000 (17:35 +0530)]
staging/ft1000: remove unnecessary assignment of ret with STATUS_SUCCESS

as ret is assigned to the return of ft1000_poll, we dont need to
initialise ret with STATUS_SUCCESS.

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/ft1000-pcmcia: fix checkpatch warnings
Devendra Naga [Sat, 7 Jul 2012 12:05:18 +0000 (17:35 +0530)]
staging/ft1000-pcmcia: fix checkpatch warnings

The below checkpatch warns fixed,

drivers/staging/ft1000/ft1000-pcmcia/ft1000_proc.c:38: ERROR: that open brace { should be on the previous line
drivers/staging/ft1000/ft1000-pcmcia/ft1000_proc.c:45: ERROR: that open brace { should be on the previous line
drivers/staging/ft1000/ft1000-pcmcia/ft1000_proc.c:57: ERROR: that open brace { should be on the previous line

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/ft1000: fix the foo * bar warning
Devendra Naga [Sat, 7 Jul 2012 12:05:17 +0000 (17:35 +0530)]
staging/ft1000: fix the foo * bar warning

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/ft1000: fix the no brace needed warning in ft1000_read_fifo_len
Devendra Naga [Sat, 7 Jul 2012 12:05:16 +0000 (17:35 +0530)]
staging/ft1000: fix the no brace needed warning in ft1000_read_fifo_len

as the if else statements enclose only one line, so braces around them are not
needed.

The following warning is fixed

drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c:100: WARNING: braces {} are not necessary for any arm of this statement

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/ft1000: fix sparse warning about the kernel thread handler function
Devendra Naga [Sat, 7 Jul 2012 12:05:15 +0000 (17:35 +0530)]
staging/ft1000: fix sparse warning about the kernel thread handler function

sparse throws warning about the ft1000_poll_thread as

drivers/staging/ft1000/ft1000-usb/ft1000_usb.c:40:5: warning: symbol 'ft1000_poll_thread' was not declared. Should it be static?

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/ft1000: fix sparse warning
Devendra Naga [Sat, 7 Jul 2012 12:05:14 +0000 (17:35 +0530)]
staging/ft1000: fix sparse warning

sparse warns about having 0 assigned to a pointer,
fix it up by using NULL.

The following sparse warning is fixed

drivers/staging/ft1000/ft1000-usb/ft1000_debug.c:170:52: warning: Using plain integer as NULL pointer

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/ft1000: fix the no brace needed warning in ft1000_interrupt
Devendra Naga [Sat, 7 Jul 2012 12:05:12 +0000 (17:35 +0530)]
staging/ft1000: fix the no brace needed warning in ft1000_interrupt

as the if statement encloses only one line braces around it are not
needed.

The following warn fixed,
WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: fix single statements block warning
Devendra Naga [Sat, 7 Jul 2012 06:03:04 +0000 (11:33 +0530)]
staging/rts_pstor: fix single statements block warning

The following warning is fixed up.
drivers/staging/rts_pstor/sd.c:190: WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: fix the no brace needed warning
Devendra Naga [Sat, 7 Jul 2012 06:02:15 +0000 (11:32 +0530)]
staging/rts_pstor: fix the no brace needed warning

for if else statements having single block no braces are needed
fixed the following checkpatch warning

drivers/staging/rts_pstor/sd.c:140: WARNING: braces {} are not necessary for any arm of this statement

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6656: iwctl: remove a few remaining redundant newlines
Jesper Juhl [Sun, 8 Jul 2012 22:59:30 +0000 (00:59 +0200)]
staging: vt6656: iwctl: remove a few remaining redundant newlines

I missed a few redundant newlines the first time.

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6656: iwctl: Fix up a few remaining brace issues
Jesper Juhl [Sun, 8 Jul 2012 22:58:52 +0000 (00:58 +0200)]
staging: vt6656: iwctl: Fix up a few remaining brace issues

My previous cleanup patches missed a few cases of
redundant/missing/replaced curly braces. This should fix up the last
ones.

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Acked-by : Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6656: iwctl: fix up spacing around operators
Jesper Juhl [Sun, 8 Jul 2012 22:58:36 +0000 (00:58 +0200)]
staging: vt6656: iwctl: fix up spacing around operators

A few cases were missed in my previous cleanup, this takes care of the
last cases of missing space (or too much space (as in a newline))
around operators ('=', '==', ',', '<').

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>