diff options
author | Krzysztof Helt <krzysztof.h1@wp.pl> | 2008-04-28 05:14:51 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-04-28 11:58:35 -0400 |
commit | 2ae09f0da1cd0c8c646edea2e68356e76789461c (patch) | |
tree | 5354faf610b582010fbe508de810c7dff9e52622 /drivers/video/pm2fb.c | |
parent | 416e74ea7813597b586eafc24f67779eeb86e12f (diff) |
pm2fb: correct error values returned from probe function
Fix error values returned in some code branches in the pm2fb_probe() function.
Signed-off-by: Krzysztof Helt <krzysztof.h1@wp.pl>
Cc: "Antonino A. Daplas" <adaplas@pol.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/video/pm2fb.c')
-rw-r--r-- | drivers/video/pm2fb.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/video/pm2fb.c b/drivers/video/pm2fb.c index 30181b593829..82aa8242f441 100644 --- a/drivers/video/pm2fb.c +++ b/drivers/video/pm2fb.c | |||
@@ -1687,10 +1687,12 @@ static int __devinit pm2fb_probe(struct pci_dev *pdev, | |||
1687 | if (!err || err == 4) | 1687 | if (!err || err == 4) |
1688 | info->var = pm2fb_var; | 1688 | info->var = pm2fb_var; |
1689 | 1689 | ||
1690 | if (fb_alloc_cmap(&info->cmap, 256, 0) < 0) | 1690 | retval = fb_alloc_cmap(&info->cmap, 256, 0); |
1691 | if (retval < 0) | ||
1691 | goto err_exit_both; | 1692 | goto err_exit_both; |
1692 | 1693 | ||
1693 | if (register_framebuffer(info) < 0) | 1694 | retval = register_framebuffer(info); |
1695 | if (retval < 0) | ||
1694 | goto err_exit_all; | 1696 | goto err_exit_all; |
1695 | 1697 | ||
1696 | printk(KERN_INFO "fb%d: %s frame buffer device, memory = %dK.\n", | 1698 | printk(KERN_INFO "fb%d: %s frame buffer device, memory = %dK.\n", |