Skip to content

Commit

Permalink
sfc_ef100: check firmware version at start-of-day
Browse files Browse the repository at this point in the history
Early in EF100 development there was a different format of event
 descriptor; if the NIC is somehow running the very old firmware
 which will use that format, fail the probe.

Signed-off-by: Edward Cree <ecree@solarflare.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Edward Cree authored and David S. Miller committed Aug 4, 2020
1 parent 215602a commit 8e73714
Showing 1 changed file with 40 additions and 0 deletions.
40 changes: 40 additions & 0 deletions drivers/net/ethernet/sfc/ef100_nic.c
Original file line number Diff line number Diff line change
Expand Up @@ -485,13 +485,44 @@ const struct efx_nic_type ef100_pf_nic_type = {

};

static int compare_versions(const char *a, const char *b)
{
int a_major, a_minor, a_point, a_patch;
int b_major, b_minor, b_point, b_patch;
int a_matched, b_matched;

a_matched = sscanf(a, "%d.%d.%d.%d", &a_major, &a_minor, &a_point, &a_patch);
b_matched = sscanf(b, "%d.%d.%d.%d", &b_major, &b_minor, &b_point, &b_patch);

if (a_matched == 4 && b_matched != 4)
return +1;

if (a_matched != 4 && b_matched == 4)
return -1;

if (a_matched != 4 && b_matched != 4)
return 0;

if (a_major != b_major)
return a_major - b_major;

if (a_minor != b_minor)
return a_minor - b_minor;

if (a_point != b_point)
return a_point - b_point;

return a_patch - b_patch;
}

/* NIC probe and remove
*/
static int ef100_probe_main(struct efx_nic *efx)
{
unsigned int bar_size = resource_size(&efx->pci_dev->resource[efx->mem_bar]);
struct net_device *net_dev = efx->net_dev;
struct ef100_nic_data *nic_data;
char fw_version[32];
int i, rc;

if (WARN_ON(bar_size == 0))
Expand Down Expand Up @@ -562,6 +593,15 @@ static int ef100_probe_main(struct efx_nic *efx)
goto fail;
efx->port_num = rc;

efx_mcdi_print_fwver(efx, fw_version, sizeof(fw_version));
netif_dbg(efx, drv, efx->net_dev, "Firmware version %s\n", fw_version);

if (compare_versions(fw_version, "1.1.0.1000") < 0) {
netif_info(efx, drv, efx->net_dev, "Firmware uses old event descriptors\n");
rc = -EINVAL;
goto fail;
}

rc = ef100_phy_probe(efx);
if (rc)
goto fail;
Expand Down

0 comments on commit 8e73714

Please sign in to comment.