- 03 Dec, 2013 40 commits
-
-
H Hartley Sweeten authored
Convert the printk(KERN_WARNING messages in this function to dev_dbg(). Signed-off-by:
H Hartley Sweeten <hsweeten@visionengravers.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
H Hartley Sweeten authored
An irq is only needed by this driver in order to support async commands. Since it is optional, modify the attach so that if the request_irq() fails the attach does not fail. Remove the printk noise about the irq. Only hookup the async command support if the request_irq() was successful. Signed-off-by:
-
H Hartley Sweeten authored
Convert this printk message into a dev_dev(). Signed-off-by:
-
H Hartley Sweeten authored
This printk could spew a lot of noise while waiting for the ai conversion to complete. Just remove it. Signed-off-by:
-
H Hartley Sweeten authored
The ai read timeout will return -ETIMEDOUT. The printk is just added noise. Remove it. Signed-off-by:
-
H Hartley Sweeten authored
The interrupt support in this driver is incomplete due to the missing comedi functions to support async commands. Since the interrupt code is already disable just remove it. Signed-off-by:
-
H Hartley Sweeten authored
This function is not used by the driver. It's also suffering from bitrot since COMEDI_MDEMAND and COMEDI_MCONTS are not defined anywhere. Just remove the code. Signed-off-by:
-
H Hartley Sweeten authored
Convert all the printk() messages in this driver to dev_dbg(). Signed-off-by:
-
H Hartley Sweeten authored
This printk is just added noise. Remove it. Signed-off-by:
-
H Hartley Sweeten authored
The code that checks the DT_S_READY status is disabled. Just remove it. Signed-off-by:
-
H Hartley Sweeten authored
Convert this printk message to a dev_err() message. Signed-off-by:
-
H Hartley Sweeten authored
An irq is only needed by this driver in order to support async commands. Since it is optional, modify the attach so that if the request_irq() fails the attach does not fail. Remove the printk noise about the irq. Only hookup the async command support if the request_irq() was successful. Signed-off-by:
-
H Hartley Sweeten authored
These printk messages are just added noise. Remove them. Signed-off-by:
-
H Hartley Sweeten authored
A timeout will return -ETIMEDOUT. The printks are just added noise. Remove them. Signed-off-by:
-
H Hartley Sweeten authored
An irq is only needed by this driver in order to support async commands. Since it is optional, modify the attach so that if an invalid irq is selected by the user, or the request_irq() fails, the attach does not fail. Remove all the printk noise about the irq. Only hookup the async command support if the request_irq() was successful. Tidy up das16m1_irq_bits() a bit. This helper returns the value that needs to be written to the DAS16M1_INTR_CONTROL register to setup the irq routing. Signed-off-by:
-
H Hartley Sweeten authored
The ai read timeout will return -ETIMEDOUT. The printk is just added noise. Remove it. Signed-off-by:
-
H Hartley Sweeten authored
This driver does not have interrupt support. Remove the commented out code that requests the irq. Signed-off-by:
-
H Hartley Sweeten authored
These commented out messages are just added noise. Remove them. Signed-off-by:
-
H Hartley Sweeten authored
The code to initialize subdevice[2] is commented out and incomplete. Just remove it. Signed-off-by:
-
H Hartley Sweeten authored
Remove the commented out message, it would just be added noise. Signed-off-by:
-
H Hartley Sweeten authored
Convert these messages to dev_dbg() and tidy them up a bit. Signed-off-by:
-
H Hartley Sweeten authored
Convert these warning messages to dev_dbg() to minimize the noise. Also, tidy them up a bit. Signed-off-by:
-
H Hartley Sweeten authored
Convert these debug messages to dev_dbg() and tidy them up a bit. Signed-off-by:
-
H Hartley Sweeten authored
The "success" message after an attach is just added noise. Remove it. The normal return value for "success" is 0. Signed-off-by:
-
H Hartley Sweeten authored
Convert these debug messages to dev_dbg(). Signed-off-by:
-
H Hartley Sweeten authored
An allocation failure will have already displayed a message. Signed-off-by:
-
Aldo Iljazi authored
Line 921: Removed the space before the semicolon. Signed-off-by:
Aldo Iljazi <mail@aldo.io> Signed-off-by:
-
Chen Gang authored
drivers: staging: ft1000: ft1000-usb: ft1000_debug.c: check return value of get_user() in ft1000_ioctl() get_user() may fail and cause 'msgsz' uninitialized, so need give a check. The related warning (with allmodconfig under hexagon): CC [M] drivers/staging/ft1000/ft1000-usb/ft1000_debug.o drivers/staging/ft1000/ft1000-usb/ft1000_debug.c: In function 'ft1000_ioctl': include/uapi/linux/swab.h:53:9: warning: 'msgsz' may be used uninitialized in this function [-Wuninitialized] drivers/staging/ft1000/ft1000-usb/ft1000_debug.c:533:17: note: 'msgsz' was declared here Signed-off-by:Chen Gang <gang.chen.5i5j@gmail.com> Signed-off-by:
-
Michal Nazarewicz authored
The rest of the code seem to imply that rmf_dumper may indeed be NULL. Change the code so that dumping is not even considered if rmf_dumper callback is not set. Signed-off-by:
Michal Nazarewicz <mina86@mina86.com> Signed-off-by:
-
Michal Nazarewicz authored
bus_info field of struct ethtool_drvinfo is an array thus it always evaluates to true in a boolean context. Therefore operation depending on it being false will never be executed. Signed-off-by:
-
Aldo Iljazi authored
Fixed the following coding style issues: Lines 66-70: Replaced spaced with tabs. Lines 75, 98, 182: Inserted space before the open parenthesis. Line 89: Inserted spaces around that '=' Signed-off-by:
-
Wei Yongjun authored
Fix to return -ENOMEM instead of 0 when the memory alloc fail in probe error handling path. Signed-off-by:
Wei Yongjun <yongjun_wei@trendmicro.com.cn> Signed-off-by:
-
Aldo Iljazi authored
Fixed a coding style issue, specifically: Line 1798: Removed parentheses since return is not a function. Signed-off-by:
-
Stephen Warren authored
The DWC2 USB controller in the BCM2835 (Raspberry Pi) needs some non- default parameters. Select these based on the compatible value from the DT node. For all other HW, fall back to the default parameters currently in use. The values in params_bcm2835[] were posted to the mailing list by Paul quite some time ago. I made a couple of minor modifications since then; to set ahbcfg instead of ahb_single, and to set uframe_sched. Signed-off-by:
Stephen Warren <swarren@wwwdotorg.org> Signed-off-by:
-
Peng Tao authored
Got below warning when building with sparc: drivers/staging/lustre/lustre/llite/file.c: In function 'll_file_aio_write': drivers/staging/lustre/lustre/llite/file.c:1247:9: warning: 'count' may be used uninitialized in this function [-Wuninitialized] drivers/staging/lustre/lustre/llite/file.c: In function 'll_file_aio_read': drivers/staging/lustre/lustre/llite/file.c:1188:9: warning: 'count' may be used uninitialized in this function [-Wuninitialized] Signed-off-by:
Peng Tao <bergwolf@gmail.com> Signed-off-by:
Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
register_sysctl_table/unregister_sysctl_table are defined regardless of CONFIG_SYSCTL true or false. so we don't need to surround them with #ifdef CONFIG_SYSCTL. Besides, current code got build warnings when CONFIG_SYSCTL is not set: drivers/staging/lustre/lustre/libcfs/linux/linux-proc.c:368:20: warning: ‘lnet_table’ defined but not used [-Wunused-variable] Signed-off-by:
-
Peng Tao authored
It is not strictly related to procfs. Besides, we already call obd_sysctl_clean outside of class_procfs_clean. Signed-off-by:
-
Peng Tao authored
Signed-off-by:
-
Peng Tao authored
No need to surround them with #ifdef LPROCFS. Signed-off-by:
-
Peng Tao authored
Got below warning when building lustre with 4.7.3. drivers/staging/lustre/lustre/obdclass/llog_test.c: In function ‘ llog_test_init’: drivers/staging/lustre/lustre/obdclass/llog_test.c:1058:28: warning: ‘ lvars.module_vars’ is used uninitialized in this function [-Wuninitialized] Signed-off-by:
-
