Commit 5f53f4e2 authored by Bjorn Helgaas's avatar Bjorn Helgaas Committed by Linus Torvalds

[PATCH] PNP: adjust pnp_register_card_driver() signature: ad1816a

Remove the assumption that pnp_register_card_driver() returns the
number of devices claimed.
Signed-off-by: default avatarBjorn Helgaas <bjorn.helgaas@hp.com>
Cc: Adam Belay <ambx1@neo.rr.com>
Cc: Jaroslav Kysela <perex@suse.cz>
Acked-by: default avatarTakashi Iwai <tiwai@suse.de>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 982c6094
...@@ -262,6 +262,8 @@ static int __devinit snd_card_ad1816a_probe(int dev, struct pnp_card_link *pcard ...@@ -262,6 +262,8 @@ static int __devinit snd_card_ad1816a_probe(int dev, struct pnp_card_link *pcard
return 0; return 0;
} }
static unsigned int __devinitdata ad1816a_devices;
static int __devinit snd_ad1816a_pnp_detect(struct pnp_card_link *card, static int __devinit snd_ad1816a_pnp_detect(struct pnp_card_link *card,
const struct pnp_card_device_id *id) const struct pnp_card_device_id *id)
{ {
...@@ -275,6 +277,7 @@ static int __devinit snd_ad1816a_pnp_detect(struct pnp_card_link *card, ...@@ -275,6 +277,7 @@ static int __devinit snd_ad1816a_pnp_detect(struct pnp_card_link *card,
if (res < 0) if (res < 0)
return res; return res;
dev++; dev++;
ad1816a_devices++;
return 0; return 0;
} }
return -ENODEV; return -ENODEV;
...@@ -297,10 +300,13 @@ static struct pnp_card_driver ad1816a_pnpc_driver = { ...@@ -297,10 +300,13 @@ static struct pnp_card_driver ad1816a_pnpc_driver = {
static int __init alsa_card_ad1816a_init(void) static int __init alsa_card_ad1816a_init(void)
{ {
int cards; int err;
err = pnp_register_card_driver(&ad1816a_pnpc_driver);
if (err)
return err;
cards = pnp_register_card_driver(&ad1816a_pnpc_driver); if (!ad1816a_devices) {
if (cards <= 0) {
pnp_unregister_card_driver(&ad1816a_pnpc_driver); pnp_unregister_card_driver(&ad1816a_pnpc_driver);
#ifdef MODULE #ifdef MODULE
printk(KERN_ERR "no AD1816A based soundcards found.\n"); printk(KERN_ERR "no AD1816A based soundcards found.\n");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment