- 20 Sep, 2016 40 commits
-
-
Georgiana Rodica Chelu authored
Reorganize the parameters to improve the readability of the code in two manners: - parameters of distinctive variable types on different lines - parameters of similar variable types on the same line Also, take advantage of the horizontal space available and place all the parameters on a single line. Issues found by checkpatch.pl script. Signed-off-by:
Georgiana Rodica Chelu <georgiana.chelu93@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Georgiana Rodica Chelu authored
Organize the parameters on a single line to improve the readability of the code. Also, indent the line to match alignment with the open parenthesis. Ensure that the modifications do not break the rule of 80 characters per line. Issues found by checkpatch.pl script. Signed-off-by:
Julia Lawall <julia.lawall@lip6.fr> Signed-off-by:
-
Rehas Sachdeva authored
Instead of storing the return value of a function call into a variable and then returning it, we can club the two into a single return statement. This change was made using the following semantic patch by Coccinelle: @@ local idexpression ret; expression e; @@ -ret = +return e; -return ret; Signed-off-by:Rehas Sachdeva <aquannie@gmail.com> Signed-off-by:
-
Georgiana Rodica Chelu authored
Remove the braces {} in the following cases: - when there is a single branch with a single statement. - when both branches have a single statement The modifications improve the coding style. Issues found by checkpatch.pl script. Signed-off-by: -
Sandhya Bankar authored
Use !x instead of x == NULL. This patch was found by checkpatch. Signed-off-by:
Sandhya Bankar <bankarsandhya512@gmail.com> Signed-off-by:
-
Sandhya Bankar authored
Use !x instead of x == NULL. This patch was found by checkpatch. Signed-off-by:
-
Sandhya Bankar authored
Use !x instead of x == NULL. This patch was found by checkpatch. Signed-off-by:
-
Sandhya Bankar authored
Use !x instead of x == NULL. This patch was found by checkpatch. Signed-off-by:
-
Namrata A Shettar authored
Insert blank line after function declaration to resolve checkpatch issue. Signed-off-by:
Namrata A Shettar <namrataashettar@gmail.com> Signed-off-by:
-
Namrata A Shettar authored
Add space around binary operator '+' to resolve checkpatch issue. Signed-off-by:
-
Namrata A Shettar authored
Remove unneccessary blank lines to resolve checkpatch issue. Signed-off-by:
-
Namrata A Shettar authored
Add space around '+' to resolve checkpatch issue. Signed-off-by:
-
Bhumika Goyal authored
Some type conversions like casting a pointer to a pointer of same type, casting to the original type using addressof(&) operator etc. are not needed. Therefore, remove them. Done using coccinelle: @@ type t; t *p; t a; @@ ( - (t)(a) + a | - (t *)(p) + p | - (t *)(&a) + &a ) Signed-off-by:
Bhumika Goyal <bhumirks@gmail.com> Signed-off-by:
-
Xavier Roumegue authored
Fix the following warnings: Symbolic permissions are not preferred. Consider using octal permissions. Signed-off-by:
Xavier Roumegue <xroumegue@gmail.com> Signed-off-by:
-
Ivan Safonov authored
GET_HAL_DATA replaced by its definition. Signed-off-by:
Ivan Safonov <insafonov@gmail.com> Signed-off-by:
-
Ivan Safonov authored
This macro does not used. Signed-off-by:
-
Ivan Safonov authored
To avoid unnecessary typecast. Signed-off-by:
-
Ivan Safonov authored
rtw_hal_set_hwreg and rtw_hal_get_hwreg does not used with HW_VAR_DM_FLAG parameter. Signed-off-by:
-
Ivan Safonov authored
usb_hal.h is empty. Signed-off-by:
-
Ivan Safonov authored
rtl8188eu_set_hal_ops only allocates HalData member of adapter structure. Signed-off-by:
-
Ivan Safonov authored
PTR_ALIGN is a bit shorter than N_BYTE_ALIGMENT. Signed-off-by:
-
Ivan Safonov authored
This function does not used. Signed-off-by:
-
Ivan Safonov authored
pkt_hdrlen has a constant value. Signed-off-by:
-
Ivan Safonov authored
xmitframe_direct is a simple wrapper around rtw_xmitframe_coalesce and rtw_dump_xframe functions. Many wrappers complicates code reading. Signed-off-by:
-
Ivan Safonov authored
usb_write_port writes only xmit_buf object data to device. In addition, an appropriate name for this argument is used. Signed-off-by:
-
Rehas Sachdeva authored
Instead of storing the return value into a variable and then returning it, we can club the two into a single return statement. This change was made using the following semantic patch by Coccinelle: @@ local idexpression ret; expression e; @@ -ret = +return e; -return ret; Signed-off-by: -
David Kershner authored
parahotplug_request_list and parahotplug_request_list_lock should be defined closer to where we are actually using them. Reported-by:
David Kershner <david.kershner@unisys.com> Reviewed-by:
Tim Sell <Timothy.Sell@unisys.com> Signed-off-by:
-
David Kershner authored
Move the visordriver_probe_device and visorbus_register_visor_driver functions lower in the file to get rid of the function prototype fix_vbus_dev_info. Signed-off-by:
-
David Kershner authored
Move the struct visorbus_type down in the file to get rid of the function prototypes visorbus_uevent and visorbus_match. Reported-by:
-
David Kershner authored
Move the deviceenabled/devicedisabled store functions so that function prototypes can be removed. This caused us to move several of the structures farther down in the file as well. Reported-by:
-
David Kershner authored
Moving the remaining_steps functions allows us to get rid of an unneeded prototype. Reported-by:
-
David Kershner authored
The textid store and show functions were moved to allow us to remove the function prototypes. Reported-by:
-
David Kershner authored
Move the show and store functions for the error DEV_ATTR_RW so that the function prototypes can be removed. Reported-by:
-
David Kershner authored
Remove the unneeded prototype parser_done. Reported-by:
-
David Kershner authored
Remove the unneeded prototype declaration for function controlvm_respond_physdev_changestate. Reported-by:
-
David Kershner authored
The function parser_init_byte_stream needs to be moved lower in the file to avoid extraneous function prototypes. Reported-by:
-
David Kershner authored
Move the boottotool show and store functions so the function prototypes can be removed. Reported-by:
-
David Kershner authored
The toolchain functions show and store needed to be moved so the prototypes were no longer needed. Reported-by:
-
David Kershner authored
Remove the unneeded controlvm_respond_chipset_init prototype by moving the functions controlvm_init_response and controlvm_respond_chipset_init up in the file. Reported-by:
-
David Kershner authored
Remove the unneeded prototype for controlvm_respond. Reported-by:
-
