diff options
| author | ariff <ariff@FreeBSD.org> | 2007-03-15 17:35:04 +0000 |
|---|---|---|
| committer | ariff <ariff@FreeBSD.org> | 2007-03-15 17:35:04 +0000 |
| commit | 9f01779680115d3d0fc6a0cce36cbef70480141d (patch) | |
| tree | 407488b50e84b26ef2ebd0fb99a22372ff72d839 /sys | |
| parent | 36b7be9f85ceba78e7ca5a5c7817e6600edb034f (diff) | |
| download | FreeBSD-src-9f01779680115d3d0fc6a0cce36cbef70480141d.zip FreeBSD-src-9f01779680115d3d0fc6a0cce36cbef70480141d.tar.gz | |
- Put some sanity break statement in few missing places.
- Remove NULL checking on snd_mtxcreate() (M_WAITOK) .
Diffstat (limited to 'sys')
| -rw-r--r-- | sys/dev/sound/pci/hda/hdac.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/sys/dev/sound/pci/hda/hdac.c b/sys/dev/sound/pci/hda/hdac.c index 8d14360..88c3e46 100644 --- a/sys/dev/sound/pci/hda/hdac.c +++ b/sys/dev/sound/pci/hda/hdac.c @@ -1718,6 +1718,8 @@ hdac_widget_pin_getconfig(struct hdac_widget *w) config &= ~HDA_CONFIG_DEFAULTCONF_DEVICE_MASK; config |= HDA_CONFIG_DEFAULTCONF_DEVICE_HP_OUT; break; + default: + break; } } else if (id == HDA_CODEC_ALC880 && (sc->pci_subvendor == CLEVO_D900T_SUBVENDOR || @@ -2828,6 +2830,8 @@ hdac_channel_trigger(kobj_t obj, void *data, int go) case PCMTRIG_ABORT: hdac_channel_stop(sc, ch); break; + default: + break; } hdac_unlock(sc); @@ -3230,12 +3234,6 @@ hdac_attach(device_t dev) } sc->lock = snd_mtxcreate(device_get_nameunit(dev), HDAC_MTX_NAME); - if (sc->lock == NULL) { - device_printf(dev, "mutex creation failed\n"); - free(sc, M_DEVBUF); - return (ENOMEM); - } - sc->dev = dev; sc->pci_subvendor = (uint32_t)pci_get_subdevice(sc->dev) << 16; sc->pci_subvendor |= (uint32_t)pci_get_subvendor(sc->dev) & 0x0000ffff; |
