aboutsummaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2013-10-26drivers/staging/lustre/lustre/osc/osc_io.c : remove space prohibited warningHimangi Saraogi1-1/+1
There was a warning on running checkpatch.pl on the file drivers/staging/lustre/lustre/osc/osc_io.c which stated: WARNING: space prohibited between function name and open parenthesis '(' 108: FILE: staging/lustre/lustre/osc/osc_io.c:108: + LIST_HEAD (list); total: 0 errors, 1 warnings, 828 lines checked To get rid of the warning the extra spaces were eliminated. Signed-off-by: Himangi Saraogi <[email protected]> Reviewed-by: Rusty Russell <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-26staging: lustre: replace IS_ERR and PTR_ERR with PTR_ERR_OR_ZEROValentina Manea1-3/+1
This patch fixes coccinelle error regarding usage of IS_ERR and PTR_ERR instead of PTR_ERR_OR_ZERO. Signed-off-by: Valentina Manea <[email protected]> Reviewed-by: Rusty Russell <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-26staging: lustre: lnet: Reformat pointer variable in lib-lnet.hLisa Nguyen1-1/+1
Reformat a pointer variable in lib-lnet.h to meet kernel coding style and eliminate pointer format warning generated by checkpatch.pl Signed-off-by: Lisa Nguyen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-26staging: lustre: lnet: Remove unnecessary () from return statementsLisa Nguyen1-6/+6
Remove unnecessary parentheses from return statements in lib-lnet.h to meet kernel coding style. Signed-off-by: Lisa Nguyen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-26staging: lustre: lnet: Remove unnecessary spaces in lib-lnet.hLisa Nguyen1-39/+39
Remove spaces between function names and open parentheses to meet kernel coding style and eliminate extra space warnings generated by checkpatch.pl Signed-off-by: Lisa Nguyen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-26staging: lustre: Remove typedef and update cfs_hash structLisa Nguyen21-249/+249
Remove typedef keyword and rename the cfs_hash_t struct to cfs_hash in libcfs_hash.h. These changes resolve the "Do not add new typedefs" warning generated by checkpatch.pl and meet kernel coding style. Struct variables in other header and source files that depend on libcfs_hash.h are updated as well. Signed-off-by: Lisa Nguyen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-26staging: lustre: Remove typedef and update cfs_hash_bd structLisa Nguyen9-124/+124
Remove typedef keyword and rename the cfs_hash_bd_t struct to cfs_hash_bd in libcfs_hash.h. These changes resolve the "Do not add new typedefs" warning generated by checkpatch.pl and meet kernel coding style. Struct variables in other header and source files that depend on libcfs_hash.h are updated as well. Signed-off-by: Lisa Nguyen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-26staging: lustre: Remove typedef and update cfs_debug_limit_state structLisa Nguyen4-7/+7
Removed typedef keyword and rename the cfs_debug_limit_state_t struct to cfs_debug_limit_state in libcfs_debug.h. These changes resolve the "Do not add new typedefs" warning generated by checkpatch.pl and meet kernel coding style. Struct variables in other header and source files that depend on libcfs_debug.h are updated as well. Signed-off-by: Lisa Nguyen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-26staging/lustre: fix tv_usec build warning on pariscPeng Tao3-6/+6
As reported by Fengguang: config: make ARCH=parisc allyesconfig All warnings: drivers/staging/lustre/lustre/llite/lproc_llite.c: In function 'll_rw_extents_stats_pp_seq_show': >> drivers/staging/lustre/lustre/llite/lproc_llite.c:1069:6: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type '__kernel_suseconds_t' [-Wformat] drivers/staging/lustre/lustre/llite/lproc_llite.c: In function 'll_rw_extents_stats_seq_show': >> drivers/staging/lustre/lustre/llite/lproc_llite.c:1133:6: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type '__kernel_suseconds_t' [-Wformat] drivers/staging/lustre/lustre/llite/lproc_llite.c: In function 'll_rw_offset_stats_seq_show': >> drivers/staging/lustre/lustre/llite/lproc_llite.c:1299:6: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type '__kernel_suseconds_t' [-Wformat] -- drivers/staging/lustre/lustre/obdclass/lprocfs_status.c: In function 'lprocfs_stats_seq_show': >> drivers/staging/lustre/lustre/obdclass/lprocfs_status.c:1070:5: warning: format '%lu' expects argument of type 'long unsigned int', but argument 5 has type '__kernel_suseconds_t' [-Wformat] -- drivers/staging/lustre/lustre/osc/lproc_osc.c: In function 'osc_rpc_stats_seq_show': >> drivers/staging/lustre/lustre/osc/lproc_osc.c:575:6: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type '__kernel_suseconds_t' [-Wformat] drivers/staging/lustre/lustre/osc/lproc_osc.c: In function 'osc_stats_seq_show': >> drivers/staging/lustre/lustre/osc/lproc_osc.c:687:6: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type '__kernel_suseconds_t' [-Wformat] Reported-by: Fengguang Wu <[email protected]> Signed-off-by: Peng Tao <[email protected]> Signed-off-by: Andreas Dilger <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-26staging/lustre/llite: fix mkdir endless loopPeng Tao1-1/+5
Running on 3.11-rc4 kernel, I got below endless loop. It turns to be that Lustre always saves the first page of a dir inode mapping at index ~0UL. And after commit 5a720394 (mm: teach truncate_inode_pages_range() to handle non page aligned ranges), truncate_inode_pages_range() _NO LONGER_ truncates the page that is sitting at index ~0UL. [16768.998006] mkdir R running task 0 2717 2716 0x00000080 [16768.998073] 000000000000000e 0000000000000000 0000000000000000 ffff88000be00460 [16768.998157] ffff88000ea65908 ffffffff810fec3e ffff88000ea65968 ffff8800229e7750 [16768.998241] ffff88000ea658b8 0000000000000000 0000000000000000 ffff88000ea65958 [16768.998326] Call Trace: [16768.998401] [<ffffffff810fc6ed>] ? rcu_read_unlock+0x1c/0x2d [16768.998473] [<ffffffff810fec3e>] ? find_get_pages+0xf5/0x11b [16768.998530] [<ffffffff811078f0>] ? pagevec_lookup+0x20/0x2a [16768.998586] [<ffffffff8110920e>] ? truncate_inode_pages_range.part.2+0x161/0x39a [16768.998680] [<ffffffffa02ad5dc>] ? ll_md_blocking_ast+0x338/0x62f [lustre] [16768.998744] [<ffffffff8110947f>] ? truncate_inode_pages_range+0x38/0x3f [16768.998805] [<ffffffff811094f8>] ? truncate_inode_pages+0x12/0x14 [16768.998871] [<ffffffffa02ad6e8>] ? ll_md_blocking_ast+0x444/0x62f [lustre] [16768.998948] [<ffffffff810981b5>] ? arch_local_irq_save+0x9/0xc [16768.999022] [<ffffffffa07ee0e8>] ? ldlm_cancel_callback+0x67/0x12a [ptlrpc] [16768.999100] [<ffffffffa07f85b2>] ? ldlm_cli_cancel_local+0xf3/0x2bc [ptlrpc] [16768.999176] [<ffffffffa07f9163>] ? ldlm_cli_cancel_list_local+0x7e/0x1e4 [ptlrpc] [16768.999268] [<ffffffffa07f9473>] ? ldlm_cancel_resource_local+0x1aa/0x1b9 [ptlrpc] [16768.999385] [<ffffffffa0657bad>] ? mdc_resource_get_unused+0xf8/0x115 [mdc] [16768.999472] [<ffffffff8109c887>] ? trace_hardirqs_on+0xd/0xf [16768.999533] [<ffffffffa06583d8>] ? mdc_create+0x11e/0x4db [mdc] [16768.999597] [<ffffffff8152ed84>] ? mutex_unlock+0xe/0x10 [16768.999654] [<ffffffffa0350e99>] ? lmv_create+0x355/0x3e9 [lmv] [16768.999712] [<ffffffff811553b7>] ? final_putname+0x35/0x39 [16768.999775] [<ffffffffa02ae167>] ? ll_new_node+0x33b/0x3ff [lustre] [16768.999841] [<ffffffffa02ae62c>] ? ll_mkdir+0xf2/0x127 [lustre] [16768.999897] [<ffffffff81156996>] ? vfs_mkdir+0x84/0xc9 [16768.999961] [<ffffffff81158cf8>] ? SyS_mkdirat+0x77/0xad [16769.000014] [<ffffffff81158d47>] ? SyS_mkdir+0x19/0x1b [16769.000066] [<ffffffff81538652>] ? system_call_fastpath+0x16/0x1b Signed-off-by: Peng Tao <[email protected]> Signed-off-by: Andreas Dilger <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-26lustre/fld: remove fld_thread_keyPeng Tao1-5/+0
It is left over when porting Lustre tree patch in commit (e62e5d92) and should be removed. Signed-off-by: Peng Tao <[email protected]> Signed-off-by: Andreas Dilger <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-26staging: lustre: Remove typedef and update cfs_hash_bucket structLisa Nguyen2-17/+17
Remove typedef keyword and rename the cfs_hash_bucket_t struct to cfs_hash_bucket in libcfs_hash.h. These changes resolve the "Do not add new typedefs" warning generated by checkpatch.pl and meet kernel coding style. The struct variables in hash.c are updated to reflect this change as well. Signed-off-by: Lisa Nguyen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-25Staging: sb105x: Fix a possible null dereferenceRashika Kheria1-3/+3
This patch fixes the following error in sb_pc_mp.c- drivers/staging/sb105x/sb_pci_mp.c:546 mp_startup() error: we previously assumed 'info->tty' could be null (see line 525) Signed-off-by: Rashika Kheria <[email protected]> Reviewed-by: Josh Triplett <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-25staging: ashmem: Fix ASHMEM_PURGE_ALL_CACHES return valueJohn Stultz1-1/+1
Hopefully this isn't too late for 3.12. In commit 7dc19d5aff (convert shrinkers to new count/scan API) the return value to PURGE_ALL_CACHES was dropped, causing -EPERM to always be returned. This patch re-adds the ret assignment, setting it to the the ashmem_shrink_count(), which is the lru_count. (Sorry this was missed in the review!) Fixes: 7dc19d5affd7 ("convert shrinkers to new count/scan API") Cc: Colin Cross <[email protected]> Cc: Android Kernel Team <[email protected]> Cc: Glauber Costa <[email protected]> Reported-by: YongQin Liu <[email protected]> Signed-off-by: John Stultz <[email protected]> Cc: stable <[email protected]> # 3.12 Acked-by: Dave Chinner <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-24iio: light: vcnl4000: Remove redundant codeSachin Kamat1-5/+1
The if check is redundant as the value obtained from iio_device_register() is already in the required format. Hence return the function directly. Signed-off-by: Sachin Kamat <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-24iio: dac: mcp4725: Remove redundant codeSachin Kamat1-7/+1
Remove an inconsequential print message and return directly thereby cleaning up some code. Signed-off-by: Sachin Kamat <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-24iio: dac: max517: Remove redundant variableSachin Kamat1-8/+1
Remove an inconsequential print message and return directly thereby eliminating an intermediate variable. Signed-off-by: Sachin Kamat <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-24iio: dac: ad5755: Remove redundant codeSachin Kamat1-7/+1
The if check is redundant as the value obtained from iio_device_register() is already in the required format. Error messages are already printed by iio_device_register(); hence not needed. Signed-off-by: Sachin Kamat <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-24iio: dac: ad5421: Remove redundant codeSachin Kamat1-7/+1
The if check is redundant as the value obtained from iio_device_register() is already in the required format. Hence return the function directly. Error messages are already printed by iio_device_register(); hence not needed. Signed-off-by: Sachin Kamat <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-24iio: adc: twl6030-gpadc: Remove redundant codeSachin Kamat1-3/+1
The if check is redundant as the value obtained from iio_device_register() is already in the required format. Hence return the function directly. Signed-off-by: Sachin Kamat <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-24iio: accel: kxsd9: Remove redundant variableSachin Kamat1-6/+1
Return directly thereby eliminating an intermediate variable. Signed-off-by: Sachin Kamat <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-24iio: core: Add misssing bracesSachin Kamat1-1/+1
Silences the following checkpatch warning: WARNING: sizeof *iio_attr should be sizeof(*iio_attr) Signed-off-by: Sachin Kamat <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-24iio: core: Use pr_err instead of printkSachin Kamat1-6/+5
Use of pr_err is preferred to printk. Signed-off-by: Sachin Kamat <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-23iio:pressure: Adds LPS001WP support also on spi interface and Kconfig fixDenis CIOCCA2-1/+2
Signed-off-by: Denis Ciocca <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-23iio: adc: ti_am335x_adc: Remove redundant of_match_ptrSachin Kamat1-1/+1
ti_adc_dt_ids is always compiled in. Hence of_match_ptr is not needed. Signed-off-by: Sachin Kamat <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-23iio: adc: nau7802: Remove redundant of_match_ptrSachin Kamat1-1/+1
nau7802_dt_ids is always compiled in. Hence of_match_ptr is not needed. Signed-off-by: Sachin Kamat <[email protected]> Cc: Maxime Ripard <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-23iio: adc: twl6030-gpadc: Remove redundant of_match_ptrSachin Kamat1-1/+1
of_twl6030_match_tbl is always compiled in. Hence of_match_ptr is not necessary. Signed-off-by: Sachin Kamat <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-23iio: Add Freescale mag3110 magnetometer driverPeter Meerwald3-0/+412
three-axis digital magnetometer with I2C interface datasheet is available from http://cache.freescale.com/files/sensors/doc/data_sheet/MAG3110.pdf Signed-off-by: Peter Meerwald <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-23staging/iio/adc: MXS_LRADC depends on INPUTRandy Dunlap1-0/+1
mxs-lradc.c uses many input_() functions so it should depend on INPUT to fix build errors. drivers/built-in.o: In function `mxs_lradc_ts_unregister': drivers/staging/iio/adc/mxs-lradc.c:905: undefined reference to `input_unregister_device' drivers/staging/iio/adc/mxs-lradc.c:905: undefined reference to `input_unregister_device' drivers/built-in.o: In function `input_report_abs': include/linux/input.h:399: undefined reference to `input_event' include/linux/input.h:399: undefined reference to `input_event' include/linux/input.h:399: undefined reference to `input_event' drivers/built-in.o: In function `input_report_key': include/linux/input.h:389: undefined reference to `input_event' drivers/built-in.o: In function `input_sync': include/linux/input.h:414: undefined reference to `input_event' drivers/built-in.o:include/linux/input.h:389: more undefined references to `input_event' follow Signed-off-by: Randy Dunlap <[email protected]> Acked-by: Marek Vasut <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-21staging: lustre: Remove typedef and update cfs_hash_lock unionLisa Nguyen2-14/+14
Remove typedef keyword and rename the cfs_hash_lock_t union to cfs_hash_lock in libcfs_hash.h. These changes resolve the "Do not add new typedefs" warning generated by checkpatch.pl and meet kernel coding style. The union variables in hash.c are updated to reflect this change as well. Signed-off-by: Lisa Nguyen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-21staging:lustre: replace foo* with foo *Dulshani Gunawardhana3-10/+10
Fix checkpatch.pl warning of "(foo*)" should be "(foo *)" in lustre/lov Signed-off-by: Dulshani Gunawardhana <[email protected]> Reviewed-by: Josh Triplett <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-21staging:lustre: Fix externs should be avoided in .c files warningDulshani Gunawardhana3-2/+4
Fix the checkpatch.pl warning 'externs should be avoided in .c files' in lustre/lov. Signed-off-by: Dulshani Gunawardhana <[email protected]> Reviewed-by: Josh Triplett <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-21staging:lustre: Fix space before open parenthesisDulshani Gunawardhana6-6/+6
Fix checkpatch.pl error "space required before the open parenthesis '('" in lustre/lov. Signed-off-by: Dulshani Gunawardhana <[email protected]> Reviewed-by: Josh Triplett <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-21staging:lustre: Replace max() with max_t()Dulshani Gunawardhana1-1/+1
Replace max() with max_t() in accordance to chekpatch.pl warning in lov_obd.c. Signed-off-by: Dulshani Gunawardhana <[email protected]> Reviewed-by: Josh Triplett <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-21staging:lustre: Fix C99 style commentsDulshani Gunawardhana1-1/+1
Fix C99 style comments in lov_obd.c in accordance to checkpatch.pl warning. Signed-off-by: Dulshani Gunawardhana <[email protected]> Reviewed-by: Josh Triplett <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-21staging:lustre: Remove space between function name and open parenthesisDulshani Gunawardhana4-38/+38
Remove checkpatch.pl warning of space prohibited between function name and open parenthesis '('in /lustre/lov. Signed-off-by: Dulshani Gunawardhana <[email protected]> Reviewed-by: Josh Triplett <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-21staging: lustre: libcfs: Remove bitmap.h fileLisa Nguyen1-110/+0
The functions in the bitmap.h header file are not being used by any source file in the lustre driver. Unless there are any objections, please remove the bitmap.h header file as it contains unused code in the kernel. Signed-off-by: Lisa Nguyen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-20iio:adc:max1363 support SMBus for 8-bit devicesVivien Didelot1-19/+57
The driver currently supports only I2C access. But supported devices with an accuracy of 8-bit are compatible with the SMBus byte access routines. This patch wraps the send and receive routines depending on the chip accuracy and fonctionnalities of its adapter. For instance, this allows us to use a MAX11603 on a ICH7 controller. This patch also simplifies the max1363_write_basic_config() routine to use the struct max1363_state fields directly. Signed-off-by: Vivien Didelot <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-20iio: Correct description how to convert microtesla to gaussPeter Meerwald1-1/+1
this just fixes the comment; however, I'm not sure if the driver reports measurements correctly; the raw values are 0.3 uT / LSB; IIO is supposed to report magnetic fields in Gauss, so the scale should be around 1/300 (ignoring ASA) -- but value and scale are returned as VAL_INT Signed-off-by: Peter Meerwald <[email protected]> Signed-off-by: Jonathan Cameron <[email protected]>
2013-10-19staging: rtl8187se: checkpatch cleanupsTeodora Baluta1-2/+2
Fix the following checkpatch.pl errors: drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c:890: "foo* bar" should be "foo *bar" drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c:944: "foo* bar" should be "foo *bar" Signed-off-by: Teodora Baluta <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-19staging: rtl8187se: fix static sparse warningsTeodora Baluta7-60/+64
This patch fixes sparse warnings regarding functions that should be static in driver rtl8187se. Signed-off-by: Teodora Baluta <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-19staging: rtl8187se: add missing __iomem annotationTeodora Baluta1-8/+8
Fix the following type of sparse warnings: drivers/staging/rtl8187se/r8180_core.c:163:48: warning: incorrect type in argument 1 (different address spaces) drivers/staging/rtl8187se/r8180_core.c:163:48: expected void const volatile [noderef] <asn:2>*addr drivers/staging/rtl8187se/r8180_core.c:163:48: got unsigned char [usertype] * Signed-off-by: Teodora Baluta <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-19staging: rtl8187se: use gfp_t instead of intTeodora Baluta1-1/+1
Fix the following type of sparse warnings: drivers/staging/rtl8187se/ieee80211/ieee80211_tx.c:249:17: warning: incorrect type in argument 2 (different base types) drivers/staging/rtl8187se/ieee80211/ieee80211_tx.c:249:17: expected restricted gfp_t [usertype] flags drivers/staging/rtl8187se/ieee80211/ieee80211_tx.c:249:17: got int [signed] gfp_mask Signed-off-by: Teodora Baluta <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-19Staging: sb105x: Remove redundant null checkRashika Kheria1-4/+1
This patch fixes the following Sparse information in sb_pci_mp.c- ~/git/kernels/linux/drivers/staging/sb105x/sb_pci_mp.c:1813 mp_unregister_driver() info: redundant null check on drv->state calling kfree() Signed-off-by: Rashika Kheria <[email protected]> Reviewed-by: Josh Triplett <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-19Staging: line6: replacing -EINVAL with retval in driver.cAshvini Varatharaj1-1/+1
drivers/staging/line6/driver.c:208 line6_send_raw_message_async_part() info: why not propagate 'retval' from usb_submit_urb() instead of (-22)? Signed-off-by: Ashvini Varatharaj <[email protected]> Reviewed-by: Josh Triplett <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-19Staging: line6: replacing -EINVAL with retvalAshvini Varatharaj1-1/+1
drivers/staging/line6/midi.c:147 send_midi_async() info: why not propagate 'retval' from usb_submit_urb() instead of (-22)? Signed-off-by: Ashvini Varatharaj <[email protected]> Reviewed-by: Josh Triplett <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-19Staging: speakup: replacing simple_strtoul with kstrtoulAshvini Varatharaj1-3/+5
Fix checkpatch warning: WARNING: simple_strtoul is obsolete, use kstrtoul instead Signed-off-by: Ashvini Varatharaj <[email protected]> Reviewed-by: Peter P Waskiewicz Jr <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-19Staging: sep: Fix Smatch Warning in sep_crypto.cRashika Kheria1-2/+2
This patch fixes the following smatch warning- ~/git/kernels/linux/drivers/staging/sep/sep_crypto.c:1137 sep_crypto_block_data() info: why not propagate 'int_error' from sep_oddball_pages() instead of (-12)? ~/git/kernels/linux/drivers/staging/sep/sep_crypto.c:1152 sep_crypto_block_data() info: why not propagate 'int_error' from sep_oddball_pages() instead of (-12)? Signed-off-by: Rashika Kheria <[email protected]> Reviewed-by: Josh Triplett <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-19staging: vt6655: Fix C99 commenting style in michael.hArchana kumari1-5/+5
This patch fixes C99 commenting style in staging:vt6655:michael.h Signed-off-by: Archana kumari <[email protected]> Reviewed-by: Josh Triplett <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2013-10-19Staging: slicoss: Fix quoted string split across linesRashika Kheria1-2/+1
This patch fixes the following checkpatch.pl warning in slicoss.c- WARNING: quoted string split across lines Signed-off-by: Rashika Kheria <[email protected]> Reviewed-by: Peter P Waskiewicz Jr <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>