diff options
author | Aditya Pakki <pakki001@umn.edu> | 2019-12-19 11:21:11 -0600 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2020-01-03 11:04:57 +0100 |
commit | d7a336d67ab5443a0ef14b8335d139e855e8a682 (patch) | |
tree | 44d95b22e01493bca491e2fdbc89f387a0942627 /drivers/staging/kpc2000/kpc_dma | |
parent | Merge 5.5-rc3 into staging-next (diff) | |
download | linux-dev-d7a336d67ab5443a0ef14b8335d139e855e8a682.tar.xz linux-dev-d7a336d67ab5443a0ef14b8335d139e855e8a682.zip |
staging: kpc2000: remove unnecessary assertions in kpc_dma_transfer
In kpc_dma_transfer(), the assertion that priv is NULL and priv->ldev
is NULL, are never satisfied. The two callers of the function,
dereference the fields before the function is called. This patch
removes the two BUG_ON calls.
Signed-off-by: Aditya Pakki <pakki001@umn.edu>
Link: https://lore.kernel.org/r/20191219172118.17456-1-pakki001@umn.edu
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/kpc2000/kpc_dma')
-rw-r--r-- | drivers/staging/kpc2000/kpc_dma/fileops.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/staging/kpc2000/kpc_dma/fileops.c b/drivers/staging/kpc2000/kpc_dma/fileops.c index cb52bd9a6d2f..40525540dde6 100644 --- a/drivers/staging/kpc2000/kpc_dma/fileops.c +++ b/drivers/staging/kpc2000/kpc_dma/fileops.c @@ -49,9 +49,7 @@ static int kpc_dma_transfer(struct dev_private_data *priv, u64 dma_addr; u64 user_ctl; - BUG_ON(priv == NULL); ldev = priv->ldev; - BUG_ON(ldev == NULL); acd = kzalloc(sizeof(*acd), GFP_KERNEL); if (!acd) { |