From c92e677b20fa389e9e81e21fa96e4520a5fd0ba5 Mon Sep 17 00:00:00 2001 From: Quytelda Kahja Date: Tue, 6 Mar 2018 17:31:41 -0800 Subject: [PATCH] staging: most: Remove unnecessary usage of BUG_ON(). There is no need for the calls to BUG_ON() in this driver, which are used to check if mbo or mbo->context are NULL; mbo is never NULL, and if mbo->context is NULL it would have already been dereferenced and oopsed before reaching the BUG_ON(). Signed-off-by: Quytelda Kahja Acked-by: Christian Gromm Signed-off-by: Greg Kroah-Hartman --- drivers/staging/most/core.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/staging/most/core.c b/drivers/staging/most/core.c index 8d311970225e..8f2833526f7f 100644 --- a/drivers/staging/most/core.c +++ b/drivers/staging/most/core.c @@ -916,7 +916,6 @@ static void arm_mbo(struct mbo *mbo) unsigned long flags; struct most_channel *c; - BUG_ON((!mbo) || (!mbo->context)); c = mbo->context; if (c->is_poisoned) { @@ -1019,8 +1018,6 @@ static void most_write_completion(struct mbo *mbo) { struct most_channel *c; - BUG_ON((!mbo) || (!mbo->context)); - c = mbo->context; if (mbo->status == MBO_E_INVAL) pr_info("WARN: Tx MBO status: invalid\n"); -- 2.30.2