- Dec 13, 2017
-
-
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:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
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:
-
Gilad Ben-Yossef authored
Fix function call parameter indentation according to coding style guide lines. Signed-off-by:
-
Gilad Ben-Yossef authored
Simplify expression by using a local variable for better code readability. Signed-off-by:
-
Gilad Ben-Yossef authored
Fix functions definition and declaration indentation according to coding style guide lines for better code readability Signed-off-by:
-
Gilad Ben-Yossef authored
Shorten parameter name for better code readability Signed-off-by:
-
Gilad Ben-Yossef authored
Fix some call sites with func params not following func name in AEAD code. Signed-off-by:
-
Gilad Ben-Yossef authored
Func definitions in the hash implementation were did not adhere to coding style. Fix them for better readability. Signed-off-by:
-
Gilad Ben-Yossef authored
The hash files were using a naming convention which was inconsistent (ssi vs. cc), included a useless prefix (ssi_hash) and often used too long function names producing monster such as ssi_ahash_get_initial_digest_len_sram_addr() that made the call site hard to read. Make the code more readable by switching to a simpler, consistent naming convention for the file. Signed-off-by:
-
Gilad Ben-Yossef authored
Remove a no longer needed abstraction around ccree hash crypto API internals that used to allow same ops to be used in synchronous and asynchronous fashion. Signed-off-by:
-
Aliaksei Karaliou authored
lu_global_init() does not check result of register_shrinker() which was tagged __must_check recently, reported by sparse. Patch also fixes missed cleanup of resources allocated prior to register_shrinker() invocation and not freed after any failure. Signed-off-by:
Aliaksei Karaliou <akaraliou.dev@gmail.com> Signed-off-by:
-
Aliaksei Karaliou authored
ldlm_pools_init() does not check result of register_shrinker() which was tagged __must_check recently, reported by sparse. Signed-off-by:
-
Aliaksei Karaliou authored
sptlrpc_enc_pool_init() does not check result of register_shrinker() which was tagged __must_check recently, reported by sparse. Signed-off-by:
-
Aliaksei Karaliou authored
osc_init() does not check result of register_shrinker() which was tagged __must_check recently, reported by sparse. Signed-off-by:
-
NeilBrown authored
This macro isn't used, and comment is about some earlier version of the lustre code that never reached the mainline kernel. Just discard it. Signed-off-by:
NeilBrown <neilb@suse.com> Acked-by:
Luis de Bethencourt <luisbg@kernel.org> Signed-off-by:
-
NeilBrown authored
This appears to be intended for assertions that only make sense in the kernel, but as this code is now kernel-only, it doesn't make sense any more. Signed-off-by:
-
NeilBrown authored
This is unused. drivers/staging/lustre/lnet/lnet/nidstrings.c does use the name, but it includes its own local definition. Signed-off-by:
-
NeilBrown authored
This is only used for tracing when some strings might be NULL. NULL strings are not a problem for tracing, vnsprintf() will report them as "(null)" which is probably better (easier to parse) than an empty string. Also remove a nearby comment that doesn't relate to the (remaining) code at all. Signed-off-by:
-
NeilBrown authored
These are all unused except cfs_size_round(). So discard the others, and use the kernel-standard round_up() function to implement cfs_size_round(). Signed-off-by:
-
NeilBrown authored
1/ Use kvmalloc() instead of kmalloc or vmalloc 2/ Discard the _GFP() interfaces that are never used. We only ever do GFP_NOFS and GFP_ATOMIC allocations, so support each of those explicitly. Signed-off-by:
-
NeilBrown authored
Using the *_entry macro simplifies the code slightly. Signed-off-by:
-
