Ping. I didn't push this through my tree because I didn't want to
create conflicts with other changes to this file. But I think this
can be merged any time.
On Fri, Dec 7, 2012 at 3:29 PM, Giridhar Malavali
<giridhar.malavali@xxxxxxxxxx> wrote:
> Acked-by: Giridhar Malavali
>
> On 12/7/12 12:17 AM, "Saurav Kashyap" <saurav.kashyap@xxxxxxxxxx> wrote:
>
>>Acked-by: Saurav Kashyap <saurav.kashyap@xxxxxxxxxx>
>>
>>Thanks,
>>~Saurav
>>
>>
>>
>>>Use the standard #defines for PCIe Link Capability register fields
>>>rather than bare numbers. This also uses the new PCI Express Capability
>>>accessor rather than reading the capability directly.
>>>
>>>Signed-off-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>
>>>CC: Andrew Vasquez <andrew.vasquez@xxxxxxxxxx>
>>>CC: linux-driver@xxxxxxxxxx
>>>CC: "James E.J. Bottomley" <JBottomley@xxxxxxxxxxxxx>
>>>CC: linux-scsi@xxxxxxxxxxxxxxx
>>>---
>>> drivers/scsi/qla2xxx/qla_os.c | 14 +++++---------
>>> 1 files changed, 5 insertions(+), 9 deletions(-)
>>>
>>>diff --git a/drivers/scsi/qla2xxx/qla_os.c
>>>b/drivers/scsi/qla2xxx/qla_os.c
>>>index d501bf5..b5d070f 100644
>>>--- a/drivers/scsi/qla2xxx/qla_os.c
>>>+++ b/drivers/scsi/qla2xxx/qla_os.c
>>>@@ -480,18 +480,14 @@ qla24xx_pci_info_str(struct scsi_qla_host *vha,
>>>char *str)
>>> static char *pci_bus_modes[] = { "33", "66", "100", "133", };
>>> struct qla_hw_data *ha = vha->hw;
>>> uint32_t pci_bus;
>>>- int pcie_reg;
>>>
>>>- pcie_reg = pci_pcie_cap(ha->pdev);
>>>- if (pcie_reg) {
>>>+ if (pci_is_pcie(ha->pdev)) {
>>> char lwstr[6];
>>>- uint16_t pcie_lstat, lspeed, lwidth;
>>>+ uint32_t lstat, lspeed, lwidth;
>>>
>>>- pcie_reg += PCI_EXP_LNKCAP;
>>>- pci_read_config_word(ha->pdev, pcie_reg, &pcie_lstat);
>>>- lspeed = pcie_lstat & (BIT_0 | BIT_1 | BIT_2 | BIT_3);
>>>- lwidth = (pcie_lstat &
>>>- (BIT_4 | BIT_5 | BIT_6 | BIT_7 | BIT_8 | BIT_9)) >> 4;
>>>+ pcie_capability_read_dword(ha->pdev, PCI_EXP_LNKCAP, &lstat);
>>>+ lspeed = lstat & PCI_EXP_LNKCAP_SLS;
>>>+ lwidth = (lstat & PCI_EXP_LNKCAP_MLW) >> 4;
>>>
>>> strcpy(str, "PCIe (");
>>> if (lspeed == 1)
>>>
>>>--
>>>To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
>>>the body of a message to majordomo@xxxxxxxxxxxxxxx
>>>More majordomo info at http://vger.kernel.org/majordomo-info.html
>>>
>>
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
