diff options
author | Jacob Meuser <jakemsr@cvs.openbsd.org> | 2008-11-28 03:18:12 +0000 |
---|---|---|
committer | Jacob Meuser <jakemsr@cvs.openbsd.org> | 2008-11-28 03:18:12 +0000 |
commit | a0ecbeaf4c06d0ff7a49b261fd3a774ab7f6a2a8 (patch) | |
tree | ee21198b074ee7cfe6673c065160285aa967deb5 /sys/dev | |
parent | 7da313a2f65e39b86488c21555c356ca3a9100e5 (diff) |
add paranoia helper function, azalia_nid_to_index. the nid is
assumed in places to be the index of the widget in the widget_t
array of the codec_t, but I'm paranoid.
use it to shrink/simplify/fortify some parts. will be used more
soon.
Diffstat (limited to 'sys/dev')
-rw-r--r-- | sys/dev/pci/azalia_codec.c | 56 |
1 files changed, 32 insertions, 24 deletions
diff --git a/sys/dev/pci/azalia_codec.c b/sys/dev/pci/azalia_codec.c index 0fc75d25624..df1d670b2f8 100644 --- a/sys/dev/pci/azalia_codec.c +++ b/sys/dev/pci/azalia_codec.c @@ -1,4 +1,4 @@ -/* $OpenBSD: azalia_codec.c,v 1.69 2008/11/27 23:30:58 jakemsr Exp $ */ +/* $OpenBSD: azalia_codec.c,v 1.70 2008/11/28 03:18:11 jakemsr Exp $ */ /* $NetBSD: azalia_codec.c,v 1.8 2006/05/10 11:17:27 kent Exp $ */ /*- @@ -92,6 +92,8 @@ int azalia_generic_set_port(codec_t *, mixer_ctrl_t *); int azalia_generic_get_port(codec_t *, mixer_ctrl_t *); int azalia_gpio_unmute(codec_t *, int); +int azalia_nid_to_index(const codec_t *, nid_t); + int azalia_alc260_mixer_init(codec_t *); int azalia_alc88x_mixer_init(codec_t *); int azalia_ad1984_init_dacgroup(codec_t *); @@ -279,6 +281,23 @@ azalia_codec_init_vtbl(codec_t *this) * ---------------------------------------------------------------- */ int +azalia_nid_to_index(const codec_t *this, nid_t nid) +{ + int i; + + FOR_EACH_WIDGET(this, i) + if (this->w[i].nid == nid) + break; + if (i == this->wend || + !VALID_WIDGET_NID(this->w[i].nid, this) || + !this->w[i].enable) + return -1; + + return i; +} + + +int azalia_generic_codec_init_dacgroup(codec_t *this) { this->dacs.ngroups = 0; @@ -363,10 +382,10 @@ azalia_generic_codec_add_convgroup(codec_t *this, convgroupset_t *group, this->w[k].nid != conv) continue; } else { - FOR_EACH_WIDGET(this, l) - if (this->w[l].nid == - conv) - break; + l = azalia_nid_to_index(this, + conv); + if (l == -1) + continue; k = azalia_generic_codec_fnode (this, w->nid, l, 0); if (k < 0 || this->w[k].nid != @@ -400,7 +419,7 @@ int azalia_generic_codec_fnode(codec_t *this, nid_t node, int index, int depth) { const widget_t *w; - int i, j, k, ret; + int i, j, ret; w = &this->w[index]; if (w->nid == node) { @@ -416,20 +435,12 @@ azalia_generic_codec_fnode(codec_t *this, nid_t node, int index, int depth) if (++depth >= 10) return -1; for (i = 0; i < w->nconnections; i++) { - j = w->connections[i]; - if (!VALID_WIDGET_NID(j, this)) + j = azalia_nid_to_index(this, w->connections[i]); + if (j == -1) continue; - FOR_EACH_WIDGET(this, k) - if (this->w[k].nid == j) - break; - if (k < this->wend) { - if (this->w[k].enable) { - ret = azalia_generic_codec_fnode(this, node, - k, depth); - if (ret >= 0) - return ret; - } - } + ret = azalia_generic_codec_fnode(this, node, j, depth); + if (ret >= 0) + return ret; } return -1; } @@ -834,11 +845,8 @@ azalia_generic_mixer_init(codec_t *this) /* sense pins */ for (i = 0; i < this->nsense_pins; i++) { - FOR_EACH_WIDGET(this, j) { - if (this->w[j].nid == this->sense_pins[i]) - break; - } - if (j == this->wend) { + j = azalia_nid_to_index(this, this->sense_pins[i]); + if (j == -1) { DPRINTF(("%s: sense pin %2.2x not found\n", __func__, this->sense_pins[i])); continue; |