diff options
| author |   Easwar Hariharan <easwar.hariharan@intel.com> | 2016-05-12 10:22:45 -0700 |
|---|---|---|
| committer |   Doug Ledford <dledford@redhat.com> | 2016-05-13 19:39:15 -0400 |
| commit | 27a340f6b206dd2a0208d3bc4605c0708b55d7b4 (patch) | |
| tree | df75b85f0613e6b744e8b831813f7da1e1c75c20 /drivers/staging/rdma/hfi1/platform.c | |
| parent | IB/hfi1: Wait for QSFP modules to initialize (diff) | |
| download | linux-dev-27a340f6b206dd2a0208d3bc4605c0708b55d7b4.tar.xz linux-dev-27a340f6b206dd2a0208d3bc4605c0708b55d7b4.zip | |
IB/hfi1: Correct external device configuration shift
The external device configuration was incorrectly shifted to byte 3 of
the 32 bit DC_HOST_COMM_SETTINGS instead of byte 0. This patch corrects
the shift and provides the cable capability information in byte 0.
Reviewed-by: Dean Luick <dean.luick@intel.com>
Signed-off-by: Easwar Hariharan <easwar.hariharan@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to '')
| -rw-r--r-- | drivers/staging/rdma/hfi1/platform.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/staging/rdma/hfi1/platform.c b/drivers/staging/rdma/hfi1/platform.c index dfa413cfa351..ae0e4985cc7e 100644 --- a/drivers/staging/rdma/hfi1/platform.c +++ b/drivers/staging/rdma/hfi1/platform.c @@ -575,8 +575,8 @@ static void apply_tunings( ret = read_8051_config(ppd->dd, DC_HOST_COMM_SETTINGS, GENERAL_CONFIG, &config_data); /* Clear, then set the external device config field */ - config_data &= ~(0xFF << 24); - config_data |= (external_device_config << 24); + config_data &= ~(u32)0xFF; + config_data |= external_device_config; ret = load_8051_config(ppd->dd, DC_HOST_COMM_SETTINGS, GENERAL_CONFIG, config_data); if (ret != HCMD_SUCCESS) |