- Aug 09, 2015
-
-
Chandra S Gorentla authored
Fixes the checkpatch.pl warning - 'unnecessary whitespace before a quoted newline'. Signed-off-by:
Chandra S Gorentla <csgorentla@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Chandra S Gorentla authored
'extern' declarations which are not referenced within the file are removed. Signed-off-by:
-
Chandra S Gorentla authored
Inserted space between nested braces. This fixes the checkpatch.pl error - space required after that close brace '}'. Signed-off-by:
-
- Aug 08, 2015
-
-
Sudip Mukherjee authored
Multiple blank lines is against the kernel coding style and checkpatch complains for that. Signed-off-by:
Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by:
-
Sudip Mukherjee authored
The function rtw_init_recvframe() was not being used. Signed-off-by:
-
Sudip Mukherjee authored
An exit label which does nothing except return, is not worth having. So remove it. Signed-off-by:
-
Sudip Mukherjee authored
_RECV_OSDEP_C_ was only defined here but never checked anywhere. Signed-off-by:
-
Sudip Mukherjee authored
Re-arrange the code to directly return success or failure, thus removing the variable used in the function. Signed-off-by:
-
Sudip Mukherjee authored
The return value of rtw_os_recv_resource_alloc() is never checked, so make it as void. Moreover as of now the function can not fail. Signed-off-by:
-
Sudip Mukherjee authored
The function rtw_os_recv_resource_alloc() only uses the argument struct recv_frame *. So remove the other unused argument. Signed-off-by:
-
Chaehyun Lim authored
Remove WILC_memcmp function because it is changed to memcmp. Signed-off-by:
Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by:
-
Chaehyun Lim authored
Use memcmp instead of WILC_memcmp. Signed-off-by:
-
Chaehyun Lim authored
Remove WILC_memset function because it is changed to memset. Signed-off-by:
-
Chaehyun Lim authored
Use memset instead of WILC_memset. Signed-off-by:
-
Andrzej Hajda authored
The patch was generated using fixed coccinelle semantic patch scripts/coccinelle/api/memdup.cocci [1]. [1]: http://permalink.gmane.org/gmane.linux.kernel/2014320 Signed-off-by:
Andrzej Hajda <a.hajda@samsung.com> Signed-off-by:
-
Joshua Clayton authored
Change the value to a name that conforms to Linux coding style. "rates" is equally expressive in this context, and I have left alone a comment and function name that describe the rates as supported rates. Signed-off-by:
Joshua Clayton <stillcompiling@gmail.com> Signed-off-by:
-
Joshua Clayton authored
Coding style fix. Get rid of typedefs NDIS_802_11_RATES and NDIS_802_11_RATES_EX Undo any casting that was done as a result of the typedef. Signed-off-by:
-
Joshua Clayton authored
Rename r8712_get_ndis_wlan_bssid_ex_sz() to r8712_get_wlan_bssid_ex_sz(), which corresponds to the struct whose size it measures. Signed-off-by:
-
Joshua Clayton authored
struct ndis_wlan_bssid_ex is a doppelganger of struct wlan_bssid_ex, and is used about a third as often. Switch all instances to wlan_bssid_ex, and remove ndis_wlan_bssid_ex This also gets rid of a use of typedef NDIS_802_11_RATES_EX Signed-off-by:
-
Joshua Clayton authored
Remove wrapper structs that just wrap struct ndis_wlan_bssid_ex, and are unused. Signed-off-by:
-
Joshua Clayton authored
The old comment refers to a typedef name which is being removed, and to a style of calculation which is no longer being used. It falsely states that IELength is variable length, instead of IEs. Change comment to simply state that the IEs field is a buffer of variable size and that IELength refers to the current size. Signed-off-by:
-
Joshua Clayton authored
replace item-by-item size calculation of a struct with the size of the struct. This gets rid of a use of typedef NDIS_802_11_RATES_EX Signed-off-by:
-
Sudip Mukherjee authored
Lets return the return value directly instead of using a variable to store the result. Signed-off-by:
-
Sudip Mukherjee authored
Rearrange the code to remove one exit label which also makes the code less indented and more readable. Signed-off-by:
-
Sudip Mukherjee authored
Multiline comments are edited to be in the kernel coding style. Signed-off-by:
-
Sudip Mukherjee authored
Kernel coding style says to have a space around the operators. Signed-off-by:
-
Sudip Mukherjee authored
Multiple blank lines are not kernel coding style and so checkpatch complains. Remove them. Signed-off-by:
-
Joshua Clayton authored
r8712_get_ndis_wlan_bssid_ex_sz has a "6 * sizeof(unsigned long)" where the underlying struct has a 6 * unsigned char. Simplify the calculation by just subtracting the variable part from the size of the struct. This also gets rid of a use of typedef NDIS_802_11_RATES_EX Signed-off-by:
-
Miguel Bernabeu Diaz authored
Removed three instances of parentheses in return calls that are unnecessary and do not contribute to readability. Signed-off-by:
Miguel Bernabeu Diaz <miguelbernadi@gmail.com> Signed-off-by:
-
Miguel Bernabeu Diaz authored
Fix checkpatch.pl error: ERROR: space prohibited before open square bracket '[' Signed-off-by:
-
Miguel Bernabeu Diaz authored
Fix several instances of checkpatch.pl warning: WARNING: space prohibited between function name and open parenthesis '(' Signed-off-by: -
Miguel Bernabeu Diaz authored
Fix checkpatch.pl error: ERROR: space prohibited before that '++' (ctx:WxO) Signed-off-by:
-
Miguel Bernabeu Diaz authored
Fix checkpatch.pl error: ERROR: code indent should use tabs where possible Signed-off-by:
-
Miguel Bernabeu Diaz authored
Fix checkpatch.pl warning: WARNING: Avoid unnecessary line continuations Signed-off-by:
-
Stuart Yoder authored
add README file providing an overview of the DPAA2 architecture and how it is integrated in Linux Signed-off-by:
Stuart Yoder <stuart.yoder@freescale.com> Signed-off-by:
-
Navy Cheng authored
The wrong warning message in vme_user_probe() will mislead developers and users. As the error message which prompt cdev_add() error already exists, just remove the unnecessary and wrong message. Signed-off-by:
Navy Cheng <navych@126.com> Signed-off-by:
-
Okash Khawaja authored
The Linux kernel coding style suggests starting every line in a block comment with an asterisk and finishing the block comment with */ on a separate line. This patch fixes those warnings, clearing all warnings and errors in this file, as reported by the checkpatch script. Signed-off-by:
Okash Khawaja <okash.khawaja@gmail.com> Signed-off-by:
-
Fabio Falzoi authored
Use a struct flexfb_lcd_controller to holds chip properties, instead of relying on a long 'if - else if' chain. This allows to: - use a simple linear search to verify if a certain LCD controller model is supported or not. - add support for a new LCD chip controller simply defining a new flexfb_lcd_controller struct. Signed-off-by:
Fabio Falzoi <fabio.falzoi84@gmail.com> Acked-by:
Noralf Trønnes <noralf@tronnes.org> Signed-off-by:
-
Johannes Postma authored
This patch improves code readability in the function rtl8723a_cal_txdesc_chksum. It improves the readability of the argument of the function le16_to_cpu. Signed-off-by:
Johannes Postma <jgmpostma@gmail.com> Acked-by:
Jes Sorensen <Jes.Sorensen@redhat.com> Signed-off-by:
-
Johannes Postma authored
usPtr is used as __le16 *, but was defined as u16 *. This was reported by sparse as: drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c:1850:29: warning: cast to restricted __le16 This patch fixes the type of usPtr. Signed-off-by:
-
