From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Henningsson Subject: Re: [PATCH] ALSA: HDA: Cirrus - fix "Surround Speaker" volume control name Date: Wed, 14 Sep 2011 13:42:52 +0200 Message-ID: <4E70933C.2030904@canonical.com> References: <4E708F43.9060302@canonical.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="------------030204040609000407000502" Return-path: Received: from youngberry.canonical.com (youngberry.canonical.com [91.189.89.112]) by alsa0.perex.cz (Postfix) with ESMTP id 3F1C8103AD9 for ; Wed, 14 Sep 2011 13:42:52 +0200 (CEST) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: alsa-devel-bounces@alsa-project.org Errors-To: alsa-devel-bounces@alsa-project.org To: Takashi Iwai Cc: ALSA Development Mailing List , Arun Raghavan List-Id: alsa-devel@alsa-project.org This is a multi-part message in MIME format. --------------030204040609000407000502 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit On 09/14/2011 01:35 PM, Takashi Iwai wrote: > At Wed, 14 Sep 2011 13:25:55 +0200, > David Henningsson wrote: >> >> > From 7d63b3c06a02d107d9f499c43d6396ada9caae98 Mon Sep 17 00:00:00 2001 >> From: David Henningsson >> Date: Wed, 14 Sep 2011 13:22:54 +0200 >> Subject: [PATCH] ALSA: HDA: Cirrus - fix "Surround Speaker" volume control >> name >> >> Commit b4dabfc452a10 fixed only the "Switch" name, not the "Volume" name. >> >> Signed-off-by: David Henningsson >> --- >> sound/pci/hda/patch_cirrus.c | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/sound/pci/hda/patch_cirrus.c b/sound/pci/hda/patch_cirrus.c >> index d6c93d9..c45f3e6 100644 >> --- a/sound/pci/hda/patch_cirrus.c >> +++ b/sound/pci/hda/patch_cirrus.c >> @@ -535,7 +535,7 @@ static int add_volume(struct hda_codec *codec, const char *name, >> int index, unsigned int pval, int dir, >> struct snd_kcontrol **kctlp) >> { >> - char tmp[32]; >> + char tmp[44]; >> struct snd_kcontrol_new knew = >> HDA_CODEC_VOLUME_IDX(tmp, index, 0, 0, HDA_OUTPUT); >> knew.private_value = pval; > > Hm, it doesn't match with the patch description... Better commit message now? -- David Henningsson, Canonical Ltd. http://launchpad.net/~diwic --------------030204040609000407000502 Content-Type: text/x-patch; name="0001-ALSA-HDA-Cirrus-fix-Surround-Speaker-volume-control-.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename*0="0001-ALSA-HDA-Cirrus-fix-Surround-Speaker-volume-control-.pa"; filename*1="tch" >>From cd0944c76e1d43e15aa5147546ede32813e85901 Mon Sep 17 00:00:00 2001 From: David Henningsson Date: Wed, 14 Sep 2011 13:22:54 +0200 Subject: [PATCH] ALSA: HDA: Cirrus - fix "Surround Speaker" volume control name This patch fixes "Surround Speaker Playback Volume" being cut off. (Commit b4dabfc452a10 was probably meant to fix this, but it fixed only the "Switch" name, not the "Volume" name.) Signed-off-by: David Henningsson --- sound/pci/hda/patch_cirrus.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/sound/pci/hda/patch_cirrus.c b/sound/pci/hda/patch_cirrus.c index d6c93d9..c45f3e6 100644 --- a/sound/pci/hda/patch_cirrus.c +++ b/sound/pci/hda/patch_cirrus.c @@ -535,7 +535,7 @@ static int add_volume(struct hda_codec *codec, const char *name, int index, unsigned int pval, int dir, struct snd_kcontrol **kctlp) { - char tmp[32]; + char tmp[44]; struct snd_kcontrol_new knew = HDA_CODEC_VOLUME_IDX(tmp, index, 0, 0, HDA_OUTPUT); knew.private_value = pval; -- 1.7.5.4 --------------030204040609000407000502 Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline --------------030204040609000407000502--