Jareer Abdel-Qader
4ef98989cb
staging/rdma/hfi1: Disable thermal polling before sensor initialization
...
During driver load the thermal sensor needs to be reset prior
to initialization of the sensor. This prevents a possible sensor lock
up which can cause the wrong temperature value to be reported.
This fix leads to remove disabling thermal polling from
reset_asic_csrs() function.
Reviewed by: Dennis Dalessandro <dennis.dalessandro@intel.com >
Reviewed by: Easwar Hariharan <easwar.hariharan@intel.com >
Signed-off-by: Jareer Abdel-Qader <jareer.h.abdel-qader@intel.com >
Signed-off-by: Ira Weiny <ira.weiny@intel.com >
Signed-off-by: Jubin John <jubin.john@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Dean Luick
65fcf55764
staging/rdma/hfi1: Always download SBus firmware
...
B0 dual port parts require the SBus firmware to always be
downloaded.
Remove reset of the SBus Master spico. It is not necessary
since the SBus firmware download already does that.
Reviewed-by: Dennis Dalessandro <dennis.dalessandro@intel.com >
Signed-off-by: Dean Luick <dean.luick@intel.com >
Signed-off-by: Jubin John <jubin.john@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Vennila Megavannan
bf70a77577
staging/rdma/hfi1: Enable WFR PCIe extended tags from the driver
...
Some BIOS implementations turn off extended tags in DevCtl (a RW
field) even though it was originally set and is advertised in DevCap
Fix is to set it in the driver
Reviewed-by: Dean Luick <dean.luick@intel.com >
Reviewed-by: Mike Marciniszyn <mike.marciniszyn@intel.com >
Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com >
Signed-off-by: Vennila Megavannan <vennila.megavannan@intel.com >
Signed-off-by: Jubin John <jubin.john@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Easwar Hariharan
72a67ba2fa
staging/rdma/hfi1: Clear the QSFP reset that is asserted on FLR
...
The FLR on driver load asserts the QSFP reset pin and the driver does
not deassert it after. This patch allows the external QSFP cable to exit
reset by writing 1 to all the QSFP pins.
Reviewed-by: Dean Luick <dean.luick@intel.com >
Signed-off-by: Easwar Hariharan <easwar.hariharan@intel.com >
Signed-off-by: Jubin John <jubin.john@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Sebastian Sanchez
6c63e4238a
staging/rdma/hfi1: Convert dd_dev_info() to hfi1_cdbg() in process startup
...
Replacing dd_dev_info() for hfi1_cdbg() to avoid generating syslog
output for every context that is open by PSM.
Reviewed-by: Mitko Haralanov <mitko.haralanov@intel.com >
Signed-off-by: Sebastian Sanchez <sebastian.sanchez@intel.com >
Signed-off-by: Jubin John <jubin.john@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Markus Elfring
e23afff948
staging: most: Delete an unnecessary check before the function call "module_put"
...
The module_put() function tests whether its argument is NULL and then
returns immediately. Thus the test around the call is not needed.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
c904ffdaf3
staging: most: rename DIM_EnqueueBuffer to dim_enqueue_buffer
...
This patch renames DIM_EnqueueBuffer to dim_enqueue_buffer to avoid
camelcase found by checkpatch.
CHECK: Avoid CamelCase: <DIM_EnqueueBuffer>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:877:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
60d5f66ce5
staging: most: rename DIM_GetChannelState to dim_get_channel_state
...
This patch renames DIM_GetChannelState to dim_get_channel_state to avoid
camelcase found by checkpatch.
CHECK: Avoid CamelCase: <DIM_GetChannelState>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:865:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
e380925312
staging: most: fix argument name of DIM_GetChannelState
...
The second argument name of DIM_GetChannelState declaration changes from
dim_ch_state_ptr to state_ptr. The DIM_GetChannelState declaration and
definition has same argument name as state_ptr.
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
0d08d54f8d
staging: most: rename DIM_ServiceChannel to dim_service_channel
...
This patch renames DIM_ServiceChannel to dim_service_channel to avoid
camelcase found by checkpatch.
CHECK: Avoid CamelCase: <DIM_ServiceChannel>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:857:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
e5baa9e99c
staging: most: rename DIM_ServiceIrq to dim_service_irq
...
This patch renames DIM_ServiceIrq to dim_service_irq to avoid camelcase
found by checkpatch.
CHECK: Avoid CamelCase: <DIM_ServiceIrq>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:819:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
a5e4d891a3
staging: most: rename DIM_DestroyChannel to dim_destroy_channel
...
This patch renames DIM_DestroyChannel to dim_destroy_channel to avoid
camelcase found by checkpatch.
CHECK: Avoid CamelCase: <DIM_DestroyChannel>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:806:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
10e5efb7b5
staging: most: rename DIM_InitSync to dim_init_sync
...
This patch renames DIM_InitSync to dim_init_sync to avoid camelcase
found by checkpatch.
CHECK: Avoid CamelCase: <DIM_InitSync>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:781:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
f1383176c9
staging: most: rename DIM_InitIsoc to dim_init_isoc
...
This patch renames DIM_InitIsoc to dim_init_isoc to avoid camelcase
found by checkpatch.
CHECK: Avoid CamelCase: <DIM_InitIsoc>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:756:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
26303150c3
staging: most: rename DIM_InitAsync to dim_init_async
...
This patch renames DIM_InitAsync to dim_init_async to avoid camelcase
found by checkpatch.
CHECK: Avoid CamelCase: <DIM_InitAsync>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:749:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
a3f3e92119
staging: most: rename DIM_InitControl to dim_init_control
...
This patch renames DIM_InitControl to dim_init_control to avoid
camelcase found by checkpatch.
CHECK: Avoid CamelCase: <DIM_InitControl>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:742:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
aff1924508
staging: most: rename DIM_NormSyncBufferSize to dim_norm_sync_buffer_size
...
This patch renames DIM_NormSyncBufferSize to dim_norm_sync_buffer_size
to avoid camelcase found by checkpatch.
CHECK: Avoid CamelCase: <DIM_NormSyncBufferSize>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:734:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
e302ca47b5
staging: most: rename DIM_NormIsocBufferSize to dim_norm_isoc_buffer_size
...
This patch renames DIM_NormIsocBufferSize to dim_norm_isoc_buffer_size
to avoid camelcase found by checkpatch.
CHECK: Avoid CamelCase: <DIM_NormIsocBufferSize>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:720:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
c64c6073e8
staging: most: rename DIM_NormCtrlAsyncBufferSize to dim_norm_ctrl_async_buffer_size
...
This patch renames DIM_NormCtrlAsyncBufferSize to
dim_norm_ctrl_async_buffer_size to avoid camelcase found by checkpatch
CHECK: Avoid CamelCase: <DIM_NormCtrlAsyncBufferSize>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:709:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
58889788fc
staging: most: rename DIMCB_IoRead to dimcb_io_read
...
This patch renames DIMCB_IoRead to dimcb_io_read to avoid camelcase
found by checkpatch.
CHECK: Avoid CamelCase: <DIMCB_IoRead>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:157:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
1efc456462
staging: most: rename DIMCB_IoWrite to dimcb_io_write
...
This patch renames DIMCB_IoWrite to dimcb_io_write to avoid camelcase
found by checkpatch.
CHECK: Avoid CamelCase: <DIMCB_IoWrite>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:154:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
de6687313d
staging: most: rename DIMCB_OnError to dimcb_on_error
...
This patch renames DIMCB_OnError to dimcb_on_error to avoid camelcase
found by checkpatch.
CHECK: Avoid CamelCase: <DIMCB_OnError>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:77:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
b724207b41
staging: most: rename DIM_GetLockState to dim_get_lock_state
...
This patch renames DIM_GetLockState to dim_get_lock_state to avoid
camelcase found by checkpatch.
CHECK: Avoid CamelCase: <DIM_GetLockState>
FILE: drivers/staging/most/hdm-dim2/dim2_hdm.c:131:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
38c3854490
staging: most: rename DIM_DetachBuffers to dim_detach_buffers
...
This patch renames DIM_DetachBuffers to dim_detach_buffers to avoid
camelcase found by checkpatch.
CHECK: Avoid CamelCase: <DIM_DetachBuffers>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:886:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
50a45b170c
staging: most: rename DIM_Shutdown to dim_shutdown
...
This patch renames DIM_Shutdown to dim_shutdown to avoid camelcase
found by checkpatch.
CHECK: Avoid CamelCase: <DIM_Shutdown>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:676:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
6417267f17
staging: most: rename DIM_Startup to dim_startup
...
This patch renames DIM_Startup to dim_startup to avoid camelcase
found by checkpatch.
CHECK: Avoid CamelCase: <DIM_Startup>
FILE: drivers/staging/most/hdm-dim2/dim2_hal.c:653:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
3c70754250
staging: most: add spaces preferred around that '<<'
...
This patch adds space around '<<' found by checkpatch.
CHECK: spaces preferred around that '<<' (ctx:VxV)
FILE: drivers/staging/most/hdm-dim2/dim2_reg.h:69:
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
23242684c1
staging: most: remove blank line after an open brace
...
This patch removes blank line after an open brace found by checkpatch.
CHECK: Blank lines aren't necessary after an open brace '{'
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Chaehyun Lim
fabb93f18f
staging: most: remove multiple blank lines
...
This patch removes multiple blank lines found by checkpatch.
CHECK: Please don't use multiple blank lines
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
25e168a4a3
staging: rtl8188eu: goto removed
...
malloc error handling moved into one place.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
4a3bda22fd
staging: rtl8188eu: goto replaced by 'else' branch
...
goto is not needed here.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
d14c07f6ff
staging: rtl8188eu: unused macros removed
...
IS_* and GET_CVID_* macros have not been used.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
940f90eae6
staging: rtl8188eu: rarely used macros replaced by their definitions
...
IS_* macros (except one) occur only once.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
08ecab13e1
staging: rtl8188eu: for loop instead of while loop used
...
The range of elements to fill with zeros is determined by using a roundup macro
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
13cab3d422
staging: rtl8188eu: offset renamed to write_address
...
This name is better suited for this variable.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
7e91b28a01
staging: rtl8188eu: pu4BytePtr renamed to dword_buffer
...
This name is better suited for this variable.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
988c514688
staging: rtl8188eu: buf_ptr renamed to byte_buffer
...
This name is better suited for this variable.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
6e84aa6c82
staging: rtl8188eu: unnessesary braces for single statement block removed
...
checkpatch fix:
WARNING: braces {} are not necessary for single statement blocks
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
429078e1f1
staging: rtl8188eu: unnecessary initialization removed
...
It is superfluous.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
645d2a8f46
staging: rtl8188eu: types of local variables conformed with types of function arguments
...
The array should not change in any case.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
1e7e93ee6d
staging: rtl8188eu: buf_ptr variable completely defined in a single line
...
It is simpler.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
ec60e037c7
staging: rtl8188eu: offset increment placed into for loop header
...
It makes the code little easier.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
f464b3a08c
staging: rtl8188eu: unnecessary branching removed
...
If the 'remain' is zero, the loop is not executed at all.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
8107b147d7
staging: rtl8188eu: unnecessary variable override removed
...
Variable value calculated in the previous loop.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
37d5579095
staging: rtl8188eu: offset variable replaced by its value
...
It is now possible to get rid of re-initializing the offset variable.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
1c48deffc7
staging: rtl8188eu: initial value placed in the variable
...
Line become shorter.
After the loop offset variable points to the location following insertion.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
d44f58f7c5
staging: rtl8188eu: assigning a value to the variable is replaced by the increment
...
It is made to the initial value could be placed in the offset variable.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
52863d83f3
staging: rtl8188eu: *(ptr + i) replaced by ptr[i] in _rtl88e_fw_block_write
...
It is better to read.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Abdul Hussain
6d9b0f00ec
Staging: rtl8188eu: fix space prohibited before that ','
...
This patch fixes the following checkpatch.pl error:
fix space prohibited before that ','
Signed-off-by: Abdul Hussain <hussain.abdul@outlook.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00
Ivan Safonov
37efd08231
staging: rtl8188eu: 'infinite' loop removed
...
The body of this loop is executed only once, so it can be removed.
In this loop no keyword 'continue', only 'break' at the end.
Signed-off-by: Ivan Safonov <insafonov@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2015-11-15 20:02:47 -08:00