Patch fixes warning and adds a blank line after declarations
Signed-off-by: Darshana Padmadas <darshanapadmadas@gmail.com>
Acked-by: Daniel Baluta <daniel.baluta@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This fixes the checkpatch.pl warning:
WARNING: else is not generally useful after a break or return
Signed-off-by: Roxana Blaj <roxanagabriela10@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This patch fixes "quoted string split across lines" checkpatch.pl
warning in ethernet-rx.c
Signed-off-by: Gulsah Kose <gulsah.1004@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This patch fixes "quoted string split across lines" checkpatch.pl
warning in ethernet-rgmii.c
Signed-off-by: Gulsah Kose <gulsah.1004@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
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 <gregkh@linuxfoundation.org>
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 <heenasirwani@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
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 <heenasirwani@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
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 <heenasirwani@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
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 <heenasirwani@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
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 <heenasirwani@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This patch fixes "Avoid unnecessary line continuations" checkpatch.pl
warning in linux-time.h
Signed-off-by: Gulsah Kose <gulsah.1004@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This patch fixes "else is not generally useful after a break or return"
checkpatch.pl warning libcfs.h
Signed-off-by: Gulsah Kose <gulsah.1004@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
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 <tapaswenipathak@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
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 <tapaswenipathak@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
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 <tapaswenipathak@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
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 <tapaswenipathak@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This patch fixes checpatch.pl error in file libcfs_time.h
ERROR: do not initialise statics to 0 or NULL
Signed-off-by: Tapasweni Pathak <tapaswenipathak@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Just switch EVENT_PC over to enum event_pc.
Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Take out those doubled blank lines in guestlinuxdebug.h.
Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
The copyright character in vbushelper.h isn't UTF-8 - swap it out for a
good, old-fashioned (C).
Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Take out the doubled blank lines in uniklog.h, too.
Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Take out those doubled blank lines in uisthread.h.
Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Clean up CamelCase names:
pSwitch => s
Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This function is not used anywhere, so remove it.
Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
