diff options
author | Karsten Keil <kkeil@linux-pingi.de> | 2012-07-12 03:01:10 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-07-18 12:40:54 -0400 |
commit | 5b999fd04af95e2cb624aaf6a7637561572b2d96 (patch) | |
tree | 7652915d0d5a8e9e27b1252da1e1c7e8c700a8b7 /drivers/isdn/hisax | |
parent | 7be7e93268447cbb42fa0e6939b6372c156a1ad5 (diff) |
ISDN: Add check for return value of pnp_activate_dev()
pnp_activate_dev() return value needs to be checked to make sure that
following calls calls to the PNP functions do work correctly.
Fix for report #44491 on bugzilla.kernel.org.
Signed-off-by: Karsten Keil <kkeil@linux-pingi.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/isdn/hisax')
-rw-r--r-- | drivers/isdn/hisax/isurf.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/isdn/hisax/isurf.c b/drivers/isdn/hisax/isurf.c index ea2717215296..c1530fe248c2 100644 --- a/drivers/isdn/hisax/isurf.c +++ b/drivers/isdn/hisax/isurf.c | |||
@@ -231,6 +231,11 @@ setup_isurf(struct IsdnCard *card) | |||
231 | } | 231 | } |
232 | pnp_disable_dev(pnp_d); | 232 | pnp_disable_dev(pnp_d); |
233 | err = pnp_activate_dev(pnp_d); | 233 | err = pnp_activate_dev(pnp_d); |
234 | if (err < 0) { | ||
235 | pr_warn("%s: pnp_activate_dev ret=%d\n", | ||
236 | __func__, err); | ||
237 | return 0; | ||
238 | } | ||
234 | cs->hw.isurf.reset = pnp_port_start(pnp_d, 0); | 239 | cs->hw.isurf.reset = pnp_port_start(pnp_d, 0); |
235 | cs->hw.isurf.phymem = pnp_mem_start(pnp_d, 1); | 240 | cs->hw.isurf.phymem = pnp_mem_start(pnp_d, 1); |
236 | cs->irq = pnp_irq(pnp_d, 0); | 241 | cs->irq = pnp_irq(pnp_d, 0); |