staging: fsl-dpaa2/eth: Defer probing if no DPIOs found
authorIoana Radulescu <ruxandra.radulescu@nxp.com>
Tue, 6 Jun 2017 15:00:33 +0000 (10:00 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 13 Jun 2017 10:02:55 +0000 (12:02 +0200)
If the Ethernet driver doesn't find any DPIO devices during probe,
it may be either because there's none available or because they
haven't been probed yet. Request deferred probing in case it's
the latter.

Signed-off-by: Bharat Bhushan <Bharat.Bhushan@nxp.com>
Signed-off-by: Ioana Radulescu <ruxandra.radulescu@nxp.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c

index fd56fbd..025b5f6 100644 (file)
@@ -1510,6 +1510,7 @@ static int setup_dpio(struct dpaa2_eth_priv *priv)
                if (!channel) {
                        dev_info(dev,
                                 "No affine channel for cpu %d and above\n", i);
+                       err = -ENODEV;
                        goto err_alloc_ch;
                }
 
@@ -1524,10 +1525,13 @@ static int setup_dpio(struct dpaa2_eth_priv *priv)
                /* Register the new context */
                err = dpaa2_io_service_register(NULL, nctx);
                if (err) {
-                       dev_info(dev, "No affine DPIO for cpu %d\n", i);
+                       dev_dbg(dev, "No affine DPIO for cpu %d\n", i);
                        /* If no affine DPIO for this core, there's probably
-                        * none available for next cores either.
+                        * none available for next cores either. Signal we want
+                        * to retry later, in case the DPIO devices weren't
+                        * probed yet.
                         */
+                       err = -EPROBE_DEFER;
                        goto err_service_reg;
                }
 
@@ -1565,7 +1569,7 @@ err_service_reg:
 err_alloc_ch:
        if (cpumask_empty(&priv->dpio_cpumask)) {
                dev_err(dev, "No cpu with an affine DPIO/DPCON\n");
-               return -ENODEV;
+               return err;
        }
 
        dev_info(dev, "Cores %*pbl available for processing ingress traffic\n",