aboutsummaryrefslogtreecommitdiffstats
path: root/sound/pci/hda/hda_codec.c
diff options
context:
space:
mode:
authorTakashi Iwai <tiwai@suse.de>2008-08-08 11:12:14 -0400
committerJaroslav Kysela <perex@perex.cz>2008-08-13 05:46:38 -0400
commitda3cec35dd3c31d8706db4bf379372ce70d92118 (patch)
tree9379edebb1c7abc7a7a92ce3be30a35b77d9aa1d /sound/pci/hda/hda_codec.c
parent622207dc31895b4e82c39100db8635d885c795e2 (diff)
ALSA: Kill snd_assert() in sound/pci/*
Kill snd_assert() in sound/pci/*, either removed or replaced with if () with snd_BUG_ON(). Signed-off-by: Takashi Iwai <tiwai@suse.de> Signed-off-by: Jaroslav Kysela <perex@perex.cz>
Diffstat (limited to 'sound/pci/hda/hda_codec.c')
-rw-r--r--sound/pci/hda/hda_codec.c21
1 files changed, 14 insertions, 7 deletions
diff --git a/sound/pci/hda/hda_codec.c b/sound/pci/hda/hda_codec.c
index d2e1093f8e97..77fbcd4a69b7 100644
--- a/sound/pci/hda/hda_codec.c
+++ b/sound/pci/hda/hda_codec.c
@@ -211,7 +211,8 @@ int snd_hda_get_connections(struct hda_codec *codec, hda_nid_t nid,
211 unsigned int shift, num_elems, mask; 211 unsigned int shift, num_elems, mask;
212 hda_nid_t prev_nid; 212 hda_nid_t prev_nid;
213 213
214 snd_assert(conn_list && max_conns > 0, return -EINVAL); 214 if (snd_BUG_ON(!conn_list || max_conns <= 0))
215 return -EINVAL;
215 216
216 parm = snd_hda_param_read(codec, nid, AC_PAR_CONNLIST_LEN); 217 parm = snd_hda_param_read(codec, nid, AC_PAR_CONNLIST_LEN);
217 if (parm & AC_CLIST_LONG) { 218 if (parm & AC_CLIST_LONG) {
@@ -407,8 +408,10 @@ int __devinit snd_hda_bus_new(struct snd_card *card,
407 .dev_free = snd_hda_bus_dev_free, 408 .dev_free = snd_hda_bus_dev_free,
408 }; 409 };
409 410
410 snd_assert(temp, return -EINVAL); 411 if (snd_BUG_ON(!temp))
411 snd_assert(temp->ops.command && temp->ops.get_response, return -EINVAL); 412 return -EINVAL;
413 if (snd_BUG_ON(!temp->ops.command || !temp->ops.get_response))
414 return -EINVAL;
412 415
413 if (busp) 416 if (busp)
414 *busp = NULL; 417 *busp = NULL;
@@ -588,8 +591,10 @@ int __devinit snd_hda_codec_new(struct hda_bus *bus, unsigned int codec_addr,
588 char component[13]; 591 char component[13];
589 int err; 592 int err;
590 593
591 snd_assert(bus, return -EINVAL); 594 if (snd_BUG_ON(!bus))
592 snd_assert(codec_addr <= HDA_MAX_CODEC_ADDRESS, return -EINVAL); 595 return -EINVAL;
596 if (snd_BUG_ON(codec_addr > HDA_MAX_CODEC_ADDRESS))
597 return -EINVAL;
593 598
594 if (bus->caddr_tbl[codec_addr]) { 599 if (bus->caddr_tbl[codec_addr]) {
595 snd_printk(KERN_ERR "hda_codec: " 600 snd_printk(KERN_ERR "hda_codec: "
@@ -2236,11 +2241,13 @@ static int __devinit set_pcm_default_values(struct hda_codec *codec,
2236 if (info->ops.close == NULL) 2241 if (info->ops.close == NULL)
2237 info->ops.close = hda_pcm_default_open_close; 2242 info->ops.close = hda_pcm_default_open_close;
2238 if (info->ops.prepare == NULL) { 2243 if (info->ops.prepare == NULL) {
2239 snd_assert(info->nid, return -EINVAL); 2244 if (snd_BUG_ON(!info->nid))
2245 return -EINVAL;
2240 info->ops.prepare = hda_pcm_default_prepare; 2246 info->ops.prepare = hda_pcm_default_prepare;
2241 } 2247 }
2242 if (info->ops.cleanup == NULL) { 2248 if (info->ops.cleanup == NULL) {
2243 snd_assert(info->nid, return -EINVAL); 2249 if (snd_BUG_ON(!info->nid))
2250 return -EINVAL;
2244 info->ops.cleanup = hda_pcm_default_cleanup; 2251 info->ops.cleanup = hda_pcm_default_cleanup;
2245 } 2252 }
2246 return 0; 2253 return 0;