- 25 Jun, 2017 4 commits
-
-
Gilad Ben-Yossef authored
Some SoC which implement CryptoCell have a dedicated clock tied to it, some do not. Implement clock support if exists based on device tree data and tie power management to it. Signed-off-by:
Gilad Ben-Yossef <gilad@benyossef.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Gilad Ben-Yossef authored
The function set_ack_last was not used anywhere. Remove it. Signed-off-by:
-
Gilad Ben-Yossef authored
The original ccree driver was registering a useless setkey method even for non-MAC hash transformations. Somewhere around v4.9 a check was added that failed hash operations if a setkey method was registered but was not called, so during the initial upstream port code was added to only register the setkey method for MAC type hash transform. Unfortunately, the ccree driver also registers non-hash based MAC transforms and the code had a logic error that stopped it registering a setkey callback even for those, thus rendering them useless. This commit fixes the logic mistake, thus correctly registering a setkey method only for MAC transformations, leaving it out for non-MAC ones, whether they are hash based on not. Fixes: 50cfbbb7 ("staging: ccree: add ahash support"). Signed-off-by:
-
Gilad Ben-Yossef authored
Hash import and export was saving and restoring the wrong context and therefore disabled. Fix it by restoring intermediate digest and additional state needed. The hash and mac transform now pass testmgr partial hash tests. Signed-off-by:
-
- 23 Jun, 2017 36 commits
-
-
Suniel Mahesh authored
le16_to_cpu() accepts argument of type __le16 and cpu_to_le16() returns an argument of type __le16. This patch fixes: (a) the type of the variable that end's up getting return from cpu_to_le16(). (b) the member types of struct hfa384x_host_scan_request_data, struct hfa384x_bytestr32 and struct hfa384x_hscan_result_sub. The following type mismatch warnings reported by sparse have been fixed: warning: incorrect type in assignment (different base types) warning: cast to restricted __le16 Signed-off-by:Suniel Mahesh <sunil.m@techveda.org> Signed-off-by:
-
Amitoj Kaur Chawla authored
Remove unnecassary casts in the argument to kfree. Found using Coccinelle. The semantic patch used to find this is as follows: //<smpl> @@ type T; expression *f; @@ - kfree((T *)(f)); + kfree(f); //</smpl> Signed-off-by:
Amitoj Kaur Chawla <amitoj1606@gmail.com> Signed-off-by:
-
simran singhal authored
This patch assigns (true/false) to boolean EDCCA_State instead of (1/0). And, there is no need of comparing EDCCA_State explicitly with constant 1. Signed-off-by:
simran singhal <singhalsimran0@gmail.com> Signed-off-by:
-
Sudip Mukherjee authored
The previous patch which updated screen resolution was tested under wrong environment. sm750 driver does not support 24bpp. It only supports 8bpp, 16bpp and 32bpp. Lets update the default screen resolution to use 32bpp for a better screen performance. Fixes: ac669251 ("staging: sm750fb: change default screen resolution") Signed-off-by:
Sudip Mukherjee <sudipm.mukherjee@gmail.com> Signed-off-by:
-
Matthew Reed authored
Remove white space after tabstop on closing bracket as suggested by checkpatch.pl. Additional white space removed for page consistency. Signed-off-by:
Matthew Reed <4d5452@gmail.com> Signed-off-by:
-
Frans Klaver authored
The header field in struct pd_message is declared as an __le16 type. The data in the message is supposed to be little endian. This means we don't have to go and shift the individual bytes into position when we're filling the buffer, we can just copy the contents right away. As an added benefit we don't get fishy results on big endian systems anymore. Signed-off-by:
Frans Klaver <fransklaver@gmail.com> Reviewed-by:
Guenter Roeck <linux@roeck-us.net> Signed-off-by:
-
Laurentiu Tudor authored
dprc.h is only used in the mc bus driver so move it together with the sources thus making it private. Signed-off-by:
Laurentiu Tudor <laurentiu.tudor@nxp.com> Signed-off-by:
-
Laurentiu Tudor authored
mc-cmd.h contains some low level functions used to encode and decode commands to the MC. They are used by the drivers so move them to the public headers and get rid of the mc-cmd.h header. Signed-off-by:
-
Laurentiu Tudor authored
mc-sys.h contains the API to send commands to the MC and is used by drivers. Move it to the public headers and get rid of the mc-sys.h header. Signed-off-by:
-
Laurentiu Tudor authored
Few files using byte order macros but did not explicitly included the required kernel header, so add it. Signed-off-by:
-
Laurentiu Tudor authored
dpmng.h & dpmng.c files expose an API of just one function which is only used by the bus driver. Move that single API in the bus source as static and remove the two files. Signed-off-by:
-
Laurentiu Tudor authored
All the mc-bus.h contents is only used privately in the bus driver so move everything to the private header and get rid of the mc-bus.h header file. Signed-off-by:
-
Laurentiu Tudor authored
Define dev_is_fsl_mc() and the bus type definition (fsl_mc_bus_type) are used externally so move them to the public header. Signed-off-by:
-
Laurentiu Tudor authored
fsl_mc_msi_create_irq_domain() will is used from the irqchip glue code so it needs to be in the public headers. Signed-off-by:
-
Laurentiu Tudor authored
They are never used outside the source they are implemented in and very likely never will, so it's safe to make them static. Signed-off-by:
-
Laurentiu Tudor authored
The function fsl_mc_bus_exists() has a prototype but is never implemented so delete it from the header file. Signed-off-by:
-
Laurentiu Tudor authored
These functions already have their prototypes in fsl-mc-private.h header file so delete them from mc-bus.h. Signed-off-by:
-
Laurentiu Tudor authored
In its current form, the public headers of the mc-bus depend only on a structure "dprc_obj_desc" defined in dprc.h. Move it to the bus public header together with its associated defines and, in order to keep the naming prefixes consistent rename it to "fsl_mc_obj_desc". This will allow making dprc.h private in future patches. Signed-off-by:
-
Laurentiu Tudor authored
These couple of header files are not needed in the source so remove them. Signed-off-by:
-
Laurentiu Tudor authored
Several macros were triggering this checkpatch.pl warning: "Macro argument reuse '$arg' - possible side-effects?" Fix the warning by turning them into real functions. Signed-off-by:
-
Laurentiu Tudor authored
dpni.c is using byte order macros and error codes but does not explicitly include the required kernel header, so add it. Signed-off-by:
Ioana Radulescu <ruxandra.radulescu@nxp.com> Signed-off-by:
-
Ioana Radulescu authored
dpaa2_io_service_register() returns zero even if qbman_swp_CDAN_set() encountered an error. Fix this by propagating the error code so the caller is informed data availability notifications are not properly set for a channel. Signed-off-by:
Bogdan Purcareata <bogdan.purcareata@nxp.com> Signed-off-by:
-
Ian Abbott authored
Centralize the "clean-up on error" handling in `comedi_init()` using `goto` statements. Also change some of the explicit `-EIO` return values to the error return values from the failing functions as there is no good reason to use `-EIO` explicitly. Signed-off-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
Ian Abbott authored
There is a clean-up bug in the core comedi module initialization functions, `comedi_init()`. If the `comedi_num_legacy_minors` module parameter is non-zero (and valid), it creates that many "legacy" devices and registers them in SysFS. A failure causes the function to clean up and return an error. Unfortunately, it fails to destroy the "comedi" class that was created earlier. Fix it by adding a call to `class_destroy(comedi_class)` at the appropriate place in the clean-up sequence. Signed-off-by:
-
Malcolm Priestley authored
Following removal of _rtl92e_ioctl this function along with associated macros, structure ieee_param and functions become dead code. Remove functions rtllib_wpa_enable, rtllib_wpa_assoc_frame, rtllib_wpa_mlme, rtllib_wpa_set_wpa_ie, rtllib_wpa_set_auth_algs, rtllib_wpa_set_param, rtllib_wpa_set_encryption and rtllib_wpa_supplicant_ioctl. Signed-off-by:
Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by:
-
Malcolm Priestley authored
A RTL_IOCTL_WPA_SUPPLICANT call is a proprietary version of wpa supplicant. All kernel calls use SIOCSIWENCODEEXT call via wireless handlers already used in this driver. Remove dead code. Signed-off-by:
-
Andrey Shvetsov authored
This adds the dev_hold and dev_put calls to the functions aim_resume_tx_channel, aim_rx_data and on_netinfo to postpone the unregistration of the used net device. Signed-off-by:
Andrey Shvetsov <andrey.shvetsov@k2l.de> Signed-off-by:
Christian Gromm <christian.gromm@microchip.com> Signed-off-by:
-
Andrey Shvetsov authored
This introduces the mutex that protects the consistency between the tx.linked, rx.linked and the presence of the net divice. Additionally, this patch optimizes the setup of the ch->linked in the function aim_probe_channel. Signed-off-by:
-
Andrey Shvetsov authored
The function aim_probe_channel calls only one of the functions alloc_netdev and register_netdev per run. Correspondingly, the function aim_disconnect_channel calls only one of the functions unregister_netdev and free_netdev per run. This patch makes it obvious by using the 'else' part of the 'if' statement. Signed-off-by:
-
Andrey Shvetsov authored
This removes redundant netdev_info/pr_info. Signed-off-by:
-
Jeremy Sowden authored
Removed spaces after opening parentheses in ssi_ivgen.c. Signed-off-by:
Jeremy Sowden <jeremy@azazel.net> Signed-off-by:
-
Derek Robson authored
Changed code indent to be tabs across whole driver Found using checkpatch Signed-off-by:
Derek Robson <robsonde@gmail.com> Acked-by:
-
Jhih-Ming Hunag authored
Move else to follow close brace '}' Signed-off-by:
Jhih-Ming Hunag <fbihjmeric@gmail.com> Acked-by:
-
Jhih-Ming Hunag authored
Remove improper space. Signed-off-by:
-
Jhih-Ming Hunag authored
Move * to close variable name instead of type. Signed-off-by:
-
Jhih-Ming Hunag authored
Move '{' to next line for function. Signed-off-by:
-
