From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1762494AbYD3Mdx (ORCPT ); Wed, 30 Apr 2008 08:33:53 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754450AbYD3Mdo (ORCPT ); Wed, 30 Apr 2008 08:33:44 -0400 Received: from mx2.suse.de ([195.135.220.15]:54004 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754394AbYD3Mdo (ORCPT ); Wed, 30 Apr 2008 08:33:44 -0400 Date: Wed, 30 Apr 2008 14:33:36 +0200 Message-ID: From: Takashi Iwai To: Rodolfo Giometti Cc: linux-kernel@vger.kernel.org, Jaroslav Kysela , James Stafford , Doug McLain Subject: Re: [PATCH] [ALSA] ice1724 - support for more that one codec. In-Reply-To: <1209547638-22570-1-git-send-email-giometti@linux.it> References: <1209547638-22570-1-git-send-email-giometti@linux.it> User-Agent: Wanderlust/2.15.5 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.7 (=?ISO-8859-4?Q?Sanj=F2?=) APEL/10.6 MULE XEmacs/21.5 (beta28) (fuki) (+CVS-20070806) (i386-suse-linux) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org At Wed, 30 Apr 2008 11:27:18 +0200, Rodolfo Giometti wrote: > > Now it's possible to specify how many codes are present on the board by > using a new filed as follow: > > ice->ac97_codecs = 4; > > For backward compatibility if "ice->ac97_codecs" is set to zero, the patch > redefines it as "1". > > Signed-off-by: Rodolfo Giometti Thanks for the patch. The change looks OK to me, but please fix the issues reported by checkpatch.pl. And, which hardware has multiple AC97 codecs? I'd like rather apply it together. Takashi > --- > sound/pci/ice1712/ice1712.c | 37 +++++++++++++++++++++++++------------ > sound/pci/ice1712/ice1712.h | 1 + > 2 files changed, 26 insertions(+), 12 deletions(-) > > diff --git a/sound/pci/ice1712/ice1712.c b/sound/pci/ice1712/ice1712.c > index 6630a0a..f78c724 100644 > --- a/sound/pci/ice1712/ice1712.c > +++ b/sound/pci/ice1712/ice1712.c > @@ -221,7 +221,7 @@ static void snd_ice1712_pro_ac97_write(struct snd_ac97 *ac97, > outb(reg, ICEMT(ice, AC97_INDEX)); > outw(val, ICEMT(ice, AC97_DATA)); > old_cmd &= ~(ICE1712_AC97_PBK_VSR | ICE1712_AC97_CAP_VSR); > - outb(old_cmd | ICE1712_AC97_WRITE, ICEMT(ice, AC97_CMD)); > + outb(old_cmd | ac97->num | ICE1712_AC97_WRITE, ICEMT(ice, AC97_CMD)); > for (tm = 0; tm < 0x10000; tm++) > if ((inb(ICEMT(ice, AC97_CMD)) & ICE1712_AC97_WRITE) == 0) > break; > @@ -244,7 +244,7 @@ static unsigned short snd_ice1712_pro_ac97_read(struct snd_ac97 *ac97, > break; > } > outb(reg, ICEMT(ice, AC97_INDEX)); > - outb(old_cmd | ICE1712_AC97_READ, ICEMT(ice, AC97_CMD)); > + outb(old_cmd | ac97->num | ICE1712_AC97_READ, ICEMT(ice, AC97_CMD)); > for (tm = 0; tm < 0x10000; tm++) > if ((inb(ICEMT(ice, AC97_CMD)) & ICE1712_AC97_READ) == 0) > break; > @@ -1504,7 +1504,7 @@ static void snd_ice1712_mixer_free_ac97(struct snd_ac97 *ac97) > > static int __devinit snd_ice1712_ac97_mixer(struct snd_ice1712 * ice) > { > - int err, bus_num = 0; > + int err, codec, bus_num = 0; > struct snd_ac97_template ac97; > struct snd_ac97_bus *pbus; > static struct snd_ac97_bus_ops con_ops = { > @@ -1532,15 +1532,28 @@ static int __devinit snd_ice1712_ac97_mixer(struct snd_ice1712 * ice) > } > > if (! (ice->eeprom.data[ICE_EEP1_ACLINK] & ICE1712_CFG_PRO_I2S)) { > - if ((err = snd_ac97_bus(ice->card, bus_num, &pro_ops, NULL, &pbus)) < 0) > - return err; > - memset(&ac97, 0, sizeof(ac97)); > - ac97.private_data = ice; > - ac97.private_free = snd_ice1712_mixer_free_ac97; > - if ((err = snd_ac97_mixer(pbus, &ac97, &ice->ac97)) < 0) > - printk(KERN_WARNING "ice1712: cannot initialize pro ac97, skipped\n"); > - else > - return 0; > + /* Backward compatibility */ > + if (!ice->ac97_codecs) > + ice->ac97_codecs = 1; > + > + for (codec = 0; codec < ice->ac97_codecs; codec++) { > + /* Sanity check */ > + if (codec >= 4) { > + printk(KERN_WARNING "ice1712: too much pro ac97 codes (max = %d), skipped\n", codec); > + break; > + } > + > + if ((err = snd_ac97_bus(ice->card, bus_num++, &pro_ops, NULL, &pbus)) < 0) > + return err; > + memset(&ac97, 0, sizeof(ac97)); > + ac97.num = codec; > + ac97.private_data = ice; > + ac97.private_free = snd_ice1712_mixer_free_ac97; > + if ((err = snd_ac97_mixer(pbus, &ac97, &ice->ac97)) < 0) > + printk(KERN_WARNING "ice1712: cannot initialize pro ac97 #%d, skipped\n", codec); > + } > + > + return 0; > } > /* I2S mixer only */ > strcat(ice->card->mixername, "ICE1712 - multitrack"); > diff --git a/sound/pci/ice1712/ice1712.h b/sound/pci/ice1712/ice1712.h > index 6ac486d..59047ad 100644 > --- a/sound/pci/ice1712/ice1712.h > +++ b/sound/pci/ice1712/ice1712.h > @@ -335,6 +335,7 @@ struct snd_ice1712 { > unsigned int force_rdma1: 1; /* VT1720/4 - RDMA1 as non-spdif */ > unsigned int num_total_dacs; /* total DACs */ > unsigned int num_total_adcs; /* total ADCs */ > + unsigned int ac97_codecs; /* total pro AC97 codecs */ > unsigned int cur_rate; /* current rate */ > > struct mutex open_mutex; > -- > 1.5.3.6 >