- 21 Mar, 2017 33 commits
-
-
David Binder authored
In an effort to create a more uniform coding style within the Unisys s-Par driver set, this patch adjusts the formatting of all #define directives within this source file to match the following template, and thereby eliminate irregular usage of whitespace: Reviewed-by:
Tim Sell <timothy.sell@unisys.com> The amount of whitespace used between the <token> and the <value> is dependent on what is needed to make the surrounding #define directives as uniform as possible. Signed-off-by:
David Binder <david.binder@unisys.com> Signed-off-by:
David Kershner <david.kershner@unisys.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
David Binder authored
In an effort to create a more uniform coding style within the Unisys s-Par driver set, this patch adjusts the formatting of all #define directives within this source file to match the following template, and thereby eliminate irregular usage of whitespace: Reviewed-by:
-
David Binder authored
In an effort to create a more uniform coding style within the Unisys s-Par driver set, this patch adjusts the formatting of all #define directives within this source file to match the following template, and thereby eliminate irregular usage of whitespace: Reviewed-by:
-
David Binder authored
In an effort to create a more uniform coding style within the Unisys s-Par driver set, this patch adjusts the formatting of all #define directives within this source file to match the following template, and thereby eliminate irregular usage of whitespace: Reviewed-by:
-
David Binder authored
In an effort to create a more uniform coding style within the Unisys s-Par driver set, this patch adjusts the formatting of all #define directives within this source file to match the following template, and thereby eliminate irregular usage of whitespace: Reviewed-by:
-
David Binder authored
In an effort to create a more uniform coding style within the Unisys s-Par driver set, this patch adjusts the formatting of all #define directives within this source file to match the following template, and thereby eliminate irregular usage of whitespace: Reviewed-by:
-
David Binder authored
In an effort to create a more uniform coding style within the Unisys s-Par driver set, this patch adjusts the formatting of all #define directives within this source file to match the following template, and thereby eliminate irregular usage of whitespace: Reviewed-by:
-
David Binder authored
In an effort to create a more uniform coding style within the Unisys s-Par driver set, this patch adjusts the formatting of all #define directives within this source file to match the following template, and thereby eliminate irregular usage of whitespace: Reviewed-by:
-
Jon Frisch authored
This patch removes the #pragma directive from visorchannel.h and adds the __packed keyword to all structs to suppress structure padding. Signed-off-by:
Jon Frisch <jon.frisch@unisys.com> Signed-off-by:
-
Jon Frisch authored
This patch removes inline functions in visornic_main.c. Signed-off-by:
-
Jon Frisch authored
This patch removes comments that are no longer relevant in channel.h. Signed-off-by:
-
Jon Frisch authored
This patch removes the unused function declarations in channel.h. Signed-off-by:
-
Jon Frisch authored
This patch removes the unused macro lengthof. Signed-off-by:
-
David Binder authored
Removes unnecessary cast of to __iomem of a non-io-mapped pointer, thereby eliminating the following sparse warning: visorchannel.c:159:17: warning: cast adds address space to expression (<asn:2>) Signed-off-by:
-
David Kershner authored
The function spar_check_channel_client shouldn't need to do readq's, it is referencing a local copy of the channel header. Simplify it to just access the fields directly. Signed-off-by:
-
David Binder authored
Adds identifier to function definition arguments to satisfy checkpatch warnings: WARNING: function definition argument 'struct visor_driver *' should also have an identifier name WARNING: function definition argument 'struct visor_driver *' should also have an identifier name Signed-off-by:
-
David Binder authored
Remove local variable on stack by directly returning the value in the array. Signed-off-by:
-
David Binder authored
Renames __IOMONINTF_H__ to __VMCALLINTERFACE_H__, which more closely reflects the naming conventions of the code. Signed-off-by:
-
David Binder authored
Move function add_physinfo_entries() to visornic_main.c, which is the only function where it is used. Signed-off-by:
-
David Binder authored
Replace references to virtpci to visornic in iochannel.h. Signed-off-by:
-
David Binder authored
Rename #define VMCALL_IO_CONTROLVM_ADDR to VMCALL_CONTROLVM_ADDR, as this vmcall can be used by any partition, not just the IO partition. Signed-off-by:
-
David Binder authored
Removes checks that all visor_device instances have an associated visorchannel. Due to the design of the s-Par drivers these checks are unnecessary. Signed-off-by:
-
David Binder authored
Removes unused #defines pertaining to visorchannel UUIDs. Signed-off-by:
-
David Binder authored
Addresses checkpatch check by wrapping macro argument in parenthesis. Signed-off-by:
-
Sameer Wadgaonkar authored
Removed inline keyword from functions sig_read_data and sig_write_data in visorchannel.c Signed-off-by:
Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com> Signed-off-by:
-
Sameer Wadgaonkar authored
Removed inline keyword from the functions in visorchipset: * issue_vmcall_io_controlvm_addr() Signed-off-by:
-
Sameer Wadgaonkar authored
Removed inline keyword from the function complete_taskmgmt_command in visorhba_main.c Signed-off-by:
-
Sameer Wadgaonkar authored
Removed unused functions issue_vmcall_update_physical_time() and issue_vmcall_query_guest_virtual_time_offset() from visorchipset.c Signed-off-by:
-
Sameer Wadgaonkar authored
Removed unused macro spar_signal_init from channel.h Signed-off-by:
-
Sameer Wadgaonkar authored
Removed unused function spar_check_channel_server from channel.h Signed-off-by:
-
Sameer Wadgaonkar authored
Removed unused structure pci_id from controlvmchannel.h Signed-off-by:
-
Alexander Curtin authored
The comment referencing virtpci was irrelevant. Signed-off-by:
Alexander Curtin <alexander.curtin@unisys.com> Signed-off-by:
-
Alexander Curtin authored
COVERQ is functionally equivalent to DIV_ROUND_UP and was only used to define the COVER macro. Signed-off-by:
-
- 17 Mar, 2017 7 commits
-
-
Dylan Leggio authored
GAS_INTIAL_REQ should be GAS_INITIAL_REQ. GAS_INTIAL_RSP should be GAS_INITIAL_RSP. Improves readability of code. Signed-off-by:
Dylan Leggio <dleggio1@binghamton.edu> Signed-off-by:
-
Colin Ian King authored
The pmkid data is meant be be copied to the previous item in the pmkidlist, however the code is just copying the data to itself because the src index into pmkidlist is the same as the dst index into pmkidlist. Fix this with i + 1 instead of i. Detected by CoverityScan,CID#13339465 ("Overlapping buffer in memory copy") Signed-off-by:Colin Ian King <colin.king@canonical.com> Signed-off-by:
-
Jérémy Lefaure authored
To use IO functions like writel, readl, ioremap_nocache and iounmap, linux/io.h should be included. Signed-off-by:
Jérémy Lefaure <jeremy.lefaure@lse.epita.fr> Signed-off-by:
-
sayli karnik authored
The patch removes unused function definition ll_get_user_pages(). The use of ll_get_user_pages() was replaced with iov_iter_get_pages_alloc() in commit 91f79c43. Signed-off-by:
sayli karnik <karniksayli1995@gmail.com> Signed-off-by:
-
Okash Khawaja authored
This moves spk_synth_immediate and spk_serial_synth_probe functions into serialio.c. These functions do outgoing serial comms. The move is a step towards collecting all serial comms in serialio.c. This also renames spk_synth_immediate to spk_serial_synth_immediate. Code inside those functions has not been changed. Along the way, this patch also fixes a couple of spots which were calling spk_synth_immediate directly, so that the calls now happen via the spk_syth struct. Signed-off-by:
Okash Khawaja <okash.khawaja@gmail.com> Reviewed-by:
Samuel Thibault <samuel.thibault@ens-lyon.org> Signed-off-by:
-
Mohsin Shan authored
Embedded function names are less appropriate to use when refactoring, can cause function renaming. Prefer the use of "%s", __func__ to embedded function names Signed-off-by:
Mohsin Shan <mohsinshan1k@gmail.com> Signed-off-by:
-
Jérémy Lefaure authored
This reverts commit fde46970. Adding a preprocessor condition is not the best solution to fix this issue. Let's revert this commit before fixing this problem in a more appropriate way. Signed-off-by:
-
