diff options
author | Ursula Braun <ursula.braun@de.ibm.com> | 2012-11-18 21:46:46 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-11-19 14:19:00 -0500 |
commit | bfd2eb3b4af1d5b80c9d0411af81b5f36cc51085 (patch) | |
tree | b408cd04d039f8750341508d0d4500d84550ad75 /drivers/s390 | |
parent | 7bf9bcff45537aad3c71f7f3a4d93e90f3d3d077 (diff) |
ctcm: remove BUG_ONs
Remove BUG_ON's in ctcm driver, since the checked error conditions
are null pointer accesses.
Signed-off-by: Ursula Braun <ursula.braun@de.ibm.com>
Signed-off-by: Frank Blaschka <frank.blaschka@de.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/s390')
-rw-r--r-- | drivers/s390/net/ctcm_main.c | 2 | ||||
-rw-r--r-- | drivers/s390/net/ctcm_mpc.c | 3 |
2 files changed, 0 insertions, 5 deletions
diff --git a/drivers/s390/net/ctcm_main.c b/drivers/s390/net/ctcm_main.c index 98ea9cc6f1aa..817b68925ddd 100644 --- a/drivers/s390/net/ctcm_main.c +++ b/drivers/s390/net/ctcm_main.c | |||
@@ -1691,8 +1691,6 @@ static void ctcm_remove_device(struct ccwgroup_device *cgdev) | |||
1691 | { | 1691 | { |
1692 | struct ctcm_priv *priv = dev_get_drvdata(&cgdev->dev); | 1692 | struct ctcm_priv *priv = dev_get_drvdata(&cgdev->dev); |
1693 | 1693 | ||
1694 | BUG_ON(priv == NULL); | ||
1695 | |||
1696 | CTCM_DBF_TEXT_(SETUP, CTC_DBF_INFO, | 1694 | CTCM_DBF_TEXT_(SETUP, CTC_DBF_INFO, |
1697 | "removing device %p, proto : %d", | 1695 | "removing device %p, proto : %d", |
1698 | cgdev, priv->protocol); | 1696 | cgdev, priv->protocol); |
diff --git a/drivers/s390/net/ctcm_mpc.c b/drivers/s390/net/ctcm_mpc.c index 05b734a2b5b7..2dbc77b5137b 100644 --- a/drivers/s390/net/ctcm_mpc.c +++ b/drivers/s390/net/ctcm_mpc.c | |||
@@ -1367,7 +1367,6 @@ static void mpc_action_go_inop(fsm_instance *fi, int event, void *arg) | |||
1367 | struct mpc_group *grp; | 1367 | struct mpc_group *grp; |
1368 | struct channel *wch; | 1368 | struct channel *wch; |
1369 | 1369 | ||
1370 | BUG_ON(dev == NULL); | ||
1371 | CTCM_PR_DEBUG("Enter %s: %s\n", __func__, dev->name); | 1370 | CTCM_PR_DEBUG("Enter %s: %s\n", __func__, dev->name); |
1372 | 1371 | ||
1373 | priv = dev->ml_priv; | 1372 | priv = dev->ml_priv; |
@@ -1472,8 +1471,6 @@ static void mpc_action_timeout(fsm_instance *fi, int event, void *arg) | |||
1472 | struct channel *wch; | 1471 | struct channel *wch; |
1473 | struct channel *rch; | 1472 | struct channel *rch; |
1474 | 1473 | ||
1475 | BUG_ON(dev == NULL); | ||
1476 | |||
1477 | priv = dev->ml_priv; | 1474 | priv = dev->ml_priv; |
1478 | grp = priv->mpcg; | 1475 | grp = priv->mpcg; |
1479 | wch = priv->channel[CTCM_WRITE]; | 1476 | wch = priv->channel[CTCM_WRITE]; |