diff options
| author |   Rasmus Villemoes <linux@rasmusvillemoes.dk> | 2019-11-28 15:55:46 +0100 |
|---|---|---|
| committer |   Li Yang <leoyang.li@nxp.com> | 2019-12-09 13:54:36 -0600 |
| commit | 4c3e565cc6237e0fc9b60e34420456a7b3fc74f6 (patch) | |
| tree | 9e2576663db19fb838a278c4343c68b2d3841e44 /drivers/soc/fsl/qe/qe.c | |
| parent | soc: fsl: qe: avoid IS_ERR_VALUE in ucc_slow.c (diff) | |
| download | linux-dev-4c3e565cc6237e0fc9b60e34420456a7b3fc74f6.tar.xz linux-dev-4c3e565cc6237e0fc9b60e34420456a7b3fc74f6.zip | |
soc: fsl: qe: drop use of IS_ERR_VALUE in qe_sdma_init()
Now that qe_muram_alloc() returns s32, adapt qe_sdma_init() and avoid
another few IS_ERR_VALUE() uses.
Reviewed-by: Timur Tabi <timur@kernel.org>
Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Signed-off-by: Li Yang <leoyang.li@nxp.com>
Diffstat (limited to 'drivers/soc/fsl/qe/qe.c')
| -rw-r--r-- | drivers/soc/fsl/qe/qe.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/soc/fsl/qe/qe.c b/drivers/soc/fsl/qe/qe.c index ec511840db3c..5bf279c679ef 100644 --- a/drivers/soc/fsl/qe/qe.c +++ b/drivers/soc/fsl/qe/qe.c @@ -365,16 +365,16 @@ EXPORT_SYMBOL(qe_put_snum); static int qe_sdma_init(void) { struct sdma __iomem *sdma = &qe_immr->sdma; - static unsigned long sdma_buf_offset = (unsigned long)-ENOMEM; + static s32 sdma_buf_offset = -ENOMEM; if (!sdma) return -ENODEV; /* allocate 2 internal temporary buffers (512 bytes size each) for * the SDMA */ - if (IS_ERR_VALUE(sdma_buf_offset)) { + if (sdma_buf_offset < 0) { sdma_buf_offset = qe_muram_alloc(512 * 2, 4096); - if (IS_ERR_VALUE(sdma_buf_offset)) + if (sdma_buf_offset < 0) return -ENOMEM; } |