diff options
| author | Meenakshi Aggarwal <meenakshi.aggarwal@freescale.com> | 2014-03-24 11:04:15 (GMT) |
|---|---|---|
| committer | Jose Rivera <German.Rivera@freescale.com> | 2014-03-26 22:36:39 (GMT) |
| commit | 474a265d43d065b6790ebcb1b59f5dc1ed4a65ef (patch) | |
| tree | e644456a2f78c9ead65a43d03474209a075dd68d /drivers/staging/fsl_qbman | |
| parent | 014304dbab977b79a5f835374e4ea01fa7b0b1a1 (diff) | |
| download | linux-fsl-qoriq-474a265d43d065b6790ebcb1b59f5dc1ed4a65ef.tar.xz | |
fsl_qbman: remove unnecessary call to put_affine_portal()
In bman_create_affine_slave() function :
- we used put_affine_portal() which should be called along with
get_affine_portal() or get_raw_affine_portal().
- unnecessary call to put_affine_portal() leaves the function in
invalid state and resulted in kernel panic.
Signed-off-by: Meenakshi Aggarwal <meenakshi.aggarwal@freescale.com>
Signed-off-by: Priyanka Jain <priyanka.jain@freescale.com>
Change-Id: I52a98c18b20d6f83f86ce462005f43e26fa1bdd9
Reviewed-on: http://git.am.freescale.net:8181/10283
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Priyanka Jain <Priyanka.Jain@freescale.com>
Reviewed-by: Jose Rivera <German.Rivera@freescale.com>
Diffstat (limited to 'drivers/staging/fsl_qbman')
| -rw-r--r-- | drivers/staging/fsl_qbman/bman_high.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/staging/fsl_qbman/bman_high.c b/drivers/staging/fsl_qbman/bman_high.c index 16e9534..16445a7 100644 --- a/drivers/staging/fsl_qbman/bman_high.c +++ b/drivers/staging/fsl_qbman/bman_high.c @@ -326,7 +326,6 @@ struct bman_portal *bman_create_affine_slave(struct bman_portal *redirect, BUG_ON(!redirect->config->public_cfg.is_shared); p->irq_sources = 0; p->sharing_redirect = redirect; - put_affine_portal(); return p; #else BUG(); |