Re: [PATCH] [ALSA] ice1724 - support for more that one codec.

[Rodolfo Giometti]

On Wed, Apr 30, 2008 at 02:33:36PM +0200, Takashi Iwai wrote:
> 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 <giometti@xxxxxxxx>
> 
> Thanks for the patch.  The change looks OK to me, but please fix the
> issues reported by checkpatch.pl.

Ok, I'll do it ASAP.

> And, which hardware has multiple AC97 codecs?  I'd like rather apply
> it together.

I'm still working on it since I'd like to record from 4 channels at
time but I can't... here my custom settings:

static unsigned char int1410_eeprom[] __devinitdata = {
        [ICE_EEP1_CODEC]        = 0x1f,
        [ICE_EEP1_ACLINK]       = 0x00,
        [ICE_EEP1_I2SID]        = 0x01,
        [ICE_EEP1_SPDIF]        = 0x03,
        [ICE_EEP1_GPIO_MASK]    = 0xff,
        [ICE_EEP1_GPIO_STATE]   = 0xff,
        [ICE_EEP1_GPIO_DIR]     = 0xff,
};

static int __devinit snd_ice1712_int1410_init(struct snd_ice1712 *ice)
{
        printk("%s\n", __FUNCTION__);

        /* Set the pro AC97 codecs */
        ice->ac97_codecs = 4;

        /* Set the analog DACs/ADCs */
        ice->num_total_dacs = 8;
        ice->num_total_adcs = 8;

        return 0;
}

/* entry point */
struct snd_ice1712_card_info snd_ice1712_eurotech_cards[] __devinitdata =
{
        {
                .subvendor      = ICE1712_SUBDEVICE_INT1410,    /* a dummy id */
                .name           = "Eurotech INT 1410",
                .model          = "int1410",
                .chip_init      = snd_ice1712_int1410_init,
                .build_controls = snd_ice1712_int1410_add_controls,
                .no_mpu401      = 1,
                .eeprom_size    = sizeof(int1410_eeprom),
                .eeprom_data    = int1410_eeprom,
                .driver         = "int1410",
        },
        { } /* terminator */
};

But I'm a bit confused regarding how to enable such feature. Again,
should I use a multitrack audio recorder as ecasound or I can just use
arecord with some magic options? :)

Thanks,

Rodolfo

-- 

GNU/Linux Solutions                  e-mail:    giometti@xxxxxxxxxxxx
Linux Device Driver                             giometti@xxxxxxxx
Embedded Systems                     phone:	+39 349 2432127
UNIX programming                     skype:     rodolfo.giometti
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/