- Dec 13, 2017
-
-
Valentin Vidic authored
Fixes checkpatch warning for error prone comparison. Signed-off-by:
Valentin Vidic <Valentin.Vidic@CARNet.hr> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Valentin Vidic authored
Fixes checkpatch warning for blank line after function/struct/union/enum declarations. Signed-off-by:
-
Valentin Vidic authored
Fixes checkpatch warning for 'transfered' and 'devided'. Signed-off-by:
-
Valentin Vidic authored
Fixes checkpatch warning for comparing value with NULL. Signed-off-by:
-
Valentin Vidic authored
Fixes checkpatch warning for void function return statements. Signed-off-by:
-
Valentin Vidic authored
Fixes checkpatch warning for hidden spaces before tabs. Signed-off-by:
-
Valentin Vidic authored
Fixes checkpatch error for trailing if statements. Signed-off-by:
-
Valentin Vidic authored
Fixes checkpatch error for whitespace idents. Signed-off-by:
-
Valentin Vidic authored
Fixes checkpatch error for open/close braces. Signed-off-by:
-
Valentin Vidic authored
Fixes checkpatch error for prohibited spaces. Signed-off-by:
-
Valentin Vidic authored
Fixes checkpatch error for missing spaces. Signed-off-by:
-
Valentin Vidic authored
Fixes checkpatch error for missing spaces. Signed-off-by:
-
Valentin Vidic authored
Fixes checkpatch error for missing spaces. Signed-off-by:
-
Simon Sandström authored
dev_dbg() already depends on DEBUG. Signed-off-by:
Simon Sandström <simon@nikanor.nu> Signed-off-by:
-
Simon Sandström authored
ftrace can be used to trace function calls, so there is no need to use dev_dbg() here. Signed-off-by:
-
Simon Sandström authored
ifdef blocks should not increase indentation level. Signed-off-by:
-
Colin Ian King authored
The functions print_links and most_match static are local to the source and do not need to be in global scope, so make them static. Cleans up sparse warnings: symbol 'print_links' was not declared. Should it be static? symbol 'most_match' was not declared. Should it be static? Signed-off-by:
Colin Ian King <colin.king@canonical.com> Signed-off-by:
-
Shreeya Patel authored
Merge the assignment and return statements to return the value directly. Done using the following semantic patch by coccinelle. @@ local idexpression ret; expression e; @@ -ret = +return e; -return ret; Also, remove the variable declaration and some braces that became useless after the merge. Signed-off-by:Shreeya Patel <shreeya.patel23498@gmail.com> Signed-off-by:
-
Jia-Ju Bai authored
The driver may sleep under a spinlock. The function call path is: update_beacon (acquire the spinlock) update_BCNTIM set_tx_beacon_cmd kzalloc(GFP_KERNEL) --> may sleep kmemdup(GFP_KERNEL) --> may sleep To fix it, GFP_KERNEL is replaced with GFP_ATOMIC. This bug is found by my static analysis tool(DSAC) and checked by my code review. Signed-off-by:Jia-Ju Bai <baijiaju1990@gmail.com> Signed-off-by:
-
Neil Singh authored
Cleanup below checkpatch issues: ERROR:CODE_INDENT: code indent should use tabs where possible 1409: FILE: rtl871x_security.c:1409: + from_timer(padapter, t, securitypriv.tkip_timer);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line 1409: FILE: rtl871x_security.c:1409: + from_timer(padapter, t, securitypriv.tkip_timer);$ Signed-off-by:
Neil Singh <neil2468@gmail.com> Signed-off-by:
-
Neil Singh authored
Cleanup below checkpatch issue: WARNING:LONG_LINE: line over 80 characters 1000: FILE: rtl871x_security.c:1000: +static void construct_ctr_preload(u8 *ctr_preload, sint a4_exists, sint qc_exists, Signed-off-by:
-
Vikash Kesarwani authored
Fixed a blank line coding style after a declaration Signed-off-by:
Vikash Kesarwani <vikash.ckiiita30@gmail.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
Nipun Gupta authored
Devices on MC bus can be deferred because of dependencies on other modules (such as IOMMU). Those are not the actual errors; as probing is again done by the kernel at later stages. So this patch avoids the error print in such case. Signed-off-by:
Nipun Gupta <nipun.gupta@nxp.com> Signed-off-by:
-
Nipun Gupta authored
Following is the current scenario when the devices are probed: FSL_MC Bus probe ---> dprc probe ---> dprc devices scan ---> probe devices in DPRC container ---> allocate IRQ's In case the devices being probed in the DPRC container need the IRQ's; probing of that device will fail. In current scenario the devices which need IRQ's such as DPIO gets deferred because they aren't registered when first time the probing of these devices is done in the dprc scan. So they are probed once IRQ's have been allocated. In case where DPRC probing itself gets deferred, which does in case IOMMU is enabled; all the devices in DPRC container gets probed before IRQ's are allocated. This causes devices using IRQ's (such as DPIO) to fail. So having IRQ's allocated before any of the devices in the DPRC container are probed is more legitimate. After this patch following is the flow of execution: FSL_MC Bus probe ---> dprc probe ---> dprc devices scan ---> allocate IRQ's ---> probe of devices in DPRC container. Signed-off-by:Bharat Bhushan <bharat.bhushan@nxp.com> Signed-off-by:
-
Gilad Ben-Yossef authored
The FIPS manager files were using a naming convention which was inconsistent (ssi vs. cc) and often too long. Make the code more readable by switching to a simpler, consistent naming convention. Signed-off-by:
Gilad Ben-Yossef <gilad@benyossef.com> Signed-off-by:
-
Gilad Ben-Yossef authored
The code freeing the SRAM memory address was zeroing the address on release although there is nothing secret about it. Simplify the code by simply calling kfree directly. Signed-off-by:
-
Gilad Ben-Yossef authored
The SRAM manager files were using a naming convention which was inconsistent (ssi vs. cc) and often too long. Make the code more readable by switching to a simpler, consistent naming convention. Signed-off-by:
-
Gilad Ben-Yossef authored
The buffer manager files were using a func naming convention which was inconsistent (ssi vs. cc) and often too long. Make the code more readable by switching to a simpler, consistent naming convention. Signed-off-by:
-
Gilad Ben-Yossef authored
Unify naming convention by renaming all ssi_ vars/structs/enums and variables to cc_* Signed-off-by:
-
Gilad Ben-Yossef authored
Unify naming convention by renaming all DX macros to CC. Signed-off-by:
-
Gilad Ben-Yossef authored
Unify naming convention by renaming all SSI macros to CC. Signed-off-by:
-
Gilad Ben-Yossef authored
Remove macro controlling build of various features. This needs to happen dynamically in registration time. Signed-off-by:
-
Gilad Ben-Yossef authored
Remove the code support for MULTI2 mode which is not supported by the current hardware. Signed-off-by:
-
Gilad Ben-Yossef authored
Fix ivgen functions definition indentation according to coding style guide lines for better code readability Signed-off-by:
-
Gilad Ben-Yossef authored
The ivgen files were using a func naming convention which was inconsistent (ssi vs. cc), included a long prefix (ssi_ivgen) and often too long. Make the code more readable by switching to a simpler, consistent naming convention. Signed-off-by:
-
Gilad Ben-Yossef authored
Fix cipher functions definition indentation according to coding style guide lines for better code readability Signed-off-by:
-
Gilad Ben-Yossef authored
The blkcipher files were using a func naming convention which was inconsistent (ssi vs. cc), included a long prefix (ssi_ablkcipher) and often too long. Make the code more readable by switching to a simpler, consistent naming convention. Signed-off-by:
-
Gilad Ben-Yossef authored
Remove the remains of no longer existing support for running blkcipher is sync mode. Signed-off-by:
-
Gilad Ben-Yossef authored
Fix request manager functions definition indentation according to coding style guide lines for better code readability Signed-off-by:
-
Gilad Ben-Yossef authored
The request manager files were using a func naming convention which was inconsistent (ssi vs. cc), included a useless prefix (ssi_request_mgr) and often too long. Make the code more readable by switching to a simpler, consistent naming convention. Signed-off-by:
-
