summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* staging: vt6656: fix module_param_named tx_buffers variableMalcolm Priestley2014-07-181-1/+1
| | | | | | | Should point to vnt_tx_buffers Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: usb_device_reset remove camel caseMalcolm Priestley2014-07-181-3/+3
| | | | | | | | Camel case change pDevice -> priv Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: bcm: Qos.c: Fix coding styleKhoa Mai2014-07-171-20/+20
| | | | | | | Switch and case should be in same indent level Signed-off-by: Khoa Mai <mdk194@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: bcm: Qos.c: Fix coding styleKhoa Mai2014-07-171-5/+5
| | | | | | | Fix C99 comment Signed-off-by: Khoa Mai <mdk194@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: gdm72xx: fix build errorsGreg Kroah-Hartman2014-07-171-3/+3
| | | | | | | | This fixes the build errors in the dev_dbg() fixes in a previous patch. Reported-by: kbuild test robot <fengguang.wu@intel.com> To: Michalis Pappas <mpappas@fastmail.fm> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* devres: Add devm_kasprintf and devm_kvasprintf APIHimangi Saraogi2014-07-172-0/+59
| | | | | | | | | | devm_kasprintf() and devm_kvasprintf() are the managed counterparts for kasprintf() and kvasprintf(). Signed-off-by: Himangi Saraogi <himangi774@gmail.com> Acked-by: Julia Lawall <julia.lawall@lip6.fr> Acked-by: Tejun Heo <tj@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: lustre: plain interger was used as NULL pointerJosep Puigdemont2014-07-171-2/+2
| | | | | | | | | This patch fixes the following warnings found when running sparse: .../lproc_echo.c:43:11: warning: Using plain integer as NULL pointer .../lproc_echo.c:49:11: warning: Using plain integer as NULL pointer Signed-off-by: Josep Puigdemont <josep.puigdemont@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: unisys: detect controlvm channel on module loadBenjamin Romer2014-07-171-121/+65
| | | | | | | | | | | The controlvm channel is not removable from a guest after the guest starts, so it makes no sense to constantly check for it. Move the channel address discovery to visorchipset_init(), and remove all of the checks for the channel address from the rest of the module, as the module will not load if the channel pointer is not valid. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: unisys: simplify controlvm channel address functionBenjamin Romer2014-07-174-94/+15
| | | | | | | | | | There is no reason to have controlvm_get_channel_address() defined in a separate file. Move it to visorchipset_main.c, remove the no-op functions controlvm_init() and controlvm_deinit() from visorchipset_main.c, and remove the controlvm.h and controlvm_direct.c files from the module. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* drivers/staging/comedi/drivers/addi-data/hwdrv_apci3200.c: drop incorrect checksAndrey Utkin2014-07-171-10/+10
| | | | | | | | | Dropped some "< 0" and ">= 0" checks on unsigned int values. Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=80501 Reported-by: David Binderman <dcb314@hotmail.com> Signed-off-by: Andrey Utkin <andrey.krieger.utkin@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: gdm72xx: replace print_hex_dump_debug() with dev_dbg()Michalis Pappas2014-07-172-12/+13
| | | | | Signed-off-by: Michalis Pappas <mpappas@fastmail.fm> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: gdm72xx: remove debug codeMichalis Pappas2014-07-171-106/+0
| | | | | | | Removed dump_eth_packet() and helper functions called upon packet tx/rx. Signed-off-by: Michalis Pappas <mpappas@fastmail.fm> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: ion: Remove left over commentSachin Kamat2014-07-171-1/+0
| | | | | | | | | | | Commit 2bb9f5034ec7 ("gpu: ion: Remove heapmask from client") removed the heap_type_mask parameter. Remove the associated kernel-doc comment too. Signed-off-by: Sachin Kamat <sachin.kamat@samsung.com> Cc: Rebecca Schultz Zavin <rebecca@android.com> Cc: John Stultz <john.stultz@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: gdm724x: gdm_tty.c: fix missing blank line after variable declarationKiran Padwal2014-07-171-0/+5
| | | | | | | Checkpatch fix - Add missing blank line after variable declaration Signed-off-by: Kiran Padwal <kiran.padwal21@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rtl8192ee: Fix setting highest n rateLarry Finger2014-07-171-2/+1
| | | | | | | | | | | | Commit 4fb6a37c3f94c1cb4b828bfcc4347771e1628f88 by Andrey Utkin <andrey.krieger.utkin@gmail.com> and entitled "staging: rtl8192ee: "Correct bitmask in comparsion" fixed what appeared to be a typo. After consultation with the Realtek engineers, merely testing for a 2T2R device is sufficient to ensure that the TX MCS map will equal 0x0c, thus the second test can be ignored. Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Remove commented out long frequency_listMalcolm Priestley2014-07-171-11/+0
| | | | | | | These frequencies are already defined in new tables channel.c Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: main_usb remove static function declarations.Malcolm Priestley2014-07-171-18/+0
| | | | | | | All these functions are in visibility order. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: main_usb remove dead module params macrosMalcolm Priestley2014-07-171-49/+0
| | | | | | | All these macros relate to module parameters already removed Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: remove dead code DEVICE_PARAMMalcolm Priestley2014-07-171-23/+0
| | | | | | | All these module parameters are dead Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: main_usb.c Replace DEVICE_PARAM TxDescriptors0Malcolm Priestley2014-07-171-2/+9
| | | | | | | | | | Replace with vnt_tx_buffers using module_param_named and MODULE_PARM_DESC Check user doe not set tx_buffer beyond the define limits of driver Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: main_usb.c Replace DEVICE_PARAM RxDescriptors0Malcolm Priestley2014-07-171-2/+10
| | | | | | | | | | Replace with vnt_rx_buffers using module_param_named and MODULE_PARM_DESC Check user doe not set rx_buffer beyond the define limits of driver Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: remove DBG_PRT kernel messages macro and enumsMalcolm Priestley2014-07-171-10/+0
| | | | | | | All dead code and nolonger used by driver. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: power.c replace DBG_PRT debug messagesMalcolm Priestley2014-07-171-3/+1
| | | | | | | Replace with dev_dbg Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: replace DBG_PRT error messagesMalcolm Priestley2014-07-171-11/+5
| | | | | | | replace with dev_err Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: main_usb replace DBG_PRT debug messagesMalcolm Priestley2014-07-171-9/+8
| | | | | | | replace with dev_dbg Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: device_init_registers remove camel caseMalcolm Priestley2014-07-171-117/+117
| | | | | | | | | | | | | | camel case changes pDevice -> priv byAntenna -> antenna ntStatus -> status byTmp-> tmp byCalibTXIQ -> calib_tx_iq byCalibTXDC -> calib_tx_dc byCalibRXIQ -> calib_rx_iq Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: device_set_options remove camel caseMalcolm Priestley2014-07-171-11/+11
| | | | | | | | Camel case changes pDevice -> priv Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: device_set_options fix opening function bracketMalcolm Priestley2014-07-171-2/+2
| | | | | Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rename BBvUpdatePreEDThreshold to vnt_update_pre_ed_thresholdMalcolm Priestley2014-07-173-5/+5
| | | | | | | Dropping BB Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rename BBvExitDeepSleep to vnt_exit_deep_sleepMalcolm Priestley2014-07-173-3/+3
| | | | | | | Dropping BB Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rename BBvSetDeepSleep to vnt_set_deep_sleepMalcolm Priestley2014-07-173-4/+4
| | | | | | | Dropping BB Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rename BBvSetVGAGainOffset to vnt_set_vga_gain_offsetMalcolm Priestley2014-07-174-4/+4
| | | | | | | Dropping BB Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rename BBvSetShortSlotTime to vnt_set_short_slot_timeMalcolm Priestley2014-07-173-4/+4
| | | | | | | Dropping BB Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rename BBbVT3184Init to vnt_vt3184_initMalcolm Priestley2014-07-173-5/+5
| | | | | | | Dropping BB Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rename BBvSetAntennaMode to vnt_set_antenna_modeMalcolm Priestley2014-07-174-7/+7
| | | | | | | Dropping BB Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: baseband.c rename awcFrameTime to vnt_frame_timeMalcolm Priestley2014-07-171-2/+2
| | | | | Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: comedi: comedi_fops: use pr_fmt()H Hartley Sweeten2014-07-171-9/+10
| | | | | | | | | | | | There are a number of pr_{level} messages in this file that are used to print kernel messages when the device pointer is not available for a dev_{level} message. Use pr_fmt() to ensure all of these messages have the module prefix. 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>
* staging: comedi: ni_labpc: tidy up labpc_ai_scan_mode()H Hartley Sweeten2014-07-171-7/+9
| | | | | | | | | | | | This function checks the cmd->chanlist to determine the scan mode used to sample the analog inputs. All possible modes are checked so the final pr_err() and return 0 can never be reached. Tidy up the function a bit and remove the unreachable code. 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>
* staging: comedi: cb_pcidas64: remove unused pr_fmt() macroH Hartley Sweeten2014-07-171-2/+0
| | | | | | | | This macro is 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>
* staging: comedi: usbduxfast: convert pr_warn() to dev_warn()H Hartley Sweeten2014-07-171-1/+1
| | | | | | | | Use dev_warn() to output the warning message. 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>
* staging: comedi: usbduxfast: convert pr_err() to dev_err()H Hartley Sweeten2014-07-171-4/+3
| | | | | | | | | Use dev_err() to output the error message and remove the unnecessary pr_fmt() 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>
* staging: comedi: remove comedi_error()H Hartley Sweeten2014-07-173-9/+0
| | | | | | | | | | | | | The comedi_error() function is just a wrapper around dev_err() that adds the dev->driver->driver_name prefix to the message and a terminating new-line character. All of the users of this function have been converted to use dev_err() directly. Remove the now unused function. 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>
* staging: comedi: hwdrv_apci1500: remove use of comedi_error()H Hartley Sweeten2014-07-171-3/+4
| | | | | | | | | | | | | | | The comedi_error() function is just a wrapper around dev_err() that adds the dev->driver->driver_name prefix to the message and a terminating new-line character. The addition of the driver_name is just added noise and some of the users of comedi_error() add unnecessary additional new-line characters. Use dev_err() directly instead of comedi_error() to avoid any confusion and so that all the comedi generated kernel messages have the same format. 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>
* staging: comedi: hwdrv_apci1564: remove use of comedi_error()H Hartley Sweeten2014-07-171-2/+1
| | | | | | | | | | | | | | | The comedi_error() function is just a wrapper around dev_err() that adds the dev->driver->driver_name prefix to the message and a terminating new-line character. The addition of the driver_name is just added noise and some of the users of comedi_error() add unnecessary additional new-line characters. Use dev_err() directly instead of comedi_error() to avoid any confusion and so that all the comedi generated kernel messages have the same format. 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>
* staging: comedi: hwdrv_apci3120: remove use of comedi_error()H Hartley Sweeten2014-07-171-13/+14
| | | | | | | | | | | | | | | The comedi_error() function is just a wrapper around dev_err() that adds the dev->driver->driver_name prefix to the message and a terminating new-line character. The addition of the driver_name is just added noise and some of the users of comedi_error() add unnecessary additional new-line characters. Use dev_err() directly instead of comedi_error() to avoid any confusion and so that all the comedi generated kernel messages have the same format. 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>
* staging: comedi: addi_apci_3501: remove use of comedi_error()H Hartley Sweeten2014-07-171-1/+1
| | | | | | | | | | | | | | | The comedi_error() function is just a wrapper around dev_err() that adds the dev->driver->driver_name prefix to the message and a terminating new-line character. The addition of the driver_name is just added noise and some of the users of comedi_error() add unnecessary additional new-line characters. Use dev_err() directly instead of comedi_error() to avoid any confusion and so that all the comedi generated kernel messages have the same format. 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>
* staging: comedi: adl_pci9118: remove use of comedi_error()H Hartley Sweeten2014-07-171-26/+23
| | | | | | | | | | | | | | | The comedi_error() function is just a wrapper around dev_err() that adds the dev->driver->driver_name prefix to the message and a terminating new-line character. The addition of the driver_name is just added noise and some of the users of comedi_error() add unnecessary additional new-line characters. Use dev_err() directly instead of comedi_error() to avoid any confusion and so that all the comedi generated kernel messages have the same format. 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>
* staging: comedi: adv_pci1724: remove use of comedi_error()H Hartley Sweeten2014-07-171-7/+8
| | | | | | | | | | | | | | | The comedi_error() function is just a wrapper around dev_err() that adds the dev->driver->driver_name prefix to the message and a terminating new-line character. The addition of the driver_name is just added noise and some of the users of comedi_error() add unnecessary additional new-line characters. Use dev_err() directly instead of comedi_error() to avoid any confusion and so that all the comedi generated kernel messages have the same format. 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>
* staging: comedi: adv_pci_dio: remove use of comedi_error()H Hartley Sweeten2014-07-171-3/+3
| | | | | | | | | | | | | | | The comedi_error() function is just a wrapper around dev_err() that adds the dev->driver->driver_name prefix to the message and a terminating new-line character. The addition of the driver_name is just added noise and some of the users of comedi_error() add unnecessary additional new-line characters. Use dev_err() directly instead of comedi_error() to avoid any confusion and so that all the comedi generated kernel messages have the same format. 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>
* staging: comedi: amplc_dio_common: remove use of comedi_error()H Hartley Sweeten2014-07-171-1/+1
| | | | | | | | | | | | | | | The comedi_error() function is just a wrapper around dev_err() that adds the dev->driver->driver_name prefix to the message and a terminating new-line character. The addition of the driver_name is just added noise and some of the users of comedi_error() add unnecessary additional new-line characters. Use dev_err() directly instead of comedi_error() to avoid any confusion and so that all the comedi generated kernel messages have the same format. 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>
OpenPOWER on IntegriCloud