diff options
| author | Adrian Bunk <bunk@stusta.de> | 2006-07-08 13:33:28 -0700 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2006-07-08 13:33:28 -0700 |
| commit | 9c6c6795eda34e4dc38ecac912a16b6314082beb (patch) | |
| tree | ed3dbc45df5794dc229bb2e439875b050ea80ab9 /drivers | |
| parent | 1252ecf63f77ea147bd40f5462c7d9e3d3ae2815 (diff) | |
| download | op-kernel-dev-9c6c6795eda34e4dc38ecac912a16b6314082beb.zip op-kernel-dev-9c6c6795eda34e4dc38ecac912a16b6314082beb.tar.gz | |
[IRDA]: fix drivers/net/irda/ali-ircc.c:ali_ircc_init()
The Coverity checker spotted, that from the changes from commit
898b1d16f8230fb912a0c2248df685735c6ceda3 the
if (ret)
platform_driver_unregister(&ali_ircc_driver);
was dead code.
This patch changes this function to what seems to have been the
intention.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/irda/ali-ircc.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/irda/ali-ircc.c b/drivers/net/irda/ali-ircc.c index bf1fca5..e3c8cd5 100644 --- a/drivers/net/irda/ali-ircc.c +++ b/drivers/net/irda/ali-ircc.c @@ -146,7 +146,7 @@ static int __init ali_ircc_init(void) { ali_chip_t *chip; chipio_t info; - int ret = -ENODEV; + int ret; int cfg, cfg_base; int reg, revision; int i = 0; @@ -160,6 +160,7 @@ static int __init ali_ircc_init(void) return ret; } + ret = -ENODEV; /* Probe for all the ALi chipsets we know about */ for (chip= chips; chip->name; chip++, i++) |
