Commit 88e41947 authored by David S. Miller's avatar David S. Miller

Merge branch 'cxgb4'

Hariprasad Shenai says:

====================
Trivial fixes for cxgb4

This patch series adds support to fix T5 adapter accessing T4 adapter registers,
issue mbox command on correct mbox for physical function, avoid dumping write
only registers, use correct length for adapter part number and support to detect
and display firmware reported errors.

The patches series is created against 'net' tree.
And includes patches on cxgb4 driver.

We have included all the maintainers of respective drivers. Kindly review the
change and let us know in case of any review comments.

Thanks

V2:
   Added description for each patch as per David Miller's comment
====================
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents cc25f0cb 5c937dd3
...@@ -1253,7 +1253,9 @@ freeout: t4_free_sge_resources(adap); ...@@ -1253,7 +1253,9 @@ freeout: t4_free_sge_resources(adap);
goto freeout; goto freeout;
} }
t4_write_reg(adap, MPS_TRC_RSS_CONTROL, t4_write_reg(adap, is_t4(adap->params.chip) ?
MPS_TRC_RSS_CONTROL :
MPS_T5_TRC_RSS_CONTROL,
RSSCONTROL(netdev2pinfo(adap->port[0])->tx_chan) | RSSCONTROL(netdev2pinfo(adap->port[0])->tx_chan) |
QUEUENUMBER(s->ethrxq[0].rspq.abs_id)); QUEUENUMBER(s->ethrxq[0].rspq.abs_id));
return 0; return 0;
...@@ -1761,7 +1763,8 @@ static void get_regs(struct net_device *dev, struct ethtool_regs *regs, ...@@ -1761,7 +1763,8 @@ static void get_regs(struct net_device *dev, struct ethtool_regs *regs,
0xd004, 0xd03c, 0xd004, 0xd03c,
0xdfc0, 0xdfe0, 0xdfc0, 0xdfe0,
0xe000, 0xea7c, 0xe000, 0xea7c,
0xf000, 0x11190, 0xf000, 0x11110,
0x11118, 0x11190,
0x19040, 0x1906c, 0x19040, 0x1906c,
0x19078, 0x19080, 0x19078, 0x19080,
0x1908c, 0x19124, 0x1908c, 0x19124,
...@@ -1968,7 +1971,8 @@ static void get_regs(struct net_device *dev, struct ethtool_regs *regs, ...@@ -1968,7 +1971,8 @@ static void get_regs(struct net_device *dev, struct ethtool_regs *regs,
0xd004, 0xd03c, 0xd004, 0xd03c,
0xdfc0, 0xdfe0, 0xdfc0, 0xdfe0,
0xe000, 0x11088, 0xe000, 0x11088,
0x1109c, 0x1117c, 0x1109c, 0x11110,
0x11118, 0x1117c,
0x11190, 0x11204, 0x11190, 0x11204,
0x19040, 0x1906c, 0x19040, 0x1906c,
0x19078, 0x19080, 0x19078, 0x19080,
...@@ -5955,7 +5959,8 @@ static int adap_init0(struct adapter *adap) ...@@ -5955,7 +5959,8 @@ static int adap_init0(struct adapter *adap)
params[3] = FW_PARAM_PFVF(CQ_END); params[3] = FW_PARAM_PFVF(CQ_END);
params[4] = FW_PARAM_PFVF(OCQ_START); params[4] = FW_PARAM_PFVF(OCQ_START);
params[5] = FW_PARAM_PFVF(OCQ_END); params[5] = FW_PARAM_PFVF(OCQ_END);
ret = t4_query_params(adap, 0, 0, 0, 6, params, val); ret = t4_query_params(adap, adap->mbox, adap->fn, 0, 6, params,
val);
if (ret < 0) if (ret < 0)
goto bye; goto bye;
adap->vres.qp.start = val[0]; adap->vres.qp.start = val[0];
...@@ -5967,7 +5972,8 @@ static int adap_init0(struct adapter *adap) ...@@ -5967,7 +5972,8 @@ static int adap_init0(struct adapter *adap)
params[0] = FW_PARAM_DEV(MAXORDIRD_QP); params[0] = FW_PARAM_DEV(MAXORDIRD_QP);
params[1] = FW_PARAM_DEV(MAXIRD_ADAPTER); params[1] = FW_PARAM_DEV(MAXIRD_ADAPTER);
ret = t4_query_params(adap, 0, 0, 0, 2, params, val); ret = t4_query_params(adap, adap->mbox, adap->fn, 0, 2, params,
val);
if (ret < 0) { if (ret < 0) {
adap->params.max_ordird_qp = 8; adap->params.max_ordird_qp = 8;
adap->params.max_ird_adapter = 32 * adap->tids.ntids; adap->params.max_ird_adapter = 32 * adap->tids.ntids;
......
...@@ -167,6 +167,34 @@ void t4_hw_pci_read_cfg4(struct adapter *adap, int reg, u32 *val) ...@@ -167,6 +167,34 @@ void t4_hw_pci_read_cfg4(struct adapter *adap, int reg, u32 *val)
t4_write_reg(adap, PCIE_CFG_SPACE_REQ, 0); t4_write_reg(adap, PCIE_CFG_SPACE_REQ, 0);
} }
/*
* t4_report_fw_error - report firmware error
* @adap: the adapter
*
* The adapter firmware can indicate error conditions to the host.
* If the firmware has indicated an error, print out the reason for
* the firmware error.
*/
static void t4_report_fw_error(struct adapter *adap)
{
static const char *const reason[] = {
"Crash", /* PCIE_FW_EVAL_CRASH */
"During Device Preparation", /* PCIE_FW_EVAL_PREP */
"During Device Configuration", /* PCIE_FW_EVAL_CONF */
"During Device Initialization", /* PCIE_FW_EVAL_INIT */
"Unexpected Event", /* PCIE_FW_EVAL_UNEXPECTEDEVENT */
"Insufficient Airflow", /* PCIE_FW_EVAL_OVERHEAT */
"Device Shutdown", /* PCIE_FW_EVAL_DEVICESHUTDOWN */
"Reserved", /* reserved */
};
u32 pcie_fw;
pcie_fw = t4_read_reg(adap, MA_PCIE_FW);
if (pcie_fw & FW_PCIE_FW_ERR)
dev_err(adap->pdev_dev, "Firmware reports adapter error: %s\n",
reason[FW_PCIE_FW_EVAL_GET(pcie_fw)]);
}
/* /*
* Get the reply to a mailbox command and store it in @rpl in big-endian order. * Get the reply to a mailbox command and store it in @rpl in big-endian order.
*/ */
...@@ -300,6 +328,7 @@ int t4_wr_mbox_meat(struct adapter *adap, int mbox, const void *cmd, int size, ...@@ -300,6 +328,7 @@ int t4_wr_mbox_meat(struct adapter *adap, int mbox, const void *cmd, int size,
dump_mbox(adap, mbox, data_reg); dump_mbox(adap, mbox, data_reg);
dev_err(adap->pdev_dev, "command %#x in mailbox %d timed out\n", dev_err(adap->pdev_dev, "command %#x in mailbox %d timed out\n",
*(const u8 *)cmd, mbox); *(const u8 *)cmd, mbox);
t4_report_fw_error(adap);
return -ETIMEDOUT; return -ETIMEDOUT;
} }
...@@ -566,6 +595,7 @@ int t4_memory_rw(struct adapter *adap, int win, int mtype, u32 addr, ...@@ -566,6 +595,7 @@ int t4_memory_rw(struct adapter *adap, int win, int mtype, u32 addr,
#define VPD_BASE 0x400 #define VPD_BASE 0x400
#define VPD_BASE_OLD 0 #define VPD_BASE_OLD 0
#define VPD_LEN 1024 #define VPD_LEN 1024
#define CHELSIO_VPD_UNIQUE_ID 0x82
/** /**
* t4_seeprom_wp - enable/disable EEPROM write protection * t4_seeprom_wp - enable/disable EEPROM write protection
...@@ -603,7 +633,14 @@ int get_vpd_params(struct adapter *adapter, struct vpd_params *p) ...@@ -603,7 +633,14 @@ int get_vpd_params(struct adapter *adapter, struct vpd_params *p)
ret = pci_read_vpd(adapter->pdev, VPD_BASE, sizeof(u32), vpd); ret = pci_read_vpd(adapter->pdev, VPD_BASE, sizeof(u32), vpd);
if (ret < 0) if (ret < 0)
goto out; goto out;
addr = *vpd == 0x82 ? VPD_BASE : VPD_BASE_OLD;
/* The VPD shall have a unique identifier specified by the PCI SIG.
* For chelsio adapters, the identifier is 0x82. The first byte of a VPD
* shall be CHELSIO_VPD_UNIQUE_ID (0x82). The VPD programming software
* is expected to automatically put this entry at the
* beginning of the VPD.
*/
addr = *vpd == CHELSIO_VPD_UNIQUE_ID ? VPD_BASE : VPD_BASE_OLD;
ret = pci_read_vpd(adapter->pdev, addr, VPD_LEN, vpd); ret = pci_read_vpd(adapter->pdev, addr, VPD_LEN, vpd);
if (ret < 0) if (ret < 0)
...@@ -667,6 +704,7 @@ int get_vpd_params(struct adapter *adapter, struct vpd_params *p) ...@@ -667,6 +704,7 @@ int get_vpd_params(struct adapter *adapter, struct vpd_params *p)
i = pci_vpd_info_field_size(vpd + sn - PCI_VPD_INFO_FLD_HDR_SIZE); i = pci_vpd_info_field_size(vpd + sn - PCI_VPD_INFO_FLD_HDR_SIZE);
memcpy(p->sn, vpd + sn, min(i, SERNUM_LEN)); memcpy(p->sn, vpd + sn, min(i, SERNUM_LEN));
strim(p->sn); strim(p->sn);
i = pci_vpd_info_field_size(vpd + pn - PCI_VPD_INFO_FLD_HDR_SIZE);
memcpy(p->pn, vpd + pn, min(i, PN_LEN)); memcpy(p->pn, vpd + pn, min(i, PN_LEN));
strim(p->pn); strim(p->pn);
...@@ -1394,15 +1432,18 @@ static void pcie_intr_handler(struct adapter *adapter) ...@@ -1394,15 +1432,18 @@ static void pcie_intr_handler(struct adapter *adapter)
int fat; int fat;
fat = t4_handle_intr_status(adapter, if (is_t4(adapter->params.chip))
PCIE_CORE_UTL_SYSTEM_BUS_AGENT_STATUS, fat = t4_handle_intr_status(adapter,
sysbus_intr_info) + PCIE_CORE_UTL_SYSTEM_BUS_AGENT_STATUS,
t4_handle_intr_status(adapter, sysbus_intr_info) +
PCIE_CORE_UTL_PCI_EXPRESS_PORT_STATUS, t4_handle_intr_status(adapter,
pcie_port_intr_info) + PCIE_CORE_UTL_PCI_EXPRESS_PORT_STATUS,
t4_handle_intr_status(adapter, PCIE_INT_CAUSE, pcie_port_intr_info) +
is_t4(adapter->params.chip) ? t4_handle_intr_status(adapter, PCIE_INT_CAUSE,
pcie_intr_info : t5_pcie_intr_info); pcie_intr_info);
else
fat = t4_handle_intr_status(adapter, PCIE_INT_CAUSE,
t5_pcie_intr_info);
if (fat) if (fat)
t4_fatal_err(adapter); t4_fatal_err(adapter);
...@@ -1521,6 +1562,9 @@ static void cim_intr_handler(struct adapter *adapter) ...@@ -1521,6 +1562,9 @@ static void cim_intr_handler(struct adapter *adapter)
int fat; int fat;
if (t4_read_reg(adapter, MA_PCIE_FW) & FW_PCIE_FW_ERR)
t4_report_fw_error(adapter);
fat = t4_handle_intr_status(adapter, CIM_HOST_INT_CAUSE, fat = t4_handle_intr_status(adapter, CIM_HOST_INT_CAUSE,
cim_intr_info) + cim_intr_info) +
t4_handle_intr_status(adapter, CIM_HOST_UPACC_INT_CAUSE, t4_handle_intr_status(adapter, CIM_HOST_UPACC_INT_CAUSE,
...@@ -1768,10 +1812,16 @@ static void ma_intr_handler(struct adapter *adap) ...@@ -1768,10 +1812,16 @@ static void ma_intr_handler(struct adapter *adap)
{ {
u32 v, status = t4_read_reg(adap, MA_INT_CAUSE); u32 v, status = t4_read_reg(adap, MA_INT_CAUSE);
if (status & MEM_PERR_INT_CAUSE) if (status & MEM_PERR_INT_CAUSE) {
dev_alert(adap->pdev_dev, dev_alert(adap->pdev_dev,
"MA parity error, parity status %#x\n", "MA parity error, parity status %#x\n",
t4_read_reg(adap, MA_PARITY_ERROR_STATUS)); t4_read_reg(adap, MA_PARITY_ERROR_STATUS));
if (is_t5(adap->params.chip))
dev_alert(adap->pdev_dev,
"MA parity error, parity status %#x\n",
t4_read_reg(adap,
MA_PARITY_ERROR_STATUS2));
}
if (status & MEM_WRAP_INT_CAUSE) { if (status & MEM_WRAP_INT_CAUSE) {
v = t4_read_reg(adap, MA_INT_WRAP_STATUS); v = t4_read_reg(adap, MA_INT_WRAP_STATUS);
dev_alert(adap->pdev_dev, "MA address wrap-around error by " dev_alert(adap->pdev_dev, "MA address wrap-around error by "
...@@ -2733,12 +2783,16 @@ int t4_fw_hello(struct adapter *adap, unsigned int mbox, unsigned int evt_mbox, ...@@ -2733,12 +2783,16 @@ int t4_fw_hello(struct adapter *adap, unsigned int mbox, unsigned int evt_mbox,
/* /*
* Issue the HELLO command to the firmware. If it's not successful * Issue the HELLO command to the firmware. If it's not successful
* but indicates that we got a "busy" or "timeout" condition, retry * but indicates that we got a "busy" or "timeout" condition, retry
* the HELLO until we exhaust our retry limit. * the HELLO until we exhaust our retry limit. If we do exceed our
* retry limit, check to see if the firmware left us any error
* information and report that if so.
*/ */
ret = t4_wr_mbox(adap, mbox, &c, sizeof(c), &c); ret = t4_wr_mbox(adap, mbox, &c, sizeof(c), &c);
if (ret < 0) { if (ret < 0) {
if ((ret == -EBUSY || ret == -ETIMEDOUT) && retries-- > 0) if ((ret == -EBUSY || ret == -ETIMEDOUT) && retries-- > 0)
goto retry; goto retry;
if (t4_read_reg(adap, MA_PCIE_FW) & FW_PCIE_FW_ERR)
t4_report_fw_error(adap);
return ret; return ret;
} }
...@@ -3742,6 +3796,7 @@ int t4_handle_fw_rpl(struct adapter *adap, const __be64 *rpl) ...@@ -3742,6 +3796,7 @@ int t4_handle_fw_rpl(struct adapter *adap, const __be64 *rpl)
lc->link_ok = link_ok; lc->link_ok = link_ok;
lc->speed = speed; lc->speed = speed;
lc->fc = fc; lc->fc = fc;
lc->supported = be16_to_cpu(p->u.info.pcap);
t4_os_link_changed(adap, port, link_ok); t4_os_link_changed(adap, port, link_ok);
} }
if (mod != pi->mod_type) { if (mod != pi->mod_type) {
......
...@@ -511,6 +511,7 @@ ...@@ -511,6 +511,7 @@
#define MEM_WRAP_CLIENT_NUM_GET(x) (((x) & MEM_WRAP_CLIENT_NUM_MASK) >> MEM_WRAP_CLIENT_NUM_SHIFT) #define MEM_WRAP_CLIENT_NUM_GET(x) (((x) & MEM_WRAP_CLIENT_NUM_MASK) >> MEM_WRAP_CLIENT_NUM_SHIFT)
#define MA_PCIE_FW 0x30b8 #define MA_PCIE_FW 0x30b8
#define MA_PARITY_ERROR_STATUS 0x77f4 #define MA_PARITY_ERROR_STATUS 0x77f4
#define MA_PARITY_ERROR_STATUS2 0x7804
#define MA_EXT_MEMORY1_BAR 0x7808 #define MA_EXT_MEMORY1_BAR 0x7808
#define EDC_0_BASE_ADDR 0x7900 #define EDC_0_BASE_ADDR 0x7900
...@@ -959,6 +960,7 @@ ...@@ -959,6 +960,7 @@
#define TRCMULTIFILTER 0x00000001U #define TRCMULTIFILTER 0x00000001U
#define MPS_TRC_RSS_CONTROL 0x9808 #define MPS_TRC_RSS_CONTROL 0x9808
#define MPS_T5_TRC_RSS_CONTROL 0xa00c
#define RSSCONTROL_MASK 0x00ff0000U #define RSSCONTROL_MASK 0x00ff0000U
#define RSSCONTROL_SHIFT 16 #define RSSCONTROL_SHIFT 16
#define RSSCONTROL(x) ((x) << RSSCONTROL_SHIFT) #define RSSCONTROL(x) ((x) << RSSCONTROL_SHIFT)
......
...@@ -2228,6 +2228,10 @@ struct fw_debug_cmd { ...@@ -2228,6 +2228,10 @@ struct fw_debug_cmd {
#define FW_PCIE_FW_MASTER(x) ((x) << FW_PCIE_FW_MASTER_SHIFT) #define FW_PCIE_FW_MASTER(x) ((x) << FW_PCIE_FW_MASTER_SHIFT)
#define FW_PCIE_FW_MASTER_GET(x) (((x) >> FW_PCIE_FW_MASTER_SHIFT) & \ #define FW_PCIE_FW_MASTER_GET(x) (((x) >> FW_PCIE_FW_MASTER_SHIFT) & \
FW_PCIE_FW_MASTER_MASK) FW_PCIE_FW_MASTER_MASK)
#define FW_PCIE_FW_EVAL_MASK 0x7
#define FW_PCIE_FW_EVAL_SHIFT 24
#define FW_PCIE_FW_EVAL_GET(x) (((x) >> FW_PCIE_FW_EVAL_SHIFT) & \
FW_PCIE_FW_EVAL_MASK)
struct fw_hdr { struct fw_hdr {
u8 ver; u8 ver;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment