diff options
author | Roland Dreier <roland@purestorage.com> | 2014-02-21 11:07:21 -0500 |
---|---|---|
committer | Jon Mason <jon.mason@intel.com> | 2014-04-07 13:58:14 -0400 |
commit | c8eee3795d68d4b7448ce9e6752de379d9fe2c23 (patch) | |
tree | e92ecb2efea8499d51b57fbf9a9d952b53f9e58d /drivers/ntb | |
parent | e8bc2ebdceea3c0c6de16d899f0679654c308739 (diff) |
NTB: Fix typo in setting one translation register
In the code for Xeon devices in back-to-back mode with xeon_errata_workaround
disabled, the downstream device puts the wrong value in SNB_B2B_XLAT_OFFSETL
(SNB_MBAR01_DSD_ADDR vs. SNB_MBAR01_USD_ADDR).
This was spotted while reading code, since the typo has no practical effect,
at least for now: the low 32 bits of both constants are actually identical
anyway. However, it's clearer and safer to use the right name.
Signed-off-by: Roland Dreier <roland@purestorage.com>
Signed-off-by: Jon Mason <jon.mason@intel.com>
Diffstat (limited to 'drivers/ntb')
-rw-r--r-- | drivers/ntb/ntb_hw.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/ntb/ntb_hw.c b/drivers/ntb/ntb_hw.c index 170e8e60cdb7..2774d356b689 100644 --- a/drivers/ntb/ntb_hw.c +++ b/drivers/ntb/ntb_hw.c | |||
@@ -785,7 +785,7 @@ static int ntb_xeon_setup(struct ntb_device *ndev) | |||
785 | /* B2B_XLAT_OFFSET is a 64bit register, but can | 785 | /* B2B_XLAT_OFFSET is a 64bit register, but can |
786 | * only take 32bit writes | 786 | * only take 32bit writes |
787 | */ | 787 | */ |
788 | writel(SNB_MBAR01_DSD_ADDR & 0xffffffff, | 788 | writel(SNB_MBAR01_USD_ADDR & 0xffffffff, |
789 | ndev->reg_base + SNB_B2B_XLAT_OFFSETL); | 789 | ndev->reg_base + SNB_B2B_XLAT_OFFSETL); |
790 | writel(SNB_MBAR01_USD_ADDR >> 32, | 790 | writel(SNB_MBAR01_USD_ADDR >> 32, |
791 | ndev->reg_base + SNB_B2B_XLAT_OFFSETU); | 791 | ndev->reg_base + SNB_B2B_XLAT_OFFSETU); |