- 08 Jun, 2015 40 commits
-
-
Don Zickus authored
Removing stuff that isn't being used. Another prepartion patch to allow us to use visor_device everywhere without the baggage of bus/dev_info. Signed-off-by:
Don Zickus <dzickus@redhat.com> Signed-off-by:
Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Don Zickus authored
The bus device and regular device were using two different structs. Let's combine them as they are not entirely different from one another. This allows us to move this creation up the stack later and actually remove bus/dev_info easily. Most of the churn is just renaming devdata -> dev and 'struct visorbus_devdata' to 'struct visor_device'. Signed-off-by:
-
Chaitanya Dhere authored
This change was detected with help of checkpatch.pl script. Signed-off-by:
Chaitanya Dhere <cvijaydh@visteon.com> Signed-off-by:
-
Chaitanya Dhere authored
In this patch, the if, else conditions are modified to remove the unnecessary equality checks. This change was detected with help of coccinelle tool. Signed-off-by:
-
Style (line over 80 chars) in drivers/staging/wlan-ng/prism2fw.c Signed-off-by:
Douglas Barbonaglia Sathler Figueiredo <eng.douglasfigueiredo@gmail.com> Signed-off-by:
-
Madhusudhanan Ravindran authored
removed the commented INFO lines. Signed-off-by:
Madhusudhanan Ravindran <mravindr@visteon.com> Signed-off-by:
-
Hari Prasath Gujulan Elango authored
Fixed indentation issue in few lines Signed-off-by:
Hari Prasath Gujulan Elango <hgujulan@visteon.com> Signed-off-by:
-
Hari Prasath Gujulan Elango authored
remove an unwanted assignment to a variable which is overwritten in the very next line.The first value assigned is not used. Signed-off-by:
-
Hari Prasath Gujulan Elango authored
Remove the ununsed Macro Signed-off-by:
-
Isaac Assegai authored
Insert Spaces after commas to rectify the following checkpatch errors in ddk750_display.c: ERROR: space required after that ',' Signed-off-by:
Isaac Assegai <isaac.a.travers@gmail.com> Signed-off-by:
-
Isaac Assegai authored
Insert Spaces after commas to rectify the following checkpatch errors in ddk750_help.c and ddk750_mode.c: ERROR: space required after that ',' Signed-off-by:
-
Isaac Assegai authored
Insert Spaces after commas to rectify the following checkpatch errors in ddk750_power.c and ddk750_swi2c.c: ERROR: space required after that ',' Signed-off-by: Isaac Assegai <isaac.a.travers@gmail.com>: Signed-off-by: -
Isaac Assegai authored
Insert Spaces after commas to rectify the following checkpatch errors in sm750_cursor.c: ERROR: space required after that ',' Signed-off-by:
-
Isaac Assegai authored
Insert Spaces after commas to rectify the following checkpatch errors in sm750_accel.c: ERROR: space required after that ',' Signed-off-by:
-
Isaac Assegai authored
Insert Spaces after commas to rectify the following checkpatch errors in sm750_hw.c: ERROR: space required after that ',' Signed-off-by:
-
Isaac Assegai authored
Insert Spaces after commas to rectify the following checkpatch errors in ddk750_hwi2c.h, ddk750_help.h and ddk750_chip.h: ERROR: space required after that ',' Signed-off-by:
-
Isaac Assegai authored
Insert Spaces after commas to rectify the following checkpatch errors in sm750_cursor.h, sm750_accel.h, ddk750_power.h and ddk750_mode.h: ERROR: space required after that ',' Signed-off-by:
-
Isaac Assegai authored
Insert Spaces after commas to rectify the following checkpatch errors in sm750.h: ERROR: space required after that ',' Signed-off-by:
-
Isaac Assegai authored
Insert Spaces after commas to rectify the following checkpatch errors in sm750_help.h: ERROR: space required after that ',' Signed-off-by:
-
Isaac Assegai authored
Insert Spaces after commas in sm750_hw.h to rectify the following checkpatch errors in sm750_hw.h: ERROR: space required after that ',' Signed-off-by:
-
Hari Prasath Gujulan Elango authored
The sm750fb driver has few Framebuffer configuration dependencies that need to be selected in order to get compiled successfully Signed-off-by:
-
David Decotigny authored
Source and destination have the same little-endian annotation: this patch removes incorrect byte-swap on non-LE cpus. This addresses the following sparse warning: drivers/staging/rtl8723au/core/rtw_mlme_ext.c:3911:56: warning: incorrect type in argument 1 (different base types) drivers/staging/rtl8723au/core/rtw_mlme_ext.c:3911:56: expected unsigned short [unsigned] [usertype] val drivers/staging/rtl8723au/core/rtw_mlme_ext.c:3911:56: got restricted __le16 [usertype] BA_timeout_value Signed-off-by:
David Decotigny <ddecotig@gmail.com> Signed-off-by:
-
David Decotigny authored
This fixes bitwise arithmetic performed on the host on a variable previously converted to little-endian, and subsequently converted again to little-endian: - issue_action_BA23a() called with "status" crafted in host byte order - "status" converted to LE - bitwise arithmetic on the (LE) "status", performed with masks and shifts in host byte order - result converted to LE (again) and stored in device structure Sparse warning addressed by this patch: drivers/staging/rtl8723au/core/rtw_mlme_ext.c:3806:16: warning: incorrect type in assignment (different base types) drivers/staging/rtl8723au/core/rtw_mlme_ext.c:3806:16: expected unsigned short [unsigned] status drivers/staging/rtl8723au/core/rtw_mlme_ext.c:3806:16: got restricted __le16 [usertype] <noident> Additional notes: initial cpu_to_le16 was introduced by kernel bulk commit 5e93f352 "staging: r8723au: Add source files for new driver - part 1", initially from github according to commit description. On github, this traces back to another bulk commit: 2896bda04353 "Add new files in core directory", which is the 1st version of the driver. Signed-off-by: -
Sudip Mukherjee authored
str was 16 bytes but was mentioned as 128 in snprintf. again msg is 128 bytes but not sufficient to hold the complete debug message of register values. Now removed the use of str, msg and print the register values from the loop. Signed-off-by:
Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by:
-
Madhusudhanan Ravindran authored
This patch attempts to simplify the debugging using standard dev_dbg call so that individual debug prints can be enabled or disbled by dynamic debugging rather than using module params. Signed-off-by:
-
Abhishek Sharma authored
Replace unnecessary conditional checks for variable 'ret' and replace by single return statement. Signed-off-by:
Abhishek Sharma <asharm14@visteon.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
Joglekar Tejas authored
This patch fix warning given by checkpatch.pl abouts spaces given before tab Signed-off-by:
Joglekar Tejas <joglekartejas@gmail.com> Reviewed-by:
-
Antonio Murdaca authored
cleanup checkpatch.pl warnings about not needed else clauses after a break or return Signed-off-by:
Antonio Murdaca <antonio.murdaca@gmail.com> Signed-off-by:
-
Dan Carpenter authored
Add some limits here so we don't corrupt memory. Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
-
Abhishek Sharma authored
Removing the commented code lines. Signed-off-by:
-
Abhishek Sharma authored
Removing the commented static variable declerations. Signed-off-by:
-
Hari Prasath Gujulan Elango authored
Remove commented code from this file. Signed-off-by:
-
Hari Prasath Gujulan Elango authored
The function WILC_WFI_InitPriv() is not used anywhere in the driver.Hence remove it. Signed-off-by:
-
Hari Prasath Gujulan yyElango authored
use the time_after_eq macro for the comparison operation Signed-off-by:
-
Hari Prasath Gujulan Elango authored
Remove dead code or commented code Signed-off-by:
-
Hari Prasath Gujulan Elango authored
Remove ununecessary typecast for kzalloc.This patch was generated by coccinelle tool Signed-off-by:
-
Chaitanya Dhere authored
In this patch, ARRAY_SIZE() macro is used to determine the size. This change was detected with the help of coccinelle tool. Signed-off-by:
-
Sudip Mukherjee authored
size_t should print using %zu and unsigned long int should use %lu but here it was using %d and hence we were getting warning while printing. Signed-off-by:
-
Julia Lawall authored
The rq_buflen field of the lov_request structure is never initialized. It is only used in the free of req->rq_oi.oi_md in lov_finish_set. But no oi_md field is ever initialized to the result of calling OBD_ALLOC_LARGE. So it seems that the call to OBD_FREE_LARGE in lov_finish_set and the rq_buflen in the lov_request structure are simply not needed. Signed-off-by:
Julia Lawall <Julia.Lawall@lip6.fr> Signed-off-by:
-
Julia Lawall authored
The fields set_oabufs and set_pga fields in the lov_request_set structure are never set, so drop them. Drop also the corresponding test and free in lov_finish_set. Signed-off-by:
-
