- 28 Mar, 2016 40 commits
-
-
H Hartley Sweeten authored
Reformat the multi-line comment in the kernel CodingStyle. Signed-off-by:
H Hartley Sweeten <hsweeten@visionengravers.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
H Hartley Sweeten authored
Convert the inline functions used to set the bits in the NI600X_IO_CFG registers into macros. Also convert the enum ni_660x_pfi_output_select into defines. This clarifies the association with the register. This also fixes a number of checkpatch.pl issues about: WARNING: Prefer 'unsigned int' to bare use of 'unsigned' Signed-off-by:
-
H Hartley Sweeten authored
Rename this CamelCase struct and the associated 'registerData' variable. Signed-off-by:
-
H Hartley Sweeten authored
This enum is used to define the, unused, 'direction' of each register in struct NI_660xRegisterData. Remove the unused member, as well as the enum. Signed-off-by:
-
H Hartley Sweeten authored
All the registers are defined struct NI_660xRegisterData and they are either 2 or 4 bytes in size. Remove the enum and just use a char member to define the size as 2 or 4 bytes. Simplify the ni_660x_{write,read}_register() functions and remove the unnecessary BUG() in each. Signed-off-by: -
H Hartley Sweeten authored
This member of the struct is not used, and just takes up space. Remove it. Instead, add the enum ni_660x_register indexes to the table to clarify, and document, the entries. Signed-off-by:
-
H Hartley Sweeten authored
The BUG_ON() in this function is unnecessary. The 'pfi_channel' will always be in range of the subdevice 'n_chan' (NUM_PFI_CHANNELS) which will return a valid 'reg'. Convert the inline function into a simple macro. Signed-off-by:
-
Leo Kim authored
This patch removes unused local variable. This variable is operation definition that back off from sending packets for some time. However, that has been deleted operation code. That is removes all relative code. Signed-off-by:
Leo Kim <leo.kim@atmel.com> Signed-off-by:
-
Leo Kim authored
This patch removes define USE_TX_BACKOFF_DELAY_IF_NO_BUFFERS and use it's feature codes. Signed-off-by:
-
Leo Kim authored
This patch removes unused define. Signed-off-by:
-
Leo Kim authored
This patch replaces memcmp with ether_addr_equal_unaligned. Warning reported by checkpatch.pl - Prefer ether_addr_equal() or ether_addr_equal_unaligned() over memcmp() Signed-off-by:
-
Leo Kim authored
This patch removes unused debug flags. Signed-off-by:
-
Leo Kim authored
This patch removes unused dead codes that define custom feature. Signed-off-by:
-
Leo Kim authored
This patch replaces frame_type with type of struct frame_reg. Signed-off-by:
-
Leo Kim authored
This patch replaces g_struct_frame_reg with frame_reg. Signed-off-by:
-
Leo Kim authored
This patch removes typedef of struct struct_frame_reg. Renames the struct_frame_reg to frame_reg as well. Signed-off-by:
-
Leo Kim authored
This patch removes unused hif_drv of wilc_frame_register function. It's perform an unnecessary null check and debug print log. Signed-off-by:
-
Leo Kim authored
This patch removes WIRELESS_EXT. Does not used the WIRELESS_EXT define. Signed-off-by:
-
Chaehyun Lim authored
struct semaphore hif_sema_driver is used to signal completion of host interface message. This patch replaces struct semaphore hif_sema_driver with struct completion hif_driver_comp. It is better to use completion than semaphore for this case. Signed-off-by:
Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by:
-
Chaehyun Lim authored
struct semaphore hif_sema_thread is used to signal completion of host interface thread. This patch replaces struct semaphore hif_sema_thread with struct completion hif_thread_comp. It is better to use completion than semaphore for this case. Signed-off-by:
-
Alexander Curtin authored
The argument for ULTRA_CHANNELCLI_STRING is supposed to be an integer representing the channel state. 'state' is a more descriptive variable name for this. Signed-off-by:
Alexander Curtin <alexander.curtin@unisys.com> Signed-off-by:
David Kershner <david.kershner@unisys.com> Signed-off-by:
-
Alexander Curtin authored
In client_bus_info_show, the variable 'x' is used to create keep track of the offset that the current 'pos' in the output buffer needs to be incremented by. Since 'off' is already taken 'shift' was used since it's used to shift the pointer. Signed-off-by:
-
Alexander Curtin authored
In the case of client_bus_info_show, the variable 'p' was used to indicate the position in the output buffer. This was changed to 'pos' to indicate that it kept track of the current position in the output buffer. Signed-off-by:
-
Alexander Curtin authored
The variable name "s" doesn't indicate the purpose of the string, which is to store the id collected from the visorchannel_zoneid function. This just replaces the name with "zoneid". Signed-off-by:
-
Alexander Curtin authored
The variable name "s" doesn't indicate the purpose of the string, which is to store the id collected from the visorchannel_id function. This just replaces the name with "typeid". Signed-off-by:
-
Erik Arfvidson authored
This patch simply removes the double negative comparison for test_bit since test_bit already preforms this check. Signed-off-by:
Erik Arfvidson <erik.arfvidson@unisys.com> Signed-off-by:
-
Erik Arfvidson authored
This patch removes the double negative comparisons for function readb. Signed-off-by:
-
James Simmons authored
Change return value to size_t. Signed-off-by:
James Simmons <uja.ornl@yahoo.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: http://review.whamcloud.com/17643Reviewed-by:
Bob Glossman <bob.glossman@intel.com> Reviewed-by:
John L. Hammond <john.hammond@intel.com> Reviewed-by:
Dmitry Eremin <dmitry.eremin@intel.com> Reviewed-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
-
James Simmons authored
Use the proper BIT macro for libcfs_ioctl_is_invalid(). Signed-off-by:
-
James Simmons authored
Return real bool values for libcfs_ioctl_is_invalid(). Signed-off-by:
-
James Simmons authored
Need a few uapi headers to make libcfs_ioctl.h compilable in userland. Signed-off-by:
-
James Simmons authored
Fix up all thw whitescapes and line up the IOCTL defines so it is readable. Signed-off-by:
-
James Simmons authored
Move the function declartions that are used only by kernel space to libcfs.h This makes libcfs_ioctl.h a offical uapi header now. Move large inline functions out of libcfs_ioctl.h to the source file linux-module.c belonging to libcfs. This code is only used by the core of libcfs and such inline functions don't belong in a uapi header file. Signed-off-by:
-
James Simmons authored
Move large inline functions out of libcfs_ioctl.h to the source file linux-module.c belonging to libcfs. This code is only used by the core of libcfs and such inline functions don't belong in a uapi header file. Signed-off-by:
-
James Simmons authored
All the macros in libcfs_ioctl.h that is needed by user land have been moved into the lustre utilities software stack. Signed-off-by:
-
Parinay Kondekar authored
With the libcfs ioctl cleanup we no longer need the libcfs pseudo device abstraction. Signed-off-by:
Parinay Kondekar <parinay.kondekar@seagate.com> Signed-off-by:
Andreas Dilger <andreas.dilger@intel.com> Reviewed-by:
-
Parinay Kondekar authored
No reason to go through the cfs_psdev_ops abstract to call libcfs_ioctl. Just call libcfs_ioctl directly. Signed-off-by:
-
Parinay Kondekar authored
With struct libcfs_device_userstate gone we can remove the remaining code of libcfs_psdev_ops.p_[open|close] as well as the libcfs_psdev_[open|release] functions. Signed-off-by:
-
Parinay Kondekar authored
The IOC_LIBCFS_MEMHOG is not needed so remove the last bits. Signed-off-by:
-
Parinay Kondekar authored
A few pieces still exist for the IOC_LIBCFS_PANIC ioctl. Remove these last bits to prevent old tools from using them. The latest lustre utilities no longer use this ioctl. Signed-off-by:
-
