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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20080502195232.GA8502@enneenne.com>
Date:	Fri, 2 May 2008 21:52:32 +0200
From:	Rodolfo Giometti <giometti@...eenne.com>
To:	Takashi Iwai <tiwai@...e.de>
Cc:	linux-kernel@...r.kernel.org, Jaroslav Kysela <perex@...e.cz>,
	James Stafford <jstafford@...ltd.com>,
	Doug McLain <nostar@...cast.net>
Subject: Re: [PATCH] [ALSA] ice1724 - support for more that one codec.

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@...ux.it>
> 
> 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@...eenne.com
Linux Device Driver                             giometti@...ux.it
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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/