Re: A: VIA Technologies Inc. VT1720/24

Re: A: VIA Technologies Inc. VT1720/24

[Philipp Matthias Hahn]

[-- Attachment #1: Type: text/plain, Size: 3780 bytes --]

Philipp Matthias Hahn wrote:
> I'm not subscribed, so please cc: me on replies.
> 
> I have several Fujitsu-Siemens Scenic W600 with an Onboard AD1885 AC97
> Audio Codec. (See
> http://www.fujitsu-siemens.co.uk/rl/servicesupport/techsupport/boards/Motherboards/FSC/D1321/D1321.htm

Wrong link, the right one is
http://www.fujitsu-siemens.co.uk/rl/servicesupport/techsupport/Boards/Motherboards/FSC/D1527/D1527.htm

> for the board description). This one is working fine with the
> snd-intel8x0 module from the Linux-2.6.18.6 kernel.
> 
> In one of those computers I plugged in an additional sound card:
> 
> 	AudioTrak Prodigy 7.1 HiFi
> 
> XXXXX:/dev# lspci -s 2:0d.0
> 02:0d.0 Multimedia audio controller: VIA Technologies Inc. VT1720/24
> [Envy24PT/HT] PCI Multi-Channel Audio Controller (rev 01)
> 
> XXXXX:/dev# lspci -s 2:0d.0 -nvxxx
> 02:0d.0 0401: 1412:1724 (rev 01)
>         Subsystem: 3138:4154
>         Flags: medium devsel, IRQ 21
>         I/O ports at 3c00 [disabled] [size=32]
>         I/O ports at 3800 [disabled] [size=128]
>         Capabilities: [80] Power Management version 1
> 00: 12 14 24 17 00 00 10 02 01 00 01 04 00 40 00 00
> 10: 01 3c 00 00 01 38 00 00 00 00 00 00 00 00 00 00
> 20: 00 00 00 00 00 00 00 00 00 00 00 00 38 31 54 41
> 30: 00 00 00 00 80 00 00 00 00 00 00 00 05 01 00 00
> 40: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> 50: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> 60: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> 70: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> 80: 01 00 01 04 00 00 00 00 00 00 00 00 00 00 00 00
> 90: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> a0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> b0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> c0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> d0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> e0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> f0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> 
> Now I have two problems:
> 
> 1. The onboard intel8x0-sound is now missing, that is, the following
> Device is no longer listed using "lspci":
> 0000:00:1f.5 Multimedia audio controller: Intel Corp. 82801DB/DBL/DBM
> (ICH4/ICH4-L/ICH4-M) AC'97 Audio Controller (rev 01)
> 
> Is there a way to re-enable it, so I can use both cards?
> (On a side node not relevant for ALSA: the board also has an onboard
> integrated graphics controller, which is also disables by plugging in an
> AGP card. Would also be nice to re-enable it.)

Solved: I had to switch the BIOS from "Auto" to "Always enable".

> 2. Loading the "snd-ice1724" module reports this failure:
> 
> ACPI: PCI Interrupt 0000:02:0d.0[A] -> GSI 21 (level, low) -> IRQ 21
> ice1724: No matching model found for ID 0x38315441
> ice1724: Invalid EEPROM (size = 120)
> ACPI: PCI interrupt for device 0000:02:0d.0 disabled
> ICE1724: probe of 0000:02:0d.0 failed with error -5
> 
> Can somebody help me to get this working?
> 
> Seemd I'm not alone, but I can't understand the language:
> http://www.google.com/search?q=cache:ZaBwG4Ge5b4J:forum.ubuntu.pl/viewtopic.php%3Ft%3D16017%26view%3Dprevious%26sid%3D0a6db6f9be5123927dfb6c3a582e3eef+ice1724+0x38315441&hl=de&gl=de&ct=clnk&cd=1&client=firefox

I did some reading and hacking myself, since it just seems to be the
same as the Prodigy 7.1 with just a different model ID. See the attached
patch for a first version. It seems to work thus far without any problems.

Sincerly

Philipp Hahn
-- 
      Dipl.-Inform. Philipp.Hahn@informatik.uni-oldenburg.de
      Abteilung Systemsoftware und verteilte Systeme, Fk. II
Carl von Ossietzky Universitaet Oldenburg, 26111 Oldenburg, Germany
    http://www.svs.informatik.uni-oldenburg.de/contact/pmhahn/
      Telefon: +49 441 798-2866    Telefax: +49 441 798-2756

[-- Attachment #2: sound_pci_ice1712.diff --]
[-- Type: text/x-patch, Size: 2901 bytes --]

From: Philipp Matthias Hahn <pmhahn@pmhahn.de>

Fix small typo in comment of Prodigy 7.1 Light
Add support for AudioTrak Prodigt 7.1 HiFi

Signed-of-by: Philipp Matthias Hahn <pmhahn@pmhahn.de>
---

--- sound/pci/ice1712/aureon.c	2006-12-17 01:21:00.000000000 +0100
+++ sound/pci/ice1712/aureon.c	2007-01-19 10:50:19.000000000 +0100
@@ -2085,7 +2085,7 @@ static unsigned char prodigy71_eeprom[] 
 };
 
 static unsigned char prodigy71lt_eeprom[] __devinitdata = {
-	0x4b,	/* SYSCINF: clock 512, spdif-in/ADC, 4DACs */
+	0x4b,	/* SYSCONF: clock 384, spdif-in/ADC, 4DACs */
 	0x80,	/* ACLINK: I2S */
 	0xfc,	/* I2S: vol, 96k, 24bit, 192k */
 	0xc3,	/* SPDIF: out-en, out-int, spdif-in */
@@ -2100,6 +2100,22 @@ static unsigned char prodigy71lt_eeprom[
 	0x00,	/* GPIO_STATE2 */
 };
 	
+static unsigned char prodigy71hifi_eeprom[] __devinitdata = {
+	[ICE_EEP2_SYSCONF]     = 0x0b,	/* clock 512, spdif-in/ADC, 4DACs */
+	[ICE_EEP2_ACLINK]      = 0x80,	/* I2S */
+	[ICE_EEP2_I2S]         = 0xfc,	/* vol, 96k, 24bit, 192k */
+	[ICE_EEP2_SPDIF]       = 0xc3,	/* out-en, out-int, spdif-in */
+	[ICE_EEP2_GPIO_DIR]    = 0xff,
+	[ICE_EEP2_GPIO_DIR1]   = 0xff,
+	[ICE_EEP2_GPIO_DIR2]   = 0x5f,
+	[ICE_EEP2_GPIO_MASK]   = 0x00,
+	[ICE_EEP2_GPIO_MASK1]  = 0x00,
+	[ICE_EEP2_GPIO_MASK2]  = 0x00,
+	[ICE_EEP2_GPIO_STATE]  = 0x00,
+	[ICE_EEP2_GPIO_STATE1] = 0x00,
+	[ICE_EEP2_GPIO_STATE2] = 0x00,
+};
+
 
 /* entry point */
 struct snd_ice1712_card_info snd_vt1724_aureon_cards[] __devinitdata = {
@@ -2153,5 +2169,15 @@ struct snd_ice1712_card_info snd_vt1724_
 		.eeprom_data = prodigy71lt_eeprom,
 		.driver = "Prodigy71LT",
 	},
+	{
+		.subvendor = VT1724_SUBDEVICE_PRODIGY71HIFI,
+		.name = "Audiotrak Prodigy 7.1 HiFi",
+		.model = "prodigy71hifi",
+		.chip_init = aureon_init,
+		.build_controls = aureon_add_controls,
+		.eeprom_size = sizeof(prodigy71hifi_eeprom),
+		.eeprom_data = prodigy71hifi_eeprom,
+		.driver = "Prodigy71HiFi", /* should be identical with Prodigy71 */
+	},
 	{ } /* terminator */
 };
--- sound/pci/ice1712/aureon.h	2006-12-17 01:21:00.000000000 +0100
+++ sound/pci/ice1712/aureon.h	2007-01-19 10:50:19.000000000 +0100
@@ -28,13 +28,15 @@
 				       "{Terratec,Aureon 7.1 Space},"\
 				       "{Terratec,Aureon 7.1 Universe}," \
 					"{AudioTrak,Prodigy 7.1}," \
-					"{AudioTrak,Prodigy 7.1 LT},"
+					"{AudioTrak,Prodigy 7.1 LT},"\
+					"{AudioTrak,Prodigy 7.1 HiFi},"
 
 #define VT1724_SUBDEVICE_AUREON51_SKY	0x3b154711	/* Aureon 5.1 Sky */
 #define VT1724_SUBDEVICE_AUREON71_SPACE	0x3b154511	/* Aureon 7.1 Space */
 #define VT1724_SUBDEVICE_AUREON71_UNIVERSE	0x3b155311	/* Aureon 7.1 Universe */
 #define VT1724_SUBDEVICE_PRODIGY71	0x33495345	/* PRODIGY 7.1 */
 #define VT1724_SUBDEVICE_PRODIGY71LT	0x32315441	/* PRODIGY 7.1 LT */
+#define VT1724_SUBDEVICE_PRODIGY71HIFI	0x38315441	/* PRODIGY 7.1 HiFi */
 
 extern struct snd_ice1712_card_info  snd_vt1724_aureon_cards[];
 


[-- Attachment #3: Type: text/plain, Size: 347 bytes --]

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

[-- Attachment #4: Type: text/plain, Size: 161 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/alsa-devel

Re: [Alsa-devel] A: VIA Technologies Inc. VT1720/24

[Takashi Iwai]

At Wed, 24 Jan 2007 13:10:40 +0100,
Philipp Matthias Hahn wrote:
> 
> From: Philipp Matthias Hahn <pmhahn@pmhahn.de>
> 
> Fix small typo in comment of Prodigy 7.1 Light
> Add support for AudioTrak Prodigt 7.1 HiFi
> 
> Signed-of-by: Philipp Matthias Hahn <pmhahn@pmhahn.de>
(snip)
>  /* entry point */
>  struct snd_ice1712_card_info snd_vt1724_aureon_cards[] __devinitdata = {
> @@ -2153,5 +2169,15 @@ struct snd_ice1712_card_info snd_vt1724_
>  		.eeprom_data = prodigy71lt_eeprom,
>  		.driver = "Prodigy71LT",
>  	},
> +	{
> +		.subvendor = VT1724_SUBDEVICE_PRODIGY71HIFI,
> +		.name = "Audiotrak Prodigy 7.1 HiFi",
> +		.model = "prodigy71hifi",
> +		.chip_init = aureon_init,
> +		.build_controls = aureon_add_controls,
> +		.eeprom_size = sizeof(prodigy71hifi_eeprom),
> +		.eeprom_data = prodigy71hifi_eeprom,
> +		.driver = "Prodigy71HiFi", /* should be identical with Prodigy71 */

I recommend you to use the same string for this driver name here.
This string is referred by alsa-lib to pick up the corresponding
config file.  As Prodigy 7.1 Hifi is almost identical with Prodigy 7.1
LT from the configuration POV, it should use "Prodigy71LT", too, so
that you don't have to change alsa-lib at all.

Otherwise changes look OK to me, but your patch conflicts with the
latest ALSA HG tree because of the patch applied prior to this.
Could you regenerate the patch to ALSA HG tree (with the fix above)?


thanks,

Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

Re: A: VIA Technologies Inc. VT1720/24

[Philipp Matthias Hahn]

Hello Takashi!

On Wed, Jan 24, 2007 at 04:12:02PM +0100, Takashi Iwai wrote:
> I recommend you to use the same string for this driver name here.

Yes.

> This string is referred by alsa-lib to pick up the corresponding
> config file.  As Prodigy 7.1 Hifi is almost identical with Prodigy 7.1
> LT from the configuration POV, it should use "Prodigy71LT", too, so
> that you don't have to change alsa-lib at all.

No, the HiFi seems to be more like the Prodigy71 (without 'LT' or 'XT')
But I might err. Is there something I can do to test is it has 3 or 4
DACs, which seems to be the main difference?

> Otherwise changes look OK to me, but your patch conflicts with the
> latest ALSA HG tree because of the patch applied prior to this.
> Could you regenerate the patch to ALSA HG tree (with the fix above)?

Yes, see below. Three more questions:
1. Since the "Aureon7.1", "Prodigy7.1" and my "Prodigy7.1HiFi"; as well
as the "Prodigy7.1LT" and the "Prodigy7.1XT" respectively seem so share
the same EEPROM-Date, wouldn't it be better to re-use/share those
eeprom-data instead of declaring them 3 respectively 2 times?
Not much space saving, but gcc seems to be too stupid to recognize this;
Even const'ifying the structures doesn't help.

2. Most arrays and structures seem to be read only, what about adding
'const' to them?

3. Do you prefer array initialization in the form of
 {1, 2, 3, } /* IDX0, IDX1, IDX2 */
or as
 { [IDX0] = 1, [IDX1] = 2, [IDX2] = 3, }

> thanks,

You're doing the main job, I have to thank you.

> Takashi

Philipp


From: Philipp Matthias Hahn <pmhahn@pmhahn.de>

Fix small typo in comment of Prodigy 7.1 Light/XT
Add support for AudioTrak Prodigy 7.1 HiFi

Signed-of-by: Philipp Matthias Hahn <pmhahn@pmhahn.de>
---

--- pci/ice1712/aureon.c~	2007-01-25 10:07:50.000000000 +0100
+++ pci/ice1712/aureon.c	2007-01-25 10:46:56.000000000 +0100
@@ -2141,6 +2141,9 @@ static unsigned char aureon71_eeprom[] _
 	0x00,	/* GPIO_STATE2 */
 };
 
+#if 1 /* Prodigy7.1 has same EEPROM as Aureon7.1 */
+#define prodigy71_eeprom aureon71_eeprom
+#else
 static unsigned char prodigy71_eeprom[] __devinitdata = {
 	0x0b,	/* SYSCONF: clock 512, spdif-in/ADC, 4DACs */
 	0x80,	/* ACLINK: I2S */
@@ -2156,9 +2159,10 @@ static unsigned char prodigy71_eeprom[] 
 	0x00,	/* GPIO_STATE1 */
 	0x00,	/* GPIO_STATE2 */
 };
+#endif
 
 static unsigned char prodigy71lt_eeprom[] __devinitdata = {
-	0x4b,	/* SYSCINF: clock 512, spdif-in/ADC, 4DACs */
+	0x4b,	/* SYSCONF: clock 384, spdif-in/ADC, 4DACs */
 	0x80,	/* ACLINK: I2S */
 	0xfc,	/* I2S: vol, 96k, 24bit, 192k */
 	0xc3,	/* SPDIF: out-en, out-int, spdif-in */
@@ -2173,8 +2177,11 @@ static unsigned char prodigy71lt_eeprom[
 	0x00,	/* GPIO_STATE2 */
 };
 
+#if 1 /* Prodigy7.1XT has same EEPROM as Prodigy7.1LT */
+#define prodigy71xt_eeprom prodigy71lt_eeprom
+#else
 static unsigned char prodigy71xt_eeprom[] __devinitdata = {
-	0x4b,	/* SYSCINF: clock 512, spdif-in/ADC, 4DACs */
+	0x4b,	/* SYSCONF: clock 384, spdif-in/ADC, 4DACs */
 	0x80,	/* ACLINK: I2S */
 	0xfc,	/* I2S: vol, 96k, 24bit, 192k */
 	0xc3,	/* SPDIF: out-en, out-int, spdif-in */
@@ -2188,6 +2195,27 @@ static unsigned char prodigy71xt_eeprom[
 	0x00,	/* GPIO_STATE1 */
 	0x00,	/* GPIO_STATE2 */
 };
+#endif
+
+#if 1 /* Prodigy7.1HiFi has same EEPROM as Aureon7.1 */
+#define prodigy71hifi_eeprom aureon71_eeprom
+#else
+static unsigned char prodigy71hifi_eeprom[] __devinitdata = {
+	[ICE_EEP2_SYSCONF]     = 0x0b,	/* clock 512, spdif-in/ADC, 4DACs */
+	[ICE_EEP2_ACLINK]      = 0x80,	/* I2S */
+	[ICE_EEP2_I2S]         = 0xfc,	/* vol, 96k, 24bit, 192k */
+	[ICE_EEP2_SPDIF]       = 0xc3,	/* out-en, out-int, spdif-in */
+	[ICE_EEP2_GPIO_DIR]    = 0xff,
+	[ICE_EEP2_GPIO_DIR1]   = 0xff,
+	[ICE_EEP2_GPIO_DIR2]   = 0x5f,
+	[ICE_EEP2_GPIO_MASK]   = 0x00,
+	[ICE_EEP2_GPIO_MASK1]  = 0x00,
+	[ICE_EEP2_GPIO_MASK2]  = 0x00,
+	[ICE_EEP2_GPIO_STATE]  = 0x00,
+	[ICE_EEP2_GPIO_STATE1] = 0x00,
+	[ICE_EEP2_GPIO_STATE2] = 0x00,
+};
+#endif
 
 /* entry point */
 struct snd_ice1712_card_info snd_vt1724_aureon_cards[] __devinitdata = {
@@ -2251,5 +2279,15 @@ struct snd_ice1712_card_info snd_vt1724_
 		.eeprom_data = prodigy71xt_eeprom,
 		.driver = "Prodigy71LT",
 	},
+	{
+		.subvendor = VT1724_SUBDEVICE_PRODIGY71HIFI,
+		.name = "Audiotrak Prodigy 7.1 HiFi",
+		.model = "prodigy71hifi",
+		.chip_init = aureon_init,
+		.build_controls = aureon_add_controls,
+		.eeprom_size = sizeof(prodigy71hifi_eeprom),
+		.eeprom_data = prodigy71hifi_eeprom,
+		.driver = "Prodigy71", /* should be identical with Aureon71 */
+	},
 	{ } /* terminator */
 };
--- pci/ice1712/aureon.h~	2007-01-25 10:07:50.000000000 +0100
+++ pci/ice1712/aureon.h	2007-01-25 10:20:04.000000000 +0100
@@ -29,7 +29,8 @@
 				       "{Terratec,Aureon 7.1 Universe}," \
 					"{AudioTrak,Prodigy 7.1}," \
 					"{AudioTrak,Prodigy 7.1 LT},"\
-					"{AudioTrak,Prodigy 7.1 XT},"
+					"{AudioTrak,Prodigy 7.1 XT},"\
+					"{AudioTrak,Prodigy 7.1 HiFi},"
 
 #define VT1724_SUBDEVICE_AUREON51_SKY	0x3b154711	/* Aureon 5.1 Sky */
 #define VT1724_SUBDEVICE_AUREON71_SPACE	0x3b154511	/* Aureon 7.1 Space */
@@ -37,6 +38,7 @@
 #define VT1724_SUBDEVICE_PRODIGY71	0x33495345	/* PRODIGY 7.1 */
 #define VT1724_SUBDEVICE_PRODIGY71LT	0x32315441	/* PRODIGY 7.1 LT */
 #define VT1724_SUBDEVICE_PRODIGY71XT	0x36315441	/* PRODIGY 7.1 XT*/
+#define VT1724_SUBDEVICE_PRODIGY71HIFI	0x38315441	/* PRODIGY 7.1 HiFi */
 
 extern struct snd_ice1712_card_info  snd_vt1724_aureon_cards[];

-- 
  / /  (_)__  __ ____  __ Philipp Hahn
 / /__/ / _ \/ // /\ \/ /
/____/_/_//_/\_,_/ /_/\_\ pmhahn@titan.lahn.de

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

Re: A: VIA Technologies Inc. VT1720/24

[Takashi Iwai]

At Thu, 25 Jan 2007 11:13:19 +0100,
Philipp Matthias Hahn wrote:
> 
> Hello Takashi!
> 
> On Wed, Jan 24, 2007 at 04:12:02PM +0100, Takashi Iwai wrote:
> > I recommend you to use the same string for this driver name here.
> 
> Yes.
> 
> > This string is referred by alsa-lib to pick up the corresponding
> > config file.  As Prodigy 7.1 Hifi is almost identical with Prodigy 7.1
> > LT from the configuration POV, it should use "Prodigy71LT", too, so
> > that you don't have to change alsa-lib at all.
> 
> No, the HiFi seems to be more like the Prodigy71 (without 'LT' or 'XT')
> But I might err. Is there something I can do to test is it has 3 or 4
> DACs, which seems to be the main difference?

Well, with our without LT doesn't matter because both are aliased to
Aureon71 in alsa-lib/src/conf/cards/alias.conf.

> > Otherwise changes look OK to me, but your patch conflicts with the
> > latest ALSA HG tree because of the patch applied prior to this.
> > Could you regenerate the patch to ALSA HG tree (with the fix above)?
> 
> Yes, see below. Three more questions:
> 1. Since the "Aureon7.1", "Prodigy7.1" and my "Prodigy7.1HiFi"; as well
> as the "Prodigy7.1LT" and the "Prodigy7.1XT" respectively seem so share
> the same EEPROM-Date, wouldn't it be better to re-use/share those
> eeprom-data instead of declaring them 3 respectively 2 times?
> Not much space saving, but gcc seems to be too stupid to recognize this;
> Even const'ifying the structures doesn't help.

Good point.  It's fine to merge them, of course.

> 2. Most arrays and structures seem to be read only, what about adding
> 'const' to them?

Yes, it's better.

> 3. Do you prefer array initialization in the form of
>  {1, 2, 3, } /* IDX0, IDX1, IDX2 */
> or as
>  { [IDX0] = 1, [IDX1] = 2, [IDX2] = 3, }

I don't care about this so much.  The former style has advantage that
you can copy the byte stream as it is while the latter is a better
coding style in general.  Though, if we change these things at once,
I'd take the latter style now.

IMO, it'd be better to split the changes to two patches, one for
cleaning up the above (merge to a single struct, add const and change 
the struct definition), and another to add 7.1 Hifi definition.

Could you work on these two patches?


thanks,

Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

Re: A: VIA Technologies Inc. VT1720/24

[Takashi Iwai]

At Fri, 26 Jan 2007 21:29:50 +0100,
Philipp Matthias Hahn wrote:
> 
> Hello Takashi!
> 
> On Thu, Jan 25, 2007 at 12:19:51PM +0100, Takashi Iwai wrote:
> > IMO, it'd be better to split the changes to two patches, one for
> > cleaning up the above (merge to a single struct, add const and change 
> > the struct definition), and another to add 7.1 Hifi definition.
> > 
> > Could you work on these two patches?
> 
> Attached are 5 patches:
> 1. Reorganize existing eeprom data
> 2. Add Prodigy 7.1 Hifi
> 3. Add "const" to files in pci/ice1712/
> 4. Add some more "const", but needs changes in i2c/other/ak4*
> 5. Add even more "const" to everything related to TLV.
> 
> The patches apply in that order. Each step in between does compile
> without warnings, but I only did test-run the version with all 5 patches
> applied.
> The first 2 are the important ones, the other three are more or less
> cosmetic.

Thanks for the patches.  Now I applied 4 clean up patches (and fixed
some warnings), committed to HG tree.

But the patch for Prodigy 7.1 Hifi is still pending since another
deveopment by Konstantin is ongoing, too.  Could you both of you
cooperate to make more solid support for that hardware?

Of course, I can take one of your patches as a prototype, and make it
merged to the standard tree as a start up.  Anyway, I'd like hear the
opinions from both of you at first.


thanks,

Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

Re: A: VIA Technologies Inc. VT1720/24

[Konstantin Kletschke]

I am here *wink* :-)

Am 2007-01-29 15:41 +0100 schrieb Takashi Iwai:

> But the patch for Prodigy 7.1 Hifi is still pending since another
> deveopment by Konstantin is ongoing, too.  Could you both of you
> cooperate to make more solid support for that hardware?

Well, I strongly recommend. As I asked before, I am still wondering
how the initial patch works on Philipps Computer. 

While the wm8776 is implementend as a SPI device in aureon.c (and used
this way in the Prodigy 7.1 Hifi driver in aureon.c here) I use it in
i2c mode with success. What still missing is spi read/write stuff for
the wm8766 I am currently working on. Also there is a cs8415a in i2c
mode connected, whereas aureon.c expects it in SPI mode currently. 

Due to that essential differences I started a (quite good working!)
skeleton based on a Hercules fortissmo4 approach I discovered in
bugtrack.alsa-project.org. There I went into a prodigy hifi thread 
https://bugtrack.alsa-project.org/alsa-bug/view.php?id=2384
and put in my prodigy_hifi.c (which envolved meanwhile) for discussion.

> Of course, I can take one of your patches as a prototype, and make
> it

I really wonder how/if Philipps stuff works. It gives a working card,
but (as I expected) the main volume mixers are not working. All
essential chips known to be connected in SPI mode need to be
initialized to speak i2c on this board.

> merged to the standard tree as a start up.  Anyway, I'd like hear the
> opinions from both of you at first.

May be Philps patch needs only slight work, my approach of a
completely additional file prodigy_hifi.c might be most probably work
of greenhorn in hacking alsa device drivers, so I am here for
discussion. At the moment I am trying to get hands onto the wm8766
chip, which needs proper SPI functions. I can nowhere adopt existing
ones as far as I can see.

Kind Regards, Konstantin Kletschke

PS.: Takashi, the mail you are replying to, has this gone to this ml
or to you directly?


-- 
GPG KeyID EF62FCEF
Fingerprint: 13C9 B16B 9844 EC15 CC2E  A080 1E69 3FDA EF62 FCEF

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

Re: A: VIA Technologies Inc. VT1720/24

[Konstantin Kletschke]

Am 2007-01-29 16:30 +0100 schrieb Konstantin Kletschke:

> May be Philps patch needs only slight work, my approach of a

What I forgot to point out: It gives basic functionality but main
volume sliders are not working. Sound is constant at moderate
level. This is expected by me, since aureon.c seems to speak in spi
mode to wm8776, it needs i2c functions.

Konsti

-- 
GPG KeyID EF62FCEF
Fingerprint: 13C9 B16B 9844 EC15 CC2E  A080 1E69 3FDA EF62 FCEF

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

Subject: Does anyone know if this chipsset VT82C686 will output 24 bit, 96 kHz to USB 1.1 or pcmcia card...using a Linux 2.4 platform

[marion rundell]

[-- Attachment #1.1: Type: text/plain, Size: 1549 bytes --]




I am trying to determine if I can get spdif out of a system with the VT82C686 chipset.  

Does anyone know if this chipset is capable of outputting, on a Linux 2.4 platform...24bit, 96kHz bandwidth to a USB 1.1 or pcmcia card.  I have only been able to get unreliable sound out so far. 

Marion

mrundell@gmail.com
  ----- Original Message ----- 
  From: Konstantin Kletschke 
  To: ALSA developers 
  Sent: Monday, January 29, 2007 9:33 AM
  Subject: Re: [Alsa-devel] A: VIA Technologies Inc. VT1720/24


  Am 2007-01-29 16:30 +0100 schrieb Konstantin Kletschke:

  > May be Philps patch needs only slight work, my approach of a

  What I forgot to point out: It gives basic functionality but main
  volume sliders are not working. Sound is constant at moderate
  level. This is expected by me, since aureon.c seems to speak in spi
  mode to wm8776, it needs i2c functions.

  Konsti

  -- 
  GPG KeyID EF62FCEF
  Fingerprint: 13C9 B16B 9844 EC15 CC2E  A080 1E69 3FDA EF62 FCEF

  -------------------------------------------------------------------------
  Take Surveys. Earn Cash. Influence the Future of IT
  Join SourceForge.net's Techsay panel and you'll get the chance to share your
  opinions on IT & business topics through brief surveys - and earn cash
  http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
  _______________________________________________
  Alsa-devel mailing list
  Alsa-devel@lists.sourceforge.net
  https://lists.sourceforge.net/lists/listinfo/alsa-devel

[-- Attachment #1.2: Type: text/html, Size: 3211 bytes --]

[-- Attachment #2: Type: text/plain, Size: 347 bytes --]

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

[-- Attachment #3: Type: text/plain, Size: 161 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/alsa-devel

Re: A: VIA Technologies Inc. VT1720/24

[Takashi Iwai]

At Mon, 29 Jan 2007 16:30:12 +0100,
Konstantin Kletschke wrote:
> 
> PS.: Takashi, the mail you are replying to, has this gone to this ml
> or to you directly?

It's directly to me, and looks like no post appearing on ML.
Maybe it's filtered out due to too many attachments.

FWIW, most of patches in that post have been already committed to ALSA
HG tree, and only the addition of the new entry for 7.1 Hifi was
missing.  It's almost identical with what Philipp posted.


Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

Re: A: VIA Technologies Inc. VT1720/24

[Konstantin Kletschke]

Am 2007-01-29 16:43 +0100 schrieb Takashi Iwai:

> It's directly to me, and looks like no post appearing on ML.

Ok, so my setup os all right. I just wondered...

> missing.  It's almost identical with what Philipp posted.

Okay, so lets see what he says about my questions :-)

Konsti


-- 
GPG KeyID EF62FCEF
Fingerprint: 13C9 B16B 9844 EC15 CC2E  A080 1E69 3FDA EF62 FCEF

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

Re: A: VIA Technologies Inc. VT1720/24

[Takashi Iwai]

At Mon, 29 Jan 2007 15:41:53 +0100,
I wrote:
> 
> At Fri, 26 Jan 2007 21:29:50 +0100,
> Philipp Matthias Hahn wrote:
> > 
> > Hello Takashi!
> > 
> > On Thu, Jan 25, 2007 at 12:19:51PM +0100, Takashi Iwai wrote:
> > > IMO, it'd be better to split the changes to two patches, one for
> > > cleaning up the above (merge to a single struct, add const and change 
> > > the struct definition), and another to add 7.1 Hifi definition.
> > > 
> > > Could you work on these two patches?
> > 
> > Attached are 5 patches:
> > 1. Reorganize existing eeprom data
> > 2. Add Prodigy 7.1 Hifi
> > 3. Add "const" to files in pci/ice1712/
> > 4. Add some more "const", but needs changes in i2c/other/ak4*
> > 5. Add even more "const" to everything related to TLV.
> > 
> > The patches apply in that order. Each step in between does compile
> > without warnings, but I only did test-run the version with all 5 patches
> > applied.
> > The first 2 are the important ones, the other three are more or less
> > cosmetic.
> 
> Thanks for the patches.  Now I applied 4 clean up patches (and fixed
> some warnings), committed to HG tree.
> 
> But the patch for Prodigy 7.1 Hifi is still pending since another
> deveopment by Konstantin is ongoing, too.  Could you both of you
> cooperate to make more solid support for that hardware?
> 
> Of course, I can take one of your patches as a prototype, and make it
> merged to the standard tree as a start up.  Anyway, I'd like hear the
> opinions from both of you at first.

Ping.

Just wondering if you have any progress wrt this hardware...


Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

Re: A: VIA Technologies Inc. VT1720/24

[Konstantin Kletschke]

Am 2007-02-14 16:30 +0100 schrieb Takashi Iwai:

> Ping.

Pong :-)

> Just wondering if you have any progress wrt this hardware...

Well, I am still at it also did philip mail me, he is alive.

I had so god damn much work recently I could not spend work into the
prodigy hifi support (spi for wm8766 and i2c for cs8415a are TODO).

Konsti

-- 
GPG KeyID EF62FCEF
Fingerprint: 13C9 B16B 9844 EC15 CC2E  A080 1E69 3FDA EF62 FCEF

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

Re: A: VIA Technologies Inc. VT1720/24

[Konstantin Kletschke]

Well actually I am fiddling around with the same card also. Since
nobody responded here in this mailing list anymore I went into

https://bugtrack.alsa-project.org/alsa-bug/view.php?id=2384

and evolved a separate prodigy_hifi.c which is quite redundant due to
the simialirities but I have to get know to als coding and wanted to
clean up later. This here looks clean and seems to work though.

I went back here Raymond pointing me to the alsa-devel archive. This
is the first mail which arrives at my mail account so I answer now
here.

I applied your stuff onto alsa-driver-1.0.14rc2.

Am 2007-01-25 11:13 +0100 schrieb Philipp Matthias Hahn:

> Yes, see below. Three more questions:
> 1. Since the "Aureon7.1", "Prodigy7.1" and my "Prodigy7.1HiFi"; as well
> as the "Prodigy7.1LT" and the "Prodigy7.1XT" respectively seem so share
> the same EEPROM-Date, wouldn't it be better to re-use/share those
> eeprom-data instead of declaring them 3 respectively 2 times?

In my opinion Prodigy7.1HiFi eeprom_ data has to be slightly
different, I need to put 

[ICE_EEP2_SYSCONF] = 0x4b

instead of

[ICE_EEP2_SYSCONF] = 0x0b

Otherwise I got speeded up sound playback.
See note (0014172) in this thread by me.

What I am currently fiddling about is the CS8415A Chip:

ALSA /XXX/alsa-kernel/pci/ice1712/aureon.c:1921: No CS8415
chip. Skipping CS8415 controls.

In the note (0014176) I am trying to search it at the i2c address it
should appear at. Still I did not found it :-)

Regards, Konsti

-- 
GPG KeyID EF62FCEF
Fingerprint: 13C9 B16B 9844 EC15 CC2E  A080 1E69 3FDA EF62 FCEF

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

Re: A: VIA Technologies Inc. VT1720/24

[Konstantin Kletschke]

Am 2007-01-25 13:36 +0100 schrieb Konstantin Kletschke:

> ALSA /XXX/alsa-kernel/pci/ice1712/aureon.c:1921: No CS8415
> chip. Skipping CS8415 controls.
> 
> In the note (0014176) I am trying to search it at the i2c address it
> should appear at. Still I did not found it :-)

Okay, I got the CS8451 at the i2c bus, I had to left shift my bitmask
to get the idea of how i2c addressing works:

        printk(KERN_DEBUG "addr_cs: %02x value 0x7F: %02x\n",
               addr_cs, snd_vt1724_read_i2c(ice, addr_cs, CS8415_ID));
        printk(KERN_DEBUG "addr_cs: %02x value 0x7F: %02x\n",
               addr_cs, snd_vt1724_read_i2c(ice, addr_cs+1,
	       CS8415_ID));
gives:

i2c_read: [0x2e,0x7f] = 0x41
addr_cs: 2e value 0x7F: 41
i2c_read: [0x2f,0x7f] = 0x41
addr_cs: 2e value 0x7F: 41

Hmmm...


-- 
GPG KeyID EF62FCEF
Fingerprint: 13C9 B16B 9844 EC15 CC2E  A080 1E69 3FDA EF62 FCEF

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

Re: A: VIA Technologies Inc. VT1720/24

[Konstantin Kletschke]

Now I inspected your approach more in detail. As I said things are
working well, but other as my approach the volume sliders (Master and
PCM) have no effect. They are moving but the sound keeps being at a
moderate level (constant). 

I suppose the wm8776 is not addressed correctly, where does aureon.c
expect it? I am still fiddling this out.

Am 2007-01-25 11:13 +0100 schrieb Philipp Matthias Hahn:

> > config file.  As Prodigy 7.1 Hifi is almost identical with Prodigy
> > 7.1

Are you both really shure? From code side of view the HiFi seems to be
the first combination of vt1724 and wm8776 and 8766 and cs8415a.

Philipp, is your prodigy 7.1 hifi really functional with your patch
mailed in this thread? Mine still needs the other initialisation
register value and cs8415a is not initialized here.

As said, now I realize the main mixer don't have effect on output
volume.

Kind Regards, Konsti

-- 
GPG KeyID EF62FCEF
Fingerprint: 13C9 B16B 9844 EC15 CC2E  A080 1E69 3FDA EF62 FCEF

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV