diff options
| author | Aditya Pakki <[email protected]> | 2019-12-19 11:21:11 -0600 |
|---|---|---|
| committer | Greg Kroah-Hartman <[email protected]> | 2020-01-03 11:04:57 +0100 |
| commit | d7a336d67ab5443a0ef14b8335d139e855e8a682 (patch) | |
| tree | 44d95b22e01493bca491e2fdbc89f387a0942627 | |
| parent | 398d999f96bf8e02573b0c7e32896c7812393bc0 (diff) | |
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 <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Greg Kroah-Hartman <[email protected]>
| -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) { |