- Dec 20, 2018
-
-
Jona Crasselt authored
Issue reported by checkpatch.pl. Signed-off-by:
Jona Crasselt <jona.crasselt@fau.de> Signed-off-by:
Felix Windsheimer <felix.windsheimer@fau.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Jona Crasselt authored
Issue reported by checkpatch.pl. Signed-off-by:
Jona Crasselt <jona.crasselt@fau.de> Signed-off-by:
Felix Windsheimer <felix.windsheimer@fau.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Jona Crasselt authored
Issue reported by checkpatch.pl. Signed-off-by:
Jona Crasselt <jona.crasselt@fau.de> Signed-off-by:
Felix Windsheimer <felix.windsheimer@fau.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Jona Crasselt authored
Fix some comment-related issues reported by checkpatch.pl: - Block comments use * on subsequent lines - Block comments use a trailing */ on a separate line Signed-off-by:
Jona Crasselt <jona.crasselt@fau.de> Signed-off-by:
Felix Windsheimer <felix.windsheimer@fau.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
- Dec 19, 2018
-
-
Daniel Bovensiepen authored
Fixed spelling in comment section. Signed-off-by:
Daniel Bovensiepen <daniel@bovensiepen.net> Acked-by:
Joel Fernandes (Google) <joel@joelfernandes.org> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
Cleanup brace coding style issues reported by checkpatch. ERROR: space required before the open brace '{' WARNING: braces {} are not necessary for single statement blocks CHECK: Unbalanced braces around else statement Signed-off-by:Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
Add spaces around '&' to follow kernel coding style. Reported by checkpatch. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
The function is_basicrate() returns true or false, so change the return type from int to bool. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
Simplfy initialization of null arrays to improve readability and save some lines. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
Change the order of array declarations in rtw_mlme_ext.c to improve readability. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
Make some arrays that are only used in rtw_mlme_ext.c static and remove the corresponding extern declarations from rtw_mlme_ext.h. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
The values of these arrays in rtw_mlme_ext.c and rtw_wlan_util.c are never changed, so make them const. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
Change the type of some arrays from unsigned char to u8. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
The array REALTEK_96B_IE is already declared in rtw_mlme.h, so remove the declaration in rtw_mlme_ext.c. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
The arrays WFD_OUI and WMM_INFO_OUI are not used anywhere, so remove them. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
Remove unnecessary parentheses, most of them reported by checkpatch. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
Remove comments from 'endif's where the corresponding 'if' is just a few lines above to improve readability. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
Add a missing blank line after declarations in rtw_mlme_ext.c. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
Remove blank lines between declarations in rtw_mlme_ext.c. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
Remove unused/commented declarations. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
Replace tabs with spaces where appropriate. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
Replace tabs with spaces and/or remove spaces in declarations. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
YueHaibing authored
strncmp() stops comparing when either the end of one of the first two arguments is reached or when 'n' characters have been compared, whichever comes first.That means that strncmp(s1, s2, n) is equivalent to strcmp(s1, s2) if n exceeds the length of s1 or the length of s2. This patch avoids that the following warning is reported by smatch: drivers/staging/fbtft/fbtft_device.c:1458 fbtft_device_init() error: strncmp() '"list"' too small (5 vs 32) Signed-off-by:
YueHaibing <yuehaibing@huawei.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Colin Ian King authored
Using snprintf without a format specifier is potentially risky if the string device_name contains format specifiers. Replace this with the safer and more efficient strscpy. Cleans up clang warning: drivers/staging/most/sound/sound.c:673:41: warning: format string is not a string literal (potentially insecure) [-Wformat-security] Signed-off-by:
Colin Ian King <colin.king@canonical.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
- Dec 17, 2018
-
-
Christian Gromm authored
This patch updates driver_usage.txt file to reflect the latest changes that this patch set introduces. Signed-off-by:
Christian Gromm <christian.gromm@microchip.com> Reviewed-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Christian Gromm authored
Adding the channel number to the name of the sound card is wrong, as the card does not represent a single streaming channel of the MOST device. Signed-off-by:
Christian Gromm <christian.gromm@microchip.com> Reviewed-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Christian Gromm authored
This patch uses a static name for the sound card's short name and long name. Having the card names configurable doesn't make sense anymore, as the card represents the same physical hardware. Signed-off-by:
Christian Gromm <christian.gromm@microchip.com> Reviewed-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Christian Gromm authored
Since the channels of a MOST device are now being represented as individual PCM devices of one sound card, the variable card_name is not suitable anymore to describe them. Therefore, this patch renames the variable to device_name. Signed-off-by:
Christian Gromm <christian.gromm@microchip.com> Reviewed-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Christian Gromm authored
This patch fixes the lable name that is used to jump to error handling section of function audio_probe_channel() in case something went wrong. Signed-off-by:
Christian Gromm <christian.gromm@microchip.com> Reviewed-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Christian Gromm authored
This patch avoids that a sound card is created and registered with ALSA every time a channel is being linked. Instead the channels are hooked on the same card, which is registered not until the final link has been added to the component. The string provided by user space that used to be the card name becomes the PCM device name. The user space API to add a link is being expanded by a "create" flag to trigger the registration. Signed-off-by:
Christian Gromm <christian.gromm@microchip.com> Reviewed-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Dominic Braun authored
Typedefing structs is not encouraged in the kernel. Signed-off-by:
Dominic Braun <inf.braun@fau.de> Signed-off-by:
Tobias Büttner <tobias.buettner@fau.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Dominic Braun authored
Typedefing structs is not encouraged in the kernel. Signed-off-by:
Dominic Braun <inf.braun@fau.de> Signed-off-by:
Tobias Büttner <tobias.buettner@fau.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Dominic Braun authored
Typedefing structs is not encouraged in the kernel. Signed-off-by:
Dominic Braun <inf.braun@fau.de> Signed-off-by:
Tobias Büttner <tobias.buettner@fau.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Dominic Braun authored
Typedefing structs is not encouraged in the kernel. Signed-off-by:
Dominic Braun <inf.braun@fau.de> Signed-off-by:
Tobias Büttner <tobias.buettner@fau.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Dominic Braun authored
Typedefing structs is not encouraged in the kernel. Signed-off-by:
Dominic Braun <inf.braun@fau.de> Signed-off-by:
Tobias Büttner <tobias.buettner@fau.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Dominic Braun authored
Typedefing structs is not encouraged in the kernel. Signed-off-by:
Dominic Braun <inf.braun@fau.de> Signed-off-by:
Tobias Büttner <tobias.buettner@fau.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Dominic Braun authored
Typedefing structs is not encouraged in the kernel. Signed-off-by:
Dominic Braun <inf.braun@fau.de> Signed-off-by:
Tobias Büttner <tobias.buettner@fau.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Dominic Braun authored
Typedefing structs is not encouraged in the kernel. Signed-off-by:
Dominic Braun <inf.braun@fau.de> Signed-off-by:
Tobias Büttner <tobias.buettner@fau.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Dominic Braun authored
Typedefing structs is not encouraged in the kernel. Signed-off-by:
Dominic Braun <inf.braun@fau.de> Signed-off-by:
Tobias Büttner <tobias.buettner@fau.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Dominic Braun authored
Typedefing structs is not encouraged in the kernel. Signed-off-by:
Dominic Braun <inf.braun@fau.de> Signed-off-by:
Tobias Büttner <tobias.buettner@fau.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-