diff options
| author | Madalin Bucur <madalin.bucur@freescale.com> | 2014-08-25 14:21:07 (GMT) |
|---|---|---|
| committer | Madalin Bucur <madalin.bucur@freescale.com> | 2014-10-08 13:19:53 (GMT) |
| commit | 6115fd88719f159ef3c818067e664324977c8589 (patch) | |
| tree | fd80bc120908c9d8778a33da9b4c33f42dd8af55 /drivers/net/ethernet/freescale/dpa/dpaa_eth_shared.c | |
| parent | bc6491027d370304c19fdfe853fdebabc235be4d (diff) | |
| download | linux-fsl-qoriq-6115fd88719f159ef3c818067e664324977c8589.tar.xz | |
dpaa_eth: fix dpa_get_channel return value check
Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com>
Change-Id: I5594b9f26210f9a7237bdd51f529d40a2b008115
Reviewed-on: http://git.am.freescale.net:8181/17400
Reviewed-by: Cristian Bercaru <cristian.bercaru@freescale.com>
Diffstat (limited to 'drivers/net/ethernet/freescale/dpa/dpaa_eth_shared.c')
| -rw-r--r-- | drivers/net/ethernet/freescale/dpa/dpaa_eth_shared.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/drivers/net/ethernet/freescale/dpa/dpaa_eth_shared.c b/drivers/net/ethernet/freescale/dpa/dpaa_eth_shared.c index 17dc86f..90d2849 100644 --- a/drivers/net/ethernet/freescale/dpa/dpaa_eth_shared.c +++ b/drivers/net/ethernet/freescale/dpa/dpaa_eth_shared.c @@ -670,7 +670,7 @@ static const struct dev_pm_ops shared_pm_ops = { static int dpaa_eth_shared_probe(struct platform_device *_of_dev) { - int err = 0, i; + int err = 0, i, channel; struct device *dev; struct device_node *dpa_node; struct dpa_bp *dpa_bp; @@ -755,13 +755,15 @@ dpaa_eth_shared_probe(struct platform_device *_of_dev) priv->mac_dev = mac_dev; - priv->channel = dpa_get_channel(); + channel = dpa_get_channel(); - if (priv->channel < 0) { - err = priv->channel; + if (channel < 0) { + err = channel; goto get_channel_failed; } + priv->channel = (uint16_t)channel; + /* Start a thread that will walk the cpus with affine portals * and add this pool channel to each's dequeue mask. */ |