diff options
author | Madalin Bucur <madalin.bucur@freescale.com> | 2014-09-16 13:39:30 (GMT) |
---|---|---|
committer | Madalin Bucur <madalin.bucur@freescale.com> | 2014-10-08 13:19:54 (GMT) |
commit | 81c870b0ad99a1ed9f9aba7dfee127ef74b0b460 (patch) | |
tree | 13e922d70f05252a3860b51a8e56c2869891e8e9 | |
parent | 10393b503d5676500445c03d62e8dcc1ae775753 (diff) | |
download | linux-fsl-qoriq-81c870b0ad99a1ed9f9aba7dfee127ef74b0b460.tar.xz |
dpaa_eth: verify return value
Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com>
Change-Id: Ife32e1984f5dc8f481dad223602163229526b861
Reviewed-on: http://git.am.freescale.net:8181/18841
Reviewed-by: Marian Cristian Rotariu <marian.rotariu@freescale.com>
-rw-r--r-- | drivers/net/ethernet/freescale/dpa/dpaa_eth_generic.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/net/ethernet/freescale/dpa/dpaa_eth_generic.c b/drivers/net/ethernet/freescale/dpa/dpaa_eth_generic.c index 07b80ba..9447d31 100644 --- a/drivers/net/ethernet/freescale/dpa/dpaa_eth_generic.c +++ b/drivers/net/ethernet/freescale/dpa/dpaa_eth_generic.c @@ -1276,16 +1276,16 @@ static int dpa_generic_fq_create(struct net_device *netdev, struct dpa_fq *fqs = NULL, *tmp = NULL; struct task_struct *kth; int err = 0; + int channel; INIT_LIST_HEAD(&priv->dpa_fq_list); list_replace_init(dpa_fq_list, &priv->dpa_fq_list); - priv->channel = dpa_get_channel(); - if (priv->channel < 0) { - err = priv->channel; - return err; - } + channel = dpa_get_channel(); + if (channel < 0) + return channel; + 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. |