- 15 May, 2018 8 commits
-
-
Sergio Paracuellos authored
This patch silence some complains of checkpatch script because of the use of long lines. Signed-off-by:
Sergio Paracuellos <sergio.paracuellos@gmail.com> Reviewed-by:
NeilBrown <neil@brown.name> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Lai Siyao authored
Currently we set LU_OBJECT_HEARD_BANSHEE on object when we want to remove object from cache, but this may lead to deadlock, because when other process lookup such object, it needs to wait for this object until release (done at last refcount put), while that process maybe already hold an LDLM lock. Now that current code can handle dying object correctly, we can just return such object in lookup, thus the above deadlock can be avoided. Signed-off-by:
Lai Siyao <lai.siyao@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-9049 Reviewed-on: https://review.whamcloud.com/26965Reviewed-by:
Alex Zhuravlev <alexey.zhuravlev@intel.com> Tested-by:
Cliff White <cliff.white@intel.com> Reviewed-by:
Fan Yong <fan.yong@intel.com> Reviewed-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
James Simmons <jsimmons@infradead.org> Reviewed-by:
-
Kamal Heib authored
Remove unused variable 'condition' which was set but not used. Signed-off-by:
Kamal Heib <kamalheib1@gmail.com> Reviewed-by:
-
Nathan Chancellor authored
After commit 6d6612de ("staging: ks7010: Remove unnecessary limit checks"), this define is not used anywhere. Remove it as well. Suggested-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Nathan Chancellor <natechancellor@gmail.com> Signed-off-by:
-
Sidong Yang authored
Move open brace to same line with enum. Remove prohibited space before ','. Signed-off-by:
Sidong Yang <realwakka@gmail.com> Signed-off-by:
-
Nathan Chancellor authored
GCC warns that vaddr is set but unused. Check the return value of ion_buffer_kmap_get to make vaddr useful and make sure everything is properly configured before beginning a DMA. Suggested-by:
Laura Abbott <labbott@redhat.com> Signed-off-by:
-
Samuel Thibault authored
Signed-off-by:
Samuel Thibault <samuel.thibault@ens-lyon.org> Signed-off-by:
-
Pratik Jain authored
Recommended indentation by Lindent on file comedi_compat32.c Signed-off-by:
Pratik Jain <pratik.jain0509@gmail.com> Signed-off-by:
-
- 14 May, 2018 32 commits
-
-
Sergio Paracuellos authored
There is an assignment inside hostif_sme_set_pmksa function which is being used together with cpu_to_le16 using uint16_t as cast type. Replace it to use 'u16' instead. Signed-off-by:
-
Sergio Paracuellos authored
There are some assignments inside hostif_sme_set_rsn function which are being used together with cpu_to_le16 using uint16_t as cast type. Replace all of them to use 'u16' instead. Signed-off-by:
-
Sergio Paracuellos authored
Parameters 'failure_count' and 'timer' was declared as unsigned short and then there was being casted to u16 inside cpu_to_le16 to make the assignation. Just declare them as 'u16' and avoid casting at all. Signed-off-by:
-
Sergio Paracuellos authored
In function hostif_bss_scan_request parameters 'scan_ssid' and 'scan_ssid_len' are declared as uint8_t. Change them to be 'u8' instead which is preferred. Also update two casts inside the same function to use 'u32' instead of 'uint32_t'. Signed-off-by:
-
Sergio Paracuellos authored
There are some assignments inside hostif_phy_information_request function which are being used together with cpu_to_le16 using uint16_t as cast type. Replace all of them to use 'u16' instead. Signed-off-by:
-
Sergio Paracuellos authored
There is an assignment inside hostif_adhoc_set_request function which is being used together with cpu_to_le16 using uint16_t as cast type. Replace it to use 'u16' instead. Signed-off-by:
-
Sergio Paracuellos authored
There are some assignments inside hostif_infrastructure_set_request function which are being used together with cpu_to_le16 using uint16_t as cast type. Replace all of them to use 'u16' instead. Signed-off-by:
-
Sergio Paracuellos authored
There is an assignment inside hostif_ps_adhoc_set_request function which is being used together with cpu_to_le16 using uint16_t as cast type. Replace it to use 'u16' instead. Signed-off-by:
-
Sergio Paracuellos authored
There are soem assignments inside init_request function which are being used together with cpu_to_le16 using uint16_t as cast type. Replace all of them to use 'u16' instead. Signed-off-by:
-
Sergio Paracuellos authored
Use u16 as cast type in hostif_start_request function replacing uint16_t which was being used. Signed-off-by:
-
Sergio Paracuellos authored
In ks_wlan_cap there is a cast to uint16_t to use cpu_to_le16 with variable 'capability' which is already defined as u16. Avoid this cast to clean code. Signed-off-by:
-
Sergio Paracuellos authored
Second parameter 'mib_attribute' in function hostif_mib_get_request is declared as unsigned long and inside the function a cast to uint32_t is being used. Just pass a u32 instead and avoid the casting. Signed-off-by:
-
Sergio Paracuellos authored
There are some castings inside the function hostif_data_request which are being using with uint16_t type. Fields which have being assigned are declared as u16. So update casts types to u16 in all of them. Signed-off-by:
-
Sergio Paracuellos authored
Field 'counter' in mic_failure struct is being assigned casting value using uint16_t. Replace with u16 which is the correct type of the field and the preferred one. Signed-off-by:
-
Sergio Paracuellos authored
The field 'wakeup_count' in 'ks_wlan_private' struct is declared as 'uint' which is not a standard type. Replace in favour of 'unsigned int' which it is. Signed-off-by:
-
Sergio Paracuellos authored
In function ks_wlan_set_rx_gain a cast to uint8_t is being used to assign reception gain. 'rx_gain' field is defined as u8 so replace the cast to the correct type Signed-off-by:
-
Sergio Paracuellos authored
In function ks_wlan_set_tx_gain a cast to uint8_t is being used to assign transmission gain. 'tx_gain' field is defined as u8 so replace the cast to the correct type. Signed-off-by:
-
Sergio Paracuellos authored
There are some casts to uint8_t in ks_wlan_set_rate function to assign values of the bitrate. Just change it to u8 which is the one defined for the field 'body' of the struct which is in use. Signed-off-by:
-
Sergio Paracuellos authored
The parameter 'size' in function ks_wlan_hw_rx is declared as uint16_t and can be declared as size_t which makes more sense. It is being passed to hif_align_size function which also expects a size_t as parameter so just change its type. Also update two casts in calls to this function. Signed-off-by:
-
Sergio Paracuellos authored
Header size and event fields of header are declared as __le16 and being casted using uint16_t in cpu_to_le16. Change cast to use preferred u16. Signed-off-by:
-
Sergio Paracuellos authored
Variable 'byte' in ks7010_upload_firmware function is declared as unsigned char and is only being used to read hardware registers which are expected in sdio_read_byteb function as u8. Change 'byte' variable type to u8 which is preferred. Signed-off-by:
-
Sergio Paracuellos authored
There was two blank lines between definitions and statements in ks7010_upload_firmware function. Remove one of them. Signed-off-by:
-
Sergio Paracuellos authored
The WPA_CAPABILITY_PREAUTH definition is not being used at all so just remove it. Signed-off-by:
-
Kilian Köppchen authored
This patch fixes the checkpatch.pl check hint: CHECK: Prefer using the BIT_ULL macro Signed-off-by:
Kilian Köppchen <kiliankoeppchen@googlemail.com> Signed-off-by:
-
Kamal Heib authored
This patch removes the ugly macro hack to make sure hw_stats and ethtool strings are consisten, instead define a new struct which will hold the stat string and his index within the hw_stats struct. Signed-off-by:
-
Ajay Singh authored
Remove 'connect_resp_info' structure as most of its elements are not used. Modified wilc_parse_assoc_resp_info() to directly parse and fill value in connect_info structure variable. Remove use of 'assoc_resp_len' variable. get_assoc_resp_cap_info() & get_asoc_id() functions are remove as its not used anymore. Signed-off-by:
Ajay Singh <ajay.kathat@microchip.com> Signed-off-by:
-
Ajay Singh authored
Cleanup patch to have variable names as per linux coding style. Signed-off-by:
-
Ajay Singh authored
Cleanup patch to remove the unused variable from 'wilc_priv' structure. Signed-off-by:
-
Ajay Singh authored
Added comments for mutex and spinlock_t to avoid checkpatch.pl script. Signed-off-by:
-
Ajay Singh authored
Remove 'flag' function parameter in wilc_mac_indicate() as only one condition was handled using that parameter. Also removed unnecessary call to wilc_mac_indicate() as no operation was performed in that function call. After above changes below macros are not required anymore. WILC_MAC_INDICATE_STATUS 0x1 WILC_MAC_INDICATE_SCAN 0x2 This changes also helped in resolving the line over 80 chars issue found by checkatpch.pl script. Signed-off-by:
-
Ajay Singh authored
Added new macro to resolve below checkpatch issues in linux_mon. "Lines should not end with a '('" Signed-off-by: -
Ajay Singh authored
Fix line over 80 chars issue found by checkpatch.pl script by placing the comment message above the macro preprocessor. Signed-off-by:
-
