diff options
author | Takashi Iwai <tiwai@suse.de> | 2015-10-01 17:59:43 +0200 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2015-10-15 14:05:28 +0200 |
commit | ded255be2276d365a91af2de7c7f8e2c233d4fa2 (patch) | |
tree | 4b12a3214ae05ccc3b1b31367c9cec53771cc251 /sound/pci/hda/patch_via.c | |
parent | 3e19fec33a5493f8a627a96ad3494d6c6dc2a624 (diff) | |
download | linux-stable-ded255be2276d365a91af2de7c7f8e2c233d4fa2.tar.gz linux-stable-ded255be2276d365a91af2de7c7f8e2c233d4fa2.tar.bz2 linux-stable-ded255be2276d365a91af2de7c7f8e2c233d4fa2.zip |
ALSA: hda - consolidate chip rename functions
A few multiple codec drivers do renaming the chip_name string but all
these are open-coded and some of them have even no error check. Let's
make common helpers to do it properly.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/pci/hda/patch_via.c')
-rw-r--r-- | sound/pci/hda/patch_via.c | 18 |
1 files changed, 4 insertions, 14 deletions
diff --git a/sound/pci/hda/patch_via.c b/sound/pci/hda/patch_via.c index da5366405eda..d714a57e9460 100644 --- a/sound/pci/hda/patch_via.c +++ b/sound/pci/hda/patch_via.c @@ -785,21 +785,11 @@ static int patch_vt1708S(struct hda_codec *codec) override_mic_boost(codec, 0x1e, 0, 3, 40); /* correct names for VT1708BCE */ - if (get_codec_type(codec) == VT1708BCE) { - kfree(codec->core.chip_name); - codec->core.chip_name = kstrdup("VT1708BCE", GFP_KERNEL); - snprintf(codec->card->mixername, - sizeof(codec->card->mixername), - "%s %s", codec->core.vendor_name, codec->core.chip_name); - } + if (get_codec_type(codec) == VT1708BCE) + snd_hda_codec_set_name(codec, "VT1708BCE"); /* correct names for VT1705 */ - if (codec->core.vendor_id == 0x11064397) { - kfree(codec->core.chip_name); - codec->core.chip_name = kstrdup("VT1705", GFP_KERNEL); - snprintf(codec->card->mixername, - sizeof(codec->card->mixername), - "%s %s", codec->core.vendor_name, codec->core.chip_name); - } + if (codec->core.vendor_id == 0x11064397) + snd_hda_codec_set_name(codec, "VT1705"); /* automatic parse from the BIOS config */ err = via_parse_auto_config(codec); |