aboutsummaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2014-10-02staging: emxx_udc: Use min_t instead of minEbru Akagunduz1-3/+3
Use min_t instead of min function in emxx_udc.c Fix checkpatch.pl warnings: WARNING: min() should probably be min_t(u32, iBufSize, ep->ep.maxpacket) WARNING: min() should probably be min_t(u32, data_size, ep->ep.maxpacket) WARNING: min() should probably be min_t(u16, udc->ctrl.wLength, sizeof(status_data)) Changes in v2: - Fixed min function call as min_t Signed-off-by: Ebru Akagunduz <[email protected]> Reviewed-by: Josh Triplett <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: emxx_udc: Fix replace printk(KERN_DEBUG ..) with dev_dbgEbru Akagunduz1-5/+5
This patch fixes "Prefer [subsystem eg: netdev]_dbg([subsystem]dev, ... then dev_dbg(dev, ... then pr_debug(... to printk(KERN_DEBUG" checkpatch.pl warning in emxx_udc.c Changes in v2: - Fixed dev_debug function call as dev_dbg Signed-off-by: Ebru Akagunduz <[email protected]> Reviewed-by: Josh Triplett <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: media: Fixed else after return or break warningYeliz Taneroglu1-2/+2
The following patch fixes the checkpatch.pl warning: drivers/staging/media/omap4iss/iss_csi2.c:811 warning: else is not generally useful after a break or return Signed-off-by: Yeliz Taneroglu <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: media: omap4iss: Fixed else after return or break warningYeliz Taneroglu1-2/+2
The following patch fixes the checkpatch.pl warning: drivers/staging/media/omap4iss/iss_ipipe.c:184 warning: else is not generally useful after a break or return Signed-off-by: Yeliz Taneroglu <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: rtl8712: Fixed else not required after returnSarah Khan1-4/+2
WARNING: Else generally not required after return checkpatch.pl warning in hal_init.c Fixed by removing else Signed-off-by: Sarah Khan <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: rtl8712: Fix missing blank line warningMelike Yurtoglu1-0/+1
Fixes "Missing a blank line after declarations" checkpatch.pl warning in usb_ops_linux.c Signed-off-by: Melike Yurtoglu <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: rtl8192e: rtl8192e: Remove spaces before the semicolonsFeyza Yavuz1-2/+2
Patch the following checkpatch.pl warnings drivers/staging/rtl8192e/rtl8192e/r8190P_rtl8256.c:232: warning: space prohibited before semicolon drivers/staging/rtl8192e/rtl8192e/r8190P_rtl8256.c:236: warning: space prohibited before semicolon Signed-off-by: Feyza Yavuz <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: rtl8192e: rtl8192e: Remove unnecessary return statementsFeyza Yavuz1-2/+0
Patch the following checkpatch.pl warnings drivers/staging/rtl8192e/rtl8192e/r8190P_rtl8256.c:304: warning: void function return statements are not generally useful drivers/staging/rtl8192e/rtl8192e/r8190P_rtl8256.c:83: warning: void function return statements are not generally useful Signed-off-by: Feyza Yavuz <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: rtl8192e: Remove unneeded void returnMelike Yurtoglu1-3/+0
Fixes "void function return statements are not generally useful"checkpatch.pl warning Signed-off-by: Melike Yurtoglu <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: rtl8192e: Fix void function return statements styleMelike Yurtoglu1-6/+0
Fixes "void function return statements are not generally useful"checkpatch.pl warning in rtl819x_HTProc.c Signed-off-by: Melike Yurtoglu <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: rtl8712: Fix unnecessary parentheses style warningYeliz Taneroglu1-1/+1
This fixes the following checkpatch.pl warning: drivers/staging/rtl8712/rtl871x_recv.c:634 warning: Unnecessary parentheses - maybe == should be = ? Signed-off-by: Yeliz Taneroglu <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: rtl8192e: Fix unnecessary space before function pointer argumentsMelike Yurtoglu1-1/+1
This patch fixes these warning messages found by checkpatch.pl: WARNING: Unnecessary space before function pointer arguments Signed-off-by: Melike Yurtoglu <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: rtl8192e: Array was made static const char * constMelike Yurtoglu1-1/+1
This patch fixes checkpatch.pl error in file rtllib.h WARNING: static const char * array should probably be static const char * const Signed-off-by: Melike Yurtoglu <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: ft1000: ft1000-usb: Removed unnecessary else statement.Gulsah Kose1-6/+4
This patch fixes "else is not generally useful after a break or return" checkpatch.pl warning in ft1000_download.c Signed-off-by: Gulsah Kose <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: ft1000: ft1000-usb: Removed unnecessary else statement.Gulsah Kose1-43/+41
This patch fixes "else is not generally useful after a break or return" checkpatch.pl warning in ft1000_hw.c Signed-off-by: Gulsah Kose <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: ft1000: ft1000-usb: Removed unnecessary parentheses.Gulsah Kose1-1/+1
This patch fixes "Unnecessary parentheses" checkpatch.pl warning in ft1000_hw.c Signed-off-by: Gulsah Kose <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: ft1000: ft1000-usb: Added new line after declarations.Gulsah Kose1-0/+1
This patch fixes "Missing a blank line after declarations" checkpatch.pl warning in ft1000_hw.c Signed-off-by: Gulsah Kose <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: vt6655: Fixed C99 // comment errors in wpactl.cEsra Altintas1-15/+15
The following patch fixes the checkpatch.pl warning: ERROR: do not use C99 // comments Signed-off-by: Esra Altintas <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: speakup: Fixed warning <linux/serial.h> instead of <asm/serial.h>Melike Yurtoglu1-1/+1
This patch fixes warning: 'use <linux/serial.h> instead <asm/serial.h>' found by checkpatch Signed-off-by: Melike Yurtoglu <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: speakup: Fix missing blank line warningMelike Yurtoglu1-0/+1
Fixes "Missing a blank line after declarations" checkpatch.pl warning in varhandlers.c Signed-off-by: Melike Yurtoglu <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: rts5208: remove unnecessary elseRoxana Blaj1-3/+2
This fixes the checkpatch.pl warning: WARNING: else is not generally useful after a break or return Signed-off-by: Roxana Blaj <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: rts5208: remove unnecessary elseRoxana Blaj1-2/+2
This fixes the checkpatch.pl warning: WARNING: else is not generally useful after a break or return Signed-off-by: Roxana Blaj <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: rts5208: Remove unneeded void returnMelike Yurtoglu1-4/+0
Fixes "void function return statements are not generally useful"checkpatch.pl warning Signed-off-by: Melike Yurtoglu <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: rts5208: Fix missing blank line warning.Melike Yurtoglu1-0/+3
Fixes "Missing a blank line after declarations" checkpatch.pl warning in rtsx_scsi.c Signed-off-by: Melike Yurtoglu <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02Staging: iio: adc: Update Kconfig entries adding module name informationVaishali Thakkar1-0/+8
This patch fixes the following checkpatch.pl warning: WARNING: please write a paragraph that describes the config symbol fully Signed-off-by: Vaishali Thakkar <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02Staging: iio: adc: Add missing blank lineVaishali Thakkar1-0/+1
This fixes following checkpatch.pl warningi: WARNING: Missing a blank line after declarations Signed-off-by: Vaishali Thakkar <[email protected]> Acked-by: Daniel Baluta <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: iio: resolver: Annotate Kconfig entries with module name informationRoberta Dobrescu1-0/+9
This fixes the following checkpatch.pl warning: WARNING: please write a paragraph that describes the config symbol fully Signed-off-by: Roberta Dobrescu <[email protected]> Acked-by: Daniel Baluta <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: iio: addac: Combined seperated strings.Gulsah Kose1-2/+1
This patch fixes "quoted string split across lines" checkpatch.pl warning in adt7316.c Signed-off-by: Gulsah Kose <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: iio: accel: Removed unnecessary else expression.Gulsah Kose1-8/+7
This patch fixes "else is not generally useful after a break or return" checkpatch.pl warning in adis16220_core.c Signed-off-by: Gulsah Kose <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02Staging: iio: meter: Remove space before start of lineDarshana Padmadas1-1/+1
This patch removes spaces before start of the line reported by checkpatch.pl Signed-off-by: Darshana Padmadas <[email protected]> Acked-by: Daniel Baluta <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02Staging: iio: meter: Delete white spaces to make line shorterDarshana Padmadas1-23/+23
Found using checkpatch warning. Signed-off-by: Darshana Padmadas <[email protected]> Acked-by: Daniel Baluta <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02Staging: iio: meter: Add blank line after declarationsDarshana Padmadas1-0/+2
This patch adds a blank line after declarations found by checkpatch warning. Signed-off-by: Darshana Padmadas <[email protected]> Acked-by: Daniel Baluta <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02Staging: iio: meter: Add blank line after declarations.Darshana Padmadas1-0/+4
This patch adds a blank line after declarations to fix checkpatch warning. Signed-off-by: Darshana Padmadas <[email protected]> Acked-by: Daniel Baluta <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02Staging: iio: meter: Add a blank line after declarationsDarshana Padmadas1-0/+3
Patch fixes warning and adds a blank line after declarations Signed-off-by: Darshana Padmadas <[email protected]> Acked-by: Daniel Baluta <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: iio: addac: remove unnecessary elseRoxana Blaj1-28/+27
This fixes the checkpatch.pl warning: WARNING: else is not generally useful after a break or return Signed-off-by: Roxana Blaj <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: octeon: Combined seperate strings.Gulsah Kose1-2/+1
This patch fixes "quoted string split across lines" checkpatch.pl warning in ethernet-rx.c Signed-off-by: Gulsah Kose <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: octeon: Combined seperate strings.Gulsah Kose1-6/+3
This patch fixes "quoted string split across lines" checkpatch.pl warning in ethernet-rgmii.c Signed-off-by: Gulsah Kose <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02Staging: lustre: lnet: selftest: Made a function static.Heena Sirwani1-1/+1
The following patch makes a function static as it was not used by other files. The patch fixes the warning: WARNING: symbol 'brw_fill_bulk' was not declared. Should it be static? The above warning was reported by sparse. Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02Staging: lustre: lnet: selftest: Made a function static.Heena Sirwani1-1/+1
The following patch makes a function static as it was not used by other files. The patch fixes the warning: WARNING: symbol 'brw_check_page' was not declared. Should it be static? The above warning was reported by sparse. Signed-off-by: Heena Sirwani <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02Staging: lustre: lnet: selftest: Made a function static.Heena Sirwani1-1/+1
The following patch makes a function static as it was not used by other files. The patch fixes the warning: WARNING: symbol 'brw_fill_page' was not declared. Should it be static? The above warning was reported by sparse. Signed-off-by: Heena Sirwani <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02Staging: lustre: lnet: selftest: Made a function static.Heena Sirwani1-1/+1
The following patch makes a function static as it is not used by other files. The patch fixes the following warning: WARNING: symbol brw_inject_one_error was not declared. Should it be static? The above warning was reported by sparse. Signed-off-by: Heena Sirwani <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02Staging: lustre: lnet: selftest: Made a function static.Heena Sirwani1-1/+1
The following patch makes a function static as it was not used by other files. The patch fixes the following warning: WARNING: symbol brw_client_init was not declared. Should it be static? The above warning was reported by sparse. Signed-off-by: Heena Sirwani <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02Staging: lustre: lnet: selftest: Made a function static.Heena Sirwani1-1/+1
The following patch makes a function static as it was not used by other files and not declared in the same file. It fixes the following warning: WARNING: symbol 'lstcon_init_acceptor_service' was not declared. Should it be static? The above warning was given by sparse. Signed-off-by: Heena Sirwani <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: lustre: include: linux: libcfs: linux: Removed line continuations.Gulsah Kose1-2/+1
This patch fixes "Avoid unnecessary line continuations" checkpatch.pl warning in linux-time.h Signed-off-by: Gulsah Kose <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: lustre: include: linux: libcfs: Removed unnecessary else expression.Gulsah Kose1-2/+1
This patch fixes "else is not generally useful after a break or return" checkpatch.pl warning libcfs.h Signed-off-by: Gulsah Kose <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: lustre: include: linux: libcfs: linux: Used "linux" instead of "asm".Gulsah Kose1-3/+3
This patch fixes "Use #include <linux/uaccess.h> instead of <asm/uaccess.h"> checkpatch.pl warning in libcfs.h Signed-off-by: Gulsah Kose <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: lustre: lustre: obdclass: linux: include <linux/*.h> instead of ↵Tapasweni Pathak1-3/+3
<asm/*.h> This patch fixes checpatch.pl warning in linux-module.c file. WARNING: Use #include <linux/io.h> instead of <asm/io.h> WARNING: Use #include <linux/poll.h> instead of <asm/poll.h> WARNING: Use #include <linux/uaccess.h> instead of <asm/uaccess.h> Signed-off-by: Tapasweni Pathak <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: lustre: lnet: lnet: Remove space between function name and open ↵Tapasweni Pathak1-117/+117
paranthesis '(' This patch fixes checpatch.pl warning in api-ni.c file. WARNING: space prohibited between function name and open parenthesis '(' Signed-off-by: Tapasweni Pathak <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: lustre: lnet: lnet: Remove unnecessary else after returnTapasweni Pathak1-4/+4
This patch fixes checpatch.pl warning in acceptor.c file. WARNING: else is not generally useful after a break or return Signed-off-by: Tapasweni Pathak <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-10-02staging: lustre: include: linux: libcfs: Remove spaces at the start of a lineTapasweni Pathak1-2/+2
This patch fixes checpatch.pl warning in file libcfs_cpu.h WARNING: please, no spaces at the start of a line Signed-off-by: Tapasweni Pathak <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
Powered by cgit, Gruvboxed by Blaster4385.