Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

cxgb4: collect serial config version from register

Collect serial config version information directly from an internal
register, instead of explicitly resizing VPD.

v2:
- Add comments on info stored in PCIE_STATIC_SPARE2 register.

Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
Reviewed-by: Heiner Kallweit <hkallweit1@gmail.com>
Reviewed-by: Alexander Duyck <alexanderduyck@fb.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

authored by

Rahul Lakkireddy and committed by
David S. Miller
24a1720a dc9d8758

+9 -24
-3
drivers/net/ethernet/chelsio/cxgb4/cudbg_entity.h
··· 220 220 u8 reserved[2]; 221 221 }; 222 222 223 - #define CUDBG_VPD_PF_SIZE 0x800 224 - #define CUDBG_SCFG_VER_ADDR 0x06 225 - #define CUDBG_SCFG_VER_LEN 4 226 223 #define CUDBG_VPD_VER_ADDR 0x18c7 227 224 #define CUDBG_VPD_VER_LEN 2 228 225
+3 -21
drivers/net/ethernet/chelsio/cxgb4/cudbg_lib.c
··· 2686 2686 struct adapter *padap = pdbg_init->adap; 2687 2687 struct cudbg_buffer temp_buff = { 0 }; 2688 2688 char vpd_str[CUDBG_VPD_VER_LEN + 1]; 2689 - u32 scfg_vers, vpd_vers, fw_vers; 2690 2689 struct cudbg_vpd_data *vpd_data; 2691 2690 struct vpd_params vpd = { 0 }; 2692 - int rc, ret; 2691 + u32 vpd_vers, fw_vers; 2692 + int rc; 2693 2693 2694 2694 rc = t4_get_raw_vpd_params(padap, &vpd); 2695 2695 if (rc) ··· 2698 2698 rc = t4_get_fw_version(padap, &fw_vers); 2699 2699 if (rc) 2700 2700 return rc; 2701 - 2702 - /* Serial Configuration Version is located beyond the PF's vpd size. 2703 - * Temporarily give access to entire EEPROM to get it. 2704 - */ 2705 - rc = pci_set_vpd_size(padap->pdev, EEPROMVSIZE); 2706 - if (rc < 0) 2707 - return rc; 2708 - 2709 - ret = cudbg_read_vpd_reg(padap, CUDBG_SCFG_VER_ADDR, CUDBG_SCFG_VER_LEN, 2710 - &scfg_vers); 2711 - 2712 - /* Restore back to original PF's vpd size */ 2713 - rc = pci_set_vpd_size(padap->pdev, CUDBG_VPD_PF_SIZE); 2714 - if (rc < 0) 2715 - return rc; 2716 - 2717 - if (ret) 2718 - return ret; 2719 2701 2720 2702 rc = cudbg_read_vpd_reg(padap, CUDBG_VPD_VER_ADDR, CUDBG_VPD_VER_LEN, 2721 2703 vpd_str); ··· 2719 2737 memcpy(vpd_data->bn, vpd.pn, PN_LEN + 1); 2720 2738 memcpy(vpd_data->na, vpd.na, MACADDR_LEN + 1); 2721 2739 memcpy(vpd_data->mn, vpd.id, ID_LEN + 1); 2722 - vpd_data->scfg_vers = scfg_vers; 2740 + vpd_data->scfg_vers = t4_read_reg(padap, PCIE_STATIC_SPARE2_A); 2723 2741 vpd_data->vpd_vers = vpd_vers; 2724 2742 vpd_data->fw_major = FW_HDR_FW_VER_MAJOR_G(fw_vers); 2725 2743 vpd_data->fw_minor = FW_HDR_FW_VER_MINOR_G(fw_vers);
+6
drivers/net/ethernet/chelsio/cxgb4/t4_regs.h
··· 884 884 #define TDUE_V(x) ((x) << TDUE_S) 885 885 #define TDUE_F TDUE_V(1U) 886 886 887 + /* SPARE2 register contains 32-bit value at offset 0x6 in Serial INIT 888 + * Configuration flashed on EEPROM. This value corresponds to 32-bit 889 + * Serial Configuration Version information. 890 + */ 891 + #define PCIE_STATIC_SPARE2_A 0x5bfc 892 + 887 893 /* registers for module MC */ 888 894 #define MC_INT_CAUSE_A 0x7518 889 895 #define MC_P_INT_CAUSE_A 0x41318