Commit 1680a370 authored by Jakub Kicinski's avatar Jakub Kicinski Committed by David S. Miller

nfp: only use direct firmware requests

request_firmware() will fallback to user space helper and may cause
long delays when driver is loaded if udev doesn't correctly handle
FW requests.  Since we never really made use of the user space
helper functionality switch to the simpler request_firmware_direct()
call.  The side effect of this change is that no warning will be
printed when the FW image does not exists.  To help users figure
out which FW file is missing print a info message when we request
each file.
Signed-off-by: default avatarJakub Kicinski <jakub.kicinski@netronome.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 9511f298
...@@ -174,6 +174,21 @@ static int nfp_pcie_sriov_configure(struct pci_dev *pdev, int num_vfs) ...@@ -174,6 +174,21 @@ static int nfp_pcie_sriov_configure(struct pci_dev *pdev, int num_vfs)
return nfp_pcie_sriov_enable(pdev, num_vfs); return nfp_pcie_sriov_enable(pdev, num_vfs);
} }
static const struct firmware *
nfp_net_fw_request(struct pci_dev *pdev, struct nfp_pf *pf, const char *name)
{
const struct firmware *fw = NULL;
int err;
err = request_firmware_direct(&fw, name, &pdev->dev);
nfp_info(pf->cpp, " %s: %s\n",
name, err ? "not found" : "found, loading...");
if (err)
return NULL;
return fw;
}
/** /**
* nfp_net_fw_find() - Find the correct firmware image for netdev mode * nfp_net_fw_find() - Find the correct firmware image for netdev mode
* @pdev: PCI Device structure * @pdev: PCI Device structure
...@@ -184,29 +199,30 @@ static int nfp_pcie_sriov_configure(struct pci_dev *pdev, int num_vfs) ...@@ -184,29 +199,30 @@ static int nfp_pcie_sriov_configure(struct pci_dev *pdev, int num_vfs)
static const struct firmware * static const struct firmware *
nfp_net_fw_find(struct pci_dev *pdev, struct nfp_pf *pf) nfp_net_fw_find(struct pci_dev *pdev, struct nfp_pf *pf)
{ {
const struct firmware *fw = NULL;
struct nfp_eth_table_port *port; struct nfp_eth_table_port *port;
const struct firmware *fw;
const char *fw_model; const char *fw_model;
char fw_name[256]; char fw_name[256];
const u8 *serial; const u8 *serial;
int spc, err = 0;
u16 interface; u16 interface;
int i, j; int spc, i, j;
nfp_info(pf->cpp, "Looking for firmware file in order of priority:\n");
/* First try to find a firmware image specific for this device */ /* First try to find a firmware image specific for this device */
interface = nfp_cpp_interface(pf->cpp); interface = nfp_cpp_interface(pf->cpp);
nfp_cpp_serial(pf->cpp, &serial); nfp_cpp_serial(pf->cpp, &serial);
sprintf(fw_name, "netronome/serial-%pMF-%02hhx-%02hhx.nffw", sprintf(fw_name, "netronome/serial-%pMF-%02hhx-%02hhx.nffw",
serial, interface >> 8, interface & 0xff); serial, interface >> 8, interface & 0xff);
err = request_firmware_direct(&fw, fw_name, &pdev->dev); fw = nfp_net_fw_request(pdev, pf, fw_name);
if (!err) if (fw)
goto done; return fw;
/* Then try the PCI name */ /* Then try the PCI name */
sprintf(fw_name, "netronome/pci-%s.nffw", pci_name(pdev)); sprintf(fw_name, "netronome/pci-%s.nffw", pci_name(pdev));
err = request_firmware_direct(&fw, fw_name, &pdev->dev); fw = nfp_net_fw_request(pdev, pf, fw_name);
if (!err) if (fw)
goto done; return fw;
/* Finally try the card type and media */ /* Finally try the card type and media */
if (!pf->eth_tbl) { if (!pf->eth_tbl) {
...@@ -241,13 +257,7 @@ nfp_net_fw_find(struct pci_dev *pdev, struct nfp_pf *pf) ...@@ -241,13 +257,7 @@ nfp_net_fw_find(struct pci_dev *pdev, struct nfp_pf *pf)
if (spc <= 0) if (spc <= 0)
return NULL; return NULL;
err = request_firmware(&fw, fw_name, &pdev->dev); return nfp_net_fw_request(pdev, pf, fw_name);
if (err)
return NULL;
done:
dev_info(&pdev->dev, "Loading FW image: %s\n", fw_name);
return fw;
} }
/** /**
......
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